Re: [Nbd] [PATCHv2] Amend NBD_OPT_SELECT (now NBD_OPT_INFO) and NBD_OPT_GO documentation
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
> for NBD_OPT_EXPORT_NAME
>
> * 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 <alex@...872...>
[...]
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 to: