Re: [PATCH v3 2/4] mm: simplify parater of function memmap_init_zone()

2021-01-07 Thread Baoquan He
On 01/05/21 at 05:53pm, David Hildenbrand wrote: > [...] > > > -void __meminit > > -memmap_init_zone(unsigned long size, int nid, unsigned long zone, > > -unsigned long start_pfn) > > +void __meminit memmap_init_zone(struct zone *zone) > > { > > + unsigned long size =

Re: [PATCH v3 2/4] mm: simplify parater of function memmap_init_zone()

2021-01-05 Thread David Hildenbrand
[...] > -void __meminit > -memmap_init_zone(unsigned long size, int nid, unsigned long zone, > - unsigned long start_pfn) > +void __meminit memmap_init_zone(struct zone *zone) > { > + unsigned long size = zone->spanned_pages; > + int nid = zone_to_nid(zone), zone_id =

[PATCH v3 2/4] mm: simplify parater of function memmap_init_zone()

2021-01-04 Thread Baoquan He
As David suggested, simply passing 'struct zone *zone' is enough. We can get all needed information from 'struct zone*' easily. Suggested-by: David Hildenbrand Signed-off-by: Baoquan He Reviewed-by: Mike Rapoport --- arch/ia64/include/asm/pgtable.h | 3 +-- arch/ia64/mm/init.c |