diff mbox series

[core-for-CI] Revert "lockdep: Enable PROVE_RAW_LOCK_NESTING with PROVE_LOCKING."

Message ID 20241211121703.2890150-1-luciano.coelho@intel.com (mailing list archive)
State New
Headers show
Series [core-for-CI] Revert "lockdep: Enable PROVE_RAW_LOCK_NESTING with PROVE_LOCKING." | expand

Commit Message

Luca Coelho Dec. 11, 2024, 12:16 p.m. UTC 
This reverts commit 560af5dc839eef08a273908f390cfefefb82aa04.

Signed-off-by: Luca Coelho <luciano.coelho@intel.com>
---
 lib/Kconfig.debug | 12 ++++++++++--
 1 file changed, 10 insertions(+), 2 deletions(-)

Comments

Lucas De Marchi Dec. 11, 2024, 5:03 p.m. UTC | #1 
On Wed, Dec 11, 2024 at 02:16:35PM +0200, Luca Coelho wrote:
>This reverts commit 560af5dc839eef08a273908f390cfefefb82aa04.
>
>Signed-off-by: Luca Coelho <luciano.coelho@intel.com>

I added some more context to the commit message and
pushed to topic/core-for-CI with Rodrigo's ack:

     81827bbe1d8b ("Revert "lockdep: Enable PROVE_RAW_LOCK_NESTING with PROVE_LOCKING."")

which should be available in drm-tip starting with

     drm-tip: 2024y-12m-11d-16h-58m-54s UTC integration manifest

thanks
Lucas De Marchi
diff mbox series

Patch

diff --git a/lib/Kconfig.debug b/lib/Kconfig.debug
index f3d723705879..de4ffe09323b 100644
--- a/lib/Kconfig.debug
+++ b/lib/Kconfig.debug
@@ -1397,14 +1397,22 @@  config PROVE_LOCKING
 	 For more details, see Documentation/locking/lockdep-design.rst.
 
 config PROVE_RAW_LOCK_NESTING
-	bool
+	bool "Enable raw_spinlock - spinlock nesting checks"
 	depends on PROVE_LOCKING
-	default y
+	default n
 	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