All of lore.kernel.org
 help / color / mirror / Atom feed
From: Wouter Verhelst <w@uter.be>
To: "Daniel P. Berrange" <berrange@redhat.com>
Cc: nbd-general@lists.sourceforge.net,
	Paolo Bonzini <pbonzini@redhat.com>,
	qemu-devel@nongnu.org, qemu-block@nongnu.org
Subject: Re: [Qemu-devel] [PATCH 00/15] Implement TLS support to QEMU NBD server & client
Date: Thu, 3 Dec 2015 15:53:14 +0100	[thread overview]
Message-ID: <20151203145314.GA7641@grep.be> (raw)
In-Reply-To: <20151127140651.GA575@grep.be>

Hi all,

On Fri, Nov 27, 2015 at 03:06:51PM +0100, Wouter Verhelst wrote:
> I have been thinking of adding a message NBD_OPT_SELECT_EXPORT to
> replace NBD_OPT_EXPORT_NAME, which would select an export but not end
> negotiation. That would also require another message to end negotiation
> and move on to the transmission phase, obviously.

I've formalized this now, and removed the PEEK_EXPORT extension (which, as you
say, can't be used safely).

> The negotation would then look something like:
> 
> NBD_OPT_SELECT_EXPORT
>  <- NBD_REP_ACK

This has become an NBD_REP_SERVER instead, with (64-bit) export size and
(16-bit) per-export flags appended (rather than yet another flags field).

(the name is also NBD_OPT_SELECT rather than NBD_OPT_SELECT_EXPORT; details)

NBD_OPT_SELECT is 6, NBD_OPT_GO is 7, NBD_REP_ERR_UNKNOWN is 2^31+6.

For more details, see the proto.md document.

Thoughts are welcome.

-- 
It is easy to love a country that is famous for chocolate and beer

  -- Barack Obama, speaking in Brussels, Belgium, 2014-03-26

  reply	other threads:[~2015-12-03 14:53 UTC|newest]

Thread overview: 23+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-11-27 12:20 [Qemu-devel] [PATCH 00/15] Implement TLS support to QEMU NBD server & client Daniel P. Berrange
2015-11-27 12:20 ` [Qemu-devel] [PATCH 01/15] qom: add user_creatable_add & user_creatable_del methods Daniel P. Berrange
2015-11-27 12:20 ` [Qemu-devel] [PATCH 02/15] qemu-nbd: add support for --object command line arg Daniel P. Berrange
2015-11-27 12:20 ` [Qemu-devel] [PATCH 03/15] nbd: convert block client to use I/O channels for connection setup Daniel P. Berrange
2015-11-27 12:20 ` [Qemu-devel] [PATCH 04/15] nbd: convert qemu-nbd server " Daniel P. Berrange
2015-11-27 12:20 ` [Qemu-devel] [PATCH 05/15] nbd: convert blockdev NBD " Daniel P. Berrange
2015-11-27 12:20 ` [Qemu-devel] [PATCH 06/15] nbd: convert to using I/O channels for actual socket I/O Daniel P. Berrange
2015-11-27 12:20 ` [Qemu-devel] [PATCH 07/15] nbd: invert client logic for negotiating protocol version Daniel P. Berrange
2015-11-27 12:20 ` [Qemu-devel] [PATCH 08/15] nbd: make server compliant with fixed newstyle spec Daniel P. Berrange
2015-11-27 12:20 ` [Qemu-devel] [PATCH 09/15] nbd: make client request fixed new style if advertized Daniel P. Berrange
2015-11-27 12:20 ` [Qemu-devel] [PATCH 10/15] nbd: allow setting of an export name for qemu-nbd server Daniel P. Berrange
2015-11-27 12:20 ` [Qemu-devel] [PATCH 11/15] nbd: pick first exported volume if no export name is requested Daniel P. Berrange
2015-11-27 12:20 ` [Qemu-devel] [PATCH 12/15] nbd: implement TLS support in the protocol negotiation Daniel P. Berrange
2015-11-28 10:28   ` Wouter Verhelst
2015-12-02 10:45     ` Daniel P. Berrange
2015-11-27 12:20 ` [Qemu-devel] [PATCH 13/15] nbd: enable use of TLS with NBD block driver Daniel P. Berrange
2015-11-27 12:20 ` [Qemu-devel] [PATCH 14/15] nbd: enable use of TLS with qemu-nbd server Daniel P. Berrange
2015-11-27 12:20 ` [Qemu-devel] [PATCH 15/15] nbd: enable use of TLS with nbd-server-start command Daniel P. Berrange
2015-11-27 14:06 ` [Qemu-devel] [PATCH 00/15] Implement TLS support to QEMU NBD server & client Wouter Verhelst
2015-12-03 14:53   ` Wouter Verhelst [this message]
2015-12-02 12:56 ` Wouter Verhelst
2015-12-02 13:37   ` Daniel P. Berrange
2015-12-02 13:45     ` Wouter Verhelst

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=20151203145314.GA7641@grep.be \
    --to=w@uter.be \
    --cc=berrange@redhat.com \
    --cc=nbd-general@lists.sourceforge.net \
    --cc=pbonzini@redhat.com \
    --cc=qemu-block@nongnu.org \
    --cc=qemu-devel@nongnu.org \
    /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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.