[lkml]   [2018]   [Apr]   [17]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
Messages in this thread
Subject[RFC v4 0/4] Centralize and unify usage of preempt/irq tracepoints
This is the next revision of preempt/irq tracepoint centralization and
unified usage across the kernel [1].
The preempt/irq tracepoints exist but not everything in the kernel is
using it. This makes things not work simultaneously (for ex, only
lockdep or irqsoff events can be used). This series is an attempt to
solve that.

I fixed several issues since last rev and I'm posting it slightly early
than I wanted to for some feedback especially for the new patches 1-3 in
this series. Also any testing is deeply appreciated.

Currently I'm noticing a performance issue with "enabling" of the
preemptoff tracer (not during its running but the enabling/startup of
it). I narrowed this down to rcu_irq_enter_irqson and
rcu_irq_exit_irqson calls in tracepoint.h. Commenting these out makes
the perf issue go away. I also notice that disabling function tracer
also makes the issue go away. Note that I ran hackbench and don't notice
any performance issues with my patch itself per-se, however the
preemptoff tracer startup seems slow and I'm debugging that. If you have
a clue why this might be happening, please let me know.

Note that I skipped v3 because I used that number already for some
internal reviews of these patches. Also I changed the subject back to
RFC in light of the lockdep issues I fixed this time and added some more
folks for review. Thanks much for any review or testing.


Joel Fernandes (4):
tracepoint: Add API to not do lockdep checks during RCU ops
softirq: reorder trace_softirqs_on to prevent lockdep splat
irqflags: Avoid unnecessary calls to trace_ if you can
tracing: Centralize preemptirq tracepoints and unify their usage

include/linux/ftrace.h | 11 +-
include/linux/irqflags.h | 32 +++--
include/linux/lockdep.h | 8 +-
include/linux/preempt.h | 2 +-
include/linux/tracepoint.h | 32 ++++-
include/trace/events/preemptirq.h | 23 ++--
init/main.c | 5 +-
kernel/locking/lockdep.c | 35 ++---
kernel/sched/core.c | 2 +-
kernel/softirq.c | 6 +-
kernel/trace/Kconfig | 22 +++-
kernel/trace/Makefile | 2 +-
kernel/trace/trace_irqsoff.c | 206 +++++++-----------------------
kernel/trace/trace_preemptirq.c | 73 +++++++++++
14 files changed, 229 insertions(+), 230 deletions(-)
create mode 100644 kernel/trace/trace_preemptirq.c

Cc: Steven Rostedt <>
Cc: Peter Zilstra <>
Cc: Ingo Molnar <>
Cc: Mathieu Desnoyers <>
Cc: Tom Zanussi <>
Cc: Namhyung Kim <>
Cc: Thomas Glexiner <>
Cc: Boqun Feng <>
Cc: Paul McKenney <>
Cc: Frederic Weisbecker <>
Cc: Randy Dunlap <>
Cc: Masami Hiramatsu <>
Cc: Fenguang Wu <>
Cc: Baohong Liu <>
Cc: Vedang Patel <>

 \ /
  Last update: 2018-04-17 06:10    [W:0.291 / U:0.236 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site