Re: [v6 02/15] x86/mm: setting fields in deferred pages

[Pasha Tatashin]

On 08/14/2017 07:43 AM, Michal Hocko wrote:
> > register_page_bootmem_info
> >   register_page_bootmem_info_node
> >    get_page_bootmem
> >     .. setting fields here ..
> >     such as: page->freelist = (void *)type;
> >
> > free_all_bootmem()
> >   free_low_memory_core_early()
> >    for_each_reserved_mem_region()
> >     reserve_bootmem_region()
> >      init_reserved_page() <- Only if this is deferred reserved page
> >       __init_single_pfn()
> >        __init_single_page()
> >            memset(0) <-- Loose the set fields here!
> OK, I have missed that part. Please make it explicit in the changelog.
> It is quite easy to get lost in the deep call chains.

Ok, will update comment.