[Qemu-devel] [PATCH v2 00/12] nbd refactoring part 1

Vladimir Sementsov-Ogievskiy posted 12 patches 6 years, 11 months ago
Failed in applying to current master (apply log)
block/nbd-client.c  |   8 +-
include/block/nbd.h |   8 +-
nbd/client.c        |  64 ++++-------
nbd/common.c        |  34 +++++-
nbd/nbd-internal.h  |  28 ++---
nbd/server.c        | 319 +++++++++++++++++-----------------------------------
6 files changed, 172 insertions(+), 289 deletions(-)
[Qemu-devel] [PATCH v2 00/12] nbd refactoring part 1
Posted by Vladimir Sementsov-Ogievskiy 6 years, 11 months ago
This is based on my "nbd: error path refactoring" series.
This is an update of first part of my "nbd errors and traces refactoring"

v2:
01: update commit msg
    grammar in comment
    add Eric's r-b
02: add Eric's r-b
03: update commit msg
    add Eric's r-b


v1:
changes from patches 01-10 of "nbd errors and traces refactoring" (mostly
proposed by Eric):
01 new patch
02-03 - split of old 01
08 and 12 - minor changes in commit messages
04-12 - add r-b by Eric

Also, of course, renaming in 01 is "visible" in other patches.

Vladimir Sementsov-Ogievskiy (12):
  nbd: rename read_sync and friends
  nbd: make nbd_drop public
  nbd/server: get rid of nbd_negotiate_read and friends
  nbd/server: get rid of ssize_t
  nbd/server: refactor nbd_co_send_reply
  nbd/server: get rid of EAGAIN dead code
  nbd/server: refactor nbd_co_receive_request
  nbd/server: remove NBDClientNewData
  nbd/server: nbd_negotiate: fix error path
  nbd/server: get rid of fail: return rc
  nbd/server: rename rc to ret
  nbd/server: refactor nbd_trip

 block/nbd-client.c  |   8 +-
 include/block/nbd.h |   8 +-
 nbd/client.c        |  64 ++++-------
 nbd/common.c        |  34 +++++-
 nbd/nbd-internal.h  |  28 ++---
 nbd/server.c        | 319 +++++++++++++++++-----------------------------------
 6 files changed, 172 insertions(+), 289 deletions(-)

-- 
2.11.1


[Qemu-devel] ping Re: [PATCH v2 00/12] nbd refactoring part 1
Posted by Vladimir Sementsov-Ogievskiy 6 years, 10 months ago
ping... What about this?

02.06.2017 18:01, Vladimir Sementsov-Ogievskiy wrote:
> This is based on my "nbd: error path refactoring" series.
> This is an update of first part of my "nbd errors and traces refactoring"
>
> v2:
> 01: update commit msg
>      grammar in comment
>      add Eric's r-b
> 02: add Eric's r-b
> 03: update commit msg
>      add Eric's r-b
>
>
> v1:
> changes from patches 01-10 of "nbd errors and traces refactoring" (mostly
> proposed by Eric):
> 01 new patch
> 02-03 - split of old 01
> 08 and 12 - minor changes in commit messages
> 04-12 - add r-b by Eric
>
> Also, of course, renaming in 01 is "visible" in other patches.
>
> Vladimir Sementsov-Ogievskiy (12):
>    nbd: rename read_sync and friends
>    nbd: make nbd_drop public
>    nbd/server: get rid of nbd_negotiate_read and friends
>    nbd/server: get rid of ssize_t
>    nbd/server: refactor nbd_co_send_reply
>    nbd/server: get rid of EAGAIN dead code
>    nbd/server: refactor nbd_co_receive_request
>    nbd/server: remove NBDClientNewData
>    nbd/server: nbd_negotiate: fix error path
>    nbd/server: get rid of fail: return rc
>    nbd/server: rename rc to ret
>    nbd/server: refactor nbd_trip
>
>   block/nbd-client.c  |   8 +-
>   include/block/nbd.h |   8 +-
>   nbd/client.c        |  64 ++++-------
>   nbd/common.c        |  34 +++++-
>   nbd/nbd-internal.h  |  28 ++---
>   nbd/server.c        | 319 +++++++++++++++++-----------------------------------
>   6 files changed, 172 insertions(+), 289 deletions(-)
>

-- 
Best regards,
Vladimir