From: Eric Blake <eblake@redhat.com>
To: qemu-devel@nongnu.org
Cc: vsementsov@virtuozzo.com, pbonzini@redhat.com, qemu-block@nongnu.org
Subject: [Qemu-devel] [PATCH v5 00/11] nbd minimal structured read
Date: Thu, 19 Oct 2017 17:26:26 -0500 [thread overview]
Message-ID: <20171019222637.17890-1-eblake@redhat.com> (raw)
I've incorporated a few more tweaks since v4:
https://lists.gnu.org/archive/html/qemu-devel/2017-10/msg03305.html
I tried to call out the biggest changes in the various commits, but
it includes bug fixes from my testing, and fixing places spotted
by Vladimir. I still haven't validated the behavior of the client
against a server that sends more than one chunk per reply, but this
is looking closer to a pull request, hopefully in time for soft freeze.
001/11:[0002] [FC] 'nbd: Include error names in trace messages'
002/11:[----] [-C] 'nbd: Move nbd_errno_to_system_errno() to public header'
003/11:[----] [--] 'nbd: Expose constants and structs for structured read'
004/11:[down] 'nbd/server: Report error for write to read-only export'
005/11:[down] 'nbd/server: Refactor zero-length option check'
006/11:[0014] [FC] 'nbd: Minimal structured read for server'
007/11:[0016] [FC] 'nbd/server: Include human-readable message in structured errors'
008/11:[0023] [FC] 'nbd/client: refactor nbd_receive_starttls'
009/11:[0004] [FC] 'nbd/client: prepare nbd_receive_reply for structured reply'
010/11:[----] [--] 'nbd: Move nbd_read() to common header'
011/11:[0282] [FC] 'nbd: Minimal structured read for client'
Eric Blake (7):
nbd: Include error names in trace messages
nbd: Move nbd_errno_to_system_errno() to public header
nbd: Expose constants and structs for structured read
nbd/server: Report error for write to read-only export
nbd/server: Refactor zero-length option check
nbd/server: Include human-readable message in structured errors
nbd: Move nbd_read() to common header
Vladimir Sementsov-Ogievskiy (4):
nbd: Minimal structured read for server
nbd/client: refactor nbd_receive_starttls
nbd/client: prepare nbd_receive_reply for structured reply
nbd: Minimal structured read for client
include/block/nbd.h | 106 +++++++++++-
nbd/nbd-internal.h | 23 +--
block/nbd-client.c | 491 ++++++++++++++++++++++++++++++++++++++++++++++++----
nbd/client.c | 215 +++++++++++++++--------
nbd/common.c | 84 +++++++++
nbd/server.c | 194 +++++++++++++++------
nbd/trace-events | 15 +-
7 files changed, 934 insertions(+), 194 deletions(-)
--
2.13.6
next reply other threads:[~2017-10-19 22:26 UTC|newest]
Thread overview: 31+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-10-19 22:26 Eric Blake [this message]
2017-10-19 22:26 ` [Qemu-devel] [PATCH v5 01/11] nbd: Include error names in trace messages Eric Blake
2017-10-19 22:26 ` [Qemu-devel] [PATCH v5 02/11] nbd: Move nbd_errno_to_system_errno() to public header Eric Blake
2017-10-19 22:26 ` [Qemu-devel] [PATCH v5 03/11] nbd: Expose constants and structs for structured read Eric Blake
2017-10-20 8:00 ` Vladimir Sementsov-Ogievskiy
2017-10-19 22:26 ` [Qemu-devel] [PATCH v5 04/11] nbd/server: Report error for write to read-only export Eric Blake
2017-10-20 8:06 ` Vladimir Sementsov-Ogievskiy
2017-10-19 22:26 ` [Qemu-devel] [PATCH v5 05/11] nbd/server: Refactor zero-length option check Eric Blake
2017-10-20 8:34 ` Vladimir Sementsov-Ogievskiy
2017-10-20 15:07 ` Eric Blake
2017-10-20 18:12 ` Vladimir Sementsov-Ogievskiy
2017-10-19 22:26 ` [Qemu-devel] [PATCH v5 06/11] nbd: Minimal structured read for server Eric Blake
2017-10-20 19:03 ` Vladimir Sementsov-Ogievskiy
2017-10-20 19:11 ` Eric Blake
2017-10-20 19:30 ` Vladimir Sementsov-Ogievskiy
2017-10-21 16:02 ` Eric Blake
2017-10-19 22:26 ` [Qemu-devel] [PATCH v5 07/11] nbd/server: Include human-readable message in structured errors Eric Blake
2017-10-20 19:08 ` Vladimir Sementsov-Ogievskiy
2017-10-19 22:26 ` [Qemu-devel] [PATCH v5 08/11] nbd/client: refactor nbd_receive_starttls Eric Blake
2017-10-20 19:26 ` Vladimir Sementsov-Ogievskiy
2017-10-20 19:33 ` Eric Blake
2017-10-19 22:26 ` [Qemu-devel] [PATCH v5 09/11] nbd/client: prepare nbd_receive_reply for structured reply Eric Blake
2017-10-19 22:26 ` [Qemu-devel] [PATCH v5 10/11] nbd: Move nbd_read() to common header Eric Blake
2017-10-19 22:26 ` [Qemu-devel] [PATCH v5 11/11] nbd: Minimal structured read for client Eric Blake
2017-10-20 19:58 ` Vladimir Sementsov-Ogievskiy
2017-10-20 20:46 ` Eric Blake
2017-10-23 11:57 ` Eric Blake
2017-10-23 12:24 ` Vladimir Sementsov-Ogievskiy
2017-10-24 7:31 ` Eric Blake
2017-10-19 23:07 ` [Qemu-devel] [PATCH v5 00/11] nbd minimal structured read no-reply
2017-10-20 15:09 ` Eric Blake
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20171019222637.17890-1-eblake@redhat.com \
--to=eblake@redhat.com \
--cc=pbonzini@redhat.com \
--cc=qemu-block@nongnu.org \
--cc=qemu-devel@nongnu.org \
--cc=vsementsov@virtuozzo.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).