qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
From: Wei Wang <wei.w.wang@intel.com>
To: "Marc-André Lureau" <marcandre.lureau@gmail.com>
Cc: mst@redhat.com, stefanha@redhat.com, pbonzini@redhat.com,
	qemu-devel@nongnu.org, virtio-dev@lists.oasis-open.org
Subject: Re: [Qemu-devel] [RFC PATCH 2/4] vhost-user: add the vhost-user extension to support the vhost-pci based inter-vm communication
Date: Fri, 11 Nov 2016 16:28:21 +0800	[thread overview]
Message-ID: <58258125.3060304@intel.com> (raw)
In-Reply-To: <CAJ+F1CLG2GFJ8bn0Eu9doe9P_8g7z=U4PaQCKUDVNB5A-0HjLg@mail.gmail.com>

On 11/10/2016 07:36 PM, Marc-André Lureau wrote:
> Hi
>
> On Thu, Nov 10, 2016 at 6:47 AM Wei Wang <wei.w.wang@intel.com 
> <mailto:wei.w.wang@intel.com>> wrote:
>
>     This is the slave part of vhost-user implemented in QEMU, with an
>     extension
>     to support vhost-pci.
>
>
> Instead of implementing "another vhost-user slave", it would be worth 
> investigating using libvhost-user instead 
> (https://lists.gnu.org/archive/html/qemu-devel/2016-10/msg03990.html). 
> This is just a suggestion, it is quite fine for vhost-pci to have its 
> own smaller/specific vhost-user slave implementation (without virtio 
> rings handlings etc). (libvhost-user is also very young, not yet in 
> qemu, we should be able shape it for vhost-pci needs)
Thanks for the suggestion. It looks possible. I will need to investigate 
more about it.

libvhost-user is still at an initial stage and not in QEMU yet, I think 
we can probably have vhost-pci as an independent slave for now - this 
will not complicate libvhost-user when it's in the initial stage (it 
might not be good to make it a quite complex design at the beginning), 
and on the other side, the vhost-pci based slave implementation draft is 
already there. We can merge them if it is necessary when they both are 
stable in QEMU.  What do you think?

Best,
Wei

  reply	other threads:[~2016-11-11  8:28 UTC|newest]

Thread overview: 9+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-11-10  2:47 [Qemu-devel] [RFC PATCH 0/4] *** Half-cooked vhost-pci implementation patch *** Wei Wang
2016-11-10  2:47 ` [Qemu-devel] [RFC PATCH 1/4] qemu-char: add the "1-server-N-client" support Wei Wang
2016-11-10 11:38   ` Marc-André Lureau
2016-11-11  8:28     ` Wei Wang
2016-11-10  2:47 ` [Qemu-devel] [RFC PATCH 2/4] vhost-user: add the vhost-user extension to support the vhost-pci based inter-vm communication Wei Wang
2016-11-10 11:36   ` Marc-André Lureau
2016-11-11  8:28     ` Wei Wang [this message]
2016-11-10  2:47 ` [Qemu-devel] [RFC PATCH 3/4] vhost-pci-net device support Wei Wang
2016-11-10  2:47 ` [Qemu-devel] [RFC PATCH 4/4] vhost-user: extend the vhost-user Master (client) part to support vhost-pci Wei Wang

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=58258125.3060304@intel.com \
    --to=wei.w.wang@intel.com \
    --cc=marcandre.lureau@gmail.com \
    --cc=mst@redhat.com \
    --cc=pbonzini@redhat.com \
    --cc=qemu-devel@nongnu.org \
    --cc=stefanha@redhat.com \
    --cc=virtio-dev@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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).