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 <[email protected]> --- 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 -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
