Re: [virtio-dev] [PATCH v8 to v9 incremental] packed ring layout spec

On 03/01/2018 12:10 AM, Michael S. Tsirkin wrote:
> Here's an incremental patch since v8, for ease of review.
> I did my best to address all outstanding comments.
> 
> Changelog:
> 
>  packed-ring: clarify wording on s/g conformance
>  split-ring: typo: aligment
>  packed-ring: typo: aligment
>  packed-ring: clarify multi-buffer requests
>  packed-ring: necessary, not sufficient condition
>  packed-ring: support skipping used descritors
>  packed-ring: fix pseudo code
>  packed-ring: format pseudocode using spaces
>  packed-ring: drop << in pseudocode
>  packed-ring: clarify event suppression format
>  packed-ring: chained descriptors are always on
>  VIRTIO_F_NOTIFICATION_DATA: switch to bitfields
>  introduction: document bitfield notation
>  VIRTIO_F_NOTIFICATION_DATA: minor cleanups
>  content: VIRTIO_F_NOTIFICATION_DATA is option
>  VIRTIO_F_NOTIFICATION_DATA: improve formatting
> 
> Note that I had to move notification format for ccw and mmio out to a
> separate file - latex seems to get confused otherwise.  A clean patchset
> to follow.
> 
> Incrementally redlined pdf is available at the
> virtio-doc repository as virtio-v1.1-w08-to-wd09-diff.pdf
> 
> ---
>

Run trough this. My first impression is changes look good. I will try to
go trough the whole thing again during the next days.

Regards,
Halil