| From: Arnaldo Carvalho de Melo <acme@redhat.com> |
| Date: Wed, 30 Mar 2016 16:51:17 -0300 |
| Subject: perf trace: Do not process PERF_RECORD_LOST twice |
| |
| commit 3ed5ca2efff70e9f589087c2013789572901112d upstream. |
| |
| We catch this record to provide a visual indication that events are |
| getting lost, then call the default method to allow extra logging shared |
| with the other tools to take place. |
| |
| This extra logging was done twice because we were continuing to the |
| "default" clause where machine__process_event() will end up calling |
| machine__process_lost_event() again, fix it. |
| |
| Cc: Adrian Hunter <adrian.hunter@intel.com> |
| Cc: David Ahern <dsahern@gmail.com> |
| Cc: Jiri Olsa <jolsa@kernel.org> |
| Cc: Namhyung Kim <namhyung@kernel.org> |
| Cc: Wang Nan <wangnan0@huawei.com> |
| Link: http://lkml.kernel.org/n/tip-wus2zlhw3qo24ye84ewu4aqw@git.kernel.org |
| Signed-off-by: Arnaldo Carvalho de Melo <acme@redhat.com> |
| Signed-off-by: Ben Hutchings <ben@decadent.org.uk> |
| --- |
| tools/perf/builtin-trace.c | 1 + |
| 1 file changed, 1 insertion(+) |
| |
| --- a/tools/perf/builtin-trace.c |
| +++ b/tools/perf/builtin-trace.c |
| @@ -1359,6 +1359,7 @@ static int trace__process_event(struct t |
| color_fprintf(trace->output, PERF_COLOR_RED, |
| "LOST %" PRIu64 " events!\n", event->lost.lost); |
| ret = machine__process_lost_event(machine, event, sample); |
| + break; |
| default: |
| ret = machine__process_event(machine, event, sample); |
| break; |