[Top][All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Qemu-devel] [Nbd] [PATCHv2] Amend NBD_OPT_SELECT (now NBD_OPT_INFO)

From: Wouter Verhelst
Subject: Re: [Qemu-devel] [Nbd] [PATCHv2] Amend NBD_OPT_SELECT (now NBD_OPT_INFO) and NBD_OPT_GO documentation
Date: Tue, 5 Apr 2016 23:03:17 +0200
User-agent: Mutt/1.5.24 (2015-08-30)

On Tue, Apr 05, 2016 at 09:42:26PM +0100, Alex Bligh wrote:
> Amend the NBD_OPT_SELECT and NBD_OPT_GO documentation as
> follows:
> * Change NBD_OPT_SELECT to be called NBD_OPT_INFO
> * Remove the 'selection' aspect of that command, so that
>   it now merely returns information. This is to avoid
>   the server storing state.
> * Allow a name to be specified on NBD_OPT_GO
> * Make clear the rules for default device selection
> * Remove the provision concerning TLS resetting device selection
> * Remove NBD_REP_ERR_INVALID as a reply to NBD_OPT_GO as there
>   is now no necessity for a prior NBD_OPT_INFO
> * Make it clear NBD_OPT_GO is in effect a better alternative
> * Make it clear the NBD_OPT_INFO and NBD_OPT_GO are in
>   essence the same command, save that NBD_OPT_GO puts you
>   into transmission mode if successful.
> * Clarify permitted option returns outside TLS to prevent
>   export enumeration.
> * Remove 'length' 32 bit quantity from
>   NBD_OPT_SELECT (and don't copy it into NBD_OPT_GO) so it
>   looks exactly like NBD_OPT_EXPORT_NAME bar the reply.
>   This length is unnecessary as it's in the option header
>   anyway.
> * Reorder the fields of an NBD_OPT_INFO / NBD_OPT_GO reply
>   so the variable length elements are at the end.
> * Make the documentation much more concise.
> Signed-off-by: Alex Bligh <address@hidden>

applied, thanks.

< ron> I mean, the main *practical* problem with C++, is there's like a dozen
       people in the world who think they really understand all of its rules,
       and pretty much all of them are just lying to themselves too.
 -- #debian-devel, OFTC, 2016-02-12

reply via email to

[Prev in Thread] Current Thread [Next in Thread]