Messages in this thread | | | Subject | Re: [PATCH] mm/page_owner: Use helper function zone_end_pfn() to get end_pfn | From | David Hildenbrand <> | Date | Tue, 26 Jan 2021 12:23:32 +0100 |
| |
On 23.01.21 08:05, Miaohe Lin wrote: > Commit 108bcc96ef70 ("mm: add & use zone_end_pfn() and zone_spans_pfn()") > introduced the helper zone_end_pfn() to calculate the zone end pfn. But > pagetypeinfo_showmixedcount_print forgot to use it. And the initialization > of local variable pfn is duplicated, remove one. > > Signed-off-by: Miaohe Lin <linmiaohe@huawei.com> > --- > mm/page_owner.c | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/mm/page_owner.c b/mm/page_owner.c > index af464bb7fbe7..d15c7c4994f5 100644 > --- a/mm/page_owner.c > +++ b/mm/page_owner.c > @@ -263,8 +263,8 @@ void pagetypeinfo_showmixedcount_print(struct seq_file *m, > struct page *page; > struct page_ext *page_ext; > struct page_owner *page_owner; > - unsigned long pfn = zone->zone_start_pfn, block_end_pfn; > - unsigned long end_pfn = pfn + zone->spanned_pages; > + unsigned long pfn, block_end_pfn; > + unsigned long end_pfn = zone_end_pfn(zone); > unsigned long count[MIGRATE_TYPES] = { 0, }; > int pageblock_mt, page_mt; > int i; >
Looks a little weird that we initialize "end_pfn" directly and "pfn" not. I'd make this consistent.
Anyhow
Reviewed-by: David Hildenbrand <david@redhat.com>
-- Thanks,
David / dhildenb
| |