Messages in this thread | | | Subject | Re: [v6 02/15] x86/mm: setting fields in deferred pages | From | Pasha Tatashin <> | Date | Mon, 14 Aug 2017 09:32:01 -0400 |
| |
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.
| |