Re: [PATCH] tools lib traceevent: Handle dynamic array's elementsize properly

From: Ingo Molnar
Date: Fri Jan 25 2013 - 02:53:21 EST



* Arnaldo Carvalho de Melo <acme@xxxxxxxxxx> wrote:

> Em Tue, Jan 22, 2013 at 02:16:47PM +0100, Jiri Olsa escreveu:
> > On Mon, Jan 21, 2013 at 11:45:42PM -0500, Steven Rostedt wrote:
> > > This is only used in this function (the field_dynamic_elem_size() is
> > > only called here). Can we not add the field->type_dyn, and just use a
> > > local variable here. You just need to make sure you free it correctly.
> >
> > ook, will send v2
>
> Waiting for this fix, just got 'perf test' crashing on it again :-)
>
> Till then I implemented 'perf test --skip':

Hm, I think perf test should fork() a testing thread and be
robust against a child catching a SIGSEGV?

That would also protect against a test corrupting perf test
itself.

Thanks,

Ingo
--
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/