Commit 708cda31 authored by Paul E. McKenney's avatar Paul E. McKenney

rcuperf: Add comments explaining the high reader overhead

This commit adds comments explaining why the readers have otherwise insane
levels of measurement overhead, namely that they are intended as a test
load for update-side performance measurements, not as a straight-up
read-side performance test.
Signed-off-by: default avatarPaul E. McKenney <paulmck@kernel.org>
parent 653ed64b
...@@ -69,6 +69,11 @@ MODULE_AUTHOR("Paul E. McKenney <paulmck@linux.ibm.com>"); ...@@ -69,6 +69,11 @@ MODULE_AUTHOR("Paul E. McKenney <paulmck@linux.ibm.com>");
* value specified by nr_cpus for a read-only test. * value specified by nr_cpus for a read-only test.
* *
* Various other use cases may of course be specified. * Various other use cases may of course be specified.
*
* Note that this test's readers are intended only as a test load for
* the writers. The reader performance statistics will be overly
* pessimistic due to the per-critical-section interrupt disabling,
* test-end checks, and the pair of calls through pointers.
*/ */
#ifdef MODULE #ifdef MODULE
...@@ -309,8 +314,10 @@ static void rcu_perf_wait_shutdown(void) ...@@ -309,8 +314,10 @@ static void rcu_perf_wait_shutdown(void)
} }
/* /*
* RCU perf reader kthread. Repeatedly does empty RCU read-side * RCU perf reader kthread. Repeatedly does empty RCU read-side critical
* critical section, minimizing update-side interference. * section, minimizing update-side interference. However, the point of
* this test is not to evaluate reader performance, but instead to serve
* as a test load for update-side performance testing.
*/ */
static int static int
rcu_perf_reader(void *arg) rcu_perf_reader(void *arg)
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment