Messages in this thread | | | Subject | Re: [PATCH 0/2] tracing: Fix synthetic event parser | From | Tom Zanussi <> | Date | Mon, 22 Oct 2018 15:14:53 -0500 |
| |
Hi Masami,
On Mon, 2018-10-22 at 00:07 +0900, Masami Hiramatsu wrote: > Hi, > > I found another bug in synthetic event. This is a small fix, but > confusingly, there is also a bug in a test case. > > Steve, since the testcase bugfix ([2/2]) breaks the test result > unless corresponding fix ([1/2]), I would like to ask you to send > these fixes from your tree. > Shuah, could you give this your ack? >
Yes, I think the silent failing mode is confusing to users, and so these changes make more sense.
Acked-by: Tom Zanussi <zanussi@linux.intel.com> Tested-by: Tom Zanussi <zanussi@linux.intel.com>
Thanks,
Tom
> Thank you, > > --- > > Masami Hiramatsu (2): > tracing: Return -ENOENT if there is no target synthetic event > selftests/ftrace: Fix synthetic event test to delete event > correctly > > > kernel/trace/trace_events_hist.c | 4 +++- > .../trigger-synthetic-event-createremove.tc | 12 ++++++-- > ---- > 2 files changed, 9 insertions(+), 7 deletions(-) > > -- > Masami Hiramatsu (Linaro) <mhiramat@kernel.org>
| |