lkml.org 
[lkml]   [2017]   [Aug]   [11]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
SubjectRe: [v6 15/15] mm: debug for raw alloctor
From
Date
>> When CONFIG_DEBUG_VM is enabled, this patch sets all the memory that is
>> returned by memblock_virt_alloc_try_nid_raw() to ones to ensure that no
>> places excpect zeroed memory.
>
> Please fold this into the patch which introduces
> memblock_virt_alloc_try_nid_raw.

OK

I am not sure CONFIG_DEBUG_VM is the
> best config because that tends to be enabled quite often. Maybe
> CONFIG_MEMBLOCK_DEBUG? Or even make it kernel command line parameter?
>

Initially, I did not want to make it CONFIG_MEMBLOCK_DEBUG because we
really benefit from this debugging code when VM debug is enabled, and
especially struct page debugging asserts which also depend on
CONFIG_DEBUG_VM.

However, now thinking about it, I will change it to
CONFIG_MEMBLOCK_DEBUG, and let users decide what other debugging configs
need to be enabled, as this is also OK.

Thank you,
Pasha

\
 
 \ /
  Last update: 2017-08-11 18:20    [W:0.043 / U:1.640 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site