diff options
| author | Vladimir Sementsov-Ogievskiy <vsementsov@virtuozzo.com> | 2020-07-27 21:47:50 +0300 |
|---|---|---|
| committer | Eric Blake <eblake@redhat.com> | 2020-07-28 09:54:43 -0500 |
| commit | 12c75e20a269ac917f4a76936d7142264e522233 (patch) | |
| tree | 3517fc68a10c75bb527a2e9295cef764c00f5fce /block | |
| parent | fbeb3e63b34a1af4a968031de1c82e5edf20bf6c (diff) | |
| download | qemu-12c75e20a269ac917f4a76936d7142264e522233.zip qemu-12c75e20a269ac917f4a76936d7142264e522233.tar.gz qemu-12c75e20a269ac917f4a76936d7142264e522233.tar.bz2 | |
block/nbd: nbd_co_reconnect_loop(): don't sleep if drained
We try to go to wakeable sleep, so that, if drain begins it will break
the sleep. But what if nbd_client_co_drain_begin() already called and
s->drained is already true? We'll go to sleep, and drain will have to
wait for the whole timeout. Let's improve it.
Signed-off-by: Vladimir Sementsov-Ogievskiy <vsementsov@virtuozzo.com>
Reviewed-by: Eric Blake <eblake@redhat.com>
Message-Id: <20200727184751.15704-5-vsementsov@virtuozzo.com>
Signed-off-by: Eric Blake <eblake@redhat.com>
Diffstat (limited to 'block')
| -rw-r--r-- | block/nbd.c | 11 |
1 files changed, 6 insertions, 5 deletions
diff --git a/block/nbd.c b/block/nbd.c index 620c97b..7bb881f 100644 --- a/block/nbd.c +++ b/block/nbd.c @@ -341,8 +341,6 @@ static coroutine_fn void nbd_co_reconnect_loop(BDRVNBDState *s) qemu_co_queue_restart_all(&s->free_sema); } - qemu_co_sleep_ns_wakeable(QEMU_CLOCK_REALTIME, timeout, - &s->connection_co_sleep_ns_state); if (s->drained) { bdrv_dec_in_flight(s->bs); s->wait_drained_end = true; @@ -354,9 +352,12 @@ static coroutine_fn void nbd_co_reconnect_loop(BDRVNBDState *s) qemu_coroutine_yield(); } bdrv_inc_in_flight(s->bs); - } - if (timeout < max_timeout) { - timeout *= 2; + } else { + qemu_co_sleep_ns_wakeable(QEMU_CLOCK_REALTIME, timeout, + &s->connection_co_sleep_ns_state); + if (timeout < max_timeout) { + timeout *= 2; + } } nbd_reconnect_attempt(s); |