2009-10-07 13:49:50

by Arnaldo Carvalho de Melo

Subject: [PATCH 1/1] perf tools: Up the verbose level for some really verbose stuff

Like printing every symbol created.

Cc: Frederic Weisbecker <[email protected]>
Cc: Peter Zijlstra <[email protected]>
Cc: Paul Mackerras <[email protected]>
Cc: Mike Galbraith <[email protected]>
Cc: Paul Mackerras <[email protected]>
Signed-off-by: Arnaldo Carvalho de Melo <[email protected]>
---
tools/perf/builtin-annotate.c | 4 ++--
tools/perf/builtin-report.c | 4 ++--
tools/perf/util/symbol.c | 4 ++--
3 files changed, 6 insertions(+), 6 deletions(-)

diff --git a/tools/perf/builtin-annotate.c b/tools/perf/builtin-annotate.c
index 35ed97b..8c84320 100644
--- a/tools/perf/builtin-annotate.c
+++ b/tools/perf/builtin-annotate.c
@@ -658,10 +658,10 @@ more:
if (dump_trace)
return 0;

- if (verbose >= 3)
+ if (verbose > 3)
threads__fprintf(stdout, &threads);

- if (verbose >= 2)
+ if (verbose > 2)
dsos__fprintf(stdout);

collapse__resort();
diff --git a/tools/perf/builtin-report.c b/tools/perf/builtin-report.c
index 12f8c86..3f0f1a4 100644
--- a/tools/perf/builtin-report.c
+++ b/tools/perf/builtin-report.c
@@ -1021,10 +1021,10 @@ done:
if (dump_trace)
return 0;

- if (verbose >= 3)
+ if (verbose > 3)
threads__fprintf(stdout, &threads);

- if (verbose >= 2)
+ if (verbose > 2)
dsos__fprintf(stdout);

collapse__resort();
diff --git a/tools/perf/util/symbol.c b/tools/perf/util/symbol.c
index 582ce72..a6887f9 100644
--- a/tools/perf/util/symbol.c
+++ b/tools/perf/util/symbol.c
@@ -74,7 +74,7 @@ static struct symbol *symbol__new(u64 start, u64 len, const char *name,
if (!self)
return NULL;

- if (v >= 2)
+ if (v > 2)
printf("new symbol: %016Lx [%08lx]: %s, hist: %p\n",
start, (unsigned long)len, name, self->hist);

@@ -685,7 +685,7 @@ static int dso__load_sym(struct dso *self, struct map *map, const char *name,
}

if (self->adjust_symbols) {
- if (v >= 2)
+ if (v > 2)
printf("adjusting symbol: st_value: %Lx sh_addr: %Lx sh_offset: %Lx\n",
(u64)sym.st_value, (u64)shdr.sh_addr, (u64)shdr.sh_offset);

--
1.6.2.5

2009-10-08 17:32:52

by Arnaldo Carvalho de Melo

Subject: [tip:perf/core] perf tools: Up the verbose level for some really verbose stuff

Commit-ID: da21d1b547cbaa2c026cf645753651c25d340923
Gitweb: http://git.kernel.org/tip/da21d1b547cbaa2c026cf645753651c25d340923
Author: Arnaldo Carvalho de Melo <[email protected]>
AuthorDate: Wed, 7 Oct 2009 10:49:00 -0300
Committer: Ingo Molnar <[email protected]>
CommitDate: Thu, 8 Oct 2009 19:27:10 +0200

perf tools: Up the verbose level for some really verbose stuff

Like printing every symbol created.

Signed-off-by: Arnaldo Carvalho de Melo <[email protected]>
Cc: Frederic Weisbecker <[email protected]>
Cc: Peter Zijlstra <[email protected]>
Cc: Paul Mackerras <[email protected]>
Cc: Mike Galbraith <[email protected]>
LKML-Reference: <[email protected]>
Signed-off-by: Ingo Molnar <[email protected]>
---
tools/perf/builtin-annotate.c | 4 ++--
tools/perf/builtin-report.c | 4 ++--
tools/perf/util/symbol.c | 4 ++--
3 files changed, 6 insertions(+), 6 deletions(-)

diff --git a/tools/perf/builtin-annotate.c b/tools/perf/builtin-annotate.c
index 35ed97b..8c84320 100644
--- a/tools/perf/builtin-annotate.c
+++ b/tools/perf/builtin-annotate.c
@@ -658,10 +658,10 @@ more:
if (dump_trace)
return 0;

- if (verbose >= 3)
+ if (verbose > 3)
threads__fprintf(stdout, &threads);

- if (verbose >= 2)
+ if (verbose > 2)
dsos__fprintf(stdout);

collapse__resort();
diff --git a/tools/perf/builtin-report.c b/tools/perf/builtin-report.c
index 87c4582..f57a23b 100644
--- a/tools/perf/builtin-report.c
+++ b/tools/perf/builtin-report.c
@@ -880,10 +880,10 @@ static int __cmd_report(void)
if (dump_trace)
return 0;

- if (verbose >= 3)
+ if (verbose > 3)
threads__fprintf(stdout, &threads);

- if (verbose >= 2)
+ if (verbose > 2)
dsos__fprintf(stdout);

collapse__resort();
diff --git a/tools/perf/util/symbol.c b/tools/perf/util/symbol.c
index 582ce72..a6887f9 100644
--- a/tools/perf/util/symbol.c
+++ b/tools/perf/util/symbol.c
@@ -74,7 +74,7 @@ static struct symbol *symbol__new(u64 start, u64 len, const char *name,
if (!self)
return NULL;

- if (v >= 2)
+ if (v > 2)
printf("new symbol: %016Lx [%08lx]: %s, hist: %p\n",
start, (unsigned long)len, name, self->hist);

@@ -685,7 +685,7 @@ static int dso__load_sym(struct dso *self, struct map *map, const char *name,
}

if (self->adjust_symbols) {
- if (v >= 2)
+ if (v > 2)
printf("adjusting symbol: st_value: %Lx sh_addr: %Lx sh_offset: %Lx\n",
(u64)sym.st_value, (u64)shdr.sh_addr, (u64)shdr.sh_offset);