diff options
author | Xishi Qiu <qiuxishi@huawei.com> | 2017-05-03 17:52:55 -0400 |
---|---|---|
committer | Linus Torvalds <torvalds@linux-foundation.org> | 2017-05-03 18:52:08 -0400 |
commit | bbf9ce9719b5c64eac7f8483d1da6e8a3d4f98f5 (patch) | |
tree | 6c9948de3bc096678c5e5e02eb39036421104c97 /mm/page_isolation.c | |
parent | a6ffdc07847e74cc244c02ab6d0351a4a5d77281 (diff) |
mm: use is_migrate_isolate_page() to simplify the code
Use is_migrate_isolate_page() to simplify the code, no functional
changes.
Link: http://lkml.kernel.org/r/58B94FB1.8020802@huawei.com
Signed-off-by: Xishi Qiu <qiuxishi@huawei.com>
Acked-by: Michal Hocko <mhocko@suse.com>
Cc: Vlastimil Babka <vbabka@suse.cz>
Cc: Mel Gorman <mgorman@techsingularity.net>
Cc: Minchan Kim <minchan@kernel.org>
Cc: Joonsoo Kim <iamjoonsoo.kim@lge.com>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Diffstat (limited to 'mm/page_isolation.c')
-rw-r--r-- | mm/page_isolation.c | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/mm/page_isolation.c b/mm/page_isolation.c index f4e17a57926a..7927bbb54a4e 100644 --- a/mm/page_isolation.c +++ b/mm/page_isolation.c | |||
@@ -88,7 +88,7 @@ static void unset_migratetype_isolate(struct page *page, unsigned migratetype) | |||
88 | 88 | ||
89 | zone = page_zone(page); | 89 | zone = page_zone(page); |
90 | spin_lock_irqsave(&zone->lock, flags); | 90 | spin_lock_irqsave(&zone->lock, flags); |
91 | if (get_pageblock_migratetype(page) != MIGRATE_ISOLATE) | 91 | if (!is_migrate_isolate_page(page)) |
92 | goto out; | 92 | goto out; |
93 | 93 | ||
94 | /* | 94 | /* |
@@ -205,7 +205,7 @@ int undo_isolate_page_range(unsigned long start_pfn, unsigned long end_pfn, | |||
205 | pfn < end_pfn; | 205 | pfn < end_pfn; |
206 | pfn += pageblock_nr_pages) { | 206 | pfn += pageblock_nr_pages) { |
207 | page = __first_valid_page(pfn, pageblock_nr_pages); | 207 | page = __first_valid_page(pfn, pageblock_nr_pages); |
208 | if (!page || get_pageblock_migratetype(page) != MIGRATE_ISOLATE) | 208 | if (!page || !is_migrate_isolate_page(page)) |
209 | continue; | 209 | continue; |
210 | unset_migratetype_isolate(page, migratetype); | 210 | unset_migratetype_isolate(page, migratetype); |
211 | } | 211 | } |
@@ -262,7 +262,7 @@ int test_pages_isolated(unsigned long start_pfn, unsigned long end_pfn, | |||
262 | */ | 262 | */ |
263 | for (pfn = start_pfn; pfn < end_pfn; pfn += pageblock_nr_pages) { | 263 | for (pfn = start_pfn; pfn < end_pfn; pfn += pageblock_nr_pages) { |
264 | page = __first_valid_page(pfn, pageblock_nr_pages); | 264 | page = __first_valid_page(pfn, pageblock_nr_pages); |
265 | if (page && get_pageblock_migratetype(page) != MIGRATE_ISOLATE) | 265 | if (page && !is_migrate_isolate_page(page)) |
266 | break; | 266 | break; |
267 | } | 267 | } |
268 | page = __first_valid_page(start_pfn, end_pfn - start_pfn); | 268 | page = __first_valid_page(start_pfn, end_pfn - start_pfn); |