Re: [PATCH v3 31/33] block/nbd: add nbd_clinent_connected() helper

2021-06-03 Thread Eric Blake
On Fri, Apr 16, 2021 at 11:09:09AM +0300, Vladimir Sementsov-Ogievskiy wrote: In the subject: s/clinent/client/ > We already have two similar helpers for other state. Let's add another > one for convenience. > > Signed-off-by: Vladimir Sementsov-Ogievskiy > --- > block/nbd.c | 25

Re: [PATCH v3 31/33] block/nbd: add nbd_clinent_connected() helper

2021-05-12 Thread Vladimir Sementsov-Ogievskiy
12.05.2021 10:06, Paolo Bonzini wrote: On 16/04/21 10:09, Vladimir Sementsov-Ogievskiy wrote: We already have two similar helpers for other state. Let's add another one for convenience. Signed-off-by: Vladimir Sementsov-Ogievskiy The whole usage of load-acquire for the connected state is

Re: [PATCH v3 31/33] block/nbd: add nbd_clinent_connected() helper

2021-05-12 Thread Paolo Bonzini
On 16/04/21 10:09, Vladimir Sementsov-Ogievskiy wrote: We already have two similar helpers for other state. Let's add another one for convenience. Signed-off-by: Vladimir Sementsov-Ogievskiy The whole usage of load-acquire for the connected state is completely meaningless, but that's not

[PATCH v3 31/33] block/nbd: add nbd_clinent_connected() helper

2021-04-16 Thread Vladimir Sementsov-Ogievskiy
We already have two similar helpers for other state. Let's add another one for convenience. Signed-off-by: Vladimir Sementsov-Ogievskiy --- block/nbd.c | 25 ++--- 1 file changed, 14 insertions(+), 11 deletions(-) diff --git a/block/nbd.c b/block/nbd.c index