RE: [PATCHv2 14/50] hexagon: Add show_stack_loglvl()

From: Brian Cain
Date: Tue Mar 17 2020 - 11:34:36 EST


Acked-by: Brian Cain <bcain@xxxxxxxxxxxxxx>

> Cc: Brian Cain <bcain@xxxxxxxxxxxxxx>
> Cc: linux-hexagon@xxxxxxxxxxxxxxx
> [1]: https://lore.kernel.org/lkml/20190528002412.1625-1-
> dima@xxxxxxxxxx/T/#u
> Signed-off-by: Dmitry Safonov <dima@xxxxxxxxxx>
> ---
> arch/hexagon/kernel/traps.c | 31 +++++++++++++++++--------------
> 1 file changed, 17 insertions(+), 14 deletions(-)
>
> diff --git a/arch/hexagon/kernel/traps.c b/arch/hexagon/kernel/traps.c
> index 69c623b14ddd..a8a3a210d781 100644
> --- a/arch/hexagon/kernel/traps.c
> +++ b/arch/hexagon/kernel/traps.c
> @@ -79,7 +79,7 @@ static const char *ex_name(int ex) }
>
> static void do_show_stack(struct task_struct *task, unsigned long *fp,
> - unsigned long ip)
> + unsigned long ip, const char *loglvl)
> {
> int kstack_depth_to_print = 24;
> unsigned long offset, size;
> @@ -93,9 +93,8 @@ static void do_show_stack(struct task_struct *task,
> unsigned long *fp,
> if (task == NULL)
> task = current;
>
> - printk(KERN_INFO "CPU#%d, %s/%d, Call Trace:\n",
> - raw_smp_processor_id(), task->comm,
> - task_pid_nr(task));
> + printk("%sCPU#%d, %s/%d, Call Trace:\n", loglvl,
> raw_smp_processor_id(),
> + task->comm, task_pid_nr(task));
>
> if (fp == NULL) {
> if (task == current) {
> @@ -108,7 +107,7 @@ static void do_show_stack(struct task_struct *task,
> unsigned long *fp,
> }
>
> if ((((unsigned long) fp) & 0x3) || ((unsigned long) fp < 0x1000)) {
> - printk(KERN_INFO "-- Corrupt frame pointer %p\n", fp);
> + printk("%s-- Corrupt frame pointer %p\n", loglvl, fp);
> return;
> }
>
> @@ -125,8 +124,7 @@ static void do_show_stack(struct task_struct *task,
> unsigned long *fp,
>
> name = kallsyms_lookup(ip, &size, &offset, &modname,
> tmpstr);
>
> - printk(KERN_INFO "[%p] 0x%lx: %s + 0x%lx", fp, ip, name,
> - offset);
> + printk("%s[%p] 0x%lx: %s + 0x%lx", loglvl, fp, ip, name,
> offset);
> if (((unsigned long) fp < low) || (high < (unsigned long)
fp))
> printk(KERN_CONT " (FP out of bounds!)");
> if (modname)
> @@ -136,8 +134,7 @@ static void do_show_stack(struct task_struct *task,
> unsigned long *fp,
> newfp = (unsigned long *) *fp;
>
> if (((unsigned long) newfp) & 0x3) {
> - printk(KERN_INFO "-- Corrupt frame pointer %p\n",
> - newfp);
> + printk("%s-- Corrupt frame pointer %p\n", loglvl,
> newfp);
> break;
> }
>
> @@ -147,7 +144,7 @@ static void do_show_stack(struct task_struct *task,
> unsigned long *fp,
> + 8);
>
> if (regs->syscall_nr != -1) {
> - printk(KERN_INFO "-- trap0 -- syscall_nr:
> %ld",
> + printk("%s-- trap0 -- syscall_nr: %ld",
loglvl,
> regs->syscall_nr);
> printk(KERN_CONT " psp: %lx elr: %lx\n",
> pt_psp(regs), pt_elr(regs));
> @@ -155,7 +152,7 @@ static void do_show_stack(struct task_struct *task,
> unsigned long *fp,
> } else {
> /* really want to see more ... */
> kstack_depth_to_print += 6;
> - printk(KERN_INFO "-- %s (0x%lx) badva:
> %lx\n",
> + printk("%s-- %s (0x%lx) badva: %lx\n",
loglvl,
> ex_name(pt_cause(regs)),
> pt_cause(regs),
> pt_badva(regs));
> }
> @@ -178,10 +175,16 @@ static void do_show_stack(struct task_struct
> *task, unsigned long *fp,
> }
> }
>
> -void show_stack(struct task_struct *task, unsigned long *fp)
> +void show_stack_loglvl(struct task_struct *task, unsigned long *fp,
> + const char *loglvl)
> {
> /* Saved link reg is one word above FP */
> - do_show_stack(task, fp, 0);
> + do_show_stack(task, fp, 0, loglvl);
> +}
> +
> +void show_stack(struct task_struct *task, unsigned long *fp) {
> + show_stack_loglvl(task, fp, 0, KERN_INFO);
> }
>
> int die(const char *str, struct pt_regs *regs, long err) @@ -207,7 +210,7
> @@ int die(const char *str, struct pt_regs *regs, long err)
>
> print_modules();
> show_regs(regs);
> - do_show_stack(current, &regs->r30, pt_elr(regs));
> + do_show_stack(current, &regs->r30, pt_elr(regs), KERN_EMERG);
>
> bust_spinlocks(0);
> add_taint(TAINT_DIE, LOCKDEP_NOW_UNRELIABLE);
> --
> 2.25.1