[<prev] [next>] [thread-next>] [day] [month] [year] [list]
Message-Id: <1440596813-12844-1-git-send-email-jolsa@kernel.org>
Date: Wed, 26 Aug 2015 15:46:42 +0200
From: Jiri Olsa <jolsa@...nel.org>
To: Arnaldo Carvalho de Melo <acme@...nel.org>
Cc: lkml <linux-kernel@...r.kernel.org>,
David Ahern <dsahern@...il.com>,
Ingo Molnar <mingo@...nel.org>,
Namhyung Kim <namhyung@...nel.org>,
Peter Zijlstra <a.p.zijlstra@...llo.nl>,
Matt Fleming <matt@...eblueprint.co.uk>,
Raphaƫl Beamonte <raphael.beamonte@...il.com>
Subject: [RFC 00/11] perf tools: Enhance parsing events tracepoint error output
hi,
enhancing parsing events tracepoint error output. Adding
more verbose output when the tracepoint is not found or
the tracing event path cannot be access.
$ sudo perf record -e sched:sched_krava ls
event syntax error: 'sched:sched_krava'
\___ unknown tracepoint
Error: File /sys/kernel/debug/tracing//tracing/events/sched/sched_krava not found.
Hint: Perhaps this kernel misses some CONFIG_ setting to enable this feature?.
Run 'perf list' for a list of valid events
...
$ perf record -e sched:sched_krava ls
event syntax error: 'sched:sched_krava'
\___ can't access trace events
Error: No permissions to read /sys/kernel/debug/tracing//tracing/events/sched/sched_krava
Hint: Try 'sudo mount -o remount,mode=755 /sys/kernel/debug'
Run 'perf list' for a list of valid events
...
I changed api/fs tracefs/debugfs related code and I'm not completely
sure what were the long term intentions with this code, so please
comment ;-)
jirka
---
Jiri Olsa (11):
tools: Add err.h with ERR_PTR PTR_ERR interface
perf tools: Add tracing_path and remove unneeded functions
perf tools: Do not change lib/api/fs/debugfs directly
perf tools: Move debugfs__strerror_open into util.c object
perf tools: Move tracing_path stuff under same namespace
perf tools: Move tracing_path interface into trace-event-path.c
perf tools: Make tracing_path_strerror_open message generic
perf tools: Do not export debugfs_mountpoint and tracefs_mountpoint
perf tools: Propagate error info for the tracepoint parsing
perf tools: Propagate error info from tp_format
perf tools: Enhance parsing events tracepoint error output
tools/include/linux/err.h | 28 +++++++++++++++++++
tools/lib/api/fs/debugfs.c | 53 +-----------------------------------
tools/lib/api/fs/debugfs.h | 5 ----
tools/lib/api/fs/tracefs.c | 2 +-
tools/lib/api/fs/tracefs.h | 2 --
tools/perf/builtin-trace.c | 4 +--
tools/perf/perf.c | 11 ++++----
tools/perf/tests/openat-syscall-all-cpus.c | 2 ++
tools/perf/tests/openat-syscall.c | 2 ++
tools/perf/util/Build | 1 +
tools/perf/util/evsel.c | 8 ++++--
tools/perf/util/parse-events.c | 58 ++++++++++++++++++++++++++++++----------
tools/perf/util/parse-events.h | 3 ++-
tools/perf/util/parse-events.y | 16 ++++++-----
tools/perf/util/trace-event-path.c | 129 ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
tools/perf/util/trace-event.c | 7 +++--
tools/perf/util/trace-event.h | 12 +++++++++
tools/perf/util/util.c | 119 ---------------------------------------------------------------------------------
tools/perf/util/util.h | 8 ------
19 files changed, 250 insertions(+), 220 deletions(-)
create mode 100644 tools/include/linux/err.h
create mode 100644 tools/perf/util/trace-event-path.c
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists