On Wed, Sep 16, 2015 at 02:39:06PM +0300, Marcel Apfelbaum wrote:
> On 09/15/2015 10:10 AM, Yuanhan Liu wrote:
> >Hi,
> >
> >Here is the updated patch set for enabling vhost-user multiple queue.
>
> Hi,
>
> I think this series will not work with the current DPDK vhost-user MQ patches,
Yes. The new patch set since v7 has made interface changes. Hence it
will not work the v4 DPDK vhost-user MQ patches.
> can you please post an RFC on the dpdk list?
I guess you could see that soon, before the end of this week.
--yliu
> If you don't have anything reedy yet, a pointer to a git would also be
> appreciated.
>
> Thanks,
> Marcel
>
> >
> >This patch set introduces 2 more vhost user messages:
> >VHOST_USER_GET_QUEUE_NUM,
> >for querying how many queues the backend supports, and
> >VHOST_USER_SET_VRING_ENABLE,
> >for enabling/disabling a specific virt queue.
> >
> >Both of the two new messages are treated as vhost protocol extension,
> >and that's why Michaels's patch "vhost-user: add protocol feature
> >negotiation" is also included here.
> >
> >Patch 1-5 are all prepare works for actually enabling multiple queue.
> >
> >Patch 6 is the major patch for enabling multiple queue, which also tries
> >to address two major concerns from Michael: no feedback from backend if
> >it can't support # of requested queues, and all messages are sent N time.
> >It also fixes a hidden bug.
> >
> >Patch 7 introduces the VHOST_USER_SET_VRING_ENABLE message, to enable
> >or disable a specific vring.
> >
> >Note that I haven't done any formal test yet, it just passes build
> >test and basic functional test, such as it does exit when backend
> >doesn't support # of requested queues. Here I sent it out just for
> >more comments, and for avoiding spending too much effort on the wrong
> >track.
> >
> >
> >v9: - Per suggested by Jason Wang, patch 5 introduces a new vhost
> > backend method: vhost_backend_get_vq_index().
> >
> > - Use qemu_find_net_clients_except() at net/vhost-user.c for
> > gathering all related ncs so that we could register chr dev
> > event handler once. Which is also suggested by Jason Wang.
> >
> >
> >Thanks.
> >
> > --yliu
> >
> >---
> >Changchun Ouyang (2):
> > vhost-user: add multiple queue support
> > vhost-user: add a new message to disable/enable a specific virt queue.
> >
> >Michael S. Tsirkin (1):
> > vhost-user: add protocol feature negotiation
> >
> >Yuanhan Liu (4):
> > vhost-user: use VHOST_USER_XXX macro for switch statement
> > vhost: rename VHOST_RESET_OWNER to VHOST_RESET_DEVICE
> > vhost-user: add VHOST_USER_GET_QUEUE_NUM message
> > vhost: introduce vhost_backend_get_vq_index method
> >
> > docs/specs/vhost-user.txt | 75 +++++++++++++++++++-
> > hw/net/vhost_net.c | 39 ++++++++--
> > hw/net/virtio-net.c | 8 +++
> > hw/virtio/vhost-backend.c | 10 ++-
> > hw/virtio/vhost-user.c | 143
> > +++++++++++++++++++++++++++++++------
> > hw/virtio/vhost.c | 20 +++---
> > include/hw/virtio/vhost-backend.h | 4 ++
> > include/hw/virtio/vhost.h | 2 +
> > include/net/vhost_net.h | 3 +
> > linux-headers/linux/vhost.h | 2 +-
> > net/vhost-user.c | 146
> > +++++++++++++++++++++++++++-----------
> > qapi-schema.json | 6 +-
> > qemu-options.hx | 5 +-
> > tests/vhost-user-test.c | 2 +-
> > 14 files changed, 380 insertions(+), 85 deletions(-)
> >
