Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit 0bde9d53 authored by Steven Rostedt (VMware)'s avatar Steven Rostedt (VMware) Committed by Greg Kroah-Hartman
Browse files

tracing: Fix userstacktrace option for instances 



commit bcee5278958802b40ee8b26679155a6d9231783e upstream.

When the instances were able to use their own options, the userstacktrace
option was left hardcoded for the top level. This made the instance
userstacktrace option bascially into a nop, and will confuse users that set
it, but nothing happens (I was confused when it happened to me!)

Cc: stable@vger.kernel.org
Fixes: 16270145 ("tracing: Add trace options for core options to instances")
Signed-off-by: default avatarSteven Rostedt (VMware) <rostedt@goodmis.org>
Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
parent af160357

kernel/trace/trace.c

+4 −3
Original line number Diff line number Diff line
@@ -2134,7 +2134,7 @@ void trace_buffer_unlock_commit_regs(struct trace_array *tr, 
	 * two. They are that meaningful.

	 */

	ftrace_trace_stack(tr, buffer, flags, regs ? 0 : 4, pc, regs);

	ftrace_trace_userstack(buffer, flags, pc);

	ftrace_trace_userstack(tr, buffer, flags, pc);

}



void
@@ -2299,14 +2299,15 @@ void trace_dump_stack(int skip) 
static DEFINE_PER_CPU(int, user_stack_count);



void

ftrace_trace_userstack(struct ring_buffer *buffer, unsigned long flags, int pc)

ftrace_trace_userstack(struct trace_array *tr,

		       struct ring_buffer *buffer, unsigned long flags, int pc)

{

	struct trace_event_call *call = &event_user_stack;

	struct ring_buffer_event *event;

	struct userstack_entry *entry;

	struct stack_trace trace;



	if (!(global_trace.trace_flags & TRACE_ITER_USERSTACKTRACE))

	if (!(tr->trace_flags & TRACE_ITER_USERSTACKTRACE))

		return;



	/*

kernel/trace/trace.h

+4 −2
Original line number Diff line number Diff line
@@ -689,13 +689,15 @@ void update_max_tr_single(struct trace_array *tr, 
#endif /* CONFIG_TRACER_MAX_TRACE */



#ifdef CONFIG_STACKTRACE

void ftrace_trace_userstack(struct ring_buffer *buffer, unsigned long flags,

void ftrace_trace_userstack(struct trace_array *tr,

			    struct ring_buffer *buffer, unsigned long flags,

			    int pc);



void __trace_stack(struct trace_array *tr, unsigned long flags, int skip,

		   int pc);

#else

static inline void ftrace_trace_userstack(struct ring_buffer *buffer,

static inline void ftrace_trace_userstack(struct trace_array *tr,

					  struct ring_buffer *buffer,

					  unsigned long flags, int pc)

{

}