diff options
author | Ingo Molnar <mingo@kernel.org> | 2015-10-19 10:04:21 +0200 |
---|---|---|
committer | Arnaldo Carvalho de Melo <acme@redhat.com> | 2015-10-19 21:03:31 +0200 |
commit | 6db175c7333e22ee818373cbea067e3eaa0236f7 (patch) | |
tree | fbbe89782cc7b6baa3e838060f5fed41827c6f25 /tools/perf/builtin-bench.c | |
parent | perf bench: Rename 'mem-memcpy.c' => 'mem-functions.c' (diff) | |
download | linux-6db175c7333e22ee818373cbea067e3eaa0236f7.tar.xz linux-6db175c7333e22ee818373cbea067e3eaa0236f7.zip |
perf bench: Remove the prefaulting complication from 'perf bench mem mem*'
So 'perf bench mem memcpy/memset' has elaborate code to measure
memcpy()/memset() performance both with freshly allocated buffers (which
includes initial page fault overhead) and with preallocated buffers.
But the thing is, the resulting bandwidth results are mostly
meaningless, because page faults dominate so much of the cost.
It might make sense to measure cache cold vs. cache hot performance, but
the code does not do this.
So remove this complication, and always prefault the ranges before using
them.
Signed-off-by: Ingo Molnar <mingo@kernel.org>
Cc: David Ahern <dsahern@gmail.com>
Cc: Hitoshi Mitake <mitake@dcl.info.waseda.ac.jp>
Cc: Jiri Olsa <jolsa@redhat.com>
Cc: Linus Torvalds <torvalds@linux-foundation.org>
Cc: Namhyung Kim <namhyung@kernel.org>
Cc: Peter Zijlstra <peterz@infradead.org>
Cc: Thomas Gleixner <tglx@linutronix.de>
Link: http://lkml.kernel.org/r/1445241870-24854-6-git-send-email-mingo@kernel.org
[ Remove --no-prefault, --only-prefault from docs, noticed by David Ahern ]
Signed-off-by: Arnaldo Carvalho de Melo <acme@redhat.com>
Diffstat (limited to 'tools/perf/builtin-bench.c')
0 files changed, 0 insertions, 0 deletions