Branch: refs/heads/master Home: https://github.com/qemu/qemu Commit: e8d3eb74bf11dff6920b8d0884c28f93e0d0d732 https://github.com/qemu/qemu/commit/e8d3eb74bf11dff6920b8d0884c28f93e0d0d732 Author: Marc-André Lureau <marcandre.lur...@redhat.com> 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 <marcandre.lur...@redhat.com> Message-Id: <20171006235023.11952-22-f4...@amsat.org> Signed-off-by: Eric Blake <ebl...@redhat.com> Commit: 4bfe4478d17679464a2aaa91ed703522ed9af8a0 https://github.com/qemu/qemu/commit/4bfe4478d17679464a2aaa91ed703522ed9af8a0 Author: Vladimir Sementsov-Ogievskiy <vsement...@virtuozzo.com> 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 <vsement...@virtuozzo.com> Reviewed-by: Eric Blake <ebl...@redhat.com> Message-Id: <20171012095319.136610-2-vsement...@virtuozzo.com> Signed-off-by: Eric Blake <ebl...@redhat.com> Commit: ed397b2fe7c37a8319e7ef89542447c86ebd67f8 https://github.com/qemu/qemu/commit/ed397b2fe7c37a8319e7ef89542447c86ebd67f8 Author: Vladimir Sementsov-Ogievskiy <vsement...@virtuozzo.com> 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 <vsement...@virtuozzo.com> Reviewed-by: Eric Blake <ebl...@redhat.com> Message-Id: <20171012095319.136610-3-vsement...@virtuozzo.com> Signed-off-by: Eric Blake <ebl...@redhat.com> Commit: 7b3158f951388428b2256fce13eb727a6afbd533 https://github.com/qemu/qemu/commit/7b3158f951388428b2256fce13eb727a6afbd533 Author: Vladimir Sementsov-Ogievskiy <vsement...@virtuozzo.com> 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 <vsement...@virtuozzo.com> Reviewed-by: Eric Blake <ebl...@redhat.com> Message-Id: <20171012095319.136610-4-vsement...@virtuozzo.com> [eblake: also tweak trace message contents] Signed-off-by: Eric Blake <ebl...@redhat.com> Commit: caad53845af2ee607913d844ce6c08a5dbbeceef https://github.com/qemu/qemu/commit/caad53845af2ee607913d844ce6c08a5dbbeceef Author: Vladimir Sementsov-Ogievskiy <vsement...@virtuozzo.com> 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 <vsement...@virtuozzo.com> Message-Id: <20171012095319.136610-5-vsement...@virtuozzo.com> [eblake: tweak and mention impact on traces, fix errp usage] Signed-off-by: Eric Blake <ebl...@redhat.com> Commit: 14cea41d394271b2f6f4c5f3ef1496de81fc0e08 https://github.com/qemu/qemu/commit/14cea41d394271b2f6f4c5f3ef1496de81fc0e08 Author: Vladimir Sementsov-Ogievskiy <vsement...@virtuozzo.com> 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 <vsement...@virtuozzo.com> Message-Id: <20171012095319.136610-6-vsement...@virtuozzo.com> [eblake: rebase to tweaks earlier in series] Signed-off-by: Eric Blake <ebl...@redhat.com> Commit: 978df1b6bf813a9fac3253073688f60bcc0a9733 https://github.com/qemu/qemu/commit/978df1b6bf813a9fac3253073688f60bcc0a9733 Author: Vladimir Sementsov-Ogievskiy <vsement...@virtuozzo.com> 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 <vsement...@virtuozzo.com> Message-Id: <20171012095319.136610-7-vsement...@virtuozzo.com> Signed-off-by: Eric Blake <ebl...@redhat.com> Commit: de79bfc36f08e9ad29866506ba58e20ff1f4b162 https://github.com/qemu/qemu/commit/de79bfc36f08e9ad29866506ba58e20ff1f4b162 Author: Vladimir Sementsov-Ogievskiy <vsement...@virtuozzo.com> 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 <vsement...@virtuozzo.com> Message-Id: <20171012095319.136610-8-vsement...@virtuozzo.com> [eblake: rebase to tweaks earlier in series] Signed-off-by: Eric Blake <ebl...@redhat.com> Commit: 92652b124336f5c08c7eb4616af202ea910dd3ea https://github.com/qemu/qemu/commit/92652b124336f5c08c7eb4616af202ea910dd3ea Author: Vladimir Sementsov-Ogievskiy <vsement...@virtuozzo.com> 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 <vsement...@virtuozzo.com> Reviewed-by: Eric Blake <ebl...@redhat.com> Message-Id: <20171012095319.136610-9-vsement...@virtuozzo.com> Signed-off-by: Eric Blake <ebl...@redhat.com> Commit: 79b2a13aa81724228166c794f48eb75bfb696b88 https://github.com/qemu/qemu/commit/79b2a13aa81724228166c794f48eb75bfb696b88 Author: Peter Maydell <peter.mayd...@linaro.org> 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 <ebl...@redhat.com>" # gpg: aka "Eric Blake (Free Software Programmer) <e...@byu.net>" # 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 <peter.mayd...@linaro.org> Compare: https://github.com/qemu/qemu/compare/48ae1f60d8c9...79b2a13aa817