diff mbox

qemu_coroutine_yield benchmark

Message ID 1379430579-4739-1-git-send-email-gabriel@kerneis.info
State New
Headers show

Commit Message

Gabriel Kerneis Sept. 17, 2013, 3:09 p.m. UTC
Current coroutine performance benchmarks test only coroutine creation,
either directly or in a nested way. This patch adds a benchmark to
evaluate the performance of qemu_coroutine_yield.

Signed-off-by: Gabriel Kerneis <gabriel@kerneis.info>
---
 tests/test-coroutine.c | 33 +++++++++++++++++++++++++++++++++
 1 file changed, 33 insertions(+)

Comments

Stefan Hajnoczi Sept. 19, 2013, 11:19 a.m. UTC | #1
On Tue, Sep 17, 2013 at 05:09:39PM +0200, Gabriel Kerneis wrote:
> Current coroutine performance benchmarks test only coroutine creation,
> either directly or in a nested way. This patch adds a benchmark to
> evaluate the performance of qemu_coroutine_yield.
> 
> Signed-off-by: Gabriel Kerneis <gabriel@kerneis.info>
> ---
>  tests/test-coroutine.c | 33 +++++++++++++++++++++++++++++++++
>  1 file changed, 33 insertions(+)

scripts/checkpatch.pl picked up on the while( syntax.  I added the space
when merging the patch.  Please run checkpatch.pl in the future.  The
easiest way is to enable it as a git-hook:
http://blog.vmsplice.net/2011/03/how-to-automatically-run-checkpatchpl.html

Thanks, applied to my block tree:
https://github.com/stefanha/qemu/commits/block

Stefan
diff mbox

Patch

diff --git a/tests/test-coroutine.c b/tests/test-coroutine.c
index 39be046..f4f90ac 100644
--- a/tests/test-coroutine.c
+++ b/tests/test-coroutine.c
@@ -202,6 +202,38 @@  static void perf_nesting(void)
         maxcycles, maxnesting, duration);
 }
 
+/*
+ * Yield benchmark
+ */
+
+static void coroutine_fn yield_loop(void *opaque)
+{
+    unsigned int *counter = opaque;
+
+    while((*counter) > 0) {
+        (*counter)--;
+        qemu_coroutine_yield();
+    }
+}
+
+static void perf_yield(void)
+{
+    unsigned int i, maxcycles;
+    double duration;
+
+    maxcycles = 100000000;
+    i = maxcycles;
+    Coroutine *coroutine = qemu_coroutine_create(yield_loop);
+
+    g_test_timer_start();
+    while(i > 0) {
+        qemu_coroutine_enter(coroutine, &i);
+    }
+    duration = g_test_timer_elapsed();
+
+    g_test_message("Yield %u iterations: %f s\n",
+        maxcycles, duration);
+}
 
 int main(int argc, char **argv)
 {
@@ -214,6 +246,7 @@  int main(int argc, char **argv)
     if (g_test_perf()) {
         g_test_add_func("/perf/lifecycle", perf_lifecycle);
         g_test_add_func("/perf/nesting", perf_nesting);
+        g_test_add_func("/perf/yield", perf_yield);
     }
     return g_test_run();
 }