On 13 Apr 2016, at 16:24, Eric Blake <eblake@...696...> wrote: >>> >>> + * `NBD_INFO_MAXIMUM_BLOCK` (4) >>> + >>> + The *info length* MUST be 4, and represents the maximum block >>> + size. See the "Block sizes" section for further requirements on >>> + its value. >>> + >>> + - 32 bits, maximum block size >> >> I like these. > > Is it worth keeping the three separate (where the server can advertise > one, but not all three, and the others fall back to defaults), or is it > easier to just always require all three to be present (since we've > documented sane defaults, a server should always be able to supply > something)? I think having them all in one option is easier. I guess we could permit '0' to mean 'the default' but TBH I'd prefer if you specified one of them you specify all of them, which would also avoid the need for the rather convoluted language I tried to suggest about the need to ensure that if you only specify some of them those ones need to be compatible with the default. -- Alex Bligh
Attachment:
signature.asc
Description: Message signed with OpenPGP using GPGMail