From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:33562) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1bxDfv-0007tS-3G for qemu-devel@nongnu.org; Thu, 20 Oct 2016 09:46:30 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1bxDfu-0004Uj-Be for qemu-devel@nongnu.org; Thu, 20 Oct 2016 09:46:27 -0400 From: Kevin Wolf Date: Thu, 20 Oct 2016 15:46:00 +0200 Message-Id: <1476971169-31604-1-git-send-email-kwolf@redhat.com> Subject: [Qemu-devel] [PATCH 0/9] block-backend: Use coroutine for flush/discard/ioctl List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: qemu-block@nongnu.org Cc: kwolf@redhat.com, mreitz@redhat.com, pbonzini@redhat.com, qemu-devel@nongnu.org Paolo, this is my attempt at implementing what you were asking for last Friday. I converted blk_(co_)flush/pdiscard/ioctl so that all interfaces (coroutine, AIO, sync) go through the same coroutine-based function already on the BlockBackend level. Where it was reasonably easy, I also removed the corresponding emulations from block/io.c IIUC, this should cover your immediate needs. Function to remove this series leaves for another day: * bdrv_aio_flush (used by blkdebug, blkverify, qed) * bdrv_flush (even more users) * bdrv_pdiscard (used by qcow2) BlockDriver callbacks to remove left for another day: * bdrv_aio_pdiscard (implemented by raw-posix and rbd) * bdrv_aio_ioctl (implemented by raw-posix and iscsi) In both cases, raw-posix is trivial to covert, but iscsi and rbd feel rather scary without a proper test setup. Kevin Wolf (9): block: Use blk_co_flush() for all BB level flushes block: Use blk_co_pdiscard() for all BB level discard block: Remove bdrv_aio_pdiscard() block: Use blk_co_ioctl() for all BB level ioctls raw-posix: Don't use bdrv_ioctl() block: Remove bdrv_ioctl() block: Introduce .bdrv_co_ioctl() driver callback raw: Implement .bdrv_co_ioctl instead of .bdrv_aio_ioctl block: Remove bdrv_aio_ioctl() block/block-backend.c | 94 ++++++++++++++++++++++++---------- block/io.c | 111 ++++------------------------------------- block/raw-posix.c | 16 ++++-- block/raw_bsd.c | 9 ++-- block/trace-events | 1 - include/block/block.h | 8 +-- include/block/block_int.h | 2 + include/sysemu/block-backend.h | 1 + 8 files changed, 98 insertions(+), 144 deletions(-) -- 1.8.3.1