diff options
author | Oscar Salvador <osalvador@techadventures.net> | 2018-01-31 19:20:11 -0500 |
---|---|---|
committer | Linus Torvalds <torvalds@linux-foundation.org> | 2018-01-31 20:18:39 -0500 |
commit | 6787c1dab1724ca0d92110d83485c8c72dbf83f4 (patch) | |
tree | 1c4fa2b4a0584654c070634cabf80c7b15a1ced0 /mm/page_owner.c | |
parent | 3c2c648842843326f8c6ace425810eb47864c6b4 (diff) |
mm/page_owner.c: clean up init_pages_in_zone()
Remove two redundant assignments in init_pages_in_zone().
[osalvador@techadventures.net: v3]
Link: http://lkml.kernel.org/r/20180117124513.GA876@techadventures.net
[akpm@linux-foundation.org: coding style tweaks]
Link: http://lkml.kernel.org/r/20180110084355.GA22822@techadventures.net
Signed-off-by: Oscar Salvador <osalvador@techadventures.net>
Acked-by: Michal Hocko <mhocko@suse.com>
Acked-by: Vlastimil Babka <vbabka@suse.cz>
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_owner.c')
-rw-r--r-- | mm/page_owner.c | 16 |
1 files changed, 7 insertions, 9 deletions
diff --git a/mm/page_owner.c b/mm/page_owner.c index 06a0055f45a6..9886c6073828 100644 --- a/mm/page_owner.c +++ b/mm/page_owner.c | |||
@@ -528,21 +528,18 @@ read_page_owner(struct file *file, char __user *buf, size_t count, loff_t *ppos) | |||
528 | 528 | ||
529 | static void init_pages_in_zone(pg_data_t *pgdat, struct zone *zone) | 529 | static void init_pages_in_zone(pg_data_t *pgdat, struct zone *zone) |
530 | { | 530 | { |
531 | struct page *page; | 531 | unsigned long pfn = zone->zone_start_pfn; |
532 | struct page_ext *page_ext; | 532 | unsigned long end_pfn = zone_end_pfn(zone); |
533 | unsigned long pfn = zone->zone_start_pfn, block_end_pfn; | ||
534 | unsigned long end_pfn = pfn + zone->spanned_pages; | ||
535 | unsigned long count = 0; | 533 | unsigned long count = 0; |
536 | 534 | ||
537 | /* Scan block by block. First and last block may be incomplete */ | ||
538 | pfn = zone->zone_start_pfn; | ||
539 | |||
540 | /* | 535 | /* |
541 | * Walk the zone in pageblock_nr_pages steps. If a page block spans | 536 | * Walk the zone in pageblock_nr_pages steps. If a page block spans |
542 | * a zone boundary, it will be double counted between zones. This does | 537 | * a zone boundary, it will be double counted between zones. This does |
543 | * not matter as the mixed block count will still be correct | 538 | * not matter as the mixed block count will still be correct |
544 | */ | 539 | */ |
545 | for (; pfn < end_pfn; ) { | 540 | for (; pfn < end_pfn; ) { |
541 | unsigned long block_end_pfn; | ||
542 | |||
546 | if (!pfn_valid(pfn)) { | 543 | if (!pfn_valid(pfn)) { |
547 | pfn = ALIGN(pfn + 1, MAX_ORDER_NR_PAGES); | 544 | pfn = ALIGN(pfn + 1, MAX_ORDER_NR_PAGES); |
548 | continue; | 545 | continue; |
@@ -551,9 +548,10 @@ static void init_pages_in_zone(pg_data_t *pgdat, struct zone *zone) | |||
551 | block_end_pfn = ALIGN(pfn + 1, pageblock_nr_pages); | 548 | block_end_pfn = ALIGN(pfn + 1, pageblock_nr_pages); |
552 | block_end_pfn = min(block_end_pfn, end_pfn); | 549 | block_end_pfn = min(block_end_pfn, end_pfn); |
553 | 550 | ||
554 | page = pfn_to_page(pfn); | ||
555 | |||
556 | for (; pfn < block_end_pfn; pfn++) { | 551 | for (; pfn < block_end_pfn; pfn++) { |
552 | struct page *page; | ||
553 | struct page_ext *page_ext; | ||
554 | |||
557 | if (!pfn_valid_within(pfn)) | 555 | if (!pfn_valid_within(pfn)) |
558 | continue; | 556 | continue; |
559 | 557 | ||