lkml.org 
[lkml]   [2020]   [Nov]   [23]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
Patch in this message
/
SubjectRe: [PATCH] mm: memory_hotplug: put migration failure information under DEBUG_VM
From
Date

Thanks Michal!
On 11/23/2020 7:43 PM, Michal Hocko wrote:
> On Mon 23-11-20 19:33:16, Charan Teja Reddy wrote:
>> When the pages are failed to get isolate or migrate, the page owner
>> information along with page info is dumped. If there are continuous
>> failures in migration(say page is pinned) or isolation, the log buffer
>> is simply getting flooded with the page owner information. As most of
>> the times page info is sufficient to know the causes for failures of
>> migration or isolation, place the page owner information under DEBUG_VM.
>
> I do not see why this path is any different from others that call
> dump_page. Page owner can add a very valuable information to debug
> the underlying reasons for failures here. It is an opt-in debugging
> feature which needs to be enabled explicitly. So I would argue users
> are ready to accept a lot of data in the kernel log.

Just thinking how frequently failures can happen in those paths. In the
memory hotplug path, we can flood the page owner logs just by making one
page pinned. Say If it is anonymous page, the page owner information
shows is something like below, which is not really telling anything
other than how the pinned page is allocated.

page last allocated via order 0, migratetype Movable, gfp_mask
0x100dca(GFP_HIGHUSER_MOVABLE|__GFP_ZERO)
prep_new_page+0x7c/0x1a4
get_page_from_freelist+0x1ac/0x1c4
__alloc_pages_nodemask+0x12c/0x378
do_anonymous_page+0xac/0x3b4
handle_pte_fault+0x2a4/0x3bc
__handle_speculative_fault+0x208/0x3c0
do_page_fault+0x280/0x508
do_translation_fault+0x3c/0x54
do_mem_abort+0x64/0xf4
el0_da+0x1c/0x20
page last free stack trace:
free_pcp_prepare+0x320/0x454
free_unref_page_list+0x9c/0x2a4
release_pages+0x370/0x3c8
free_pages_and_swap_cache+0xdc/0x10c
tlb_flush_mmu+0x110/0x134
tlb_finish_mmu+0x48/0xc0
unmap_region+0x104/0x138
__do_munmap+0x2ec/0x3b4
__arm64_sys_munmap+0x80/0xd8

I see at some places in the kernel where they put the dump_page under
DEBUG_VM, but in the end I agree that it is up to the users need. Then
there are some users who don't care for these page owner logs.

And an issue on Embedded systems with these continuous logs being
printed to the console is the watchdog timeouts, because console logging
happens by disabling the interrupts.

>
>> Signed-off-by: Charan Teja Reddy <charante@codeaurora.org>
>> ---
>> mm/memory_hotplug.c | 10 ++++++++--
>> 1 file changed, 8 insertions(+), 2 deletions(-)
>>
>> diff --git a/mm/memory_hotplug.c b/mm/memory_hotplug.c
>> index 63b2e46..f48f30d 100644
>> --- a/mm/memory_hotplug.c
>> +++ b/mm/memory_hotplug.c
>> @@ -1326,7 +1326,10 @@ do_migrate_range(unsigned long start_pfn, unsigned long end_pfn)
>>
>> } else {
>> pr_warn("failed to isolate pfn %lx\n", pfn);
>> - dump_page(page, "isolation failed");
>> + __dump_page(page, "isolation failed");
>> +#if defined(CONFIG_DEBUG_VM)
>> + dump_page_owner(page);
>> +#endif
>> }
>> put_page(page);
>> }
>> @@ -1357,7 +1360,10 @@ do_migrate_range(unsigned long start_pfn, unsigned long end_pfn)
>> list_for_each_entry(page, &source, lru) {
>> pr_warn("migrating pfn %lx failed ret:%d ",
>> page_to_pfn(page), ret);
>> - dump_page(page, "migration failure");
>> + __dump_page(page, "migration failure");
>> +#if defined(CONFIG_DEBUG_VM)
>> + dump_page_owner(page);
>> +#endif
>> }
>> putback_movable_pages(&source);
>> }
>> --
>> QUALCOMM INDIA, on behalf of Qualcomm Innovation Center, Inc. is a
>> member of the Code Aurora Forum, hosted by The Linux Foundation
>

--
The Qualcomm Innovation Center, Inc. is a member of the Code Aurora
Forum, a Linux Foundation Collaborative Project

\
 
 \ /
  Last update: 2020-11-23 16:13    [W:3.189 / U:1.196 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site