Commit ea7ab982 authored by Michal Hocko's avatar Michal Hocko Committed by Linus Torvalds

mm, compaction: change COMPACT_ constants into enum

Compaction code is doing weird dances between COMPACT_FOO -> int ->
unsigned long

But there doesn't seem to be any reason for that.  All functions which
return/use one of those constants are not expecting any other value so it
really makes sense to define an enum for them and make it clear that no
other values are expected.

This is a pure cleanup and shouldn't introduce any functional changes.
Signed-off-by: default avatarMichal Hocko <mhocko@suse.com>
Acked-by: default avatarVlastimil Babka <vbabka@suse.cz>
Acked-by: default avatarHillf Danton <hillf.zj@alibaba-inc.com>
Cc: David Rientjes <rientjes@google.com>
Cc: Johannes Weiner <hannes@cmpxchg.org>
Cc: Joonsoo Kim <js1304@gmail.com>
Cc: Mel Gorman <mgorman@techsingularity.net>
Cc: Tetsuo Handa <penguin-kernel@I-love.SAKURA.ne.jp>
Cc: Vladimir Davydov <vdavydov@virtuozzo.com>
Signed-off-by: default avatarAndrew Morton <akpm@linux-foundation.org>
Signed-off-by: default avatarLinus Torvalds <torvalds@linux-foundation.org>
parent b6459cc1
...@@ -2,21 +2,29 @@ ...@@ -2,21 +2,29 @@
#define _LINUX_COMPACTION_H #define _LINUX_COMPACTION_H
/* Return values for compact_zone() and try_to_compact_pages() */ /* Return values for compact_zone() and try_to_compact_pages() */
/* compaction didn't start as it was deferred due to past failures */
#define COMPACT_DEFERRED 0
/* compaction didn't start as it was not possible or direct reclaim was more suitable */
#define COMPACT_SKIPPED 1
/* compaction should continue to another pageblock */
#define COMPACT_CONTINUE 2
/* direct compaction partially compacted a zone and there are suitable pages */
#define COMPACT_PARTIAL 3
/* The full zone was compacted */
#define COMPACT_COMPLETE 4
/* For more detailed tracepoint output */
#define COMPACT_NO_SUITABLE_PAGE 5
#define COMPACT_NOT_SUITABLE_ZONE 6
#define COMPACT_CONTENDED 7
/* When adding new states, please adjust include/trace/events/compaction.h */ /* When adding new states, please adjust include/trace/events/compaction.h */
enum compact_result {
/* compaction didn't start as it was deferred due to past failures */
COMPACT_DEFERRED,
/*
* compaction didn't start as it was not possible or direct reclaim
* was more suitable
*/
COMPACT_SKIPPED,
/* compaction should continue to another pageblock */
COMPACT_CONTINUE,
/*
* direct compaction partially compacted a zone and there are suitable
* pages
*/
COMPACT_PARTIAL,
/* The full zone was compacted */
COMPACT_COMPLETE,
/* For more detailed tracepoint output */
COMPACT_NO_SUITABLE_PAGE,
COMPACT_NOT_SUITABLE_ZONE,
COMPACT_CONTENDED,
};
/* Used to signal whether compaction detected need_sched() or lock contention */ /* Used to signal whether compaction detected need_sched() or lock contention */
/* No contention detected */ /* No contention detected */
...@@ -38,12 +46,13 @@ extern int sysctl_extfrag_handler(struct ctl_table *table, int write, ...@@ -38,12 +46,13 @@ extern int sysctl_extfrag_handler(struct ctl_table *table, int write,
extern int sysctl_compact_unevictable_allowed; extern int sysctl_compact_unevictable_allowed;
extern int fragmentation_index(struct zone *zone, unsigned int order); extern int fragmentation_index(struct zone *zone, unsigned int order);
extern unsigned long try_to_compact_pages(gfp_t gfp_mask, unsigned int order, extern enum compact_result try_to_compact_pages(gfp_t gfp_mask,
unsigned int order,
unsigned int alloc_flags, const struct alloc_context *ac, unsigned int alloc_flags, const struct alloc_context *ac,
enum migrate_mode mode, int *contended); enum migrate_mode mode, int *contended);
extern void compact_pgdat(pg_data_t *pgdat, int order); extern void compact_pgdat(pg_data_t *pgdat, int order);
extern void reset_isolation_suitable(pg_data_t *pgdat); extern void reset_isolation_suitable(pg_data_t *pgdat);
extern unsigned long compaction_suitable(struct zone *zone, int order, extern enum compact_result compaction_suitable(struct zone *zone, int order,
unsigned int alloc_flags, int classzone_idx); unsigned int alloc_flags, int classzone_idx);
extern void defer_compaction(struct zone *zone, int order); extern void defer_compaction(struct zone *zone, int order);
...@@ -57,7 +66,7 @@ extern void kcompactd_stop(int nid); ...@@ -57,7 +66,7 @@ extern void kcompactd_stop(int nid);
extern void wakeup_kcompactd(pg_data_t *pgdat, int order, int classzone_idx); extern void wakeup_kcompactd(pg_data_t *pgdat, int order, int classzone_idx);
#else #else
static inline unsigned long try_to_compact_pages(gfp_t gfp_mask, static inline enum compact_result try_to_compact_pages(gfp_t gfp_mask,
unsigned int order, int alloc_flags, unsigned int order, int alloc_flags,
const struct alloc_context *ac, const struct alloc_context *ac,
enum migrate_mode mode, int *contended) enum migrate_mode mode, int *contended)
...@@ -73,7 +82,7 @@ static inline void reset_isolation_suitable(pg_data_t *pgdat) ...@@ -73,7 +82,7 @@ static inline void reset_isolation_suitable(pg_data_t *pgdat)
{ {
} }
static inline unsigned long compaction_suitable(struct zone *zone, int order, static inline enum compact_result compaction_suitable(struct zone *zone, int order,
int alloc_flags, int classzone_idx) int alloc_flags, int classzone_idx)
{ {
return COMPACT_SKIPPED; return COMPACT_SKIPPED;
......
...@@ -1229,7 +1229,7 @@ static inline bool is_via_compact_memory(int order) ...@@ -1229,7 +1229,7 @@ static inline bool is_via_compact_memory(int order)
return order == -1; return order == -1;
} }
static int __compact_finished(struct zone *zone, struct compact_control *cc, static enum compact_result __compact_finished(struct zone *zone, struct compact_control *cc,
const int migratetype) const int migratetype)
{ {
unsigned int order; unsigned int order;
...@@ -1292,7 +1292,8 @@ static int __compact_finished(struct zone *zone, struct compact_control *cc, ...@@ -1292,7 +1292,8 @@ static int __compact_finished(struct zone *zone, struct compact_control *cc,
return COMPACT_NO_SUITABLE_PAGE; return COMPACT_NO_SUITABLE_PAGE;
} }
static int compact_finished(struct zone *zone, struct compact_control *cc, static enum compact_result compact_finished(struct zone *zone,
struct compact_control *cc,
const int migratetype) const int migratetype)
{ {
int ret; int ret;
...@@ -1312,7 +1313,7 @@ static int compact_finished(struct zone *zone, struct compact_control *cc, ...@@ -1312,7 +1313,7 @@ static int compact_finished(struct zone *zone, struct compact_control *cc,
* COMPACT_PARTIAL - If the allocation would succeed without compaction * COMPACT_PARTIAL - If the allocation would succeed without compaction
* COMPACT_CONTINUE - If compaction should run now * COMPACT_CONTINUE - If compaction should run now
*/ */
static unsigned long __compaction_suitable(struct zone *zone, int order, static enum compact_result __compaction_suitable(struct zone *zone, int order,
unsigned int alloc_flags, unsigned int alloc_flags,
int classzone_idx) int classzone_idx)
{ {
...@@ -1358,11 +1359,11 @@ static unsigned long __compaction_suitable(struct zone *zone, int order, ...@@ -1358,11 +1359,11 @@ static unsigned long __compaction_suitable(struct zone *zone, int order,
return COMPACT_CONTINUE; return COMPACT_CONTINUE;
} }
unsigned long compaction_suitable(struct zone *zone, int order, enum compact_result compaction_suitable(struct zone *zone, int order,
unsigned int alloc_flags, unsigned int alloc_flags,
int classzone_idx) int classzone_idx)
{ {
unsigned long ret; enum compact_result ret;
ret = __compaction_suitable(zone, order, alloc_flags, classzone_idx); ret = __compaction_suitable(zone, order, alloc_flags, classzone_idx);
trace_mm_compaction_suitable(zone, order, ret); trace_mm_compaction_suitable(zone, order, ret);
...@@ -1372,9 +1373,9 @@ unsigned long compaction_suitable(struct zone *zone, int order, ...@@ -1372,9 +1373,9 @@ unsigned long compaction_suitable(struct zone *zone, int order,
return ret; return ret;
} }
static int compact_zone(struct zone *zone, struct compact_control *cc) static enum compact_result compact_zone(struct zone *zone, struct compact_control *cc)
{ {
int ret; enum compact_result ret;
unsigned long start_pfn = zone->zone_start_pfn; unsigned long start_pfn = zone->zone_start_pfn;
unsigned long end_pfn = zone_end_pfn(zone); unsigned long end_pfn = zone_end_pfn(zone);
const int migratetype = gfpflags_to_migratetype(cc->gfp_mask); const int migratetype = gfpflags_to_migratetype(cc->gfp_mask);
...@@ -1530,11 +1531,11 @@ static int compact_zone(struct zone *zone, struct compact_control *cc) ...@@ -1530,11 +1531,11 @@ static int compact_zone(struct zone *zone, struct compact_control *cc)
return ret; return ret;
} }
static unsigned long compact_zone_order(struct zone *zone, int order, static enum compact_result compact_zone_order(struct zone *zone, int order,
gfp_t gfp_mask, enum migrate_mode mode, int *contended, gfp_t gfp_mask, enum migrate_mode mode, int *contended,
unsigned int alloc_flags, int classzone_idx) unsigned int alloc_flags, int classzone_idx)
{ {
unsigned long ret; enum compact_result ret;
struct compact_control cc = { struct compact_control cc = {
.nr_freepages = 0, .nr_freepages = 0,
.nr_migratepages = 0, .nr_migratepages = 0,
...@@ -1572,7 +1573,7 @@ int sysctl_extfrag_threshold = 500; ...@@ -1572,7 +1573,7 @@ int sysctl_extfrag_threshold = 500;
* *
* This is the main entry point for direct page compaction. * This is the main entry point for direct page compaction.
*/ */
unsigned long try_to_compact_pages(gfp_t gfp_mask, unsigned int order, enum compact_result try_to_compact_pages(gfp_t gfp_mask, unsigned int order,
unsigned int alloc_flags, const struct alloc_context *ac, unsigned int alloc_flags, const struct alloc_context *ac,
enum migrate_mode mode, int *contended) enum migrate_mode mode, int *contended)
{ {
...@@ -1580,7 +1581,7 @@ unsigned long try_to_compact_pages(gfp_t gfp_mask, unsigned int order, ...@@ -1580,7 +1581,7 @@ unsigned long try_to_compact_pages(gfp_t gfp_mask, unsigned int order,
int may_perform_io = gfp_mask & __GFP_IO; int may_perform_io = gfp_mask & __GFP_IO;
struct zoneref *z; struct zoneref *z;
struct zone *zone; struct zone *zone;
int rc = COMPACT_DEFERRED; enum compact_result rc = COMPACT_DEFERRED;
int all_zones_contended = COMPACT_CONTENDED_LOCK; /* init for &= op */ int all_zones_contended = COMPACT_CONTENDED_LOCK; /* init for &= op */
*contended = COMPACT_CONTENDED_NONE; *contended = COMPACT_CONTENDED_NONE;
...@@ -1594,7 +1595,7 @@ unsigned long try_to_compact_pages(gfp_t gfp_mask, unsigned int order, ...@@ -1594,7 +1595,7 @@ unsigned long try_to_compact_pages(gfp_t gfp_mask, unsigned int order,
/* Compact each zone in the list */ /* Compact each zone in the list */
for_each_zone_zonelist_nodemask(zone, z, ac->zonelist, ac->high_zoneidx, for_each_zone_zonelist_nodemask(zone, z, ac->zonelist, ac->high_zoneidx,
ac->nodemask) { ac->nodemask) {
int status; enum compact_result status;
int zone_contended; int zone_contended;
if (compaction_deferred(zone, order)) if (compaction_deferred(zone, order))
......
...@@ -3188,7 +3188,7 @@ __alloc_pages_direct_compact(gfp_t gfp_mask, unsigned int order, ...@@ -3188,7 +3188,7 @@ __alloc_pages_direct_compact(gfp_t gfp_mask, unsigned int order,
enum migrate_mode mode, int *contended_compaction, enum migrate_mode mode, int *contended_compaction,
bool *deferred_compaction) bool *deferred_compaction)
{ {
unsigned long compact_result; enum compact_result compact_result;
struct page *page; struct page *page;
if (!order) if (!order)
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment