On Thu, Dec 21, 2023 at 08:26:58AM +0100, Paolo Bonzini wrote:
> On 12/21/23 02:49, Stefan Hajnoczi wrote:
> > nbd_client_receive_next_request(client);
> > +
> > + qemu_mutex_unlock(&client->lock);
> > +
> > if (ret == -EIO) {
> > goto disconnect;
> > }
>
> I think I slightly prefer if disconnect is reached with lock taken, for
> consistency with the "done" label. It does not complicate the code,
> because you can just move qio_channel_set_cork() and replace:Yes, that makes the code easier to follow. Will fix in v2. Stefan
signature.asc
Description: PGP signature
