Commit b4c97124 authored by Yue Hu's avatar Yue Hu Committed by Will Deacon

arm64: traps: Add str of description to panic() in die()

Currently, there are different description strings in die() such as
die("Oops",,), die("Oops - BUG",,). And panic() called by die() will
always show "Fatal exception" or "Fatal exception in interrupt".

Note that panic() will run any panic handler via panic_notifier_list.
And the string above will be formatted and placed in static buf[]
which will be passed to handler.

So panic handler can not distinguish which Oops it is from the buf if
we want to do some things like reserve the string in memory or panic
statistics. It's not benefit to debug. We need to add more codes to
troubleshoot. Let's utilize existing resource to make debug much simpler.
Signed-off-by: default avatarYue Hu <huyue2@yulong.com>
Link: https://lore.kernel.org/r/20200804085347.10720-1-zbestahu@gmail.comSigned-off-by: default avatarWill Deacon <will@kernel.org>
parent f75aef39
...@@ -200,9 +200,9 @@ void die(const char *str, struct pt_regs *regs, int err) ...@@ -200,9 +200,9 @@ void die(const char *str, struct pt_regs *regs, int err)
oops_exit(); oops_exit();
if (in_interrupt()) if (in_interrupt())
panic("Fatal exception in interrupt"); panic("%s: Fatal exception in interrupt", str);
if (panic_on_oops) if (panic_on_oops)
panic("Fatal exception"); panic("%s: Fatal exception", str);
raw_spin_unlock_irqrestore(&die_lock, flags); raw_spin_unlock_irqrestore(&die_lock, flags);
......
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