On Mon, Jan 26, 2015 at 06:38:23PM +0800, Wang Nan wrote: > Introduce a priv field to 'struct format_field' for futher expansion. > > (In https://lkml.org/lkml/2015/1/21/383 , Jiri Olsa gives a suggestion > about changing lib traceevent to solve a bug of perf-convert-to-ctf, > which is related to duplicated field names. I think his suggestion > should be something like this patch. )
Hi Steven, any feedback on this one? thanks, jirka > > Signed-off-by: Wang Nan <[email protected]> > --- > tools/lib/traceevent/event-parse.c | 2 ++ > tools/lib/traceevent/event-parse.h | 2 ++ > 2 files changed, 4 insertions(+) > > diff --git a/tools/lib/traceevent/event-parse.c > b/tools/lib/traceevent/event-parse.c > index cf3a44b..5f76003 100644 > --- a/tools/lib/traceevent/event-parse.c > +++ b/tools/lib/traceevent/event-parse.c > @@ -5909,6 +5909,8 @@ static void free_format_fields(struct format_field > *field) > free(field->type); > free(field->name); > free(field); > + if (field->destroy_priv) > + field->destroy_priv(field); > field = next; > } > } > diff --git a/tools/lib/traceevent/event-parse.h > b/tools/lib/traceevent/event-parse.h > index 7a3873f..928d801 100644 > --- a/tools/lib/traceevent/event-parse.h > +++ b/tools/lib/traceevent/event-parse.h > @@ -190,6 +190,8 @@ struct format_field { > unsigned int arraylen; > unsigned int elementsize; > unsigned long flags; > + void *priv; > + void (*destroy_priv)(struct format_field *); > }; > > struct format { > -- > 1.8.4 > -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
