Messages in this thread | | | Date | Fri, 25 Jan 2013 08:53:30 +0100 | From | Ingo Molnar <> | Subject | Re: [PATCH] tools lib traceevent: Handle dynamic array's element size properly |
| |
* Arnaldo Carvalho de Melo <acme@redhat.com> 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
| |