Skip to content

Commit 6787c1d

Browse files
leberustorvalds
authored andcommitted
mm/page_owner.c: clean up init_pages_in_zone()
Remove two redundant assignments in init_pages_in_zone(). [[email protected]: v3] Link: http://lkml.kernel.org/r/[email protected] [[email protected]: coding style tweaks] Link: http://lkml.kernel.org/r/[email protected] Signed-off-by: Oscar Salvador <[email protected]> Acked-by: Michal Hocko <[email protected]> Acked-by: Vlastimil Babka <[email protected]> Cc: Joonsoo Kim <[email protected]> Signed-off-by: Andrew Morton <[email protected]> Signed-off-by: Linus Torvalds <[email protected]>
1 parent 3c2c648 commit 6787c1d

File tree

1 file changed

+7
-9
lines changed

1 file changed

+7
-9
lines changed

mm/page_owner.c

Lines changed: 7 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -528,21 +528,18 @@ read_page_owner(struct file *file, char __user *buf, size_t count, loff_t *ppos)
528528

529529
static void init_pages_in_zone(pg_data_t *pgdat, struct zone *zone)
530530
{
531-
struct page *page;
532-
struct page_ext *page_ext;
533-
unsigned long pfn = zone->zone_start_pfn, block_end_pfn;
534-
unsigned long end_pfn = pfn + zone->spanned_pages;
531+
unsigned long pfn = zone->zone_start_pfn;
532+
unsigned long end_pfn = zone_end_pfn(zone);
535533
unsigned long count = 0;
536534

537-
/* Scan block by block. First and last block may be incomplete */
538-
pfn = zone->zone_start_pfn;
539-
540535
/*
541536
* Walk the zone in pageblock_nr_pages steps. If a page block spans
542537
* a zone boundary, it will be double counted between zones. This does
543538
* not matter as the mixed block count will still be correct
544539
*/
545540
for (; pfn < end_pfn; ) {
541+
unsigned long block_end_pfn;
542+
546543
if (!pfn_valid(pfn)) {
547544
pfn = ALIGN(pfn + 1, MAX_ORDER_NR_PAGES);
548545
continue;
@@ -551,9 +548,10 @@ static void init_pages_in_zone(pg_data_t *pgdat, struct zone *zone)
551548
block_end_pfn = ALIGN(pfn + 1, pageblock_nr_pages);
552549
block_end_pfn = min(block_end_pfn, end_pfn);
553550

554-
page = pfn_to_page(pfn);
555-
556551
for (; pfn < block_end_pfn; pfn++) {
552+
struct page *page;
553+
struct page_ext *page_ext;
554+
557555
if (!pfn_valid_within(pfn))
558556
continue;
559557

0 commit comments

Comments
 (0)