diff options
author | Arnaldo Carvalho de Melo <[email protected]> | 2023-06-02 15:38:25 -0300 |
---|---|---|
committer | Arnaldo Carvalho de Melo <[email protected]> | 2023-06-05 10:36:58 -0300 |
commit | 16203e9cd01896b4244100a8e3fb9f6e612ab2b1 (patch) | |
tree | 4f0b46a35e42d22437d0ad0053d4e2a657360de6 | |
parent | fe8e04348727f992f6fce3709639fb6d92a81137 (diff) |
perf bench: Add missing setlocale() call to allow usage of %'d style formatting
Without this we were not getting the thousands separator for big
numbers.
Noticed while developing 'perf bench uprobe', but the use of %' predates
that, for instance 'perf bench syscall' uses it.
Before:
# perf bench uprobe all
# Running uprobe/baseline benchmark...
# Executed 1000 usleep(1000) calls
Total time: 1054082243ns
1054082.243000 nsecs/op
#
After:
# perf bench uprobe all
# Running uprobe/baseline benchmark...
# Executed 1,000 usleep(1000) calls
Total time: 1,053,715,144ns
1,053,715.144000 nsecs/op
#
Fixes: c2a08203052f8975 ("perf bench: Add basic syscall benchmark")
Cc: Adrian Hunter <[email protected]>
Cc: Andre Fredette <[email protected]>
Cc: Clark Williams <[email protected]>
Cc: Dave Tucker <[email protected]>
Cc: Davidlohr Bueso <[email protected]>
Cc: Derek Barbosa <[email protected]>
Cc: Ian Rogers <[email protected]>
Cc: Jiri Olsa <[email protected]>
Cc: Namhyung Kim <[email protected]>
Cc: Tiezhu Yang <[email protected]>
Link: https://lore.kernel.org/lkml/[email protected]
Signed-off-by: Arnaldo Carvalho de Melo <[email protected]>
-rw-r--r-- | tools/perf/builtin-bench.c | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/tools/perf/builtin-bench.c b/tools/perf/builtin-bench.c index 58f1cfe1eb34..db435b791a09 100644 --- a/tools/perf/builtin-bench.c +++ b/tools/perf/builtin-bench.c @@ -21,6 +21,7 @@ #include "builtin.h" #include "bench/bench.h" +#include <locale.h> #include <stdio.h> #include <stdlib.h> #include <string.h> @@ -260,6 +261,7 @@ int cmd_bench(int argc, const char **argv) /* Unbuffered output */ setvbuf(stdout, NULL, _IONBF, 0); + setlocale(LC_ALL, ""); if (argc < 2) { /* No collection specified. */ |