[v2] NBD fixes before softfreeze

[Qemu-devel] [PATCH v2 0/2] NBD fixes before softfreeze

Eric Blake posted 2 patches 6 years, 9 months ago

Diff against v1
Download series mbox

Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/20170717192635.17880-1-eblake@redhat.com

Test FreeBSD passed

Test checkpatch passed

Test docker passed

Test s390x passed

nbd/nbd-internal.h |  8 ++++++--
nbd/client.c       |  3 ++-
nbd/server.c       | 18 ++++++++++--------
nbd/trace-events   |  2 +-
4 files changed, 19 insertions(+), 12 deletions(-)

Expand all Fold all

[Qemu-devel] [PATCH v2 0/2] NBD fixes before softfreeze

Posted by Eric Blake 6 years, 9 months ago

Paolo's inclusion of my NBD series in his miscellaneous tree
ended up with a couple of bugs that were pointed out in the
thread after the pull request was first created; these are the
fixes.  They are small enough that I don't mind using maintainer
status to bundle them up as a PULL request for NBD sometime on
Monday (in order to make soft freeze), whether or not they've
been reviewed by then; but of course, review is appreciated.

https://lists.gnu.org/archive/html/qemu-devel/2017-07/msg01983.html
https://lists.gnu.org/archive/html/qemu-devel/2017-07/msg01981.html

since v1:
- more defines and comments to make patch 2 more legible [John]

001/2:[----] [--] 'nbd: Trace client command being sent'
002/2:[0026] [FC] 'nbd: Fix server reply to NBD_OPT_EXPORT_NAME of older clients'

Eric Blake (2):
  nbd: Trace client command being sent
  nbd: Fix server reply to NBD_OPT_EXPORT_NAME of older clients

 nbd/nbd-internal.h |  8 ++++++--
 nbd/client.c       |  3 ++-
 nbd/server.c       | 18 ++++++++++--------
 nbd/trace-events   |  2 +-
 4 files changed, 19 insertions(+), 12 deletions(-)

-- 
2.13.3

Re: [Qemu-devel] [PATCH v2 0/2] NBD fixes before softfreeze

Posted by Eric Blake 6 years, 9 months ago

On 07/17/2017 02:26 PM, Eric Blake wrote:
> Paolo's inclusion of my NBD series in his miscellaneous tree
> ended up with a couple of bugs that were pointed out in the
> thread after the pull request was first created; these are the
> fixes.  They are small enough that I don't mind using maintainer
> status to bundle them up as a PULL request for NBD sometime on
> Monday (in order to make soft freeze), whether or not they've
> been reviewed by then; but of course, review is appreciated.
> 
> https://lists.gnu.org/archive/html/qemu-devel/2017-07/msg01983.html
> https://lists.gnu.org/archive/html/qemu-devel/2017-07/msg01981.html
> 
> since v1:
> - more defines and comments to make patch 2 more legible [John]
> 
> 001/2:[----] [--] 'nbd: Trace client command being sent'
> 002/2:[0026] [FC] 'nbd: Fix server reply to NBD_OPT_EXPORT_NAME of older clients'
> 
> Eric Blake (2):
>   nbd: Trace client command being sent
>   nbd: Fix server reply to NBD_OPT_EXPORT_NAME of older clients

Thanks; queued on my NBD branch:
git://repo.or.cz/qemu/ericb.git nbd

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