All of lore.kernel.org
 help / color / mirror / Atom feed
From: Nikos Dragazis <ndragazis@arrikto.com>
To: "Michael S. Tsirkin" <mst@redhat.com>
Cc: virtio-dev@lists.oasis-open.org,
	Stefan Hajnoczi <stefanha@gmail.com>,
	"Dr . David Alan Gilbert" <dgilbert@redhat.com>,
	Vangelis Koukis <vkoukis@arrikto.com>,
	Stojaczyk Dariusz <dariusz.stojaczyk@intel.com>
Subject: Re: [virtio-dev] [PATCH v3 0/4] introduce virtio vhost-user backend device type
Date: Wed, 31 Jul 2019 17:49:57 +0300	[thread overview]
Message-ID: <d37d5a6e-f591-5156-126c-bf9ee07ec4aa@arrikto.com> (raw)
In-Reply-To: <20190621164315-mutt-send-email-mst@kernel.org>

On 21/6/19 11:43 μ.μ., Michael S. Tsirkin wrote:
> On Wed, Jun 19, 2019 at 03:54:30PM +0100, Stefan Hajnoczi wrote:
>> On Sat, May 11, 2019 at 09:47:36PM +0300, Nikos Dragazis wrote:
>>> Hi everyone,
>>>
>>> this PATCH presents an updated version of the RFC virtio device spec for
>>> the virtio-vhost-user device. The initial RFC implementation can be
>>> found here: [1].
>>>
>>> This PATCH is split into four parts:
>>>
>>> 1. the first commit is just a refactored version of the initial RFC
>>> implementation [1]. I just moved the device spec into a separate .tex
>>> file and changed the device id from #24 to #28 since the ids #24-#27 are
>>> reserved.
>>>
>>> 2. the second commit fixes some minor issues with the device spec. This
>>> is quite straightforward.
>>>
>>> 3. the third commit enhances the notification capability with some
>>> device/driver requirements. This makes sense because the notification
>>> capability behaves similarly to the MSI-X capability, thereby having
>>> some requirements that need to be mentioned.
>>>
>>> 4. the fourth commit synchronizes the shared memory capability with a
>>> recent patch [2] that attempts to standardize a standalone
>>> SHARED_MEMORY_CFG virtio capability.
>>>
>>> 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
>>>
>>> Looking forward to your comments.
>> I'm happy with v3.
>>
>> Any other comments or shall we proceed to a vote?
>>
>> Stefan
> I think we need to wait for shared memory part to be finalized, right?
>

Michael,

given that David's patchset for the shared memory regions has been
approved, I think we can start discussing on the spec for the
virtio-vhost-user device. A link to the latest version is here: [1].

I really think that this device is useful and should be part of the
VIRTIO specification. I will submit a fifth version of the patchset soon
with some minor changes that seem reasonable to me.

Best regards,
Nikos

[1] https://lists.oasis-open.org/archives/virtio-dev/201906/msg00036.html

---------------------------------------------------------------------
To unsubscribe, e-mail: virtio-dev-unsubscribe@lists.oasis-open.org
For additional commands, e-mail: virtio-dev-help@lists.oasis-open.org


  parent reply	other threads:[~2019-07-31 14:50 UTC|newest]

Thread overview: 15+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2019-05-11 18:47 [virtio-dev] [PATCH v3 0/4] introduce virtio vhost-user backend device type Nikos Dragazis
2019-05-11 18:47 ` [virtio-dev] [PATCH v3 1/4] vhost-user: add vhost-user " Nikos Dragazis
2019-05-11 18:47 ` [virtio-dev] [PATCH v3 2/4] vhost-user: minor fixes Nikos Dragazis
2019-05-11 18:47 ` [virtio-dev] [PATCH v3 3/4] vhost-user: add requirements for the notification capability Nikos Dragazis
2019-05-11 18:47 ` [virtio-dev] [PATCH v3 4/4] vhost-user: update shared memory capability Nikos Dragazis
2019-05-31 16:59   ` [virtio-dev] " Dr. David Alan Gilbert
2019-06-07 12:05     ` Nikos Dragazis
2019-05-20 22:03 ` [virtio-dev] [PATCH v3 0/4] introduce virtio vhost-user backend device type Nikos Dragazis
2019-06-19 14:54 ` Stefan Hajnoczi
2019-06-21 20:43   ` Michael S. Tsirkin
2019-06-24 15:32     ` Nikos Dragazis
2019-07-31 14:49     ` Nikos Dragazis [this message]
2019-07-31 20:19       ` Michael S. Tsirkin
2019-08-04 22:32         ` Nikos Dragazis
2019-08-05  5:22           ` 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=d37d5a6e-f591-5156-126c-bf9ee07ec4aa@arrikto.com \
    --to=ndragazis@arrikto.com \
    --cc=dariusz.stojaczyk@intel.com \
    --cc=dgilbert@redhat.com \
    --cc=mst@redhat.com \
    --cc=stefanha@gmail.com \
    --cc=virtio-dev@lists.oasis-open.org \
    --cc=vkoukis@arrikto.com \
    /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.