diff options
| author | Dave Jones <[email protected]> | 2006-12-07 04:37:09 +0000 |
|---|---|---|
| committer | Linus Torvalds <[email protected]> | 2006-12-07 16:39:35 +0000 |
| commit | 8f63fdbbd6de7d734c036948bf7c4b2bebe3ad99 (patch) | |
| tree | 0cd665ca44f188922064eb4dc42625463975b9e7 /lib/list_debug.c | |
| parent | [PATCH] Correct misc_register return code handling in several drivers (diff) | |
| download | kernel-8f63fdbbd6de7d734c036948bf7c4b2bebe3ad99.tar.gz kernel-8f63fdbbd6de7d734c036948bf7c4b2bebe3ad99.zip | |
[PATCH] More list debugging context
Print the other (hopefully) known good pointer when list_head debugging
too, which may yield additional clues.
Also fix for 80-columns to win akpm brownie points.
Signed-off-by: Dave Jones <[email protected]>
Signed-off-by: Andrew Morton <[email protected]>
Signed-off-by: Linus Torvalds <[email protected]>
Diffstat (limited to 'lib/list_debug.c')
| -rw-r--r-- | lib/list_debug.c | 10 |
1 files changed, 6 insertions, 4 deletions
diff --git a/lib/list_debug.c b/lib/list_debug.c index 7ba9d823d388..4350ba9655bd 100644 --- a/lib/list_debug.c +++ b/lib/list_debug.c @@ -21,13 +21,15 @@ void __list_add(struct list_head *new, struct list_head *next) { if (unlikely(next->prev != prev)) { - printk(KERN_ERR "list_add corruption. next->prev should be %p, but was %p\n", - prev, next->prev); + printk(KERN_ERR "list_add corruption. next->prev should be " + "prev (%p), but was %p. (next=%p).\n", + prev, next->prev, next); BUG(); } if (unlikely(prev->next != next)) { - printk(KERN_ERR "list_add corruption. prev->next should be %p, but was %p\n", - next, prev->next); + printk(KERN_ERR "list_add corruption. prev->next should be " + "next (%p), but was %p. (prev=%p).\n", + next, prev->next, prev); BUG(); } next->prev = new; |