Commit-ID:  69c71252298acad7a25e9499929f723790f48e26
Gitweb:     https://git.kernel.org/tip/69c71252298acad7a25e9499929f723790f48e26
Author:     Arnaldo Carvalho de Melo <[email protected]>
AuthorDate: Thu, 26 Oct 2017 09:51:13 -0300
Committer:  Arnaldo Carvalho de Melo <[email protected]>
CommitDate: Fri, 27 Oct 2017 09:10:09 -0300

perf script: Add a few missing conversions to fprintf style

In a1a587073ccd ("perf script: Use fprintf like printing uniformly")
there were a few cases that were missed, fix it.

Reported-by: yuzhoujian <[email protected]>
Cc: Adrian Hunter <[email protected]>
Cc: David Ahern <[email protected]>
Cc: Jiri Olsa <[email protected]>
Cc: Namhyung Kim <[email protected]>
Cc: Wang Nan <[email protected]>
Link: http://lkml.kernel.org/n/[email protected]
Signed-off-by: Arnaldo Carvalho de Melo <[email protected]>
---
 tools/perf/builtin-script.c | 9 ++++-----
 1 file changed, 4 insertions(+), 5 deletions(-)

diff --git a/tools/perf/builtin-script.c b/tools/perf/builtin-script.c
index a3add2c..db4ff15 100644
--- a/tools/perf/builtin-script.c
+++ b/tools/perf/builtin-script.c
@@ -1447,7 +1447,7 @@ static void process_event(struct perf_script *script,
        perf_sample__fprintf_start(sample, thread, evsel, fp);
 
        if (PRINT_FIELD(PERIOD))
-               printf("%10" PRIu64 " ", sample->period);
+               fprintf(fp, "%10" PRIu64 " ", sample->period);
 
        if (PRINT_FIELD(EVNAME)) {
                const char *evname = perf_evsel__name(evsel);
@@ -1455,8 +1455,7 @@ static void process_event(struct perf_script *script,
                if (!script->name_width)
                        script->name_width = 
perf_evlist__max_name_len(script->session->evlist);
 
-               printf("%*s: ", script->name_width,
-                      evname ? evname : "[unknown]");
+               fprintf(fp, "%*s: ", script->name_width, evname ?: "[unknown]");
        }
 
        if (print_flags)
@@ -1513,8 +1512,8 @@ static void process_event(struct perf_script *script,
        perf_sample__fprintf_insn(sample, attr, thread, machine, fp);
 
        if (PRINT_FIELD(PHYS_ADDR))
-               printf("%16" PRIx64, sample->phys_addr);
-       printf("\n");
+               fprintf(fp, "%16" PRIx64, sample->phys_addr);
+       fprintf(fp, "\n");
 }
 
 static struct scripting_ops    *scripting_ops;

Reply via email to