Linux PARISC architecture development
 help / color / mirror / Atom feed
From: Meelis Roos <mroos@linux.ee>
To: John David Anglin <dave.anglin@bell.net>
Cc: linux-parisc@vger.kernel.org
Subject: Re: parisc 4.13-git, BUG: sleeping function called from invalid context at mm/slab.h:416
Date: Tue, 1 Aug 2017 10:52:20 +0300 (EEST)	[thread overview]
Message-ID: <alpine.LRH.2.20.1708011050580.27473@math.ut.ee> (raw)
In-Reply-To: <F79536F1-AB2B-435C-A127-A1DFB4AC9CE0@bell.net>

> Since this code hasn't changed in a long time, it should be possible to bisect problem.

I know the patch works, but out of curiosity I bisected it and found 
that the BUG is brough out with the following commit. Seems to make 
sense.

ma1c3c5eab171590f86edd8d31389d61dd1efe3037 is the first bad commit
commit 1c3c5eab171590f86edd8d31389d61dd1efe3037
Author: Thomas Gleixner <tglx@linutronix.de>
Date:   Tue May 16 20:42:48 2017 +0200

    sched/core: Enable might_sleep() and smp_processor_id() checks early
    
    might_sleep() and smp_processor_id() checks are enabled after the boot
    process is done. That hides bugs in the SMP bringup and driver
    initialization code.
    
    Enable it right when the scheduler starts working, i.e. when init task and
    kthreadd have been created and right before the idle task enables
    preemption.
    
    Tested-by: Mark Rutland <mark.rutland@arm.com>
    Signed-off-by: Thomas Gleixner <tglx@linutronix.de>
    Signed-off-by: Peter Zijlstra (Intel) <peterz@infradead.org>
    Acked-by: Mark Rutland <mark.rutland@arm.com>
    Cc: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
    Cc: Linus Torvalds <torvalds@linux-foundation.org>
    Cc: Peter Zijlstra <peterz@infradead.org>
    Cc: Steven Rostedt <rostedt@goodmis.org>
    Link: http://lkml.kernel.org/r/20170516184736.272225698@linutronix.de
    Signed-off-by: Ingo Molnar <mingo@kernel.org>



-- 
Meelis Roos (mroos@linux.ee)

  parent reply	other threads:[~2017-08-01  7:52 UTC|newest]

Thread overview: 14+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-07-29 19:09 parisc 4.13-git, BUG: sleeping function called from invalid context at mm/slab.h:416 Meelis Roos
2017-07-29 19:53 ` John David Anglin
2017-07-29 20:27   ` James Bottomley
2017-07-30  6:47   ` Meelis Roos
2017-07-30 12:27     ` John David Anglin
2017-07-30 15:30       ` James Bottomley
2017-07-30 18:12         ` Meelis Roos
2017-07-30 18:21           ` John David Anglin
2017-07-30 18:37             ` James Bottomley
2017-07-30 18:53               ` Helge Deller
2017-07-30 19:27                 ` John David Anglin
2017-07-30 19:08               ` Meelis Roos
2017-08-01  7:52       ` Meelis Roos [this message]
2017-08-01 14:26         ` James Bottomley

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=alpine.LRH.2.20.1708011050580.27473@math.ut.ee \
    --to=mroos@linux.ee \
    --cc=dave.anglin@bell.net \
    --cc=linux-parisc@vger.kernel.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox