Commit 3f1756dc authored by Steven Rostedt (VMware)'s avatar Steven Rostedt (VMware)
Browse files

tracing: More reverting of "tracing: Centralize preemptirq tracepoints and unify their usage" 

Joel Fernandes created a nice patch that cleaned up the duplicate hooks used
by lockdep and irqsoff latency tracer. It made both use tracepoints. But the
latency tracer is triggering warnings when using tracepoints to call into
the latency tracer's routines. Mainly, they can be called from NMI context.
If that happens, then the SRCU may not work properly because on some
architectures, SRCU is not safe to be called in both NMI and non-NMI
context.

This is a partial revert of the clean up patch c3bc8fd6 ("tracing:
Centralize preemptirq tracepoints and unify their usage") that adds back the
direct calls into the latency tracer. It also only calls the trace events
when not in NMI.

Link: http://lkml.kernel.org/r/20180809210654.622445925@goodmis.org


Reviewed-by: default avatarJoel Fernandes (Google) <joel@joelfernandes.org>
Fixes: c3bc8fd6 ("tracing: Centralize preemptirq tracepoints and unify their usage")
Signed-off-by: default avatarSteven Rostedt (VMware) <rostedt@goodmis.org>
parent f27107fa

kernel/trace/trace.h

+15 −0
Original line number Original line Diff line number Diff line
@@ -1827,6 +1827,21 @@ static inline int tracing_alloc_snapshot_instance(struct trace_array *tr) 
}

}

#endif

#endif





#ifdef CONFIG_PREEMPT_TRACER

void tracer_preempt_on(unsigned long a0, unsigned long a1);

void tracer_preempt_off(unsigned long a0, unsigned long a1);

#else

static inline void tracer_preempt_on(unsigned long a0, unsigned long a1) { }

static inline void tracer_preempt_off(unsigned long a0, unsigned long a1) { }

#endif

#ifdef CONFIG_IRQSOFF_TRACER

void tracer_hardirqs_on(unsigned long a0, unsigned long a1);

void tracer_hardirqs_off(unsigned long a0, unsigned long a1);

#else

static inline void tracer_hardirqs_on(unsigned long a0, unsigned long a1) { }

static inline void tracer_hardirqs_off(unsigned long a0, unsigned long a1) { }

#endif



extern struct trace_iterator *tracepoint_print_iter;

extern struct trace_iterator *tracepoint_print_iter;





#endif /* _LINUX_KERNEL_TRACE_H */
 
#endif /* _LINUX_KERNEL_TRACE_H */

kernel/trace/trace_irqsoff.c

+4 −44
Original line number Original line Diff line number Diff line
@@ -605,40 +605,18 @@ static void irqsoff_tracer_stop(struct trace_array *tr) 
/*

/*

 * We are only interested in hardirq on/off events:

 * We are only interested in hardirq on/off events:

 */

 */

static void tracer_hardirqs_on(void *none, unsigned long a0, unsigned long a1)

void tracer_hardirqs_on(unsigned long a0, unsigned long a1)

{

{

	unsigned int pc = preempt_count();

	unsigned int pc = preempt_count();





	/*

	 * Tracepoint probes are expected to be called with preempt disabled,

	 * We don't care about being called with preempt disabled but we need

	 * to know in the future if that changes so we can remove the next

	 * preempt_enable.

	 */

	WARN_ON_ONCE(pc < PREEMPT_DISABLE_OFFSET);



	/* Use PREEMPT_DISABLE_OFFSET to handle !CONFIG_PREEMPT cases */

	pc -= PREEMPT_DISABLE_OFFSET;



	if (!preempt_trace(pc) && irq_trace())

	if (!preempt_trace(pc) && irq_trace())

		stop_critical_timing(a0, a1, pc);

		stop_critical_timing(a0, a1, pc);

}

}





static void tracer_hardirqs_off(void *none, unsigned long a0, unsigned long a1)

void tracer_hardirqs_off(unsigned long a0, unsigned long a1)

{

{

	unsigned int pc = preempt_count();

	unsigned int pc = preempt_count();





	/*

	 * Tracepoint probes are expected to be called with preempt disabled,

	 * We don't care about being called with preempt disabled but we need

	 * to know in the future if that changes so we can remove the next

	 * preempt_enable.

	 */

	WARN_ON_ONCE(pc < PREEMPT_DISABLE_OFFSET);



	/* Use PREEMPT_DISABLE_OFFSET to handle !CONFIG_PREEMPT cases */

	pc -= PREEMPT_DISABLE_OFFSET;



	if (!preempt_trace(pc) && irq_trace())

	if (!preempt_trace(pc) && irq_trace())

		start_critical_timing(a0, a1, pc);

		start_critical_timing(a0, a1, pc);

}

}
@@ -647,15 +625,11 @@ static int irqsoff_tracer_init(struct trace_array *tr) 
{

{

	trace_type = TRACER_IRQS_OFF;

	trace_type = TRACER_IRQS_OFF;





	register_trace_irq_disable(tracer_hardirqs_off, NULL);

	register_trace_irq_enable(tracer_hardirqs_on, NULL);

	return __irqsoff_tracer_init(tr);

	return __irqsoff_tracer_init(tr);

}

}





static void irqsoff_tracer_reset(struct trace_array *tr)

static void irqsoff_tracer_reset(struct trace_array *tr)

{

{

	unregister_trace_irq_disable(tracer_hardirqs_off, NULL);

	unregister_trace_irq_enable(tracer_hardirqs_on, NULL);

	__irqsoff_tracer_reset(tr);

	__irqsoff_tracer_reset(tr);

}

}
@@ -681,7 +655,7 @@ static struct tracer irqsoff_tracer __read_mostly = 
#endif /*  CONFIG_IRQSOFF_TRACER */

#endif /*  CONFIG_IRQSOFF_TRACER */





#ifdef CONFIG_PREEMPT_TRACER

#ifdef CONFIG_PREEMPT_TRACER

static void tracer_preempt_on(void *none, unsigned long a0, unsigned long a1)

void tracer_preempt_on(unsigned long a0, unsigned long a1)

{

{

	int pc = preempt_count();

	int pc = preempt_count();
@@ -689,7 +663,7 @@ static void tracer_preempt_on(void *none, unsigned long a0, unsigned long a1) 
		stop_critical_timing(a0, a1, pc);

		stop_critical_timing(a0, a1, pc);

}

}





static void tracer_preempt_off(void *none, unsigned long a0, unsigned long a1)

void tracer_preempt_off(unsigned long a0, unsigned long a1)

{

{

	int pc = preempt_count();

	int pc = preempt_count();
@@ -701,15 +675,11 @@ static int preemptoff_tracer_init(struct trace_array *tr) 
{

{

	trace_type = TRACER_PREEMPT_OFF;

	trace_type = TRACER_PREEMPT_OFF;





	register_trace_preempt_disable(tracer_preempt_off, NULL);

	register_trace_preempt_enable(tracer_preempt_on, NULL);

	return __irqsoff_tracer_init(tr);

	return __irqsoff_tracer_init(tr);

}

}





static void preemptoff_tracer_reset(struct trace_array *tr)

static void preemptoff_tracer_reset(struct trace_array *tr)

{

{

	unregister_trace_preempt_disable(tracer_preempt_off, NULL);

	unregister_trace_preempt_enable(tracer_preempt_on, NULL);

	__irqsoff_tracer_reset(tr);

	__irqsoff_tracer_reset(tr);

}

}
@@ -740,21 +710,11 @@ static int preemptirqsoff_tracer_init(struct trace_array *tr) 
{

{

	trace_type = TRACER_IRQS_OFF | TRACER_PREEMPT_OFF;

	trace_type = TRACER_IRQS_OFF | TRACER_PREEMPT_OFF;





	register_trace_irq_disable(tracer_hardirqs_off, NULL);

	register_trace_irq_enable(tracer_hardirqs_on, NULL);

	register_trace_preempt_disable(tracer_preempt_off, NULL);

	register_trace_preempt_enable(tracer_preempt_on, NULL);



	return __irqsoff_tracer_init(tr);

	return __irqsoff_tracer_init(tr);

}

}





static void preemptirqsoff_tracer_reset(struct trace_array *tr)

static void preemptirqsoff_tracer_reset(struct trace_array *tr)

{

{

	unregister_trace_irq_disable(tracer_hardirqs_off, NULL);

	unregister_trace_irq_enable(tracer_hardirqs_on, NULL);

	unregister_trace_preempt_disable(tracer_preempt_off, NULL);

	unregister_trace_preempt_enable(tracer_preempt_on, NULL);



	__irqsoff_tracer_reset(tr);

	__irqsoff_tracer_reset(tr);

}

}

kernel/trace/trace_preemptirq.c

+19 −6
Original line number Original line Diff line number Diff line
@@ -9,6 +9,7 @@ 
#include <linux/uaccess.h>

#include <linux/uaccess.h>

#include <linux/module.h>

#include <linux/module.h>

#include <linux/ftrace.h>

#include <linux/ftrace.h>

#include "trace.h"





#define CREATE_TRACE_POINTS

#define CREATE_TRACE_POINTS

#include <trace/events/preemptirq.h>

#include <trace/events/preemptirq.h>
@@ -20,7 +21,9 @@ static DEFINE_PER_CPU(int, tracing_irq_cpu); 
void trace_hardirqs_on(void)

void trace_hardirqs_on(void)

{

{

	if (this_cpu_read(tracing_irq_cpu)) {

	if (this_cpu_read(tracing_irq_cpu)) {

		if (!in_nmi())

			trace_irq_enable_rcuidle(CALLER_ADDR0, CALLER_ADDR1);

			trace_irq_enable_rcuidle(CALLER_ADDR0, CALLER_ADDR1);

		tracer_hardirqs_on(CALLER_ADDR0, CALLER_ADDR1);

		this_cpu_write(tracing_irq_cpu, 0);

		this_cpu_write(tracing_irq_cpu, 0);

	}

	}
@@ -32,6 +35,8 @@ void trace_hardirqs_off(void) 
{

{

	if (!this_cpu_read(tracing_irq_cpu)) {

	if (!this_cpu_read(tracing_irq_cpu)) {

		this_cpu_write(tracing_irq_cpu, 1);

		this_cpu_write(tracing_irq_cpu, 1);

		tracer_hardirqs_off(CALLER_ADDR0, CALLER_ADDR1);

		if (!in_nmi())

			trace_irq_disable_rcuidle(CALLER_ADDR0, CALLER_ADDR1);

			trace_irq_disable_rcuidle(CALLER_ADDR0, CALLER_ADDR1);

	}

	}
@@ -42,7 +47,9 @@ EXPORT_SYMBOL(trace_hardirqs_off); 
__visible void trace_hardirqs_on_caller(unsigned long caller_addr)

__visible void trace_hardirqs_on_caller(unsigned long caller_addr)

{

{

	if (this_cpu_read(tracing_irq_cpu)) {

	if (this_cpu_read(tracing_irq_cpu)) {

		if (!in_nmi())

			trace_irq_enable_rcuidle(CALLER_ADDR0, caller_addr);

			trace_irq_enable_rcuidle(CALLER_ADDR0, caller_addr);

		tracer_hardirqs_on(CALLER_ADDR0, caller_addr);

		this_cpu_write(tracing_irq_cpu, 0);

		this_cpu_write(tracing_irq_cpu, 0);

	}

	}
@@ -54,6 +61,8 @@ __visible void trace_hardirqs_off_caller(unsigned long caller_addr) 
{

{

	if (!this_cpu_read(tracing_irq_cpu)) {

	if (!this_cpu_read(tracing_irq_cpu)) {

		this_cpu_write(tracing_irq_cpu, 1);

		this_cpu_write(tracing_irq_cpu, 1);

		tracer_hardirqs_off(CALLER_ADDR0, caller_addr);

		if (!in_nmi())

			trace_irq_disable_rcuidle(CALLER_ADDR0, caller_addr);

			trace_irq_disable_rcuidle(CALLER_ADDR0, caller_addr);

	}

	}
@@ -66,11 +75,15 @@ EXPORT_SYMBOL(trace_hardirqs_off_caller); 




void trace_preempt_on(unsigned long a0, unsigned long a1)

void trace_preempt_on(unsigned long a0, unsigned long a1)

{

{

	if (!in_nmi())

		trace_preempt_enable_rcuidle(a0, a1);

		trace_preempt_enable_rcuidle(a0, a1);

	tracer_preempt_on(a0, a1);

}

}





void trace_preempt_off(unsigned long a0, unsigned long a1)

void trace_preempt_off(unsigned long a0, unsigned long a1)

{

{

	if (!in_nmi())

		trace_preempt_disable_rcuidle(a0, a1);

		trace_preempt_disable_rcuidle(a0, a1);

	tracer_preempt_off(a0, a1);

}

}

#endif
 
#endif

CRA Git | Maintained and supported by SUSTech CRA and CCSE