perf buildid-list: Always show the DSO name

Porcelain can ignore it, humans can make more sense of it.

Suggested-by: Frederic Weisbecker <fweisbec@gmail.com>
Suggested-by: Ingo Molnar <mingo@elte.hu>
Suggested-by: Peter Zijlstra <peterz@infradead.org>
Signed-off-by: Arnaldo Carvalho de Melo <acme@redhat.com>
Cc: Frederic Weisbecker <fweisbec@gmail.com>
Cc: Mike Galbraith <efault@gmx.de>
Cc: Paul Mackerras <paulus@samba.org>
Cc: Peter Zijlstra <peterz@infradead.org>
LKML-Reference: <1258415125-15019-2-git-send-email-acme@infradead.org>
Signed-off-by: Ingo Molnar <mingo@elte.hu>

diff --git a/tools/perf/Documentation/perf-buildid-list.txt b/tools/perf/Documentation/perf-buildid-list.txt
index abab34e055769c42d2f0438e824d17cee60658e4..01b642c0bf8f974aedf2560df4e91cc5dd514a37 100644 (file)
--- a/tools/perf/Documentation/perf-buildid-list.txt
+++ b/tools/perf/Documentation/perf-buildid-list.txt
@@ -26,7 +26,7 @@ OPTIONS
        Don't do ownership validation.
 -v::
 --verbose::
-       Be more verbose, showing the name of the DSOs after the buildids.
+       Be more verbose.
 
 SEE ALSO
 --------

diff --git a/tools/perf/builtin-buildid-list.c b/tools/perf/builtin-buildid-list.c
index 2e377e1be43582c2adea29c2d5a3a3d645394d9a..7dee9d19ab7a8036494373fb71e2258f6c225fa1 100644 (file)
--- a/tools/perf/builtin-buildid-list.c
+++ b/tools/perf/builtin-buildid-list.c
@@ -28,7 +28,7 @@ static const struct option options[] = {
                    "input file name"),
        OPT_BOOLEAN('f', "force", &force, "don't complain, do it"),
        OPT_BOOLEAN('v', "verbose", &verbose,
-                   "be more verbose (show counter open errors, etc)"),
+                   "be more verbose"),
        OPT_END()
 };
 

diff --git a/tools/perf/util/symbol.c b/tools/perf/util/symbol.c
index 53de9c4488d8fd76b9a5f1dbf38d8a015d03f4b6..1b77e81b38de2fe7008fc09cd51aa753f4959ff9 100644 (file)
--- a/tools/perf/util/symbol.c
+++ b/tools/perf/util/symbol.c
@@ -1442,10 +1442,7 @@ size_t dsos__fprintf_buildid(FILE *fp)
 
        list_for_each_entry(pos, &dsos, node) {
                ret += dso__fprintf_buildid(pos, fp);
-               if (verbose)
-                       ret += fprintf(fp, " %s\n", pos->long_name);
-               else
-                       ret += fprintf(fp, "\n");
+               ret += fprintf(fp, " %s\n", pos->long_name);
        }
        return ret;
 }