Skip to content

nbd minimal structured read

Vladimir Sementsov-Ogievskiy qemu-devel 1 version In review

Get series with b4
b4 am [email protected]

Versions newest first

  1. v1 Latest8 patches 20 repliesIn review

Patches in v1 No applies recorded yet

#Subject and filesSizeRepliesApplied
1/8block/nbd-client: assert qiov len once in nbd_co_requestblock/nbd-client.c+1 -21 reply
2/8block/nbd-client: refactor nbd_co_receive_replyblock/nbd-client.c+4 -41 reply
3/8nbd: rename NBD_REPLY_MAGIC to NBD_SIMPLE_REPLY_MAGICnbd/client.c nbd/nbd-internal.h nbd/server.c +1 more+7 -70 replies
4/8nbd-server: refactor simple reply sendinginclude/block/nbd.h nbd/nbd-internal.h nbd/server.c+56 -560 replies
5/8nbd: header constants indentinginclude/block/nbd.h nbd/nbd-internal.h+25 -240 replies
6/8nbd: Minimal structured read for serverinclude/block/nbd.h nbd/nbd-internal.h nbd/server.c+126 -60 replies
7/8nbd/client: refactor nbd_receive_starttlsnbd/client.c nbd/trace-events+50 -210 replies
8/8nbd: Minimal structured read for clientblock/nbd-client.c block/nbd-client.h include/block/nbd.h +1 more+249 -3419 replies

Reviews trailers per patch

2 of 8 patches have a Reviewed-by or Acked-by.

PatchEric Blake
1/8
2/8R-b
3/8R-b
4/8
5/8
6/8
7/8
8/8

Trailers given in replies are filled; those carried in the patches from earlier versions are outlined with dashes.

Read the v1 thread

Cover letter [Qemu-devel] [PATCH 0/8] nbd minimal structured read

Hi all. Here is minimal structured-read extension implementation for nbd. It is needed mostly to implement nbd block-status extension over it, so it is minimal.

It's based on my three patches, currently on pull-request stage:
block/nbd-client: refactor nbd_co_receive_reply block/nbd-client: simplify check in nbd_co_receive_reply block/nbd-client: nbd_co_send_request: fix return code

clone: tag up-nbd-minimal-structured-read-v1 from https://src.openvz.org/scm/~vsementsov/qemu.git
online: https://src.openvz.org/users/vsementsov/repos/qemu/browse?at=up-nbd-minimal-structured-read-v1

Note: actually it's a continuation of a part of my series "nbd: BLOCK_STATUS", but it was sent about half a year ago, so let's consider it as new series.

Vladimir Sementsov-Ogievskiy (8):
block/nbd-client: assert qiov len once in nbd_co_request block/nbd-client: refactor nbd_co_receive_reply
nbd: rename NBD_REPLY_MAGIC to NBD_SIMPLE_REPLY_MAGIC
nbd-server: refactor simple reply sending
nbd: header constants indenting
nbd: Minimal structured read for server nbd/client: refactor nbd_receive_starttls
nbd: Minimal structured read for client

 block/nbd-client.h                       |   2 +
 include/block/nbd.h                      |  67 +++++++--
 nbd/nbd-internal.h                       |  44 +++---
 block/nbd-client.c                       | 104 +++++++++++---
 nbd/client.c                             | 233 ++++++++++++++++++++++++++-----
 nbd/server.c                             | 189 +++++++++++++++++--------
 nbd/trace-events                         |   7 +-
 tests/qemu-iotests/nbd-fault-injector.py |   4 +-
 8 files changed, 507 insertions(+), 143 deletions(-)
Signature · 1 line
2.11.1