diff options
| author | Paolo Bonzini <pbonzini@redhat.com> | 2017-03-14 12:11:56 +0100 |
|---|---|---|
| committer | Max Reitz <mreitz@redhat.com> | 2017-03-27 16:50:36 +0200 |
| commit | a12a712a7dfbd2e2f4882ef2c90a9b2162166dd7 (patch) | |
| tree | e1eae825cb75759a62acc3e0ddd75ed064fc26b6 /accel.c | |
| parent | c919297379e9980c2bcc4d2053addbc1fd6d762b (diff) | |
nbd-client: fix handling of hungup connections
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 <mreitz@redhat.com>
Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
Message-id: 20170314111157.14464-1-pbonzini@redhat.com
Reviewed-by: Max Reitz <mreitz@redhat.com>
Signed-off-by: Max Reitz <mreitz@redhat.com>
Diffstat (limited to 'accel.c')
0 files changed, 0 insertions, 0 deletions