[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH] doc: Mention proper use of handle
From: |
Wouter Verhelst |
Subject: |
Re: [Qemu-devel] [PATCH] doc: Mention proper use of handle |
Date: |
Tue, 29 Mar 2016 09:11:24 +0200 |
User-agent: |
Mutt/1.5.24 (2015-08-30) |
On Mon, Mar 28, 2016 at 07:59:15AM -0600, Eric Blake wrote:
> Although the proper use of the handle field during transmission
> phase was implied, it never hurts to make it more explicit that
> clients should alter the handle on each message, and the server
> repeat the handle unchanged, in order for the client to track
> when the server is sending replies out of order.
>
> Signed-off-by: Eric Blake <address@hidden>
> ---
> doc/proto.md | 6 +++++-
> 1 file changed, 5 insertions(+), 1 deletion(-)
>
> diff --git a/doc/proto.md b/doc/proto.md
> index 6d1cb34..d0102e0 100644
> --- a/doc/proto.md
> +++ b/doc/proto.md
> @@ -200,7 +200,11 @@ S: 64 bits, handle
> S: (*length* bytes of data if the request is of type `NBD_CMD_READ`)
>
> Replies need not be sent in the same order as requests (i.e., requests
> -may be handled by the server asynchronously).
> +may be handled by the server asynchronously). Clients SHOULD send a
> +different value of handle for each request, and the server MUST use the
> +same value for handle as was sent by the client for each request that
> +the server is replying to, so that the client may correlate which
> +request is receiving a response.
NAK. This implies that a client should not ever reuse handles, while it
is legal for a client (and in fact the kernel does this) to reuse
handles once the server has ack'd the request.
--
< 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
- Re: [Qemu-devel] [Nbd] [PATCH 3/1] doc: Propose Structured Replies extension, (continued)
- Re: [Qemu-devel] [Nbd] [PATCH 3/1] doc: Propose Structured Replies extension, Wouter Verhelst, 2016/03/29
- Re: [Qemu-devel] [Nbd] [PATCH 3/1] doc: Propose Structured Replies extension, Wouter Verhelst, 2016/03/29
- Re: [Qemu-devel] [Nbd] [PATCH 3/1] doc: Propose Structured Replies extension, Alex Bligh, 2016/03/29
- Re: [Qemu-devel] [Nbd] [PATCH 3/1] doc: Propose Structured Replies extension, Eric Blake, 2016/03/29
- Re: [Qemu-devel] [Nbd] [PATCH 3/1] doc: Propose Structured Replies extension, Alex Bligh, 2016/03/29
- Re: [Qemu-devel] [Nbd] [PATCH 3/1] doc: Propose Structured Replies extension, Alex Bligh, 2016/03/29
- Re: [Qemu-devel] [Nbd] [PATCH 3/1] doc: Propose Structured Replies extension, Wouter Verhelst, 2016/03/29
- Re: [Qemu-devel] [Nbd] [PATCH 3/1] doc: Propose Structured Replies extension, Alex Bligh, 2016/03/29
- Re: [Qemu-devel] [Nbd] [PATCH 3/1] doc: Propose Structured Replies extension, Wouter Verhelst, 2016/03/29
- Re: [Qemu-devel] [Nbd] [PATCH 3/1] doc: Propose Structured Replies extension, Alex Bligh, 2016/03/29
Re: [Qemu-devel] [PATCH] doc: Mention proper use of handle,
Wouter Verhelst <=
[Qemu-devel] [PATCH v2 0/3] NBD Structured Read, Eric Blake, 2016/03/29
- [Qemu-devel] [PATCH v2 2/3] doc: Mention proper use of handle, Eric Blake, 2016/03/29
- [Qemu-devel] [PATCH v2 3/3] doc: Propose Structured Read extension, Eric Blake, 2016/03/29
- Re: [Qemu-devel] [PATCH v2 3/3] doc: Propose Structured Read extension, Eric Blake, 2016/03/29
- Re: [Qemu-devel] [PATCH v2 3/3] doc: Propose Structured Read extension, Alex Bligh, 2016/03/30
- Re: [Qemu-devel] [PATCH v2 3/3] doc: Propose Structured Read extension, Eric Blake, 2016/03/30
- Re: [Qemu-devel] [Nbd] [PATCH v2 3/3] doc: Propose Structured Read extension, Wouter Verhelst, 2016/03/30
- Re: [Qemu-devel] [Nbd] [PATCH v2 3/3] doc: Propose Structured Read extension, Eric Blake, 2016/03/30
- Re: [Qemu-devel] [Nbd] [PATCH v2 3/3] doc: Propose Structured Read extension, Wouter Verhelst, 2016/03/30