[lkml]   [2009]   [Jun]   [8]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
Messages in this thread
SubjectRe: [PATCH 2/2] convert to syscall tracepoints

* Jason Baron <> wrote:

> +#ifdef __NR_time
> +trace_event_syscall(1, time, time_t __user *, tloc);
> +#endif
> +
> +#ifdef __NR_stime
> +trace_event_syscall(1, stime, time_t __user *, tptr);
> +#endif
> +
> +#ifdef __NR_gettimeofday
> +trace_event_syscall(2, gettimeofday, struct timeval __user *, tv, struct timezone __user *, tz);
> +#endif

This could be reduced to a single line: just add a Kconfig entry
(say TRACE_SYSCALL_TRACEPOINTS) wether an arch supports syscall
tracepoints, enable it on a sane arch, make sure it has all the
syscalls and list them ...

As more architectures turn on SYSCALL_TRACEPOINTS, they'll have to
resolve any deviations in syscall entry points. Ideally we'd have
one generic table that covers 95% of all syscalls, and the remaining
5% in some architecture specific #ifdef section.

But, more generally, i'm not at all convinced that we need _any_ of
this enumeration. Look how much the above lines duplicate
DEFINE_SYSCALL macros. Why arent those macros re-used?

We dont need too smart pretty-printing i think - we only want to
know the field size and the field name - nothing else. Duplicating
all those definitions looks outright wrong to me. Do we really,
really, really have to do it?


 \ /
  Last update: 2009-06-08 22:43    [W:0.037 / U:2.796 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site