Re: [Nbd] [PATCH v4 04/11] nbd: Improve server handling of bogus commands
- To: Wouter Verhelst <w@...112...>
- Cc: "nbd-general@lists.sourceforge.net" <nbd-general@lists.sourceforge.net>, Paolo Bonzini <pbonzini@...696...>, qemu block <qemu-block@...530...>, "qemu-devel@...530..." <qemu-devel@...530...>
- Subject: Re: [Nbd] [PATCH v4 04/11] nbd: Improve server handling of bogus commands
- From: Alex Bligh <alex@...872...>
- Date: Wed, 15 Jun 2016 09:52:21 +0100
- Message-id: <077010D7-81BE-4AD5-B36A-408844852006@...872...>
- In-reply-to: <20160615080346.GA6994@...3...>
- References: <1463006384-7734-1-git-send-email-eblake@...696...> <1463006384-7734-5-git-send-email-eblake@...696...> <852e526a-5235-499a-741e-a695f5e74f83@...696...> <575EA656.80508@...696...> <6DD06745-C91C-4BFB-BFE5-92E5982ACB42@...872...> <11f620d2-a51d-5235-5abd-4ced314c9090@...696...> <38ABE56B-CA23-4372-A413-CDA72BDAE86A@...872...> <20160615070522.GC3787@...3...> <20160615080346.GA6994@...3...>
> On 15 Jun 2016, at 09:03, Wouter Verhelst <w@...112...> wrote:
>
> On Wed, Jun 15, 2016 at 09:05:22AM +0200, Wouter Verhelst wrote:
>> There are more clients than the Linux and qemu ones, but I think it's
>> fair to say that those two are the most important ones. If they agree
>> that a read reply which errors should come without payload, then I think
>> we should update the standard to say that, too.
>
> I've just pushed a commit that changes the spec (and the implementation)
> so that if a server encounters a read error, it does not send a payload.
>
> In other words, the current behaviour of qemu is correct, is now
> documented to be correct, and should not be changed.
So what should those servers do (like 2 of mine) which don't buffer
the entire read, if they get an error having already sent some data?
--
Alex Bligh
Attachment:
signature.asc
Description: Message signed with OpenPGP using GPGMail
Reply to: