cryptospore / rpms / qemu-kvm

Forked from rpms/qemu-kvm 2 years ago
Clone
76daa3
From 398bebe1d3c1feea6e4578b1ce40366ac39f933e Mon Sep 17 00:00:00 2001
76daa3
From: =?UTF-8?q?Marc-Andr=C3=A9=20Lureau?= <marcandre.lureau@redhat.com>
76daa3
Date: Wed, 2 Aug 2017 15:13:18 +0200
76daa3
Subject: [PATCH 3/4] pegas: add --disable-vhost-user
76daa3
MIME-Version: 1.0
76daa3
Content-Type: text/plain; charset=UTF-8
76daa3
Content-Transfer-Encoding: 8bit
76daa3
76daa3
RH-Author: Marc-André Lureau <marcandre.lureau@redhat.com>
76daa3
Message-id: <20170802151319.16790-2-marcandre.lureau@redhat.com>
76daa3
Patchwork-id: 75884
76daa3
O-Subject: [RHEL-ALT-7.4 qemu-kvm PATCH v2 1/2] pegas: add --disable-vhost-user
76daa3
Bugzilla: 1455269
76daa3
RH-Acked-by: Michael S. Tsirkin <mst@redhat.com>
76daa3
RH-Acked-by: Eduardo Habkost <ehabkost@redhat.com>
76daa3
RH-Acked-by: Miroslav Rezanina <mrezanin@redhat.com>
76daa3
76daa3
Learn to disable vhost-user for Pegas builds.
76daa3
76daa3
When trying to make a vhost-user netdev, it gives the following error:
76daa3
76daa3
-netdev vhost-user,id=foo,chardev=chr-test: Parameter 'type' expects a netdev backend type
76daa3
76daa3
And similar error with the HMP/QMP monitors.
76daa3
76daa3
Upstream-status:
76daa3
https://lists.nongnu.org/archive/html/qemu-devel/2017-07/msg09016.html
76daa3
76daa3
Signed-off-by: Marc-André Lureau <marcandre.lureau@redhat.com>
76daa3
Signed-off-by: Miroslav Rezanina <mrezanin@redhat.com>
76daa3
---
76daa3
 configure              | 14 +++++++++++++-
76daa3
 hw/net/vhost_net.c     | 10 ++++++++--
76daa3
 hw/net/virtio-net.c    |  4 ++++
76daa3
 net/Makefile.objs      |  3 ++-
76daa3
 net/hub.c              |  2 ++
76daa3
 net/net.c              |  4 +++-
76daa3
 qemu-options.hx        |  4 ++++
76daa3
 tests/Makefile.include |  6 +++---
76daa3
 8 files changed, 39 insertions(+), 8 deletions(-)
76daa3
76daa3
diff --git a/configure b/configure
76daa3
index 9f200c2..ca82854 100755
76daa3
--- a/configure
76daa3
+++ b/configure
76daa3
@@ -227,6 +227,7 @@ xfs=""
76daa3
 vhost_net="no"
76daa3
 vhost_scsi="no"
76daa3
 vhost_vsock="no"
76daa3
+vhost_user="yes"
76daa3
 kvm="no"
76daa3
 hax="no"
76daa3
 rdma=""
76daa3
@@ -1204,6 +1205,10 @@ for opt do
76daa3
   ;;
76daa3
   --enable-vtd) vtd="yes"
76daa3
   ;;
76daa3
+  --disable-vhost-user) vhost_user="no"
76daa3
+  ;;
76daa3
+  --enable-vhost-user) vhost_user="yes"
76daa3
+  ;;
76daa3
   *)
76daa3
       echo "ERROR: unknown option $opt"
76daa3
       echo "Try '$0 --help' for more information"
76daa3
@@ -1452,6 +1457,7 @@ disabled with --disable-FEATURE, default is enabled if available:
76daa3
   tools           build qemu-io, qemu-nbd and qemu-image tools
76daa3
   vxhs            Veritas HyperScale vDisk backend support
76daa3
   vtd             Emulated VT-d support (only affects x86 targets)
76daa3
+  vhost-user      vhost-user support
76daa3
 
76daa3
 NOTE: The object files are built at the place where configure is launched
76daa3
 EOF
76daa3
@@ -5135,6 +5141,7 @@ echo "libcap-ng support $cap_ng"
76daa3
 echo "vhost-net support $vhost_net"
76daa3
 echo "vhost-scsi support $vhost_scsi"
76daa3
 echo "vhost-vsock support $vhost_vsock"
76daa3
+echo "vhost-user support $vhost_user"
76daa3
 echo "Trace backends    $trace_backends"
76daa3
 if have_backend "simple"; then
76daa3
 echo "Trace output file $trace_file-<pid>"
76daa3
@@ -5557,6 +5564,9 @@ fi
76daa3
 if test "$vhost_vsock" = "yes" ; then
76daa3
   echo "CONFIG_VHOST_VSOCK=y" >> $config_host_mak
76daa3
 fi
76daa3
+if test "$vhost_user" = "yes" ; then
76daa3
+  echo "CONFIG_VHOST_USER=y" >> $config_host_mak
76daa3
+fi
76daa3
 if test "$blobs" = "yes" ; then
76daa3
   echo "INSTALL_BLOBS=yes" >> $config_host_mak
76daa3
 fi
76daa3
@@ -6159,7 +6169,9 @@ case "$target_name" in
76daa3
       echo "CONFIG_KVM=y" >> $config_target_mak
76daa3
       if test "$vhost_net" = "yes" ; then
76daa3
         echo "CONFIG_VHOST_NET=y" >> $config_target_mak
76daa3
-        echo "CONFIG_VHOST_NET_TEST_$target_name=y" >> $config_host_mak
76daa3
+        if test "$vhost_user" = "yes" ; then
76daa3
+          echo "CONFIG_VHOST_USER_TEST_$target_name=y" >> $config_host_mak
76daa3
+        fi
76daa3
       fi
76daa3
     fi
76daa3
 esac
76daa3
diff --git a/hw/net/vhost_net.c b/hw/net/vhost_net.c
76daa3
index e037db6..40438a8 100644
76daa3
--- a/hw/net/vhost_net.c
76daa3
+++ b/hw/net/vhost_net.c
76daa3
@@ -95,9 +95,11 @@ static const int *vhost_net_get_feature_bits(struct vhost_net *net)
76daa3
     case NET_CLIENT_DRIVER_TAP:
76daa3
         feature_bits = kernel_feature_bits;
76daa3
         break;
76daa3
+#ifdef CONFIG_VHOST_USER
76daa3
     case NET_CLIENT_DRIVER_VHOST_USER:
76daa3
         feature_bits = user_feature_bits;
76daa3
         break;
76daa3
+#endif
76daa3
     default:
76daa3
         error_report("Feature bits not defined for this type: %d",
76daa3
                 net->nc->info->type);
76daa3
@@ -193,6 +195,7 @@ struct vhost_net *vhost_net_init(VhostNetOptions *options)
76daa3
         }
76daa3
     }
76daa3
 
76daa3
+#ifdef CONFIG_VHOST_USER
76daa3
     /* Set sane init value. Override when guest acks. */
76daa3
     if (net->nc->info->type == NET_CLIENT_DRIVER_VHOST_USER) {
76daa3
         features = vhost_user_get_acked_features(net->nc);
76daa3
@@ -203,7 +206,7 @@ struct vhost_net *vhost_net_init(VhostNetOptions *options)
76daa3
             goto fail;
76daa3
         }
76daa3
     }
76daa3
-
76daa3
+#endif
76daa3
     vhost_net_ack_features(net, features);
76daa3
 
76daa3
     return net;
76daa3
@@ -309,6 +312,7 @@ int vhost_net_start(VirtIODevice *dev, NetClientState *ncs,
76daa3
         net = get_vhost_net(ncs[i].peer);
76daa3
         vhost_net_set_vq_index(net, i * 2);
76daa3
 
76daa3
+#ifdef CONFIG_VHOST_USER
76daa3
         /* Suppress the masking guest notifiers on vhost user
76daa3
          * because vhost user doesn't interrupt masking/unmasking
76daa3
          * properly.
76daa3
@@ -316,8 +320,8 @@ int vhost_net_start(VirtIODevice *dev, NetClientState *ncs,
76daa3
         if (net->nc->info->type == NET_CLIENT_DRIVER_VHOST_USER) {
76daa3
             dev->use_guest_notifier_mask = false;
76daa3
         }
76daa3
+#endif
76daa3
      }
76daa3
-
76daa3
     r = k->set_guest_notifiers(qbus->parent, total_queues * 2, true);
76daa3
     if (r < 0) {
76daa3
         error_report("Error binding guest notifier: %d", -r);
76daa3
@@ -414,10 +418,12 @@ VHostNetState *get_vhost_net(NetClientState *nc)
76daa3
     case NET_CLIENT_DRIVER_TAP:
76daa3
         vhost_net = tap_get_vhost_net(nc);
76daa3
         break;
76daa3
+#ifdef CONFIG_VHOST_USER
76daa3
     case NET_CLIENT_DRIVER_VHOST_USER:
76daa3
         vhost_net = vhost_user_get_vhost_net(nc);
76daa3
         assert(vhost_net);
76daa3
         break;
76daa3
+#endif
76daa3
     default:
76daa3
         break;
76daa3
     }
76daa3
diff --git a/hw/net/virtio-net.c b/hw/net/virtio-net.c
76daa3
index 39c336e..9f61432 100644
76daa3
--- a/hw/net/virtio-net.c
76daa3
+++ b/hw/net/virtio-net.c
76daa3
@@ -502,9 +502,11 @@ static int peer_attach(VirtIONet *n, int index)
76daa3
         return 0;
76daa3
     }
76daa3
 
76daa3
+#ifdef CONFIG_VHOST_USER
76daa3
     if (nc->peer->info->type == NET_CLIENT_DRIVER_VHOST_USER) {
76daa3
         vhost_set_vring_enable(nc->peer, 1);
76daa3
     }
76daa3
+#endif
76daa3
 
76daa3
     if (nc->peer->info->type != NET_CLIENT_DRIVER_TAP) {
76daa3
         return 0;
76daa3
@@ -525,9 +527,11 @@ static int peer_detach(VirtIONet *n, int index)
76daa3
         return 0;
76daa3
     }
76daa3
 
76daa3
+#ifdef CONFIG_VHOST_USER
76daa3
     if (nc->peer->info->type == NET_CLIENT_DRIVER_VHOST_USER) {
76daa3
         vhost_set_vring_enable(nc->peer, 0);
76daa3
     }
76daa3
+#endif
76daa3
 
76daa3
     if (nc->peer->info->type !=  NET_CLIENT_DRIVER_TAP) {
76daa3
         return 0;
76daa3
diff --git a/net/Makefile.objs b/net/Makefile.objs
76daa3
index 2e2fd43..a8f6556 100644
76daa3
--- a/net/Makefile.objs
76daa3
+++ b/net/Makefile.objs
76daa3
@@ -3,7 +3,8 @@ common-obj-y += socket.o
76daa3
 common-obj-y += dump.o
76daa3
 common-obj-y += eth.o
76daa3
 common-obj-$(CONFIG_L2TPV3) += l2tpv3.o
76daa3
-common-obj-$(CONFIG_POSIX) += tap.o vhost-user.o
76daa3
+common-obj-$(CONFIG_POSIX) += tap.o
76daa3
+common-obj-$(CONFIG_VHOST_USER) += vhost-user.o
76daa3
 common-obj-$(CONFIG_LINUX) += tap-linux.o
76daa3
 common-obj-$(CONFIG_WIN32) += tap-win32.o
76daa3
 common-obj-$(CONFIG_BSD) += tap-bsd.o
76daa3
diff --git a/net/hub.c b/net/hub.c
76daa3
index 32d8cf5..ec06275 100644
76daa3
--- a/net/hub.c
76daa3
+++ b/net/hub.c
76daa3
@@ -322,7 +322,9 @@ void net_hub_check_clients(void)
76daa3
             case NET_CLIENT_DRIVER_TAP:
76daa3
             case NET_CLIENT_DRIVER_SOCKET:
76daa3
             case NET_CLIENT_DRIVER_VDE:
76daa3
+#ifdef CONFIG_VHOST_USER
76daa3
             case NET_CLIENT_DRIVER_VHOST_USER:
76daa3
+#endif
76daa3
                 has_host_dev = 1;
76daa3
                 break;
76daa3
             default:
76daa3
diff --git a/net/net.c b/net/net.c
76daa3
index 0ac3b9e..4da90c1 100644
76daa3
--- a/net/net.c
76daa3
+++ b/net/net.c
76daa3
@@ -955,7 +955,7 @@ static int (* const net_client_init_fun[NET_CLIENT_DRIVER__MAX])(
76daa3
         [NET_CLIENT_DRIVER_BRIDGE]    = net_init_bridge,
76daa3
 #endif
76daa3
         [NET_CLIENT_DRIVER_HUBPORT]   = net_init_hubport,
76daa3
-#ifdef CONFIG_VHOST_NET_USED
76daa3
+#if defined(CONFIG_VHOST_NET_USED) && defined(CONFIG_VHOST_USER)
76daa3
         [NET_CLIENT_DRIVER_VHOST_USER] = net_init_vhost_user,
76daa3
 #endif
76daa3
 #ifdef CONFIG_L2TPV3
76daa3
@@ -1031,10 +1031,12 @@ static int net_client_init1(const void *object, bool is_netdev, Error **errp)
76daa3
             legacy.type = NET_CLIENT_DRIVER_NETMAP;
76daa3
             legacy.u.netmap = opts->u.netmap;
76daa3
             break;
76daa3
+#ifdef CONFIG_VHOST_USER
76daa3
         case NET_LEGACY_OPTIONS_TYPE_VHOST_USER:
76daa3
             legacy.type = NET_CLIENT_DRIVER_VHOST_USER;
76daa3
             legacy.u.vhost_user = opts->u.vhost_user;
76daa3
             break;
76daa3
+#endif
76daa3
         default:
76daa3
             abort();
76daa3
         }
76daa3
diff --git a/qemu-options.hx b/qemu-options.hx
76daa3
index 2f25547..30b6256 100644
76daa3
--- a/qemu-options.hx
76daa3
+++ b/qemu-options.hx
76daa3
@@ -1741,8 +1741,10 @@ DEF("netdev", HAS_ARG, QEMU_OPTION_netdev,
76daa3
     "                VALE port (created on the fly) called 'name' ('nmname' is name of the \n"
76daa3
     "                netmap device, defaults to '/dev/netmap')\n"
76daa3
 #endif
76daa3
+#ifdef CONFIG_VHOST_USER
76daa3
     "-netdev vhost-user,id=str,chardev=dev[,vhostforce=on|off]\n"
76daa3
     "                configure a vhost-user network, backed by a chardev 'dev'\n"
76daa3
+#endif
76daa3
     "-netdev hubport,id=str,hubid=n\n"
76daa3
     "                configure a hub port on QEMU VLAN 'n'\n", QEMU_ARCH_ALL)
76daa3
 DEF("net", HAS_ARG, QEMU_OPTION_net,
76daa3
@@ -2175,6 +2177,7 @@ The hubport netdev lets you connect a NIC to a QEMU "vlan" instead of a single
76daa3
 netdev.  @code{-net} and @code{-device} with parameter @option{vlan} create the
76daa3
 required hub automatically.
76daa3
 
76daa3
+#ifdef CONFIG_VHOST_USER
76daa3
 @item -netdev vhost-user,chardev=@var{id}[,vhostforce=on|off][,queues=n]
76daa3
 
76daa3
 Establish a vhost-user netdev, backed by a chardev @var{id}. The chardev should
76daa3
@@ -2192,6 +2195,7 @@ qemu-kvm -m 512 -object memory-backend-file,id=mem,size=512M,mem-path=/hugetlbfs
76daa3
          -netdev type=vhost-user,id=net0,chardev=chr0 \
76daa3
          -device virtio-net-pci,netdev=net0
76daa3
 @end example
76daa3
+#endif
76daa3
 
76daa3
 @item -net dump[,vlan=@var{n}][,file=@var{file}][,len=@var{len}]
76daa3
 Dump network traffic on VLAN @var{n} to file @var{file} (@file{qemu-vlan0.pcap} by default).
76daa3
diff --git a/tests/Makefile.include b/tests/Makefile.include
76daa3
index 0b94a20..3e3870f 100644
76daa3
--- a/tests/Makefile.include
76daa3
+++ b/tests/Makefile.include
76daa3
@@ -230,9 +230,9 @@ gcov-files-i386-y += hw/usb/hcd-xhci.c
76daa3
 check-qtest-i386-y += tests/pc-cpu-test$(EXESUF)
76daa3
 check-qtest-i386-y += tests/q35-test$(EXESUF)
76daa3
 gcov-files-i386-y += hw/pci-host/q35.c
76daa3
-check-qtest-i386-$(CONFIG_VHOST_NET_TEST_i386) += tests/vhost-user-test$(EXESUF)
76daa3
-ifeq ($(CONFIG_VHOST_NET_TEST_i386),)
76daa3
-check-qtest-x86_64-$(CONFIG_VHOST_NET_TEST_x86_64) += tests/vhost-user-test$(EXESUF)
76daa3
+check-qtest-i386-$(CONFIG_VHOST_USER_TEST_i386) += tests/vhost-user-test$(EXESUF)
76daa3
+ifeq ($(CONFIG_VHOST_USER_TEST_i386),)
76daa3
+check-qtest-x86_64-$(CONFIG_VHOST_USER_TEST_x86_64) += tests/vhost-user-test$(EXESUF)
76daa3
 endif
76daa3
 check-qtest-i386-y += tests/test-netfilter$(EXESUF)
76daa3
 check-qtest-i386-y += tests/test-filter-mirror$(EXESUF)
76daa3
-- 
76daa3
1.8.3.1
76daa3