• Home
  • Line#
  • Scopes#
  • Navigate#
  • Raw
  • Download
1Runtime locking correctness validator
2=====================================
3
4started by Ingo Molnar <mingo@redhat.com>
5additions by Arjan van de Ven <arjan@linux.intel.com>
6
7Lock-class
8----------
9
10The basic object the validator operates upon is a 'class' of locks.
11
12A class of locks is a group of locks that are logically the same with
13respect to locking rules, even if the locks may have multiple (possibly
14tens of thousands of) instantiations. For example a lock in the inode
15struct is one class, while each inode has its own instantiation of that
16lock class.
17
18The validator tracks the 'state' of lock-classes, and it tracks
19dependencies between different lock-classes. The validator maintains a
20rolling proof that the state and the dependencies are correct.
21
22Unlike an lock instantiation, the lock-class itself never goes away: when
23a lock-class is used for the first time after bootup it gets registered,
24and all subsequent uses of that lock-class will be attached to this
25lock-class.
26
27State
28-----
29
30The validator tracks lock-class usage history into 4n + 1 separate state bits:
31
32- 'ever held in STATE context'
33- 'ever held as readlock in STATE context'
34- 'ever held with STATE enabled'
35- 'ever held as readlock with STATE enabled'
36
37Where STATE can be either one of (kernel/lockdep_states.h)
38 - hardirq
39 - softirq
40 - reclaim_fs
41
42- 'ever used'                                       [ == !unused        ]
43
44When locking rules are violated, these state bits are presented in the
45locking error messages, inside curlies. A contrived example:
46
47   modprobe/2287 is trying to acquire lock:
48    (&sio_locks[i].lock){-.-...}, at: [<c02867fd>] mutex_lock+0x21/0x24
49
50   but task is already holding lock:
51    (&sio_locks[i].lock){-.-...}, at: [<c02867fd>] mutex_lock+0x21/0x24
52
53
54The bit position indicates STATE, STATE-read, for each of the states listed
55above, and the character displayed in each indicates:
56
57   '.'  acquired while irqs disabled and not in irq context
58   '-'  acquired in irq context
59   '+'  acquired with irqs enabled
60   '?'  acquired in irq context with irqs enabled.
61
62Unused mutexes cannot be part of the cause of an error.
63
64
65Single-lock state rules:
66------------------------
67
68A softirq-unsafe lock-class is automatically hardirq-unsafe as well. The
69following states are exclusive, and only one of them is allowed to be
70set for any lock-class:
71
72 <hardirq-safe> and <hardirq-unsafe>
73 <softirq-safe> and <softirq-unsafe>
74
75The validator detects and reports lock usage that violate these
76single-lock state rules.
77
78Multi-lock dependency rules:
79----------------------------
80
81The same lock-class must not be acquired twice, because this could lead
82to lock recursion deadlocks.
83
84Furthermore, two locks may not be taken in different order:
85
86 <L1> -> <L2>
87 <L2> -> <L1>
88
89because this could lead to lock inversion deadlocks. (The validator
90finds such dependencies in arbitrary complexity, i.e. there can be any
91other locking sequence between the acquire-lock operations, the
92validator will still track all dependencies between locks.)
93
94Furthermore, the following usage based lock dependencies are not allowed
95between any two lock-classes:
96
97   <hardirq-safe>   ->  <hardirq-unsafe>
98   <softirq-safe>   ->  <softirq-unsafe>
99
100The first rule comes from the fact the a hardirq-safe lock could be
101taken by a hardirq context, interrupting a hardirq-unsafe lock - and
102thus could result in a lock inversion deadlock. Likewise, a softirq-safe
103lock could be taken by an softirq context, interrupting a softirq-unsafe
104lock.
105
106The above rules are enforced for any locking sequence that occurs in the
107kernel: when acquiring a new lock, the validator checks whether there is
108any rule violation between the new lock and any of the held locks.
109
110When a lock-class changes its state, the following aspects of the above
111dependency rules are enforced:
112
113- if a new hardirq-safe lock is discovered, we check whether it
114  took any hardirq-unsafe lock in the past.
115
116- if a new softirq-safe lock is discovered, we check whether it took
117  any softirq-unsafe lock in the past.
118
119- if a new hardirq-unsafe lock is discovered, we check whether any
120  hardirq-safe lock took it in the past.
121
122- if a new softirq-unsafe lock is discovered, we check whether any
123  softirq-safe lock took it in the past.
124
125(Again, we do these checks too on the basis that an interrupt context
126could interrupt _any_ of the irq-unsafe or hardirq-unsafe locks, which
127could lead to a lock inversion deadlock - even if that lock scenario did
128not trigger in practice yet.)
129
130Exception: Nested data dependencies leading to nested locking
131-------------------------------------------------------------
132
133There are a few cases where the Linux kernel acquires more than one
134instance of the same lock-class. Such cases typically happen when there
135is some sort of hierarchy within objects of the same type. In these
136cases there is an inherent "natural" ordering between the two objects
137(defined by the properties of the hierarchy), and the kernel grabs the
138locks in this fixed order on each of the objects.
139
140An example of such an object hierarchy that results in "nested locking"
141is that of a "whole disk" block-dev object and a "partition" block-dev
142object; the partition is "part of" the whole device and as long as one
143always takes the whole disk lock as a higher lock than the partition
144lock, the lock ordering is fully correct. The validator does not
145automatically detect this natural ordering, as the locking rule behind
146the ordering is not static.
147
148In order to teach the validator about this correct usage model, new
149versions of the various locking primitives were added that allow you to
150specify a "nesting level". An example call, for the block device mutex,
151looks like this:
152
153enum bdev_bd_mutex_lock_class
154{
155       BD_MUTEX_NORMAL,
156       BD_MUTEX_WHOLE,
157       BD_MUTEX_PARTITION
158};
159
160 mutex_lock_nested(&bdev->bd_contains->bd_mutex, BD_MUTEX_PARTITION);
161
162In this case the locking is done on a bdev object that is known to be a
163partition.
164
165The validator treats a lock that is taken in such a nested fashion as a
166separate (sub)class for the purposes of validation.
167
168Note: When changing code to use the _nested() primitives, be careful and
169check really thoroughly that the hierarchy is correctly mapped; otherwise
170you can get false positives or false negatives.
171
172Proof of 100% correctness:
173--------------------------
174
175The validator achieves perfect, mathematical 'closure' (proof of locking
176correctness) in the sense that for every simple, standalone single-task
177locking sequence that occurred at least once during the lifetime of the
178kernel, the validator proves it with a 100% certainty that no
179combination and timing of these locking sequences can cause any class of
180lock related deadlock. [*]
181
182I.e. complex multi-CPU and multi-task locking scenarios do not have to
183occur in practice to prove a deadlock: only the simple 'component'
184locking chains have to occur at least once (anytime, in any
185task/context) for the validator to be able to prove correctness. (For
186example, complex deadlocks that would normally need more than 3 CPUs and
187a very unlikely constellation of tasks, irq-contexts and timings to
188occur, can be detected on a plain, lightly loaded single-CPU system as
189well!)
190
191This radically decreases the complexity of locking related QA of the
192kernel: what has to be done during QA is to trigger as many "simple"
193single-task locking dependencies in the kernel as possible, at least
194once, to prove locking correctness - instead of having to trigger every
195possible combination of locking interaction between CPUs, combined with
196every possible hardirq and softirq nesting scenario (which is impossible
197to do in practice).
198
199[*] assuming that the validator itself is 100% correct, and no other
200    part of the system corrupts the state of the validator in any way.
201    We also assume that all NMI/SMM paths [which could interrupt
202    even hardirq-disabled codepaths] are correct and do not interfere
203    with the validator. We also assume that the 64-bit 'chain hash'
204    value is unique for every lock-chain in the system. Also, lock
205    recursion must not be higher than 20.
206
207Performance:
208------------
209
210The above rules require _massive_ amounts of runtime checking. If we did
211that for every lock taken and for every irqs-enable event, it would
212render the system practically unusably slow. The complexity of checking
213is O(N^2), so even with just a few hundred lock-classes we'd have to do
214tens of thousands of checks for every event.
215
216This problem is solved by checking any given 'locking scenario' (unique
217sequence of locks taken after each other) only once. A simple stack of
218held locks is maintained, and a lightweight 64-bit hash value is
219calculated, which hash is unique for every lock chain. The hash value,
220when the chain is validated for the first time, is then put into a hash
221table, which hash-table can be checked in a lockfree manner. If the
222locking chain occurs again later on, the hash table tells us that we
223dont have to validate the chain again.
224
225Troubleshooting:
226----------------
227
228The validator tracks a maximum of MAX_LOCKDEP_KEYS number of lock classes.
229Exceeding this number will trigger the following lockdep warning:
230
231	(DEBUG_LOCKS_WARN_ON(id >= MAX_LOCKDEP_KEYS))
232
233By default, MAX_LOCKDEP_KEYS is currently set to 8191, and typical
234desktop systems have less than 1,000 lock classes, so this warning
235normally results from lock-class leakage or failure to properly
236initialize locks.  These two problems are illustrated below:
237
2381.	Repeated module loading and unloading while running the validator
239	will result in lock-class leakage.  The issue here is that each
240	load of the module will create a new set of lock classes for
241	that module's locks, but module unloading does not remove old
242	classes (see below discussion of reuse of lock classes for why).
243	Therefore, if that module is loaded and unloaded repeatedly,
244	the number of lock classes will eventually reach the maximum.
245
2462.	Using structures such as arrays that have large numbers of
247	locks that are not explicitly initialized.  For example,
248	a hash table with 8192 buckets where each bucket has its own
249	spinlock_t will consume 8192 lock classes -unless- each spinlock
250	is explicitly initialized at runtime, for example, using the
251	run-time spin_lock_init() as opposed to compile-time initializers
252	such as __SPIN_LOCK_UNLOCKED().  Failure to properly initialize
253	the per-bucket spinlocks would guarantee lock-class overflow.
254	In contrast, a loop that called spin_lock_init() on each lock
255	would place all 8192 locks into a single lock class.
256
257	The moral of this story is that you should always explicitly
258	initialize your locks.
259
260One might argue that the validator should be modified to allow
261lock classes to be reused.  However, if you are tempted to make this
262argument, first review the code and think through the changes that would
263be required, keeping in mind that the lock classes to be removed are
264likely to be linked into the lock-dependency graph.  This turns out to
265be harder to do than to say.
266
267Of course, if you do run out of lock classes, the next thing to do is
268to find the offending lock classes.  First, the following command gives
269you the number of lock classes currently in use along with the maximum:
270
271	grep "lock-classes" /proc/lockdep_stats
272
273This command produces the following output on a modest system:
274
275	 lock-classes:                          748 [max: 8191]
276
277If the number allocated (748 above) increases continually over time,
278then there is likely a leak.  The following command can be used to
279identify the leaking lock classes:
280
281	grep "BD" /proc/lockdep
282
283Run the command and save the output, then compare against the output from
284a later run of this command to identify the leakers.  This same output
285can also help you find situations where runtime lock initialization has
286been omitted.
287