From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:40802) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1ZbPZw-0007os-17 for qemu-devel@nongnu.org; Mon, 14 Sep 2015 04:57:36 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1ZbPZs-0001Zk-RD for qemu-devel@nongnu.org; Mon, 14 Sep 2015 04:57:35 -0400 Received: from mga02.intel.com ([134.134.136.20]:53884) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1ZbPZs-0001Zb-LU for qemu-devel@nongnu.org; Mon, 14 Sep 2015 04:57:32 -0400 From: Yuanhan Liu Date: Mon, 14 Sep 2015 16:58:06 +0800 Message-Id: <1442221092-4545-1-git-send-email-yuanhan.liu@linux.intel.com> Subject: [Qemu-devel] [PATCH 0/6 v8] vhost-user multiple queue support List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: qemu-devel@nongnu.org Cc: jasowang@redhat.com, changchun.ouyang@intel.com, Yuanhan Liu , mst@redhat.com Hi, Here is the updated patch set for enabling vhost-user multiple queue. 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-4 are all prepare works for actually enabling multiple queue. Patch 5 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 6 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. v8: - set vq_index for vhost-user inside vhost_net_init() based on the net->nc->queue_index field. Per Jason Wang suggested, we can't move the vhost_net_set_vq_index() ahead to inside of vhost_net_init(), as it will break vhost-kernel multiple queue support(with tap). And that's why it's one patch less in this version. - Rename VHOST_USER_SET_VRING_FLAG to VHOST_USER_SET_VRING_ENABLE. 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 (3): 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 docs/specs/vhost-user.txt | 75 +++++++++++++++++++- hw/net/vhost_net.c | 34 ++++++++- hw/net/virtio-net.c | 8 +++ hw/virtio/vhost-user.c | 140 ++++++++++++++++++++++++++++++++------ include/hw/virtio/vhost-backend.h | 2 + include/hw/virtio/vhost.h | 2 + include/net/net.h | 1 + include/net/vhost_net.h | 3 + linux-headers/linux/vhost.h | 2 +- net/vhost-user.c | 140 +++++++++++++++++++++++++++----------- qapi-schema.json | 6 +- qemu-options.hx | 5 +- tests/vhost-user-test.c | 2 +- 13 files changed, 350 insertions(+), 70 deletions(-) -- 1.9.0