lockdep: fix overflow in the hlock shrinkage code
authorPeter Zijlstra <a.p.zijlstra@chello.nl>
Mon, 11 Aug 2008 10:34:42 +0000 (12:34 +0200)
committerIngo Molnar <mingo@elte.hu>
Mon, 11 Aug 2008 10:34:42 +0000 (12:34 +0200)
There is a overflow by 1 case in the new shrunken hlock code.

Signed-off-by: Peter Zijlstra <a.p.zijlstra@chello.nl>
Signed-off-by: Ingo Molnar <mingo@elte.hu>
include/linux/lockdep.h

index 67f42b300c65ce81ec09d03b70053c6691f4b6fc..c88aa3d8e87fef2c048b9ea39a7f0c1a7bad989b 100644 (file)
@@ -191,7 +191,12 @@ struct lock_chain {
 };
 
 #define MAX_LOCKDEP_KEYS_BITS          11
-#define MAX_LOCKDEP_KEYS               (1UL << MAX_LOCKDEP_KEYS_BITS)
+/*
+ * Subtract one because we offset hlock->class_idx by 1 in order
+ * to make 0 mean no class. This avoids overflowing the class_idx
+ * bitfield and hitting the BUG in hlock_class().
+ */
+#define MAX_LOCKDEP_KEYS               ((1UL << MAX_LOCKDEP_KEYS_BITS) - 1)
 
 struct held_lock {
        /*