Commit fc91a3c4 authored by Joel Fernandes (Google)'s avatar Joel Fernandes (Google) Committed by Thomas Gleixner

debugobjects: Make stack check warning more informative

While debugging an issue debugobject tracking warned about an annotation
issue of an object on stack. It turned out that the issue was due to the
object in concern being on a different stack which was due to another
issue.

Thomas suggested to print the pointers and the location of the stack for
the currently running task. This helped to figure out that the object was
on the wrong stack.

As this is general useful information for debugging similar issues, make
the error message more informative by printing the pointers.

[ tglx: Massaged changelog ]
Signed-off-by: default avatarJoel Fernandes (Google) <joel@joelfernandes.org>
Signed-off-by: default avatarThomas Gleixner <tglx@linutronix.de>
Acked-by: default avatarWaiman Long <longman@redhat.com>
Acked-by: default avatarYang Shi <yang.shi@linux.alibaba.com>
Cc: kernel-team@android.com
Cc: Arnd Bergmann <arnd@arndb.de>
Cc: astrachan@google.com
Link: https://lkml.kernel.org/r/20180723212531.202328-1-joel@joelfernandes.org
parent acb18725
...@@ -360,9 +360,12 @@ static void debug_object_is_on_stack(void *addr, int onstack) ...@@ -360,9 +360,12 @@ static void debug_object_is_on_stack(void *addr, int onstack)
limit++; limit++;
if (is_on_stack) if (is_on_stack)
pr_warn("object is on stack, but not annotated\n"); pr_warn("object %p is on stack %p, but NOT annotated.\n", addr,
task_stack_page(current));
else else
pr_warn("object is not on stack, but annotated\n"); pr_warn("object %p is NOT on stack %p, but annotated.\n", addr,
task_stack_page(current));
WARN_ON(1); WARN_ON(1);
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment