lkml.org 
[lkml]   [2019]   [Aug]   [6]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
From
SubjectRE: [PATCH V2] fork: Improve error message for corrupted page tables
Date
> >> Without patch:
> >> --------------
> >> [ 204.836425] mm/pgtable-generic.c:29: bad p4d
> >> 0000000089eb4e92(800000025f941467)
> >> [ 204.836544] BUG: Bad rss-counter state mm:00000000f75895ea idx:0
> >> val:2 [ 204.836615] BUG: Bad rss-counter state mm:00000000f75895ea
> >> idx:1 val:5 [ 204.836685] BUG: non-zero pgtables_bytes on freeing
> >> mm: 20480
> >>
> >> With patch:
> >> -----------
> >> [ 69.815453] mm/pgtable-generic.c:29: bad p4d
> 0000000084653642(800000025ca37467)
> >> [ 69.815872] BUG: Bad rss-counter state mm:00000000014a6c03
> type:MM_FILEPAGES val:2
> >> [ 69.815962] BUG: Bad rss-counter state mm:00000000014a6c03
> type:MM_ANONPAGES val:5
> >> [ 69.816050] BUG: non-zero pgtables_bytes on freeing mm: 20480
> >>
> >> Also, change print function (from printk(KERN_ALERT, ..) to
> >> pr_alert()) so that it matches the other print statement.
> >>
> >> Cc: Ingo Molnar <mingo@kernel.org>
> >> Cc: Vlastimil Babka <vbabka@suse.cz>
> >> Cc: Peter Zijlstra <peterz@infradead.org>
> >> Cc: Andrew Morton <akpm@linux-foundation.org>
> >> Cc: Anshuman Khandual <anshuman.khandual@arm.com>
> >> Acked-by: Dave Hansen <dave.hansen@intel.com>
> >> Suggested-by: Dave Hansen <dave.hansen@intel.com>
> >> Signed-off-by: Sai Praneeth Prakhya <sai.praneeth.prakhya@intel.com>
> >
> > Acked-by: Vlastimil Babka <vbabka@suse.cz>
> >
> > I would also add something like this to reduce risk of breaking it in
> > the
> > future:
> >
> > ----8<----
> > diff --git a/include/linux/mm_types_task.h
> > b/include/linux/mm_types_task.h index d7016dcb245e..a6f83cbe4603
> > 100644
> > --- a/include/linux/mm_types_task.h
> > +++ b/include/linux/mm_types_task.h
> > @@ -36,6 +36,9 @@ struct vmacache {
> > struct vm_area_struct *vmas[VMACACHE_SIZE]; };
> >
> > +/*
> > + * When touching this, update also resident_page_types in
> > +kernel/fork.c */
> > enum {
> > MM_FILEPAGES, /* Resident file mapping pages */
> > MM_ANONPAGES, /* Resident anonymous pages */
> >
>
> Agreed and with that
>
> Reviewed-by: Anshuman Khandual <anshuman.khandual@arm.com>

Thanks for the review and helping me in improving the patch :)

Regards,
Sai
\
 
 \ /
  Last update: 2019-08-06 18:13    [W:0.071 / U:0.072 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site