On 18/5/20 11:37 μ.μ., Nikos Dragazis wrote:
Hi everyone,
this patch presents an updated version of the virtio device spec for the
virtio-vhost-user device. The initial RFC implementation can be found
here: [1]. It’s been a long time since the last revision. I’m sending
this patch to kick off the discussion again. I'm Cc-ing Stefan and
Michael who had engaged with this in the past.
In this fifth revision of the patch I have made some minor and major
changes. The most important is the drop of the device-specific PCI
capabilities, as Michael suggested [2]. This guarantees portability
across transports, i.e., the device spec is not bound to the PCI
transport, as it used to be up to this point.
In more detail, the changes in this revision are outlined below. You can
find an HTML version here: [3]. Note that I have kept the commits
separate so that it is easy to track the changes from the previous
revision [4].
Looking forward to your comments.
Thanks,
Nikos
v5 changes:
* Change device id from #28 to #32
* Fix definitions of VIRTIO_VHOST_USER_STATUS_{MASTER,SLAVE}_UP bits
* Remove the extra PCI capabilities and use existing virtio registers
instead
* Intercept slave's reply to VHOST_USER_GET_PROTOCOL_FEATURES
* Some more minor fixes (typos, etc.)
v4 changes:
* Add comformance targets for the virtio-vhost-user device.
* Update the reference link for the vhost-user protocol specification.
v3 changes:
* Device Requirements for the Notification Capability: point out the
difference between the MSI-X Table Size stored in the Message Control
register of the MSI-X capability structure and the actual MSI-X Table
Size
v2 changes:
* Change device id from #25 to #28
[1] https://lists.oasis-open.org/archives/virtio-dev/201801/msg00110.html
[2] https://lists.oasis-open.org/archives/virtio-dev/201908/msg00014.html
[3] https://ndragazis.github.io/virtio/vvu/v5/virtio-v1.1-cs01.html#x1-49600014
[4] https://lists.oasis-open.org/archives/virtio-dev/201906/msg00036.html
Nikos Dragazis (9):
vhost-user: minor fixes
vhost-user: add requirements for the notification capability
vhost-user: update shared memory capability
vhost-user: add conformance targets and clauses
vhost-user: change the device id
vhost-user: minor fix on status register
vhost-user: remove the extra PCI capabilities
vhost-user: intercept slave's reply to
VHOST_USER_GET_PROTOCOL_FEATURES
vhost-user: clarify that we are talking about slave-initiated messages
Stefan Hajnoczi (1):
vhost-user: add vhost-user device type
conformance.tex | 11 +-
content.tex | 3 +
introduction.tex | 1 +
virtio-vhost-user.tex | 282 ++++++++++++++++++++++++++++++++++++++++++
4 files changed, 295 insertions(+), 2 deletions(-)
create mode 100644 virtio-vhost-user.tex
Kind reminder on this.
Did anyone have the chance to look at this patchset? Is there anything
that seems off or doesn't make sense? Is there anything else I can do to
help?
Any feedback is more than welcome.
Thanks,
Nikos
---------------------------------------------------------------------
To unsubscribe, e-mail: virtio-dev-unsubscr...@lists.oasis-open.org
For additional commands, e-mail: virtio-dev-h...@lists.oasis-open.org