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

Pasha Tatashin
2017-08-14T13:33:18Z

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.