diff options
| author | Benjamin Herrenschmidt <benh@kernel.crashing.org> | 2017-12-20 13:16:23 +1100 |
|---|---|---|
| committer | Stewart Smith <stewart@linux.vnet.ibm.com> | 2017-12-20 22:15:36 -0600 |
| commit | 76d9bcdca58936d761458f8f05960239c4dd8dec (patch) | |
| tree | c8377b11be33e62d312810645b8044d3a6f427aa /hdata | |
| parent | ca612b802adac0c72cd0f10c51a51275e5914101 (diff) | |
| download | skiboot-76d9bcdca58936d761458f8f05960239c4dd8dec.zip skiboot-76d9bcdca58936d761458f8f05960239c4dd8dec.tar.gz skiboot-76d9bcdca58936d761458f8f05960239c4dd8dec.tar.bz2 | |
lock: Add additional lock auditing code
Keep track of lock owner name and replace lock_depth counter
with a per-cpu list of locks held by the cpu.
This allows us to print the actual locks held in case we hit
the (in)famous message about opal_pollers being run with a
lock held.
It also allows us to warn (and drop them) if locks are still
held when returning to the OS or completing a scheduled job.
Signed-off-by: Benjamin Herrenschmidt <benh@kernel.crashing.org>
Reviewed-by: Nicholas Piggin <npiggin@gmail.com>
[stewart: fix unit tests]
Signed-off-by: Stewart Smith <stewart@linux.vnet.ibm.com>
Diffstat (limited to 'hdata')
| -rw-r--r-- | hdata/test/stubs.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/hdata/test/stubs.c b/hdata/test/stubs.c index abeb832..92098bf 100644 --- a/hdata/test/stubs.c +++ b/hdata/test/stubs.c @@ -109,7 +109,7 @@ static bool false_stub(void) { return false; } #define NOOP_STUB FALSE_STUB TRUE_STUB(lock_held_by_me); -NOOP_STUB(lock); +NOOP_STUB(lock_caller); NOOP_STUB(unlock); NOOP_STUB(early_uart_init); NOOP_STUB(mem_reserve_fw); |