From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:49552) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1cdFDs-000432-1n for qemu-devel@nongnu.org; Mon, 13 Feb 2017 06:55:13 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1cdFDn-0006am-5O for qemu-devel@nongnu.org; Mon, 13 Feb 2017 06:55:12 -0500 Received: from mail-wm0-x242.google.com ([2a00:1450:400c:c09::242]:34508) by eggs.gnu.org with esmtps (TLS1.0:RSA_AES_128_CBC_SHA1:16) (Exim 4.71) (envelope-from ) id 1cdFDm-0006a0-QY for qemu-devel@nongnu.org; Mon, 13 Feb 2017 06:55:07 -0500 Received: by mail-wm0-x242.google.com with SMTP id c85so19031355wmi.1 for ; Mon, 13 Feb 2017 03:55:06 -0800 (PST) Sender: Paolo Bonzini References: <20170213100906.23486-1-pbonzini@redhat.com> From: Paolo Bonzini Message-ID: Date: Mon, 13 Feb 2017 12:54:52 +0100 MIME-Version: 1.0 In-Reply-To: <20170213100906.23486-1-pbonzini@redhat.com> Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 7bit Subject: Re: [Qemu-devel] [PATCH v4 00/18] aio_context_acquire/release pushdown, part 2 List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: qemu-devel@nongnu.org Cc: famz@redhat.com, stefanha@redhat.com On 13/02/2017 11:08, Paolo Bonzini wrote: > Same as v3 except for tests/Makefile.include rebase in patch 1. Nope, this one was about a week old so it needs yet another rebase. Paolo > The next part is CoMutex thread-safety, which is relatively > small (10 patches). > > Paolo > > Paolo Bonzini (18): > block: move AioContext, QEMUTimer, main-loop to libqemuutil > aio: introduce aio_co_schedule and aio_co_wake > block-backend: allow blk_prw from coroutine context > test-thread-pool: use generic AioContext infrastructure > io: add methods to set I/O handlers on AioContext > io: make qio_channel_yield aware of AioContexts > nbd: convert to use qio_channel_yield > coroutine-lock: reschedule coroutine on the AioContext it was running > on > blkdebug: reschedule coroutine on the AioContext it is running on > qed: introduce qed_aio_start_io and qed_aio_next_io_cb > aio: push aio_context_acquire/release down to dispatching > block: explicitly acquire aiocontext in timers that need it > block: explicitly acquire aiocontext in callbacks that need it > block: explicitly acquire aiocontext in bottom halves that need it > block: explicitly acquire aiocontext in aio callbacks that need it > aio-posix: partially inline aio_dispatch into aio_poll > async: remove unnecessary inc/dec pairs > block: document fields protected by AioContext lock > > Makefile.objs | 4 - > block/blkdebug.c | 9 +- > block/blkreplay.c | 2 +- > block/block-backend.c | 13 ++- > block/curl.c | 44 ++++++-- > block/gluster.c | 9 +- > block/io.c | 38 ++----- > block/iscsi.c | 15 ++- > block/linux-aio.c | 10 +- > block/mirror.c | 12 +- > block/nbd-client.c | 117 +++++++++----------- > block/nbd-client.h | 2 +- > block/nfs.c | 9 +- > block/qed-cluster.c | 2 + > block/qed-table.c | 12 +- > block/qed.c | 58 +++++++--- > block/qed.h | 3 + > block/sheepdog.c | 29 ++--- > block/ssh.c | 29 ++--- > block/throttle-groups.c | 2 + > block/win32-aio.c | 9 +- > dma-helpers.c | 2 + > hw/block/virtio-blk.c | 19 +++- > hw/scsi/scsi-bus.c | 2 + > hw/scsi/scsi-disk.c | 15 +++ > hw/scsi/scsi-generic.c | 20 +++- > hw/scsi/virtio-scsi.c | 6 + > include/block/aio.h | 38 ++++++- > include/block/block_int.h | 64 ++++++----- > include/io/channel.h | 72 +++++++++++- > include/qemu/coroutine_int.h | 11 +- > include/sysemu/block-backend.h | 14 ++- > io/channel-command.c | 13 +++ > io/channel-file.c | 11 ++ > io/channel-socket.c | 16 ++- > io/channel-tls.c | 12 ++ > io/channel-watch.c | 6 + > io/channel.c | 97 ++++++++++++---- > nbd/client.c | 2 +- > nbd/common.c | 9 +- > nbd/server.c | 94 +++++----------- > stubs/Makefile.objs | 1 + > stubs/linux-aio.c | 32 ++++++ > stubs/set-fd-handler.c | 11 -- > tests/Makefile.include | 19 ++-- > tests/iothread.c | 91 +++++++++++++++ > tests/iothread.h | 25 +++++ > tests/test-aio-multithread.c | 213 ++++++++++++++++++++++++++++++++++++ > tests/test-thread-pool.c | 12 +- > trace-events | 4 + > util/Makefile.objs | 6 +- > aio-posix.c => util/aio-posix.c | 60 +++------- > aio-win32.c => util/aio-win32.c | 30 ++--- > util/aiocb.c | 55 ++++++++++ > async.c => util/async.c | 84 ++++++++++++-- > iohandler.c => util/iohandler.c | 0 > main-loop.c => util/main-loop.c | 0 > util/qemu-coroutine-lock.c | 5 +- > util/qemu-coroutine-sleep.c | 2 +- > util/qemu-coroutine.c | 8 ++ > qemu-timer.c => util/qemu-timer.c | 0 > thread-pool.c => util/thread-pool.c | 6 +- > util/trace-events | 1 - > 63 files changed, 1161 insertions(+), 455 deletions(-) > create mode 100644 stubs/linux-aio.c > create mode 100644 tests/iothread.c > create mode 100644 tests/iothread.h > create mode 100644 tests/test-aio-multithread.c > rename aio-posix.c => util/aio-posix.c (94%) > rename aio-win32.c => util/aio-win32.c (95%) > create mode 100644 util/aiocb.c > rename async.c => util/async.c (82%) > rename iohandler.c => util/iohandler.c (100%) > rename main-loop.c => util/main-loop.c (100%) > rename qemu-timer.c => util/qemu-timer.c (100%) > rename thread-pool.c => util/thread-pool.c (98%) >