Messages in this thread | | | Date | Fri, 4 Oct 2013 12:25:24 -0300 | From | Arnaldo Carvalho de Melo <> | Subject | Re: [PATCH] perf session: Fix infinite loop on invalid perf.data file |
| |
Em Tue, Oct 01, 2013 at 10:59:06AM -0600, David Ahern escreveu: > On 10/1/13 10:54 AM, Arnaldo Carvalho de Melo wrote: > >Em Tue, Oct 01, 2013 at 05:33:09PM +0200, Ingo Molnar escreveu: > >>* David Ahern <dsahern@gmail.com> wrote: > >>>On 10/1/13 8:21 AM, Ingo Molnar wrote: > >I liked this last one: > > > >> WARNING: The perf.data file's data size field is 0 which is unexpected. > >> Was the 'perf record' command properly terminated? > > > >Can I patch that up into Namhyung's latest patch? > > > >Sonny, David, from your replies I think I can add Tested-by: tags for > >both of you? > > I'm fine with the last message. And yes a Tested by
Got to process this now, changing it a bit to replace "perf.data" with "%s" + session->filename, ack?
Otherwise the user may have a 'perf.data' file and despite having informed -i with a different file name still get confused... ;-)
- Arnaldo
| |