Messages in this thread | | | Subject | Re: [PATCH 3/3] tracing/perf: Avoid perf_trace_buf_*() in perf_trace_##call() when possible | From | Steven Rostedt <> | Date | Tue, 18 Jun 2013 16:02:53 -0400 |
| |
On Tue, 2013-06-18 at 21:22 +0200, Oleg Nesterov wrote: > perf_trace_buf_prepare() + perf_trace_buf_submit(task => NULL) > make no sense if hlist_empty(head). Change perf_trace_##call() > to check ->perf_events beforehand and do nothing if it is empty. > > However, we can only do this if __task == NULL, so we also add > the __builtin_constant_p(__task) check. > > Signed-off-by: Oleg Nesterov <oleg@redhat.com> > --- > include/trace/ftrace.h | 7 ++++++- > 1 files changed, 6 insertions(+), 1 deletions(-) > > diff --git a/include/trace/ftrace.h b/include/trace/ftrace.h > index 8886877..04455b8 100644 > --- a/include/trace/ftrace.h > +++ b/include/trace/ftrace.h > @@ -663,6 +663,12 @@ perf_trace_##call(void *__data, proto) \ > int rctx; \ > \ > __data_size = ftrace_get_offsets_##call(&__data_offsets, args); \ > + \ > + head = this_cpu_ptr(event_call->perf_events); \ > + if (__builtin_constant_p(!__task) && !__task && \
I'm trying to wrap my head around this:
__builtin_constant_p(!task)
is this the same as:
!__builtin_constant_p(task)
Or is it the same as:
__builtin_constant_p(task)
?
Because that '!' is confusing the heck out of me.
If !task is a constant, wouldn't task be a constant too, and if task is not a constant then I would also assume !task is not a constant as well.
If this is the case, can we nuke the '!' from the builtin_consant_p(). The code is confusing enough as is. Or is it that the code is very confusing and in keeping with the coding style, you are trying to come up with new ways of adding to the confusion.
Or is this your way to confuse me as much as my code has confused you? ;-)
-- Steve
> + hlist_empty(head)) \ > + return; \ > + \ > __entry_size = ALIGN(__data_size + sizeof(*entry) + sizeof(u32),\ > sizeof(u64)); \ > __entry_size -= sizeof(u32); \ > @@ -677,7 +683,6 @@ perf_trace_##call(void *__data, proto) \ > \ > { assign; } \ > \ > - head = this_cpu_ptr(event_call->perf_events); \ > perf_trace_buf_submit(entry, __entry_size, rctx, __addr, \ > __count, &__regs, head, __task); \ > }
| |