On 05/17/2016 10:26 AM, Alex Bligh wrote: > > On 17 May 2016, at 16:54, Richard W.M. Jones <rjones@...696...> wrote: > >> On Tue, May 17, 2016 at 04:22:06PM +0100, Alex Bligh wrote: >>> nbdkit is non-compliant in that case. Support of NBD_OPT_LIST is >>> compulsory, even if you support it by returning a nameless export >>> (or default). Moreover support of export names is compulsory >>> (even if you have a single fixed one called "" or "default"). >> >> The protocol doc doesn't mention "default" (assuming you mean that >> literal string). It says: > > As per my message to Eric, I meant use an arbitrary piece of > text in your reply to NBD_OPT_LIST (it doesn't matter what it is, > 'default' is not special). It doesn't matter because you ignore > what is passed in NBD_OPT_EXPORT_NAME. I was just suggesting making > things more readable for clients that did a list. > >>> Whilst that is fair enough, NBD_OPT_LIST is a mandatory part of the >>> specification. If a server doesn't implement mandatory parts of >>> the specification, then bad things will happen. >> >> It implements it, it's just that there wasn't a way to implement >> anything other than returning an error since we accept anything as an >> export name. > > In which case you should just return an export name. Any > export name will work. The fact you don't have names for your > exports doesn't mean (in my book) you can error the list > request. You have one export. You don't know what it's name is, > but you should list it. Except that the qemu 2.6 client is specifically checking that the name it is looking for is present in the list, and gives up trying if the name was not found. And there is no way for the server to know a priori which name the client is trying to match. An arbitrary name is less likely to be found; the empty name "" is the most likely arbitrary name, but even then, there is no guarantee that the qemu client is using the "" default name, and since NBD_OPT_EXPORT_NAME doesn't care, I think it's nicer to state that NBD_OPT_LIST should ONLY be supported if names matter to NBD_OPT_EXPORT_NAME (and should return NBD_REP_ERR_UNSUP when names don't matter). -- Eric Blake eblake redhat com +1-919-301-3266 Libvirt virtualization library http://libvirt.org
Attachment:
signature.asc
Description: OpenPGP digital signature