Re: [PATCH] kmemcheck: add scheduling point to kmemleak_scan

2017-11-20 Thread Yisheng Xie
Hi Catalin, On 2017/11/18 2:27, Catalin Marinas wrote: > Please fix the subject as the tool is called "kmemleak" rather than > "kmemcheck". Yeah, this really is a terrible typo. > > On Fri, Nov 17, 2017 at 03:03:56PM +0800, Yisheng Xie wrote: >> diff --git a/mm/kmemleak.c b/mm/kmemleak.c >> ind

Re: [PATCH] kmemcheck: add scheduling point to kmemleak_scan

2017-11-17 Thread Catalin Marinas
Please fix the subject as the tool is called "kmemleak" rather than "kmemcheck". On Fri, Nov 17, 2017 at 03:03:56PM +0800, Yisheng Xie wrote: > diff --git a/mm/kmemleak.c b/mm/kmemleak.c > index e4738d5..e9f2e86 100644 > --- a/mm/kmemleak.c > +++ b/mm/kmemleak.c > @@ -1523,6 +1523,8 @@ static void

[PATCH] kmemcheck: add scheduling point to kmemleak_scan

2017-11-16 Thread Yisheng Xie
kmemleak_scan will scan struct page for each node and it can be really large and resulting in a soft lockup. We have seen a soft lockup when do scan while compile kernel: [ 220.561051] watchdog: BUG: soft lockup - CPU#53 stuck for 22s! [bash:10287] [...] [ 220.753837] Call Trace: [ 220.7562