Re: [Qemu-devel] [Nbd] [PATCH v4] doc: Add NBD_CMD_BLOCK_STATUS extension

2016-12-12 Thread Wouter Verhelst
On Mon, Dec 12, 2016 at 09:26:15PM +0100, Wouter Verhelst wrote: > > Do we still want to require servers to always send 16 extents (when not > > limited to exactly 1), or is it better to just state that as long as the > > server sends at least one extent (so that the client can make progress), > >

Re: [Qemu-devel] [Nbd] [PATCH v4] doc: Add NBD_CMD_BLOCK_STATUS extension

2016-12-12 Thread Eric Blake
On 12/12/2016 02:26 PM, Wouter Verhelst wrote: >>> >>> This section describes the value and meaning of constants (other than >>> @@ -768,8 +814,6 @@ The field has the following format: >>>to that command to the client. In the absense of this flag, clients >> >> drive-by typo fix applicable

Re: [Qemu-devel] [Nbd] [PATCH v4] doc: Add NBD_CMD_BLOCK_STATUS extension

2016-12-12 Thread Wouter Verhelst
On Mon, Dec 12, 2016 at 01:58:26PM -0600, Eric Blake wrote: > On 12/12/2016 12:21 PM, Wouter Verhelst wrote: > > diff from v3 to v4: > > - Remove some repetitive wording (some sections were written more than > > once); > > - Rework the text to remove all lingering remains of the "extension" > >