On Wed, Apr 07, 2021 at 01:46:24PM +0300, Vladimir Sementsov-Ogievskiy wrote: > The field is actually unused. Let's make things a bit simpler dropping > it and corresponding logic. > > Signed-off-by: Vladimir Sementsov-Ogievskiy <vsement...@virtuozzo.com> > --- > block/nbd.c | 9 ++------- > 1 file changed, 2 insertions(+), 7 deletions(-) > > diff --git a/block/nbd.c b/block/nbd.c > index c26dc5a54f..a47d6cfea3 100644 > --- a/block/nbd.c > +++ b/block/nbd.c > @@ -122,7 +122,6 @@ typedef struct BDRVNBDState { > int in_flight; > NBDClientState state; > int connect_status;
This field is write-only too. Do you have any plans on using it later? Otherwise you may want to nuke it in this patch too. Roman.