Em Tue, Jan 05, 2016 at 07:58:34PM +0900, Namhyung Kim escreveu: > When the 'trace_fields' sort key is used explicitly for non-tracepoint > events, it'll get segfault since it assumed evsel->tp_format was set. > Skip those events in add_all_dynamic_fields(). > > Acked-by: Jiri Olsa <jo...@kernel.org> > Signed-off-by: Namhyung Kim <namhy...@kernel.org>
Merged with the patch introducing this problem, that I hadn't sent to Ingo, - Arnaldo > --- > tools/perf/util/sort.c | 3 +++ > 1 file changed, 3 insertions(+) > > diff --git a/tools/perf/util/sort.c b/tools/perf/util/sort.c > index e558e87cafaf..59c4c8586d79 100644 > --- a/tools/perf/util/sort.c > +++ b/tools/perf/util/sort.c > @@ -1955,6 +1955,9 @@ static int add_all_dynamic_fields(struct perf_evlist > *evlist, bool raw_trace) > struct perf_evsel *evsel; > > evlist__for_each(evlist, evsel) { > + if (evsel->attr.type != PERF_TYPE_TRACEPOINT) > + continue; > + > ret = add_evsel_fields(evsel, raw_trace); > if (ret < 0) > return ret; > -- > 2.6.4 -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/