[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 07/12] nbd: Increase bs->in_flight during AioCon
From: |
Paolo Bonzini |
Subject: |
Re: [Qemu-devel] [PATCH 07/12] nbd: Increase bs->in_flight during AioContext switch |
Date: |
Tue, 19 Feb 2019 15:12:35 +0100 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:60.0) Gecko/20100101 Thunderbird/60.4.0 |
On 19/02/19 12:11, Kevin Wolf wrote:
>> 2) maybe instead of aio_co_schedul-ing client->connection_co and having
>> the s->aio_ctx_switch flag, you could go through a bottom half that does
>> the bdrv_inc_in_flight and then enters client->connection_co?
> That would be too easy. :-)
>
> But I agree, that might indeed be the better solution.
>
> I think I'd keep patch 6 anyway so that we know the exact yield that
> we'll interrupt, even if it's not strictly necessary as long as we know
> that nbd_receive_reply() can only yield in places that are safe to be
> interrupted. While intuitively I think it's true, I don't feel like
> actually auditing the code, and at some point we'd probably fail to
> check that new code won't violate this invariant.
Yes, I agree with keeping patch 6.
Paolo
- [Qemu-devel] [PATCH 00/12] block: bdrv_set_aio_context() related fixes, Kevin Wolf, 2019/02/18
- [Qemu-devel] [PATCH 01/12] block-backend: Make blk_inc/dec_in_flight public, Kevin Wolf, 2019/02/18
- [Qemu-devel] [PATCH 08/12] block: Don't poll in bdrv_set_aio_context(), Kevin Wolf, 2019/02/18
- [Qemu-devel] [PATCH 02/12] virtio-blk: Increase in_flight for request restart BH, Kevin Wolf, 2019/02/18
- [Qemu-devel] [PATCH 10/12] test-bdrv-drain: AioContext switch in drained section, Kevin Wolf, 2019/02/18
- [Qemu-devel] [PATCH 07/12] nbd: Increase bs->in_flight during AioContext switch, Kevin Wolf, 2019/02/18
- [Qemu-devel] [PATCH 09/12] block: Fix AioContext switch for drained node, Kevin Wolf, 2019/02/18
- [Qemu-devel] [PATCH 03/12] nbd: Restrict connection_co reentrance, Kevin Wolf, 2019/02/18
- [Qemu-devel] [PATCH 11/12] block: Use normal drain for bdrv_set_aio_context(), Kevin Wolf, 2019/02/18
[Qemu-devel] [PATCH 12/12] aio-posix: Assert that aio_poll() is always called in home thread, Kevin Wolf, 2019/02/18