[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-block] [PATCH] nbd-client: fix handling of hungup connections
From: |
Max Reitz |
Subject: |
Re: [Qemu-block] [PATCH] nbd-client: fix handling of hungup connections |
Date: |
Wed, 22 Mar 2017 16:50:32 +0100 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:45.0) Gecko/20100101 Thunderbird/45.8.0 |
On 14.03.2017 12:11, Paolo Bonzini wrote:
> After the switch to reading replies in a coroutine, nothing is
> reentering pending receive coroutines if the connection hangs.
> Move nbd_recv_coroutines_enter_all to the reply read coroutine,
> which is the place where hangups are detected. nbd_teardown_connection
> can simply wait for the reply read coroutine to detect the hangup
> and clean up after itself.
>
> This wouldn't be enough though because nbd_receive_reply returns 0
> (rather than -EPIPE or similar) when reading from a hung connection.
> Fix the return value check in nbd_read_reply_entry.
>
> This fixes qemu-iotests 083.
>
> Reported-by: Max Reitz <address@hidden>
> Signed-off-by: Paolo Bonzini <address@hidden>
> ---
> block/nbd-client.c | 12 ++++++------
> nbd/client.c | 2 +-
> 2 files changed, 7 insertions(+), 7 deletions(-)
Thanks, I've applied the patch to my block branch:
https://github.com/XanClic/qemu/commits/block
Max
signature.asc
Description: OpenPGP digital signature