Commit b8030906 authored by Ingo Molnar's avatar Ingo Molnar

x86: dumpstack: Clean up the x86_stack_ids[][] initalization and other details

Make the initialization more readable, plus tidy up a few small
visual details as well.

No change in functionality.

LKML-Reference: <new-submission>
Signed-off-by: default avatarIngo Molnar <mingo@elte.hu>
parent 27c13ece
...@@ -10,9 +10,9 @@ ...@@ -10,9 +10,9 @@
#include <linux/module.h> #include <linux/module.h>
#include <linux/ptrace.h> #include <linux/ptrace.h>
#include <linux/kexec.h> #include <linux/kexec.h>
#include <linux/sysfs.h>
#include <linux/bug.h> #include <linux/bug.h>
#include <linux/nmi.h> #include <linux/nmi.h>
#include <linux/sysfs.h>
#include <asm/stacktrace.h> #include <asm/stacktrace.h>
...@@ -35,6 +35,7 @@ void dump_trace(struct task_struct *task, struct pt_regs *regs, ...@@ -35,6 +35,7 @@ void dump_trace(struct task_struct *task, struct pt_regs *regs,
if (!stack) { if (!stack) {
unsigned long dummy; unsigned long dummy;
stack = &dummy; stack = &dummy;
if (task && task != current) if (task && task != current)
stack = (unsigned long *)task->thread.sp; stack = (unsigned long *)task->thread.sp;
...@@ -57,8 +58,7 @@ void dump_trace(struct task_struct *task, struct pt_regs *regs, ...@@ -57,8 +58,7 @@ void dump_trace(struct task_struct *task, struct pt_regs *regs,
context = (struct thread_info *) context = (struct thread_info *)
((unsigned long)stack & (~(THREAD_SIZE - 1))); ((unsigned long)stack & (~(THREAD_SIZE - 1)));
bp = print_context_stack(context, stack, bp, ops, bp = print_context_stack(context, stack, bp, ops, data, NULL, &graph);
data, NULL, &graph);
stack = (unsigned long *)context->previous_esp; stack = (unsigned long *)context->previous_esp;
if (!stack) if (!stack)
...@@ -156,4 +156,3 @@ int is_valid_bugaddr(unsigned long ip) ...@@ -156,4 +156,3 @@ int is_valid_bugaddr(unsigned long ip)
return ud2 == 0x0b0f; return ud2 == 0x0b0f;
} }
...@@ -10,26 +10,28 @@ ...@@ -10,26 +10,28 @@
#include <linux/module.h> #include <linux/module.h>
#include <linux/ptrace.h> #include <linux/ptrace.h>
#include <linux/kexec.h> #include <linux/kexec.h>
#include <linux/sysfs.h>
#include <linux/bug.h> #include <linux/bug.h>
#include <linux/nmi.h> #include <linux/nmi.h>
#include <linux/sysfs.h>
#include <asm/stacktrace.h> #include <asm/stacktrace.h>
#include "dumpstack.h" #include "dumpstack.h"
#define N_EXCEPTION_STACKS_END \
(N_EXCEPTION_STACKS + DEBUG_STKSZ/EXCEPTION_STKSZ - 2)
static char x86_stack_ids[][8] = { static char x86_stack_ids[][8] = {
[DEBUG_STACK - 1] = "#DB", [ DEBUG_STACK-1 ] = "#DB",
[NMI_STACK - 1] = "NMI", [ NMI_STACK-1 ] = "NMI",
[DOUBLEFAULT_STACK - 1] = "#DF", [ DOUBLEFAULT_STACK-1 ] = "#DF",
[STACKFAULT_STACK - 1] = "#SS", [ STACKFAULT_STACK-1 ] = "#SS",
[MCE_STACK - 1] = "#MC", [ MCE_STACK-1 ] = "#MC",
#if DEBUG_STKSZ > EXCEPTION_STKSZ #if DEBUG_STKSZ > EXCEPTION_STKSZ
[N_EXCEPTION_STACKS ... [ N_EXCEPTION_STACKS ...
N_EXCEPTION_STACKS + DEBUG_STKSZ / EXCEPTION_STKSZ - 2] = "#DB[?]" N_EXCEPTION_STACKS_END ] = "#DB[?]"
#endif #endif
}; };
int x86_is_stack_id(int id, char *name) int x86_is_stack_id(int id, char *name)
{ {
...@@ -303,4 +305,3 @@ int is_valid_bugaddr(unsigned long ip) ...@@ -303,4 +305,3 @@ int is_valid_bugaddr(unsigned long ip)
return ud2 == 0x0b0f; return ud2 == 0x0b0f;
} }
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