All of lore.kernel.org
 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 1/4] qemu-char: add the "1-server-N-client" support
Date: Fri, 11 Nov 2016 16:28:58 +0800	[thread overview]
Message-ID: <5825814A.8060902@intel.com> (raw)
In-Reply-To: <CAJ+F1CLFuvpa1ubAMyAbfd4vBfSc7hpH_D++eCv4enDd71-_qg@mail.gmail.com>

On 11/10/2016 07:38 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 patch enables a qemu server socket to be connected by multiple
>     client sockets.
>
> Thanks for sharing this early version of the series, I hope some early 
> feedback will help you. I'll be waiting for a more complete 
> implementation for detailed review.
>
> Is this patch necessary as a first step? I would rather start with  a 
> vhost-pci 1-1 Master-Slave series. Keep 1-n for a following 
> improvement. This would also probably post-pone the discussion 
> regarding connection-id, or uuid.
>
> In short, I think it would help if you can break your proposal in 
> smaller independant steps.
>

OK, we can leave this QEMU socket patch to the 2nd step.

So, I think we can have a two-step plan:
Step1: 1-1 QEMU socket based vhost-pci-net design
    Each QEMU socket manages (create/hotplug/destruction) only 1 
vhost-pci-net device. To create more vhost-pci-net devices, the slave VM 
needs to have more server sockets created at booting time.
Step2: 1-server-N-client based vhost-pci design
   A single QEMU server socket manages all the vhost-pci devices (may 
also in different device types).

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 [this message]
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
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=5825814A.8060902@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 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.