Re: Another Obsolete Fix me in trace.h?

From: Radim KrÄmÃÅ
Date: Wed Nov 26 2014 - 07:40:52 EST


2014-11-24 22:49+0100, Radim KrÄmÃÅ:
> 2014-11-24 16:19-0500, Steven Rostedt:
> > That wouldn't be too hard to implement.
>
> I'll look at the patch tommorrow.

The kernel part is trivial.
Most of the code is going to be in tools/lib/traceevent/event-parse.c.

I wasn't sure whether to
1) define new 'enum print_arg_type' for our function and do exactly what
other __print* did
2) extend existing PRINT_FUNC with variable arguments and register it as
a "global" event handler

as (2) makes more sense to me, but we already had it when some __print*
was implemented ... (and I didn't want to dig too deep into the project)
--
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/