diff options
| author | Dave Jones <davej@redhat.com> | 2006-12-06 23:37:09 -0500 |
|---|---|---|
| committer | Linus Torvalds <torvalds@woody.osdl.org> | 2006-12-07 11:39:35 -0500 |
| commit | 8f63fdbbd6de7d734c036948bf7c4b2bebe3ad99 (patch) | |
| tree | 0cd665ca44f188922064eb4dc42625463975b9e7 /lib/list_debug.c | |
| parent | 5d469ec0f40d65b2a0a704402990a43b2dafe197 (diff) | |
[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 <davej@redhat.com>
Signed-off-by: Andrew Morton <akpm@osdl.org>
Signed-off-by: Linus Torvalds <torvalds@osdl.org>
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, | |||
| 21 | struct list_head *next) | 21 | struct list_head *next) |
| 22 | { | 22 | { |
| 23 | if (unlikely(next->prev != prev)) { | 23 | if (unlikely(next->prev != prev)) { |
| 24 | printk(KERN_ERR "list_add corruption. next->prev should be %p, but was %p\n", | 24 | printk(KERN_ERR "list_add corruption. next->prev should be " |
| 25 | prev, next->prev); | 25 | "prev (%p), but was %p. (next=%p).\n", |
| 26 | prev, next->prev, next); | ||
| 26 | BUG(); | 27 | BUG(); |
| 27 | } | 28 | } |
| 28 | if (unlikely(prev->next != next)) { | 29 | if (unlikely(prev->next != next)) { |
| 29 | printk(KERN_ERR "list_add corruption. prev->next should be %p, but was %p\n", | 30 | printk(KERN_ERR "list_add corruption. prev->next should be " |
| 30 | next, prev->next); | 31 | "next (%p), but was %p. (prev=%p).\n", |
| 32 | next, prev->next, prev); | ||
| 31 | BUG(); | 33 | BUG(); |
| 32 | } | 34 | } |
| 33 | next->prev = new; | 35 | next->prev = new; |
