lkml.org 
[lkml]   [2011]   [Nov]   [11]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    Date
    From
    SubjectRe: [PATCH 1/4] mm: remove debug_pagealloc_enabled
    On Fri, Nov 11, 2011 at 01:36:31PM +0100, Stanislaw Gruszka wrote:
    > After we finish (no)bootmem, pages are passed to buddy allocator. Since
    > debug_pagealloc_enabled is not set, we do not protect pages, what is
    > not what we want with CONFIG_DEBUG_PAGEALLOC=y. That could be fixed by
    > calling enable_debug_pagealloc() before free_all_bootmem(), but actually
    > I do not see any reason why we need that global variable. Hence patch
    > remove it.
    >
    > Signed-off-by: Stanislaw Gruszka <sgruszka@redhat.com>
    > ---
    > arch/x86/mm/pageattr.c | 6 ------
    > include/linux/mm.h | 10 ----------
    > init/main.c | 5 -----
    > mm/debug-pagealloc.c | 3 ---
    > 4 files changed, 0 insertions(+), 24 deletions(-)
    >
    > diff --git a/arch/x86/mm/pageattr.c b/arch/x86/mm/pageattr.c
    > index f9e5267..5031eef 100644
    > --- a/arch/x86/mm/pageattr.c
    > +++ b/arch/x86/mm/pageattr.c
    > @@ -1334,12 +1334,6 @@ void kernel_map_pages(struct page *page, int numpages, int enable)
    > }
    >
    > /*
    > - * If page allocator is not up yet then do not call c_p_a():
    > - */
    > - if (!debug_pagealloc_enabled)
    > - return;
    > -
    > - /*

    According to commit [12d6f21e: x86: do not PSE on
    CONFIG_DEBUG_PAGEALLOC=y], the intention of debug_pagealloc_enabled
    was to force additional testing of splitting large pages due to
    cpa. Presumably this was because when bootmem was retired, all the
    pages would be mapped forcing the protection to be applied later
    while the system was running and races would be more interesting.

    This patch is trading additional CPA testing for better detecting
    of memory corruption with DEBUG_PAGEALLOC. I see no issue with this
    per-se, but I'm cc'ing Ingo for comment as it was his patch and this
    is something that should go by the x86 maintainers.

    --
    Mel Gorman
    SUSE Labs


    \
     
     \ /
      Last update: 2011-11-11 15:15    [W:0.023 / U:0.200 seconds]
    ©2003-2016 Jasper Spaans. hosted at Digital OceanAdvertise on this site