backport of mainline commit dd04b452f532ca100f7c557295ffcbc049c77171
Author: Jean Delvare <[email protected]>
Date:   Wed Jul 3 15:08:47 2013 -0700

    idr: print a stack dump after ida_remove warning

    We print a dump stack after idr_remove warning.  This is useful to find
    the faulty piece of code.  Let's do the same for ida_remove, as it would
    be equally useful there.

    [[email protected]: convert the open-coded printk+dump_stack into 
WARN()]
    Signed-off-by: Jean Delvare <[email protected]>
    Cc: Tejun Heo <[email protected]>
    Cc: Takashi Iwai <[email protected]>
    Signed-off-by: Andrew Morton <[email protected]>
    Signed-off-by: Linus Torvalds <[email protected]>

https://bugzilla.openvz.org/show_bug.cgi?id=3117
Signed-off-by: Vasily Averin <[email protected]>
---
 lib/idr.c | 7 ++-----
 1 file changed, 2 insertions(+), 5 deletions(-)

diff --git a/lib/idr.c b/lib/idr.c
index a3bfde8..674c30b 100644
--- a/lib/idr.c
+++ b/lib/idr.c
@@ -524,9 +524,7 @@ EXPORT_SYMBOL(idr_alloc_cyclic);
 
 static void idr_remove_warning(int id)
 {
-       printk(KERN_WARNING
-               "idr_remove called for id=%d which is not allocated.\n", id);
-       dump_stack();
+       WARN(1, "idr_remove called for id=%d which is not allocated.\n", id);
 }
 
 static void sub_remove(struct idr *idp, int shift, int id)
@@ -1062,8 +1060,7 @@ void ida_remove(struct ida *ida, int id)
        return;
 
  err:
-       printk(KERN_WARNING
-              "ida_remove called for id=%d which is not allocated.\n", id);
+       WARN(1, "ida_remove called for id=%d which is not allocated.\n", id);
 }
 EXPORT_SYMBOL(ida_remove);
 
-- 
1.9.1

_______________________________________________
Devel mailing list
[email protected]
https://lists.openvz.org/mailman/listinfo/devel

Reply via email to