x86/dumpstack: make show_trace_log_lvl() static
Subsystems (2)
PATCH 1/1 x86/dumpstack: make show_trace_log_lvl() static
show_trace_log_lvl() won't be used by other files, so make it static and remove the declaration from the header file.
Signed-off-by Hui Su <[email protected]>
--- arch/x86/include/asm/stacktrace.h | 3 --- arch/x86/kernel/dumpstack.c | 2 +- 2 files changed, 1 insertion(+), 4 deletions(-)
arch/x86/include/asm/stacktrace.h +0 -3
@@ -88,9 +88,6 @@ get_stack_pointer(struct task_struct *task, struct pt_regs *regs)return (unsigned long *)task->thread.sp; } -void show_trace_log_lvl (struct task_struct *task,struct pt_regs *regs, -unsigned long *stack,const char *log_lvl); -/* The form of the top of the frame on the stack */ struct stack_frame {struct stack_frame *next_frame;
arch/x86/kernel/dumpstack.c +1 -1
@@ -168,7 +168,7 @@ static void show_regs_if_on_stack(struct stack_info *info, struct pt_regs *regs, } } -void show_trace_log_lvl (struct task_struct *task,struct pt_regs *regs, +static void show_trace_log_lvl (struct task_struct *task,struct pt_regs *regs,unsigned long *stack,const char *log_lvl) {struct unwind_state state;
Signature · 1 line
2.29.0
[tip: x86/cleanups] x86/dumpstack: Make show_trace_log_lvl() static
The following commit has been merged into the x86/cleanups branch of tip:
The following commit has been merged into the x86/cleanups branch of tip:
Commit-ID: 09a217c10504bcaef911cf2af74e424338efe629 Gitweb: https://git.kernel.org/tip/09a217c10504bcaef911cf2af74e424338efe629 Author: Hui Su <[email protected]> AuthorDate: Fri, 13 Nov 2020 21:39:43 +08:00 Committer: Borislav Petkov <[email protected]> CommitterDate: Tue, 17 Nov 2020 19:05:32 +01:00
x86/dumpstack: Make show_trace_log_lvl() static
show_trace_log_lvl() is not used by other compilation units so make it static and remove the declaration from the header file.
Signed-off-by Hui Su <[email protected]>
Signed-off-by Borislav Petkov <[email protected]>
--- arch/x86/include/asm/stacktrace.h | 3 --- arch/x86/kernel/dumpstack.c | 2 +- 2 files changed, 1 insertion(+), 4 deletions(-)
arch/x86/include/asm/stacktrace.h +0 -3
@@ -88,9 +88,6 @@ get_stack_pointer(struct task_struct *task, struct pt_regs *regs)return (unsigned long *)task->thread.sp; } -void show_trace_log_lvl (struct task_struct *task,struct pt_regs *regs, -unsigned long *stack,const char *log_lvl); -/* The form of the top of the frame on the stack */ struct stack_frame {struct stack_frame *next_frame;
arch/x86/kernel/dumpstack.c +1 -1
@@ -168,7 +168,7 @@ static void show_regs_if_on_stack(struct stack_info *info, struct pt_regs *regs, } } -void show_trace_log_lvl (struct task_struct *task,struct pt_regs *regs, +static void show_trace_log_lvl (struct task_struct *task,struct pt_regs *regs,unsigned long *stack,const char *log_lvl) {struct unwind_state state;
Quoted 5 lines · Hui Su
show_trace_log_lvl()won't be used by other files, so make it static and remove the declaration from the header file.Signed-off-by Hui Su <[email protected]>
I would make sense to add:
Fixes 16d1cb0bc43642a4d9 ("x86/dumpstack: Unify show_regs()")
Othrewise, it looks good:
Reviewed-by Petr Mladek <[email protected]>
Best Regards,
Petr