[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH 1/8] virtio_queue_aio_attach_host_notifier: remove AioContext
From: |
Stefan Hajnoczi |
Subject: |
Re: [PATCH 1/8] virtio_queue_aio_attach_host_notifier: remove AioContext lock |
Date: |
Tue, 5 Jul 2022 15:11:49 +0100 |
On Thu, Jun 09, 2022 at 10:37:20AM -0400, Emanuele Giuseppe Esposito wrote:
> @@ -146,7 +147,6 @@ int virtio_scsi_dataplane_start(VirtIODevice *vdev)
>
> s->dataplane_starting = false;
> s->dataplane_started = true;
> - aio_context_release(s->ctx);
> return 0;
This looks risky because s->dataplane_started is accessed by IO code and
there is a race condition here. Maybe you can refactor the code along
the lines of virtio-blk to avoid the race.
signature.asc
Description: PGP signature
- Re: [PATCH 1/8] virtio_queue_aio_attach_host_notifier: remove AioContext lock,
Stefan Hajnoczi <=