[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v2 0/7] coroutine: optimizations
From: |
Stefan Hajnoczi |
Subject: |
Re: [Qemu-devel] [PATCH v2 0/7] coroutine: optimizations |
Date: |
Tue, 6 Jan 2015 15:39:08 +0000 |
User-agent: |
Mutt/1.5.23 (2014-03-12) |
On Tue, Dec 02, 2014 at 12:05:43PM +0100, Paolo Bonzini wrote:
> As discussed in the other thread, this brings speedups from
> dropping the coroutine mutex (which serializes multiple iothreads,
> too) and using ELF thread-local storage.
>
> The speedup in perf/cost is about 50% (190->125). Windows port tested
> with tests/test-coroutine.exe under Wine.
>
> Paolo
>
> v1->v2: include the noinline attribute [many...]
> do not mention SwitchToFiber [Kevin]
> rename run_main_iothread_exit -> run_main_thread_exit
> leave personal opinions out of commit messages :) [Kevin]
> mention gain from patch 7 [Peter]
> change "alloc_pool_size +=" to "alloc_pool_size =" [Peter]
>
> Paolo Bonzini (7):
> coroutine-ucontext: use __thread
> qemu-thread: add per-thread atexit functions
> test-coroutine: avoid overflow on 32-bit systems
> QSLIST: add lock-free operations
> coroutine: rewrite pool to avoid mutex
> coroutine: drop qemu_coroutine_adjust_pool_size
> coroutine: try harder not to delete coroutines
>
> block/block-backend.c | 4 --
> coroutine-ucontext.c | 64 +++++++---------------------
> include/block/coroutine.h | 10 -----
> include/qemu/queue.h | 15 ++++++-
> include/qemu/thread.h | 4 ++
> qemu-coroutine.c | 104
> ++++++++++++++++++++++------------------------
> tests/test-coroutine.c | 2 +-
> util/qemu-thread-posix.c | 37 +++++++++++++++++
> util/qemu-thread-win32.c | 48 ++++++++++++++++-----
> 9 files changed, 157 insertions(+), 131 deletions(-)
>
> --
> 2.1.0
>
>
Thanks, applied to my block tree:
https://github.com/stefanha/qemu/commits/block
Stefan
pgpszq7sOlHrM.pgp
Description: PGP signature
[Prev in Thread] |
Current Thread |
[Next in Thread] |
- Re: [Qemu-devel] [PATCH v2 0/7] coroutine: optimizations,
Stefan Hajnoczi <=