Re: [PATCH v2 1/2] tracing: Move tracepoint callbacks into DEFINE

[Josh Stone]

On 08/21/2009 10:52 AM, Jason Baron wrote:
> this means that when CONFIG_EVENT_TRACING is set, the 'generic' syscall
> enter/exit will show up as events in the debugfs, but enabling them
> wouldn't do anything. I think we should simply drop the
> 'CONFIG_FTRACE_SYSCALLS' 'ifdef' and 'else' clause. That will give us
> what we want - tying these callbacks directly to tracepoint.

But only x86 and s390 have TIF_SYSCALL_FTRACE, while kernel/tracepoint.c
must still compile everywhere that has CONFIG_TRACEPOINTS=y.

Maybe it would be better to make that #ifdef TIF_SYSCALL_FTRACE, and
then also #ifdef the TRACE_EVENT declaration, so it will only show up on
kernels that actually support it.

Also, since this event is now usable outside of ftrace, would you object
to renaming the flag TIF_SYSCALL_TRACEPOINT?

Josh
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/