Re: [Nbd] [PATCH] Further tidy-up on block status
- To: Vladimir Sementsov-Ogievskiy <vsementsov@...2319...>
- Cc: "nbd-general@lists.sourceforge.net" <nbd-general@lists.sourceforge.net>, Kevin Wolf <kwolf@...696...>, "Denis V . Lunev" <den@...2317...>, "qemu-devel@...530..." <qemu-devel@...530...>, "Stefan stefanha@...2321... com" <stefanha@...696...>, Paolo Bonzini <pbonzini@...696...>, Wouter Verhelst <w@...112...>, John Snow <jsnow@...696...>
- Subject: Re: [Nbd] [PATCH] Further tidy-up on block status
- From: Alex Bligh <alex@...872...>
- Date: Thu, 29 Dec 2016 16:08:02 +0000
- Message-id: <0D5EAC18-B7BE-46AF-85CE-8488AB6C9448@...872...>
- In-reply-to: <d7e58b6e-815c-ffb6-ac68-ef52b0082c50@...2319...>
- References: <20161214150840.10899-1-alex@...872...> <31576d46-c0ed-29b9-71a0-5aca1790799a@...2319...> <6D1B30FC-FD7E-474C-A8E3-FD87E7AA1364@...872...> <5e9150ed-2127-f2e8-f9db-a514e8f0ddf8@...2319...> <5E697C22-5FBB-49A2-A018-A6B96E29FE84@...872...> <94ef3ef2-b76f-fa5d-cbaf-8990ce2b1be8@...2319...> <20161214181323.mehzfmlf6z4pyajp@...3...> <D71CECFD-E513-4372-BB54-BF8D8D521292@...872...> <20161214184937.uv5gwkijaofhjfd7@...3...> <95CEC42B-486F-45F2-8879-ACF48C2380FD@...872...> <d7e58b6e-815c-ffb6-ac68-ef52b0082c50@...2319...>
Vladimir,
> On 26 Dec 2016, at 15:57, Vladimir Sementsov-Ogievskiy <vsementsov@...2723...9...> wrote:
>
>> OK, so first of all, one of the changes I made earlier was that now
>> each of the commands carries a list of queries, the way you list
>> everything is not 'having a query that doesn't contain a namespace'
>> but rather doing a _LIST_ with no queries at all. But that's semantics
>> and orthogonal to the main point.
>>
>> What I've proposed (and pushed - but feel free to alter it) is that
>
> I think, an overview in "## Metadata querying" should be updated too.
Sorry, which bit needs updating?
The only bit in that more general section that mentions _LIST_ is:
> First, during negotiation, if the client wishes to query metadata
> during transmission, the client MUST select one or more metadata
> contexts with the NBD_OPT_SET_META_CONTEXT command. If needed, the
> client can use NBD_OPT_LIST_META_CONTEXT to list contexts that the
> server supports.
That still seems correct.
--
Alex Bligh
Reply to: