qemu-commits
[Top][All Lists]
Advanced
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[Qemu-commits] [qemu/qemu] e8d3eb: NBD: use g_new() family of functions

From: GitHub
Subject: [Qemu-commits] [qemu/qemu] e8d3eb: NBD: use g_new() family of functions
Date: Mon, 16 Oct 2017 08:40:43 -0700 
  Branch: refs/heads/master
  Home:   https://github.com/qemu/qemu
  Commit: e8d3eb74bf11dff6920b8d0884c28f93e0d0d732
      
https://github.com/qemu/qemu/commit/e8d3eb74bf11dff6920b8d0884c28f93e0d0d732
  Author: Marc-André Lureau <address@hidden>
  Date:   2017-10-12 (Thu, 12 Oct 2017)

  Changed paths:
    M nbd/server.c

  Log Message:
  -----------
  NBD: use g_new() family of functions

Signed-off-by: Marc-André Lureau <address@hidden>
Message-Id: <address@hidden>
Signed-off-by: Eric Blake <address@hidden>


  Commit: 4bfe4478d17679464a2aaa91ed703522ed9af8a0
      
https://github.com/qemu/qemu/commit/4bfe4478d17679464a2aaa91ed703522ed9af8a0
  Author: Vladimir Sementsov-Ogievskiy <address@hidden>
  Date:   2017-10-12 (Thu, 12 Oct 2017)

  Changed paths:
    M block/nbd-client.c

  Log Message:
  -----------
  block/nbd-client: assert qiov len once in nbd_co_request

Also improve the assertion: check that qiov is NULL for other commands
than CMD_READ and CMD_WRITE.

Signed-off-by: Vladimir Sementsov-Ogievskiy <address@hidden>
Reviewed-by: Eric Blake <address@hidden>
Message-Id: <address@hidden>
Signed-off-by: Eric Blake <address@hidden>


  Commit: ed397b2fe7c37a8319e7ef89542447c86ebd67f8
      
https://github.com/qemu/qemu/commit/ed397b2fe7c37a8319e7ef89542447c86ebd67f8
  Author: Vladimir Sementsov-Ogievskiy <address@hidden>
  Date:   2017-10-12 (Thu, 12 Oct 2017)

  Changed paths:
    M block/nbd-client.c

  Log Message:
  -----------
  block/nbd-client: refactor nbd_co_receive_reply

Pass handle parameter directly, as the whole request isn't needed.

Signed-off-by: Vladimir Sementsov-Ogievskiy <address@hidden>
Reviewed-by: Eric Blake <address@hidden>
Message-Id: <address@hidden>
Signed-off-by: Eric Blake <address@hidden>


  Commit: 7b3158f951388428b2256fce13eb727a6afbd533
      
https://github.com/qemu/qemu/commit/7b3158f951388428b2256fce13eb727a6afbd533
  Author: Vladimir Sementsov-Ogievskiy <address@hidden>
  Date:   2017-10-12 (Thu, 12 Oct 2017)

  Changed paths:
    M nbd/client.c
    M nbd/nbd-internal.h
    M nbd/server.c
    M nbd/trace-events
    M tests/qemu-iotests/nbd-fault-injector.py

  Log Message:
  -----------
  nbd: rename some simple-request related objects to be _simple_

To be consistent when their _structured_ analogs will be introduced.

Signed-off-by: Vladimir Sementsov-Ogievskiy <address@hidden>
Reviewed-by: Eric Blake <address@hidden>
Message-Id: <address@hidden>
[eblake: also tweak trace message contents]
Signed-off-by: Eric Blake <address@hidden>


  Commit: caad53845af2ee607913d844ce6c08a5dbbeceef
      
https://github.com/qemu/qemu/commit/caad53845af2ee607913d844ce6c08a5dbbeceef
  Author: Vladimir Sementsov-Ogievskiy <address@hidden>
  Date:   2017-10-12 (Thu, 12 Oct 2017)

  Changed paths:
    M include/block/nbd.h
    M nbd/server.c
    M nbd/trace-events

  Log Message:
  -----------
  nbd/server: structurize simple reply header sending

Use packed structure instead of pointer arithmetics.

Also, merge two redundant traces into one.

Signed-off-by: Vladimir Sementsov-Ogievskiy <address@hidden>
Message-Id: <address@hidden>
[eblake: tweak and mention impact on traces, fix errp usage]
Signed-off-by: Eric Blake <address@hidden>


  Commit: 14cea41d394271b2f6f4c5f3ef1496de81fc0e08
      
https://github.com/qemu/qemu/commit/14cea41d394271b2f6f4c5f3ef1496de81fc0e08
  Author: Vladimir Sementsov-Ogievskiy <address@hidden>
  Date:   2017-10-13 (Fri, 13 Oct 2017)

  Changed paths:
    M nbd/server.c

  Log Message:
  -----------
  nbd/server: do not use NBDReply structure

NBDReply structure will be upgraded in future patches to handle both
simple and structured replies and will be used only in the client

Signed-off-by: Vladimir Sementsov-Ogievskiy <address@hidden>
Message-Id: <address@hidden>
[eblake: rebase to tweaks earlier in series]
Signed-off-by: Eric Blake <address@hidden>


  Commit: 978df1b6bf813a9fac3253073688f60bcc0a9733
      
https://github.com/qemu/qemu/commit/978df1b6bf813a9fac3253073688f60bcc0a9733
  Author: Vladimir Sementsov-Ogievskiy <address@hidden>
  Date:   2017-10-13 (Fri, 13 Oct 2017)

  Changed paths:
    M nbd/server.c

  Log Message:
  -----------
  nbd/server: refactor nbd_co_send_simple_reply parameters

Pass client and buffer (*data) parameters directly, to make the function
consistent with further structured reply sending functions.

Signed-off-by: Vladimir Sementsov-Ogievskiy <address@hidden>
Message-Id: <address@hidden>
Signed-off-by: Eric Blake <address@hidden>


  Commit: de79bfc36f08e9ad29866506ba58e20ff1f4b162
      
https://github.com/qemu/qemu/commit/de79bfc36f08e9ad29866506ba58e20ff1f4b162
  Author: Vladimir Sementsov-Ogievskiy <address@hidden>
  Date:   2017-10-13 (Fri, 13 Oct 2017)

  Changed paths:
    M nbd/server.c

  Log Message:
  -----------
  nbd/server: simplify reply transmission

Send qiov via qio_channel_writev_all instead of calling nbd_write twice
with a cork.

Signed-off-by: Vladimir Sementsov-Ogievskiy <address@hidden>
Message-Id: <address@hidden>
[eblake: rebase to tweaks earlier in series]
Signed-off-by: Eric Blake <address@hidden>


  Commit: 92652b124336f5c08c7eb4616af202ea910dd3ea
      
https://github.com/qemu/qemu/commit/92652b124336f5c08c7eb4616af202ea910dd3ea
  Author: Vladimir Sementsov-Ogievskiy <address@hidden>
  Date:   2017-10-13 (Fri, 13 Oct 2017)

  Changed paths:
    M include/block/nbd.h
    M nbd/nbd-internal.h

  Log Message:
  -----------
  nbd: header constants indenting

Prepare indenting for the following commit.

Signed-off-by: Vladimir Sementsov-Ogievskiy <address@hidden>
Reviewed-by: Eric Blake <address@hidden>
Message-Id: <address@hidden>
Signed-off-by: Eric Blake <address@hidden>


  Commit: 79b2a13aa81724228166c794f48eb75bfb696b88
      
https://github.com/qemu/qemu/commit/79b2a13aa81724228166c794f48eb75bfb696b88
  Author: Peter Maydell <address@hidden>
  Date:   2017-10-16 (Mon, 16 Oct 2017)

  Changed paths:
    M block/nbd-client.c
    M include/block/nbd.h
    M nbd/client.c
    M nbd/nbd-internal.h
    M nbd/server.c
    M nbd/trace-events
    M tests/qemu-iotests/nbd-fault-injector.py

  Log Message:
  -----------
  Merge remote-tracking branch 'remotes/ericb/tags/pull-nbd-2017-10-14' into 
staging

nbd patches for 2017-10-14

- Marc-André Lureau - NBD: use g_new() family of functions
- Vladimir Sementsov-Ogievskiy - first half of 00/13 nbd minimal structured read

# gpg: Signature made Sun 15 Oct 2017 01:38:47 BST
# gpg:                using RSA key 0xA7A16B4A2527436A
# gpg: Good signature from "Eric Blake <address@hidden>"
# gpg:                 aka "Eric Blake (Free Software Programmer) 
<address@hidden>"
# gpg:                 aka "[jpeg image of size 6874]"
# Primary key fingerprint: 71C2 CC22 B1C4 6029 27D2  F3AA A7A1 6B4A 2527 436A

* remotes/ericb/tags/pull-nbd-2017-10-14:
  nbd: header constants indenting
  nbd/server: simplify reply transmission
  nbd/server: refactor nbd_co_send_simple_reply parameters
  nbd/server: do not use NBDReply structure
  nbd/server: structurize simple reply header sending
  nbd: rename some simple-request related objects to be _simple_
  block/nbd-client: refactor nbd_co_receive_reply
  block/nbd-client: assert qiov len once in nbd_co_request
  NBD: use g_new() family of functions

Signed-off-by: Peter Maydell <address@hidden>


Compare: https://github.com/qemu/qemu/compare/48ae1f60d8c9...79b2a13aa817
reply via email to
[Prev in Thread] Current Thread [Next in Thread]