* [PATCH v2 0/2] tracing: Make synthetic events a separate option
@ 2020-05-28 19:32 Tom Zanussi
2020-05-28 19:32 ` [PATCH v2 2/2] selftests/ftrace: Distinguish between hist and synthetic event checks Tom Zanussi
0 siblings, 1 reply; 4+ messages in thread
From: Tom Zanussi @ 2020-05-28 19:32 UTC (permalink / raw)
To: rostedt; +Cc: mhiramat, skhan, linux-kernel, linux-rt-users
Hi Steve,
These are the same patches as v1, but after posting v1 I realized that
if you applied them on top of the other patchset I posted awhile back
("[PATCH 0/3] tracing: histogram internals doc, hist_debug, and misc")
[1], you wouldn't be able to apply these.
So assuming you apply the other patchset first, these should apply
cleanly on top of those.
Thanks,
Tom
[1] https://lore.kernel.org/lkml/cover.1585941485.git.zanussi@kernel.org/
v1 text:
Since synthetic events can now be used separately from hist triggers,
it makes sense to actually separate them into different options. It
also makes sense in terms of file bloat - trace_events_hist.c was
getting very large and this makes it smaller and easier to follow.
Thanks,
Tom
The following changes since commit 77914c22b0ba493d9783c53bbfbc6087d6a7e1b1:
tracing: Add hist_debug trace event files for histogram debugging (2020-04-03 13:46:05 -0500)
are available in the Git repository at:
git://git.kernel.org/pub/scm/linux/kernel/git/zanussi/linux-trace.git ftrace/separate-synth-v2
Tom Zanussi (2):
tracing: Move synthetic events to a separate file
selftests/ftrace: Distinguish between hist and synthetic event checks
kernel/trace/Kconfig | 20 +-
kernel/trace/Makefile | 1 +
kernel/trace/trace_events_hist.c | 2071 ++---------------
kernel/trace/trace_events_synth.c | 1789 ++++++++++++++
kernel/trace/trace_synth.h | 36 +
.../trigger-field-variable-support.tc | 5 +
.../trigger-inter-event-combined-hist.tc | 5 +
.../trigger-multi-actions-accept.tc | 5 +
.../trigger-onmatch-action-hist.tc | 5 +
.../trigger-onmatch-onmax-action-hist.tc | 5 +
.../inter-event/trigger-onmax-action-hist.tc | 5 +
.../trigger-snapshot-action-hist.tc | 5 +
.../inter-event/trigger-trace-action-hist.tc | 5 +
13 files changed, 2029 insertions(+), 1928 deletions(-)
create mode 100644 kernel/trace/trace_events_synth.c
create mode 100644 kernel/trace/trace_synth.h
--
2.17.1
^ permalink raw reply [flat|nested] 4+ messages in thread* [PATCH v2 2/2] selftests/ftrace: Distinguish between hist and synthetic event checks 2020-05-28 19:32 [PATCH v2 0/2] tracing: Make synthetic events a separate option Tom Zanussi @ 2020-05-28 19:32 ` Tom Zanussi 2020-05-29 14:38 ` Masami Hiramatsu 0 siblings, 1 reply; 4+ messages in thread From: Tom Zanussi @ 2020-05-28 19:32 UTC (permalink / raw) To: rostedt; +Cc: mhiramat, skhan, linux-kernel, linux-rt-users With synthetic events now a separate config item as a result of 'tracing: Move synthetic events to a separate file', tests that use both need to explicitly check for hist trigger support rather than relying on hist triggers to pull in synthetic events. Add an additional hist trigger check to all the trigger tests that now require it, otherwise they'll fail if synthetic events but not hist triggers are enabled. Signed-off-by: Tom Zanussi <zanussi@kernel.org> --- .../trigger/inter-event/trigger-field-variable-support.tc | 5 +++++ .../trigger/inter-event/trigger-inter-event-combined-hist.tc | 5 +++++ .../trigger/inter-event/trigger-multi-actions-accept.tc | 5 +++++ .../trigger/inter-event/trigger-onmatch-action-hist.tc | 5 +++++ .../trigger/inter-event/trigger-onmatch-onmax-action-hist.tc | 5 +++++ .../test.d/trigger/inter-event/trigger-onmax-action-hist.tc | 5 +++++ .../trigger/inter-event/trigger-snapshot-action-hist.tc | 5 +++++ .../test.d/trigger/inter-event/trigger-trace-action-hist.tc | 5 +++++ 8 files changed, 40 insertions(+) diff --git a/tools/testing/selftests/ftrace/test.d/trigger/inter-event/trigger-field-variable-support.tc b/tools/testing/selftests/ftrace/test.d/trigger/inter-event/trigger-field-variable-support.tc index 77be6e1f6e7b..e232059a8ab2 100644 --- a/tools/testing/selftests/ftrace/test.d/trigger/inter-event/trigger-field-variable-support.tc +++ b/tools/testing/selftests/ftrace/test.d/trigger/inter-event/trigger-field-variable-support.tc @@ -17,6 +17,11 @@ if [ ! -f synthetic_events ]; then exit_unsupported fi +if [ ! -f events/sched/sched_process_fork/hist ]; then + echo "hist trigger is not supported" + exit_unsupported +fi + echo "Test field variable support" echo 'wakeup_latency u64 lat; pid_t pid; int prio; char comm[16]' > synthetic_events diff --git a/tools/testing/selftests/ftrace/test.d/trigger/inter-event/trigger-inter-event-combined-hist.tc b/tools/testing/selftests/ftrace/test.d/trigger/inter-event/trigger-inter-event-combined-hist.tc index f3eb8aacec0e..07cfcb8157b6 100644 --- a/tools/testing/selftests/ftrace/test.d/trigger/inter-event/trigger-inter-event-combined-hist.tc +++ b/tools/testing/selftests/ftrace/test.d/trigger/inter-event/trigger-inter-event-combined-hist.tc @@ -17,6 +17,11 @@ if [ ! -f synthetic_events ]; then exit_unsupported fi +if [ ! -f events/sched/sched_process_fork/hist ]; then + echo "hist trigger is not supported" + exit_unsupported +fi + echo "Test create synthetic event" echo 'waking_latency u64 lat pid_t pid' > synthetic_events diff --git a/tools/testing/selftests/ftrace/test.d/trigger/inter-event/trigger-multi-actions-accept.tc b/tools/testing/selftests/ftrace/test.d/trigger/inter-event/trigger-multi-actions-accept.tc index d281f056f980..73e413c2ca26 100644 --- a/tools/testing/selftests/ftrace/test.d/trigger/inter-event/trigger-multi-actions-accept.tc +++ b/tools/testing/selftests/ftrace/test.d/trigger/inter-event/trigger-multi-actions-accept.tc @@ -17,6 +17,11 @@ if [ ! -f synthetic_events ]; then exit_unsupported fi +if [ ! -f events/sched/sched_process_fork/hist ]; then + echo "hist trigger is not supported" + exit_unsupported +fi + echo "Test multiple actions on hist trigger" echo 'wakeup_latency u64 lat; pid_t pid' >> synthetic_events TRIGGER1=events/sched/sched_wakeup/trigger diff --git a/tools/testing/selftests/ftrace/test.d/trigger/inter-event/trigger-onmatch-action-hist.tc b/tools/testing/selftests/ftrace/test.d/trigger/inter-event/trigger-onmatch-action-hist.tc index a708f0e7858a..ebe0ad827f9f 100644 --- a/tools/testing/selftests/ftrace/test.d/trigger/inter-event/trigger-onmatch-action-hist.tc +++ b/tools/testing/selftests/ftrace/test.d/trigger/inter-event/trigger-onmatch-action-hist.tc @@ -17,6 +17,11 @@ if [ ! -f synthetic_events ]; then exit_unsupported fi +if [ ! -f events/sched/sched_process_fork/hist ]; then + echo "hist trigger is not supported" + exit_unsupported +fi + echo "Test create synthetic event" echo 'wakeup_latency u64 lat pid_t pid char comm[16]' > synthetic_events diff --git a/tools/testing/selftests/ftrace/test.d/trigger/inter-event/trigger-onmatch-onmax-action-hist.tc b/tools/testing/selftests/ftrace/test.d/trigger/inter-event/trigger-onmatch-onmax-action-hist.tc index dfce6932d8be..2a2ef767249e 100644 --- a/tools/testing/selftests/ftrace/test.d/trigger/inter-event/trigger-onmatch-onmax-action-hist.tc +++ b/tools/testing/selftests/ftrace/test.d/trigger/inter-event/trigger-onmatch-onmax-action-hist.tc @@ -17,6 +17,11 @@ if [ ! -f synthetic_events ]; then exit_unsupported fi +if [ ! -f events/sched/sched_process_fork/hist ]; then + echo "hist trigger is not supported" + exit_unsupported +fi + echo "Test create synthetic event" echo 'wakeup_latency u64 lat pid_t pid char comm[16]' > synthetic_events diff --git a/tools/testing/selftests/ftrace/test.d/trigger/inter-event/trigger-onmax-action-hist.tc b/tools/testing/selftests/ftrace/test.d/trigger/inter-event/trigger-onmax-action-hist.tc index 0035995c2194..98d73bfb0296 100644 --- a/tools/testing/selftests/ftrace/test.d/trigger/inter-event/trigger-onmax-action-hist.tc +++ b/tools/testing/selftests/ftrace/test.d/trigger/inter-event/trigger-onmax-action-hist.tc @@ -17,6 +17,11 @@ if [ ! -f synthetic_events ]; then exit_unsupported fi +if [ ! -f events/sched/sched_process_fork/hist ]; then + echo "hist trigger is not supported" + exit_unsupported +fi + echo "Test create synthetic event" echo 'wakeup_latency u64 lat pid_t pid char comm[16]' > synthetic_events diff --git a/tools/testing/selftests/ftrace/test.d/trigger/inter-event/trigger-snapshot-action-hist.tc b/tools/testing/selftests/ftrace/test.d/trigger/inter-event/trigger-snapshot-action-hist.tc index f546c1b66a9b..01b01b9c4e07 100644 --- a/tools/testing/selftests/ftrace/test.d/trigger/inter-event/trigger-snapshot-action-hist.tc +++ b/tools/testing/selftests/ftrace/test.d/trigger/inter-event/trigger-snapshot-action-hist.tc @@ -12,6 +12,11 @@ if [ ! -f set_event ]; then exit_unsupported fi +if [ ! -f events/sched/sched_process_fork/hist ]; then + echo "hist trigger is not supported" + exit_unsupported +fi + if [ ! -f snapshot ]; then echo "snapshot is not supported" exit_unsupported diff --git a/tools/testing/selftests/ftrace/test.d/trigger/inter-event/trigger-trace-action-hist.tc b/tools/testing/selftests/ftrace/test.d/trigger/inter-event/trigger-trace-action-hist.tc index 8021d60aafec..c3baa486aeb4 100644 --- a/tools/testing/selftests/ftrace/test.d/trigger/inter-event/trigger-trace-action-hist.tc +++ b/tools/testing/selftests/ftrace/test.d/trigger/inter-event/trigger-trace-action-hist.tc @@ -17,6 +17,11 @@ if [ ! -f synthetic_events ]; then exit_unsupported fi +if [ ! -f events/sched/sched_process_fork/hist ]; then + echo "hist trigger is not supported" + exit_unsupported +fi + grep -q "trace(<synthetic_event>" README || exit_unsupported # version issue echo "Test create synthetic event" -- 2.17.1 ^ permalink raw reply related [flat|nested] 4+ messages in thread
* Re: [PATCH v2 2/2] selftests/ftrace: Distinguish between hist and synthetic event checks 2020-05-28 19:32 ` [PATCH v2 2/2] selftests/ftrace: Distinguish between hist and synthetic event checks Tom Zanussi @ 2020-05-29 14:38 ` Masami Hiramatsu 2020-05-29 14:53 ` Steven Rostedt 0 siblings, 1 reply; 4+ messages in thread From: Masami Hiramatsu @ 2020-05-29 14:38 UTC (permalink / raw) To: Tom Zanussi; +Cc: rostedt, mhiramat, skhan, linux-kernel, linux-rt-users Hi Tom, On Thu, 28 May 2020 14:32:38 -0500 Tom Zanussi <zanussi@kernel.org> wrote: > With synthetic events now a separate config item as a result of > 'tracing: Move synthetic events to a separate file', tests that use > both need to explicitly check for hist trigger support rather than > relying on hist triggers to pull in synthetic events. > > Add an additional hist trigger check to all the trigger tests that now > require it, otherwise they'll fail if synthetic events but not hist > triggers are enabled. OK, this looks good to me. And if you don't want to repeat it, you can also put the check function into the test.d/functions. Acked-by: Masami Hiramatsu <mhiramat@kernel.org> Thank you, > > Signed-off-by: Tom Zanussi <zanussi@kernel.org> > --- > .../trigger/inter-event/trigger-field-variable-support.tc | 5 +++++ > .../trigger/inter-event/trigger-inter-event-combined-hist.tc | 5 +++++ > .../trigger/inter-event/trigger-multi-actions-accept.tc | 5 +++++ > .../trigger/inter-event/trigger-onmatch-action-hist.tc | 5 +++++ > .../trigger/inter-event/trigger-onmatch-onmax-action-hist.tc | 5 +++++ > .../test.d/trigger/inter-event/trigger-onmax-action-hist.tc | 5 +++++ > .../trigger/inter-event/trigger-snapshot-action-hist.tc | 5 +++++ > .../test.d/trigger/inter-event/trigger-trace-action-hist.tc | 5 +++++ > 8 files changed, 40 insertions(+) > > diff --git a/tools/testing/selftests/ftrace/test.d/trigger/inter-event/trigger-field-variable-support.tc b/tools/testing/selftests/ftrace/test.d/trigger/inter-event/trigger-field-variable-support.tc > index 77be6e1f6e7b..e232059a8ab2 100644 > --- a/tools/testing/selftests/ftrace/test.d/trigger/inter-event/trigger-field-variable-support.tc > +++ b/tools/testing/selftests/ftrace/test.d/trigger/inter-event/trigger-field-variable-support.tc > @@ -17,6 +17,11 @@ if [ ! -f synthetic_events ]; then > exit_unsupported > fi > > +if [ ! -f events/sched/sched_process_fork/hist ]; then > + echo "hist trigger is not supported" > + exit_unsupported > +fi > + > echo "Test field variable support" > > echo 'wakeup_latency u64 lat; pid_t pid; int prio; char comm[16]' > synthetic_events > diff --git a/tools/testing/selftests/ftrace/test.d/trigger/inter-event/trigger-inter-event-combined-hist.tc b/tools/testing/selftests/ftrace/test.d/trigger/inter-event/trigger-inter-event-combined-hist.tc > index f3eb8aacec0e..07cfcb8157b6 100644 > --- a/tools/testing/selftests/ftrace/test.d/trigger/inter-event/trigger-inter-event-combined-hist.tc > +++ b/tools/testing/selftests/ftrace/test.d/trigger/inter-event/trigger-inter-event-combined-hist.tc > @@ -17,6 +17,11 @@ if [ ! -f synthetic_events ]; then > exit_unsupported > fi > > +if [ ! -f events/sched/sched_process_fork/hist ]; then > + echo "hist trigger is not supported" > + exit_unsupported > +fi > + > echo "Test create synthetic event" > > echo 'waking_latency u64 lat pid_t pid' > synthetic_events > diff --git a/tools/testing/selftests/ftrace/test.d/trigger/inter-event/trigger-multi-actions-accept.tc b/tools/testing/selftests/ftrace/test.d/trigger/inter-event/trigger-multi-actions-accept.tc > index d281f056f980..73e413c2ca26 100644 > --- a/tools/testing/selftests/ftrace/test.d/trigger/inter-event/trigger-multi-actions-accept.tc > +++ b/tools/testing/selftests/ftrace/test.d/trigger/inter-event/trigger-multi-actions-accept.tc > @@ -17,6 +17,11 @@ if [ ! -f synthetic_events ]; then > exit_unsupported > fi > > +if [ ! -f events/sched/sched_process_fork/hist ]; then > + echo "hist trigger is not supported" > + exit_unsupported > +fi > + > echo "Test multiple actions on hist trigger" > echo 'wakeup_latency u64 lat; pid_t pid' >> synthetic_events > TRIGGER1=events/sched/sched_wakeup/trigger > diff --git a/tools/testing/selftests/ftrace/test.d/trigger/inter-event/trigger-onmatch-action-hist.tc b/tools/testing/selftests/ftrace/test.d/trigger/inter-event/trigger-onmatch-action-hist.tc > index a708f0e7858a..ebe0ad827f9f 100644 > --- a/tools/testing/selftests/ftrace/test.d/trigger/inter-event/trigger-onmatch-action-hist.tc > +++ b/tools/testing/selftests/ftrace/test.d/trigger/inter-event/trigger-onmatch-action-hist.tc > @@ -17,6 +17,11 @@ if [ ! -f synthetic_events ]; then > exit_unsupported > fi > > +if [ ! -f events/sched/sched_process_fork/hist ]; then > + echo "hist trigger is not supported" > + exit_unsupported > +fi > + > echo "Test create synthetic event" > > echo 'wakeup_latency u64 lat pid_t pid char comm[16]' > synthetic_events > diff --git a/tools/testing/selftests/ftrace/test.d/trigger/inter-event/trigger-onmatch-onmax-action-hist.tc b/tools/testing/selftests/ftrace/test.d/trigger/inter-event/trigger-onmatch-onmax-action-hist.tc > index dfce6932d8be..2a2ef767249e 100644 > --- a/tools/testing/selftests/ftrace/test.d/trigger/inter-event/trigger-onmatch-onmax-action-hist.tc > +++ b/tools/testing/selftests/ftrace/test.d/trigger/inter-event/trigger-onmatch-onmax-action-hist.tc > @@ -17,6 +17,11 @@ if [ ! -f synthetic_events ]; then > exit_unsupported > fi > > +if [ ! -f events/sched/sched_process_fork/hist ]; then > + echo "hist trigger is not supported" > + exit_unsupported > +fi > + > echo "Test create synthetic event" > > echo 'wakeup_latency u64 lat pid_t pid char comm[16]' > synthetic_events > diff --git a/tools/testing/selftests/ftrace/test.d/trigger/inter-event/trigger-onmax-action-hist.tc b/tools/testing/selftests/ftrace/test.d/trigger/inter-event/trigger-onmax-action-hist.tc > index 0035995c2194..98d73bfb0296 100644 > --- a/tools/testing/selftests/ftrace/test.d/trigger/inter-event/trigger-onmax-action-hist.tc > +++ b/tools/testing/selftests/ftrace/test.d/trigger/inter-event/trigger-onmax-action-hist.tc > @@ -17,6 +17,11 @@ if [ ! -f synthetic_events ]; then > exit_unsupported > fi > > +if [ ! -f events/sched/sched_process_fork/hist ]; then > + echo "hist trigger is not supported" > + exit_unsupported > +fi > + > echo "Test create synthetic event" > > echo 'wakeup_latency u64 lat pid_t pid char comm[16]' > synthetic_events > diff --git a/tools/testing/selftests/ftrace/test.d/trigger/inter-event/trigger-snapshot-action-hist.tc b/tools/testing/selftests/ftrace/test.d/trigger/inter-event/trigger-snapshot-action-hist.tc > index f546c1b66a9b..01b01b9c4e07 100644 > --- a/tools/testing/selftests/ftrace/test.d/trigger/inter-event/trigger-snapshot-action-hist.tc > +++ b/tools/testing/selftests/ftrace/test.d/trigger/inter-event/trigger-snapshot-action-hist.tc > @@ -12,6 +12,11 @@ if [ ! -f set_event ]; then > exit_unsupported > fi > > +if [ ! -f events/sched/sched_process_fork/hist ]; then > + echo "hist trigger is not supported" > + exit_unsupported > +fi > + > if [ ! -f snapshot ]; then > echo "snapshot is not supported" > exit_unsupported > diff --git a/tools/testing/selftests/ftrace/test.d/trigger/inter-event/trigger-trace-action-hist.tc b/tools/testing/selftests/ftrace/test.d/trigger/inter-event/trigger-trace-action-hist.tc > index 8021d60aafec..c3baa486aeb4 100644 > --- a/tools/testing/selftests/ftrace/test.d/trigger/inter-event/trigger-trace-action-hist.tc > +++ b/tools/testing/selftests/ftrace/test.d/trigger/inter-event/trigger-trace-action-hist.tc > @@ -17,6 +17,11 @@ if [ ! -f synthetic_events ]; then > exit_unsupported > fi > > +if [ ! -f events/sched/sched_process_fork/hist ]; then > + echo "hist trigger is not supported" > + exit_unsupported > +fi > + > grep -q "trace(<synthetic_event>" README || exit_unsupported # version issue > > echo "Test create synthetic event" > -- > 2.17.1 > -- Masami Hiramatsu <mhiramat@kernel.org> ^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH v2 2/2] selftests/ftrace: Distinguish between hist and synthetic event checks 2020-05-29 14:38 ` Masami Hiramatsu @ 2020-05-29 14:53 ` Steven Rostedt 0 siblings, 0 replies; 4+ messages in thread From: Steven Rostedt @ 2020-05-29 14:53 UTC (permalink / raw) To: Masami Hiramatsu; +Cc: Tom Zanussi, skhan, linux-kernel, linux-rt-users On Fri, 29 May 2020 23:38:45 +0900 Masami Hiramatsu <mhiramat@kernel.org> wrote: > Hi Tom, > > On Thu, 28 May 2020 14:32:38 -0500 > Tom Zanussi <zanussi@kernel.org> wrote: > > > With synthetic events now a separate config item as a result of > > 'tracing: Move synthetic events to a separate file', tests that use > > both need to explicitly check for hist trigger support rather than > > relying on hist triggers to pull in synthetic events. > > > > Add an additional hist trigger check to all the trigger tests that now > > require it, otherwise they'll fail if synthetic events but not hist > > triggers are enabled. > > OK, this looks good to me. And if you don't want to repeat it, > you can also put the check function into the test.d/functions. > > Acked-by: Masami Hiramatsu <mhiramat@kernel.org> Thanks Tom and Masami. I'm running tests on these now. -- Steve ^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2020-05-29 14:53 UTC | newest] Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2020-05-28 19:32 [PATCH v2 0/2] tracing: Make synthetic events a separate option Tom Zanussi 2020-05-28 19:32 ` [PATCH v2 2/2] selftests/ftrace: Distinguish between hist and synthetic event checks Tom Zanussi 2020-05-29 14:38 ` Masami Hiramatsu 2020-05-29 14:53 ` Steven Rostedt
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).