diff options
author | Steven Rostedt <srostedt@redhat.com> | 2009-10-14 15:43:41 -0400 |
---|---|---|
committer | Ingo Molnar <mingo@elte.hu> | 2009-10-15 04:42:39 -0400 |
commit | 0d1da915c76838c9ee7af7cdefbcb2bae9424161 (patch) | |
tree | 5aac3e7f4c9c44814e6a1833e5d5f02b0b7ae9d7 /tools/perf | |
parent | ffa1895561645103d8f8059b35d9c06e6eeead2e (diff) |
perf tools: Handle both versions of ftrace output
The ftrace output events can have either arguments or no
arguments. The parser needs to be able to handle both.
Signed-off-by: Steven Rostedt <srostedt@redhat.com>
Cc: Peter Zijlstra <peterz@infradead.org>
Cc: Frederic Weisbecker <fweisbec@gmail.com>
Cc: Arnaldo Carvalho de Melo <acme@redhat.com>
LKML-Reference: <20091014194359.790221427@goodmis.org>
Signed-off-by: Ingo Molnar <mingo@elte.hu>
Diffstat (limited to 'tools/perf')
-rw-r--r-- | tools/perf/util/trace-event-parse.c | 5 |
1 files changed, 4 insertions, 1 deletions
diff --git a/tools/perf/util/trace-event-parse.c b/tools/perf/util/trace-event-parse.c index 93a82fead958..c174765d4056 100644 --- a/tools/perf/util/trace-event-parse.c +++ b/tools/perf/util/trace-event-parse.c | |||
@@ -1819,7 +1819,7 @@ static int event_read_print(struct event *event) | |||
1819 | if (ret < 0) | 1819 | if (ret < 0) |
1820 | return -1; | 1820 | return -1; |
1821 | 1821 | ||
1822 | return 0; | 1822 | return ret; |
1823 | 1823 | ||
1824 | fail: | 1824 | fail: |
1825 | free_token(token); | 1825 | free_token(token); |
@@ -3088,6 +3088,9 @@ int parse_ftrace_file(char *buf, unsigned long size) | |||
3088 | if (ret < 0) | 3088 | if (ret < 0) |
3089 | die("failed to read ftrace event print fmt"); | 3089 | die("failed to read ftrace event print fmt"); |
3090 | 3090 | ||
3091 | /* New ftrace handles args */ | ||
3092 | if (ret > 0) | ||
3093 | return 0; | ||
3091 | /* | 3094 | /* |
3092 | * The arguments for ftrace files are parsed by the fields. | 3095 | * The arguments for ftrace files are parsed by the fields. |
3093 | * Set up the fields as their arguments. | 3096 | * Set up the fields as their arguments. |