Messages in this thread Patch in this message | | | From | He Kuang <> | Subject | [PATCH 1/4] perf record: Use %ld for long type sample counter | Date | Wed, 1 Jul 2015 09:05:36 +0000 |
| |
Since we post-process and count (long rec->samples) to show precise number of samples instead of estimate on (u64 rec->bytes_written), the format string of that should be changed to %ld accordingly, otherwise the value don't show right on 32bit machine.
Before this patch:
$ perf record -e syscalls:sys_enter_write -- dd if=/dev/zero of=/dev/null bs=4k count=1000 1000+0 records in 1000+0 records out [ perf record: Woken up 1 times to write data ] [ perf record: Captured and wrote 0.077 MB perf.data (1 samples) ]
After this patch:
$ perf record -e syscalls:sys_enter_write -- dd if=/dev/zero of=/dev/null bs=4k count=1000 1000+0 records in 1000+0 records out [ perf record: Woken up 1 times to write data ] [ perf record: Captured and wrote 0.077 MB perf.data (1001 samples) ]
Signed-off-by: He Kuang <hekuang@huawei.com> --- tools/perf/builtin-record.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/tools/perf/builtin-record.c b/tools/perf/builtin-record.c index de165a1..65e632d 100644 --- a/tools/perf/builtin-record.c +++ b/tools/perf/builtin-record.c @@ -737,7 +737,7 @@ out_child: if (rec->samples && !rec->opts.full_auxtrace) scnprintf(samples, sizeof(samples), - " (%" PRIu64 " samples)", rec->samples); + " (%ld samples)", rec->samples); else samples[0] = '\0'; -- 1.8.5.2
| |