[Qemu-devel] [PATCH v5 0/6] NBD reconnect: prep. refactoring

Vladimir Sementsov-Ogievskiy posted 6 patches 5 years, 2 months ago
Test checkpatch passed
Failed in applying to current master (apply log)
block/nbd-client.h |   6 +-
block/nbd-client.c | 196 +++++++++++++++++++++++++++++----------------
block/nbd.c        |  40 +--------
3 files changed, 131 insertions(+), 111 deletions(-)
[Qemu-devel] [PATCH v5 0/6] NBD reconnect: prep. refactoring
Posted by Vladimir Sementsov-Ogievskiy 5 years, 2 months ago
Hi all.

Here is most of preparatory refactoring for NBD reconnect, rebased
on master. Let's push it.

Sorry for a long delay in answering on v4 review and for v4 which
I pinged a lot can't be applied on master directly :(

v5:
02: rebased on master, object_unref() moved to fail: block
05: tiny fixes in commit message [Eric]

Vladimir Sementsov-Ogievskiy (6):
  block/nbd-client: split channel errors from export errors
  block/nbd: move connection code from block/nbd to block/nbd-client
  block/nbd-client: split connection from initialization
  block/nbd-client: fix nbd_reply_chunk_iter_receive
  block/nbd-client: don't check ioc
  block/nbd-client: rename read_reply_co to connection_co

 block/nbd-client.h |   6 +-
 block/nbd-client.c | 196 +++++++++++++++++++++++++++++----------------
 block/nbd.c        |  40 +--------
 3 files changed, 131 insertions(+), 111 deletions(-)

-- 
2.18.0


Re: [Qemu-devel] [PATCH v5 0/6] NBD reconnect: prep. refactoring
Posted by Eric Blake 5 years, 2 months ago
On 2/1/19 7:01 AM, Vladimir Sementsov-Ogievskiy wrote:
> Hi all.
> 
> Here is most of preparatory refactoring for NBD reconnect, rebased
> on master. Let's push it.

Thanks; queued for my next NBD pull request.

> 
> Sorry for a long delay in answering on v4 review and for v4 which
> I pinged a lot can't be applied on master directly :(
> 
> v5:
> 02: rebased on master, object_unref() moved to fail: block
> 05: tiny fixes in commit message [Eric]
> 
> Vladimir Sementsov-Ogievskiy (6):
>   block/nbd-client: split channel errors from export errors
>   block/nbd: move connection code from block/nbd to block/nbd-client
>   block/nbd-client: split connection from initialization
>   block/nbd-client: fix nbd_reply_chunk_iter_receive
>   block/nbd-client: don't check ioc
>   block/nbd-client: rename read_reply_co to connection_co
> 
>  block/nbd-client.h |   6 +-
>  block/nbd-client.c | 196 +++++++++++++++++++++++++++++----------------
>  block/nbd.c        |  40 +--------
>  3 files changed, 131 insertions(+), 111 deletions(-)
> 

-- 
Eric Blake, Principal Software Engineer
Red Hat, Inc.           +1-919-301-3226
Virtualization:  qemu.org | libvirt.org