2021-09-02 19:00:19

by Matthew Wilcox

[permalink] [raw]
Subject: Re: slub: BUG: Invalid wait context

On Thu, Sep 02, 2021 at 10:22:23AM -0600, Jens Axboe wrote:
> Hi,
>
> Booting current -git yields the below splat. I'm assuming this is
> related to the new RT stuff, where spin_lock() can sleep. This obviously
> won't fly off IPI.

You want to turn off PROVE_RAW_LOCK_NESTING for the moment.


2021-09-02 19:02:15

by Jens Axboe

[permalink] [raw]
Subject: Re: slub: BUG: Invalid wait context

On 9/2/21 12:56 PM, Matthew Wilcox wrote:
> On Thu, Sep 02, 2021 at 10:22:23AM -0600, Jens Axboe wrote:
>> Hi,
>>
>> Booting current -git yields the below splat. I'm assuming this is
>> related to the new RT stuff, where spin_lock() can sleep. This obviously
>> won't fly off IPI.
>
> You want to turn off PROVE_RAW_LOCK_NESTING for the moment.

Because?

--
Jens Axboe

2021-09-02 19:07:42

by Matthew Wilcox

[permalink] [raw]
Subject: Re: slub: BUG: Invalid wait context

On Thu, Sep 02, 2021 at 12:59:33PM -0600, Jens Axboe wrote:
> On 9/2/21 12:56 PM, Matthew Wilcox wrote:
> > On Thu, Sep 02, 2021 at 10:22:23AM -0600, Jens Axboe wrote:
> >> Hi,
> >>
> >> Booting current -git yields the below splat. I'm assuming this is
> >> related to the new RT stuff, where spin_lock() can sleep. This obviously
> >> won't fly off IPI.
> >
> > You want to turn off PROVE_RAW_LOCK_NESTING for the moment.
>
> Because?

help
Enable the raw_spinlock vs. spinlock nesting checks which ensure
that the lock nesting rules for PREEMPT_RT enabled kernels are
not violated.

NOTE: There are known nesting problems. So if you enable this
option expect lockdep splats until these problems have been fully
addressed which is work in progress. This config switch allows to
identify and analyze these problems. It will be removed and the
check permanently enabled once the main issues have been fixed.

If unsure, select N.

2021-09-02 19:08:10

by Jens Axboe

[permalink] [raw]
Subject: Re: slub: BUG: Invalid wait context

On 9/2/21 1:02 PM, Matthew Wilcox wrote:
> On Thu, Sep 02, 2021 at 12:59:33PM -0600, Jens Axboe wrote:
>> On 9/2/21 12:56 PM, Matthew Wilcox wrote:
>>> On Thu, Sep 02, 2021 at 10:22:23AM -0600, Jens Axboe wrote:
>>>> Hi,
>>>>
>>>> Booting current -git yields the below splat. I'm assuming this is
>>>> related to the new RT stuff, where spin_lock() can sleep. This obviously
>>>> won't fly off IPI.
>>>
>>> You want to turn off PROVE_RAW_LOCK_NESTING for the moment.
>>
>> Because?
>
> help
> Enable the raw_spinlock vs. spinlock nesting checks which ensure
> that the lock nesting rules for PREEMPT_RT enabled kernels are
> not violated.
>
> NOTE: There are known nesting problems. So if you enable this
> option expect lockdep splats until these problems have been fully
> addressed which is work in progress. This config switch allows to
> identify and analyze these problems. It will be removed and the
> check permanently enabled once the main issues have been fixed.
>
> If unsure, select N.

This isn't a new text, it's been like that for quite a while. The warning
is new on -git with that option.

--
Jens Axboe

2021-09-02 23:43:47

by Thomas Gleixner

[permalink] [raw]
Subject: Re: slub: BUG: Invalid wait context

On Thu, Sep 02 2021 at 13:05, Jens Axboe wrote:
> On 9/2/21 1:02 PM, Matthew Wilcox wrote:
>> NOTE: There are known nesting problems. So if you enable this
>> option expect lockdep splats until these problems have been fully
>> addressed which is work in progress. This config switch allows to
>> identify and analyze these problems. It will be removed and the
>> check permanently enabled once the main issues have been fixed.
>>
>> If unsure, select N.
>
> This isn't a new text, it's been like that for quite a while. The warning
> is new on -git with that option.

I'm looking into it...