diff options
author | Andrew Morton <akpm@osdl.org> | 2006-10-01 02:26:59 -0400 |
---|---|---|
committer | Linus Torvalds <torvalds@g5.osdl.org> | 2006-10-01 03:39:17 -0400 |
commit | ab8e92efcf3f7972f30033cad75f180aef4f3abc (patch) | |
tree | 3205746553d1981de43cdaa07739f910fd25ed2d /lib/list_debug.c | |
parent | 1bdfd554be94def718323659173517c5d4a69d25 (diff) |
[PATCH] list_del-debug fix
These two BUG_ON()s are redundant and undesired: we're checking for this
condition further on in the function, only better.
Cc: Dave Jones <davej@codemonkey.org.uk>
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 | 3 |
1 files changed, 0 insertions, 3 deletions
diff --git a/lib/list_debug.c b/lib/list_debug.c index e80d27c97898..7ba9d823d388 100644 --- a/lib/list_debug.c +++ b/lib/list_debug.c | |||
@@ -59,9 +59,6 @@ EXPORT_SYMBOL(list_add); | |||
59 | */ | 59 | */ |
60 | void list_del(struct list_head *entry) | 60 | void list_del(struct list_head *entry) |
61 | { | 61 | { |
62 | BUG_ON(entry->prev->next != entry); | ||
63 | BUG_ON(entry->next->prev != entry); | ||
64 | |||
65 | if (unlikely(entry->prev->next != entry)) { | 62 | if (unlikely(entry->prev->next != entry)) { |
66 | printk(KERN_ERR "list_del corruption. prev->next should be %p, " | 63 | printk(KERN_ERR "list_del corruption. prev->next should be %p, " |
67 | "but was %p\n", entry, entry->prev->next); | 64 | "but was %p\n", entry, entry->prev->next); |