diff options
author | Michael Petlan <mpetlan@redhat.com> | 2015-11-27 08:48:09 -0500 |
---|---|---|
committer | Arnaldo Carvalho de Melo <acme@redhat.com> | 2015-11-27 10:24:46 -0500 |
commit | c4e079494f9258a3d2045779a3c9d6599c773dd9 (patch) | |
tree | c574f6737ed8a7dbfe83021b941ce6642dc1c7f5 /tools | |
parent | 3f3b1a46bfdbce97dd3f9594d8a95db82baa554b (diff) |
perf buildid-list: Show running kernel build id fix
The --kernel option of perf buildid-list tool should show the running
kernel buildid. The functionality has been lost during other changes of
the related code.
The build_id__sprintf() function should return length of the build-id
string, but it was the length of the build-id raw data instead. Due to
that, some return value checking caused that the final string was not
printed out.
With this patch the build_id__sprintf() returns the correct value, so
the --kernel option works again.
Before:
# perf buildid-list --kernel
#
After:
# perf buildid-list --kernel
972c1edab5bdc06cc224af45d510af662a3c6972
#
Signed-off-by: Michael Petlan <mpetlan@redhat.com>
Cc: Jiri Olsa <jolsa@redhat.com>
Cc: Masami Hiramatsu <masami.hiramatsu.pt@hitachi.com>
LPU-Reference: 1448632089.24573.114.camel@redhat.com
Signed-off-by: Arnaldo Carvalho de Melo <acme@redhat.com>
Diffstat (limited to 'tools')
-rw-r--r-- | tools/perf/util/build-id.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/tools/perf/util/build-id.c b/tools/perf/util/build-id.c index 217b5a60e2ab..6a7e273a514a 100644 --- a/tools/perf/util/build-id.c +++ b/tools/perf/util/build-id.c | |||
@@ -91,7 +91,7 @@ int build_id__sprintf(const u8 *build_id, int len, char *bf) | |||
91 | bid += 2; | 91 | bid += 2; |
92 | } | 92 | } |
93 | 93 | ||
94 | return raw - build_id; | 94 | return (bid - bf) + 1; |
95 | } | 95 | } |
96 | 96 | ||
97 | int sysfs__sprintf_build_id(const char *root_dir, char *sbuild_id) | 97 | int sysfs__sprintf_build_id(const char *root_dir, char *sbuild_id) |