Re: [PATCH 1/1] lockdep: Enable PROVE_RAW_LOCK_NESTING with PROVE_LOCKING.

[Boqun Feng]

On Wed, Oct 09, 2024 at 12:50:39PM -0400, Waiman Long wrote:
> 
> On 10/9/24 11:45 AM, Sebastian Andrzej Siewior wrote:
> > With the printk issues solved, the last known splat created by
> > PROVE_RAW_LOCK_NESTING is gone.
> > 
> > Enable PROVE_RAW_LOCK_NESTING by default as part of PROVE_LOCKING. Keep
> > the defines around in case something serious pops up and it needs to be
> > disabled.
> > 
> > Signed-off-by: Sebastian Andrzej Siewior <bigeasy@xxxxxxxxxxxxx>
> > ---
> >   lib/Kconfig.debug | 12 ++----------
> >   1 file changed, 2 insertions(+), 10 deletions(-)
> > 
> > diff --git a/lib/Kconfig.debug b/lib/Kconfig.debug
> > index 7315f643817ae..5b67816f4a62f 100644
> > --- a/lib/Kconfig.debug
> > +++ b/lib/Kconfig.debug
> > @@ -1409,22 +1409,14 @@ config PROVE_LOCKING
> >   	 For more details, see Documentation/locking/lockdep-design.rst.
> >   config PROVE_RAW_LOCK_NESTING
> > -	bool "Enable raw_spinlock - spinlock nesting checks"
> > +	bool
> >   	depends on PROVE_LOCKING
> > -	default n
> > +	default y
> >   	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.
> > -
> >   config LOCK_STAT
> >   	bool "Lock usage statistics"
> >   	depends on DEBUG_KERNEL && LOCK_DEBUGGING_SUPPORT
> Acked-by: Waiman Long <longman@xxxxxxxxxx>
> 

Thanks!

Queued in my lockdep branch:

	https://git.kernel.org/pub/scm/linux/kernel/git/boqun/linux.git/log/?h=lockdep-for-tip

will send a PR to tip between -rc3 and -rc4.

Regards,
Boqun