[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 0/9] block-backend: Use coroutine for flush/disc
From: |
Paolo Bonzini |
Subject: |
Re: [Qemu-devel] [PATCH 0/9] block-backend: Use coroutine for flush/discard/ioctl |
Date: |
Thu, 20 Oct 2016 10:37:01 -0400 (EDT) |
----- Original Message -----
> From: "Kevin Wolf" <address@hidden>
> To: address@hidden
> Cc: address@hidden, address@hidden, address@hidden, address@hidden
> Sent: Thursday, October 20, 2016 3:46:00 PM
> Subject: [PATCH 0/9] block-backend: Use coroutine for flush/discard/ioctl
>
> 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.
Thanks!
Paolo
- [Qemu-devel] [PATCH 3/9] block: Remove bdrv_aio_pdiscard(), (continued)
- [Qemu-devel] [PATCH 3/9] block: Remove bdrv_aio_pdiscard(), Kevin Wolf, 2016/10/20
- [Qemu-devel] [PATCH 6/9] block: Remove bdrv_ioctl(), Kevin Wolf, 2016/10/20
- [Qemu-devel] [PATCH 8/9] raw: Implement .bdrv_co_ioctl instead of .bdrv_aio_ioctl, Kevin Wolf, 2016/10/20
- [Qemu-devel] [PATCH 5/9] raw-posix: Don't use bdrv_ioctl(), Kevin Wolf, 2016/10/20
- [Qemu-devel] [PATCH 9/9] block: Remove bdrv_aio_ioctl(), Kevin Wolf, 2016/10/20
- Re: [Qemu-devel] [PATCH 0/9] block-backend: Use coroutine for flush/discard/ioctl,
Paolo Bonzini <=
- Re: [Qemu-devel] [PATCH 0/9] block-backend: Use coroutine for flush/discard/ioctl, no-reply, 2016/10/20