diff options
author | Jiri Olsa <jolsa@kernel.org> | 2014-12-01 14:06:23 -0500 |
---|---|---|
committer | Arnaldo Carvalho de Melo <acme@redhat.com> | 2014-12-09 07:14:34 -0500 |
commit | 498922adf1173ddeebd155f82646d4a9d518d606 (patch) | |
tree | bf7d80d6c7438e5de0658bb47dfb4a4dabf58ef1 /tools/perf/util/build-id.c | |
parent | cfd31d85bb42b96449157bd57c638dc779070753 (diff) |
perf buildid-cache: Remove extra debugdir variables
There's no need to copy over the buildid_dir into separate variable with
no change.
This is leftover from commit:
45de34bbe3e1 perf buildid: add perfconfig option to specify buildid cache dir
that added global buildid_dir variable that holds cache directory, but
did not cleanup the debugdir copies.
Signed-off-by: Jiri Olsa <jolsa@kernel.org>
Cc: Corey Ashford <cjashfor@linux.vnet.ibm.com>
Cc: David Ahern <dsahern@gmail.com>
Cc: Frederic Weisbecker <fweisbec@gmail.com>
Cc: Ingo Molnar <mingo@kernel.org>
Cc: Namhyung Kim <namhyung@kernel.org>
Cc: Paul Mackerras <paulus@samba.org>
Cc: Peter Zijlstra <a.p.zijlstra@chello.nl>
Cc: Stephane Eranian <eranian@google.com>
Cc: Steven Rostedt <rostedt@goodmis.org>
Link: http://lkml.kernel.org/r/1417460789-13874-3-git-send-email-jolsa@kernel.org
Signed-off-by: Arnaldo Carvalho de Melo <acme@redhat.com>
Diffstat (limited to 'tools/perf/util/build-id.c')
-rw-r--r-- | tools/perf/util/build-id.c | 9 |
1 files changed, 3 insertions, 6 deletions
diff --git a/tools/perf/util/build-id.c b/tools/perf/util/build-id.c index e8d79e5bfaf7..0c72680a977f 100644 --- a/tools/perf/util/build-id.c +++ b/tools/perf/util/build-id.c | |||
@@ -410,21 +410,18 @@ int perf_session__cache_build_ids(struct perf_session *session) | |||
410 | { | 410 | { |
411 | struct rb_node *nd; | 411 | struct rb_node *nd; |
412 | int ret; | 412 | int ret; |
413 | char debugdir[PATH_MAX]; | ||
414 | 413 | ||
415 | if (no_buildid_cache) | 414 | if (no_buildid_cache) |
416 | return 0; | 415 | return 0; |
417 | 416 | ||
418 | snprintf(debugdir, sizeof(debugdir), "%s", buildid_dir); | 417 | if (mkdir(buildid_dir, 0755) != 0 && errno != EEXIST) |
419 | |||
420 | if (mkdir(debugdir, 0755) != 0 && errno != EEXIST) | ||
421 | return -1; | 418 | return -1; |
422 | 419 | ||
423 | ret = machine__cache_build_ids(&session->machines.host, debugdir); | 420 | ret = machine__cache_build_ids(&session->machines.host, buildid_dir); |
424 | 421 | ||
425 | for (nd = rb_first(&session->machines.guests); nd; nd = rb_next(nd)) { | 422 | for (nd = rb_first(&session->machines.guests); nd; nd = rb_next(nd)) { |
426 | struct machine *pos = rb_entry(nd, struct machine, rb_node); | 423 | struct machine *pos = rb_entry(nd, struct machine, rb_node); |
427 | ret |= machine__cache_build_ids(pos, debugdir); | 424 | ret |= machine__cache_build_ids(pos, buildid_dir); |
428 | } | 425 | } |
429 | return ret ? -1 : 0; | 426 | return ret ? -1 : 0; |
430 | } | 427 | } |