lkml.org 
[lkml]   [2013]   [Jun]   [18]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
SubjectRe: [PATCH 3/3] tracing/perf: Avoid perf_trace_buf_*() in perf_trace_##call() when possible
From
Date
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); \
> }




\
 
 \ /
  Last update: 2013-06-18 22:41    [W:0.092 / U:1.796 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site