From: "Michael S. Tsirkin" <mst@redhat.com>
To: virtio@lists.oasis-open.org, virtio-dev@lists.oasis-open.org
Cc: Cornelia Huck <cohuck@redhat.com>,
Halil Pasic <pasic@linux.vnet.ibm.com>,
Tiwei Bie <tiwei.bie@intel.com>,
Stefan Hajnoczi <stefanha@redhat.com>,
"Dhanoa, Kully" <kully.dhanoa@intel.com>
Subject: [virtio] [PATCH v9 00/16] packed ring layout spec
Date: Thu, 1 Mar 2018 01:31:18 +0200 [thread overview]
Message-ID: <1519860484-7936-1-git-send-email-mst@redhat.com> (raw)
This addresses comments on v8.
Thanks a lot to all reviewers of v8 - I hope we are
finally there or almost there.
A compiled version can
be found under https://github.com/oasis-tcs/virtio-docs.git
virtio-v1.1-packed-wd09-diff.pdf virtio-v1.1-packed-wd09.pdf
for redline and clean versions, respectively.
If you are interested in changes from v8, that's in
virtio-v1.1-w08-to-wd09-diff.pdf
in this directory.
Note: please do not try to edit the pdf and post comments
in the edited file. Please post comments in a text
format, as pdfs are not archived with the list.
TODO: support for actual passthrough devices will likely
require more new features, such as requirement for
stronger memory barriers.
Note: should this proposal be accepted and approved, one or more
claims disclosed to the TC admin and listed on the Virtio TC
IPR page https://github.com/oasis-tcs/virtio-admin/blob/master/IPR.md
might become Essential Claims.
Changes from v8:
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
Changes from v7:
- new notitfication_data feature, supported for all
transports and formats
- addressed all outstanding comments
Changes from v6:
- isolate in-order feature to a separate set of patches
(reduces scope in case there's more discussion around it)
- support in-order option for split rings
- update all references to available/used ring in spec
to a format-agnostic terminology
- minor changes to event suppression format
- minor changes to notification format
- lots of new conformance clauses
Changes from v5:
- scope reductions (see below). We can add more
features down the road, hopefully reduced scope will be enough
to finalize spec soon.
- cleanup and integrate in the spec
- pseudo-code
Deferred features:
- dropped _F_DESC_LIST, 1.0 includes this unconditionally, we
can do same
- dropped event structure change notifications - needed for
efficient hardware implementations but let's add this on top
3 1st patches just move text around so all virtio 1.0
things are in the same place. 2 last ones add the new layout
Option to mark descriptors as not generating events isn't
yet implemented. Again, let's add this on top.
I also note that for hardware implementations, a different
set of memory barriers is needed. Again, let's add this on top
Michael S. Tsirkin (16):
introduction: document bitfield notation
content: move 1.0 queue format out to a separate section
content: move ring text out to a separate file
content: move virtqueue operation description
content: len -> used length, used ring -> vq
content: generalize transport ring part naming
content: generalize rest of text
split-ring: generalize text
split-ring: typo: aligment
packed virtqueues: more efficient virtqueue layout
content: in-order buffer use
packed-ring: add in order support
split-ring: in order feature
VIRTIO_F_NOTIFICATION_DATA: extra data to devices
makediff: update to show diff from master
REVISION: set to 1.1 packed wd09
REVISION | 2 +-
conformance.tex | 5 +-
content.tex | 919 ++++++++++++++-----------------------------------------
introduction.tex | 18 ++
makediff.sh | 3 +-
notifications.c | 3 +
packed-ring.tex | 704 ++++++++++++++++++++++++++++++++++++++++++
split-ring.tex | 689 +++++++++++++++++++++++++++++++++++++++++
8 files changed, 1655 insertions(+), 688 deletions(-)
create mode 100644 notifications.c
create mode 100644 packed-ring.tex
create mode 100644 split-ring.tex
--
MST
---------------------------------------------------------------------
To unsubscribe from this mail list, you must leave the OASIS TC that
generates this mail. Follow this link to all your TCs in OASIS at:
https://www.oasis-open.org/apps/org/workgroup/portal/my_workgroups.php
next reply other threads:[~2018-02-28 23:31 UTC|newest]
Thread overview: 36+ messages / expand[flat|nested] mbox.gz Atom feed top
2018-02-28 23:31 Michael S. Tsirkin [this message]
2018-02-28 23:31 ` [virtio] [PATCH v9 01/16] introduction: document bitfield notation Michael S. Tsirkin
2018-02-28 23:31 ` [virtio] [PATCH v9 02/16] content: move 1.0 queue format out to a separate section Michael S. Tsirkin
2018-02-28 23:31 ` [virtio] [PATCH v9 03/16] content: move ring text out to a separate file Michael S. Tsirkin
2018-02-28 23:31 ` [virtio] [PATCH v9 04/16] content: move virtqueue operation description Michael S. Tsirkin
2018-02-28 23:31 ` [virtio] [PATCH v9 05/16] content: len -> used length, used ring -> vq Michael S. Tsirkin
2018-02-28 23:31 ` [virtio] [PATCH v9 06/16] content: generalize transport ring part naming Michael S. Tsirkin
2018-02-28 23:31 ` [virtio] [PATCH v9 07/16] content: generalize rest of text Michael S. Tsirkin
2018-02-28 23:31 ` [virtio] [PATCH v9 08/16] split-ring: generalize text Michael S. Tsirkin
2018-03-07 11:00 ` [virtio] " Cornelia Huck
2018-03-09 22:56 ` Michael S. Tsirkin
2018-02-28 23:31 ` [virtio] [PATCH v9 09/16] split-ring: typo: aligment Michael S. Tsirkin
2018-03-07 10:56 ` [virtio] " Cornelia Huck
2018-02-28 23:31 ` [virtio] [PATCH v9 11/16] content: in-order buffer use Michael S. Tsirkin
2018-02-28 23:31 ` [virtio] [PATCH v9 10/16] packed virtqueues: more efficient virtqueue layout Michael S. Tsirkin
2018-03-07 13:47 ` [virtio] " Michael S. Tsirkin
2018-02-28 23:31 ` [virtio] [PATCH v9 12/16] packed-ring: add in order support Michael S. Tsirkin
2018-02-28 23:31 ` [virtio] [PATCH v9 13/16] split-ring: in order feature Michael S. Tsirkin
2018-02-28 23:31 ` [virtio] [PATCH v9 14/16] VIRTIO_F_NOTIFICATION_DATA: extra data to devices Michael S. Tsirkin
2018-03-07 11:11 ` [virtio] " Cornelia Huck
2018-03-07 14:09 ` Michael S. Tsirkin
2018-03-07 14:49 ` Cornelia Huck
2018-03-07 15:10 ` Michael S. Tsirkin
2018-03-07 15:13 ` Cornelia Huck
2018-03-07 16:05 ` Halil Pasic
2018-03-07 16:14 ` Cornelia Huck
2018-03-07 19:53 ` Michael S. Tsirkin
2018-03-08 13:03 ` [virtio] Re: [virtio-dev] " Halil Pasic
2018-03-08 16:19 ` Michael S. Tsirkin
2018-03-09 8:16 ` Cornelia Huck
2018-03-09 12:25 ` Halil Pasic
2018-03-09 16:52 ` Michael S. Tsirkin
2018-03-07 19:27 ` Michael S. Tsirkin
2018-02-28 23:31 ` [virtio] [PATCH v9 15/16] makediff: update to show diff from master Michael S. Tsirkin
2018-02-28 23:31 ` [virtio] [PATCH dont commit v9 16/16] REVISION: set to 1.1 packed wd09 Michael S. Tsirkin
2018-03-09 17:06 ` [virtio] Re: [PATCH v9 00/16] packed ring layout spec Michael S. Tsirkin
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=1519860484-7936-1-git-send-email-mst@redhat.com \
--to=mst@redhat.com \
--cc=cohuck@redhat.com \
--cc=kully.dhanoa@intel.com \
--cc=pasic@linux.vnet.ibm.com \
--cc=stefanha@redhat.com \
--cc=tiwei.bie@intel.com \
--cc=virtio-dev@lists.oasis-open.org \
--cc=virtio@lists.oasis-open.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.