[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[PATCH 7/7] block/nbd: stop manipulating in_flight counter
From: |
Roman Kagan |
Subject: |
[PATCH 7/7] block/nbd: stop manipulating in_flight counter |
Date: |
Mon, 15 Mar 2021 09:06:11 +0300 |
As the reconnect logic no longer interferes with drained sections, it
appears unnecessary to explicitly manipulate the in_flight counter.
Fixes: 5ad81b4946 ("nbd: Restrict connection_co reentrance")
Signed-off-by: Roman Kagan <rvkagan@yandex-team.ru>
---
block/nbd.c | 6 ------
nbd/client.c | 2 --
2 files changed, 8 deletions(-)
diff --git a/block/nbd.c b/block/nbd.c
index a5a9e4aca5..3a22f5d897 100644
--- a/block/nbd.c
+++ b/block/nbd.c
@@ -311,7 +311,6 @@ static void nbd_client_attach_aio_context_bh(void *opaque)
if (s->connection_co) {
qemu_aio_coroutine_enter(bs->aio_context, s->connection_co);
}
- bdrv_dec_in_flight(bs);
}
static void nbd_client_attach_aio_context(BlockDriverState *bs,
@@ -327,7 +326,6 @@ static void nbd_client_attach_aio_context(BlockDriverState
*bs,
qio_channel_attach_aio_context(QIO_CHANNEL(s->ioc), new_context);
}
- bdrv_inc_in_flight(bs);
/*
* Need to wait here for the BH to run because the BH must run while the
@@ -643,11 +641,9 @@ static coroutine_fn void
nbd_reconnect_attempt(BDRVNBDState *s)
goto out;
}
- bdrv_dec_in_flight(s->bs);
ret = nbd_client_handshake(s->bs, &local_err);
- bdrv_inc_in_flight(s->bs);
out:
s->connect_status = ret;
@@ -759,7 +755,6 @@ static coroutine_fn void nbd_connection_entry(void *opaque)
qemu_co_queue_restart_all(&s->free_sema);
nbd_recv_coroutines_wake_all(s);
- bdrv_dec_in_flight(s->bs);
s->connection_co = NULL;
if (s->ioc) {
@@ -2307,7 +2302,6 @@ static int nbd_open(BlockDriverState *bs, QDict *options,
int flags,
nbd_init_connect_thread(s);
s->connection_co = qemu_coroutine_create(nbd_connection_entry, s);
- bdrv_inc_in_flight(bs);
aio_co_schedule(bdrv_get_aio_context(bs), s->connection_co);
return 0;
diff --git a/nbd/client.c b/nbd/client.c
index 0c2db4bcba..30d5383cb1 100644
--- a/nbd/client.c
+++ b/nbd/client.c
@@ -1434,9 +1434,7 @@ nbd_read_eof(BlockDriverState *bs, QIOChannel *ioc, void
*buffer, size_t size,
len = qio_channel_readv(ioc, &iov, 1, errp);
if (len == QIO_CHANNEL_ERR_BLOCK) {
- bdrv_dec_in_flight(bs);
qio_channel_yield(ioc, G_IO_IN);
- bdrv_inc_in_flight(bs);
continue;
} else if (len < 0) {
return -EIO;
--
2.30.2
- [PATCH 2/7] block/nbd: use uniformly nbd_client_connecting_wait, (continued)
[PATCH 3/7] block/nbd: assert attach/detach runs in the proper context, Roman Kagan, 2021/03/15
[PATCH 7/7] block/nbd: stop manipulating in_flight counter,
Roman Kagan <=
Re: [PATCH 0/7] block/nbd: decouple reconnect from drain, Vladimir Sementsov-Ogievskiy, 2021/03/15
Re: [PATCH 0/7] block/nbd: decouple reconnect from drain, Eric Blake, 2021/03/16
Re: [PATCH 0/7] block/nbd: decouple reconnect from drain, Vladimir Sementsov-Ogievskiy, 2021/03/17