ae23c9
From d32439f6eb5f0d628c1a9d559007dda3083ec7b8 Mon Sep 17 00:00:00 2001
ae23c9
From: "plai@redhat.com" <plai@redhat.com>
ae23c9
Date: Thu, 21 Jun 2018 18:54:40 +0200
ae23c9
Subject: [PATCH 161/268] vhost-user-bridge: support host notifier
ae23c9
ae23c9
RH-Author: plai@redhat.com
ae23c9
Message-id: <1529607285-9942-6-git-send-email-plai@redhat.com>
ae23c9
Patchwork-id: 80934
ae23c9
O-Subject: [RHEL7.6 PATCH BZ 1526645 05/10] vhost-user-bridge: support host notifier
ae23c9
Bugzilla: 1526645
ae23c9
RH-Acked-by: Michael S. Tsirkin <mst@redhat.com>
ae23c9
RH-Acked-by: Maxime Coquelin <maxime.coquelin@redhat.com>
ae23c9
RH-Acked-by: Laurent Vivier <lvivier@redhat.com>
ae23c9
ae23c9
From: Tiwei Bie <tiwei.bie@intel.com>
ae23c9
ae23c9
This patch introduces the host notifier support in
ae23c9
vhost-user-bridge. A new option (-H) is added to use
ae23c9
the host notifier. This is mainly used to test the
ae23c9
host notifier implementation in vhost user.
ae23c9
ae23c9
Signed-off-by: Tiwei Bie <tiwei.bie@intel.com>
ae23c9
Reviewed-by: Michael S. Tsirkin <mst@redhat.com>
ae23c9
Signed-off-by: Michael S. Tsirkin <mst@redhat.com>
ae23c9
(cherry picked from commit e3af2928f8270498097af163bb023b633c0d8d1c)
ae23c9
Signed-off-by: Paul Lai <plai@redhat.com>
ae23c9
Signed-off-by: Miroslav Rezanina <mrezanin@redhat.com>
ae23c9
---
ae23c9
 tests/vhost-user-bridge.c | 98 +++++++++++++++++++++++++++++++++++++++++++++--
ae23c9
 1 file changed, 95 insertions(+), 3 deletions(-)
ae23c9
ae23c9
diff --git a/tests/vhost-user-bridge.c b/tests/vhost-user-bridge.c
ae23c9
index e0605a5..0884294 100644
ae23c9
--- a/tests/vhost-user-bridge.c
ae23c9
+++ b/tests/vhost-user-bridge.c
ae23c9
@@ -29,6 +29,7 @@
ae23c9
 
ae23c9
 #define _FILE_OFFSET_BITS 64
ae23c9
 
ae23c9
+#include "qemu/atomic.h"
ae23c9
 #include "qemu/osdep.h"
ae23c9
 #include "qemu/iov.h"
ae23c9
 #include "standard-headers/linux/virtio_net.h"
ae23c9
@@ -65,6 +66,11 @@ typedef struct VubrDev {
ae23c9
     int sock;
ae23c9
     int ready;
ae23c9
     int quit;
ae23c9
+    struct {
ae23c9
+        int fd;
ae23c9
+        void *addr;
ae23c9
+        pthread_t thread;
ae23c9
+    } notifier;
ae23c9
 } VubrDev;
ae23c9
 
ae23c9
 static void
ae23c9
@@ -445,14 +451,22 @@ static uint64_t
ae23c9
 vubr_get_features(VuDev *dev)
ae23c9
 {
ae23c9
     return 1ULL << VIRTIO_NET_F_GUEST_ANNOUNCE |
ae23c9
-        1ULL << VIRTIO_NET_F_MRG_RXBUF;
ae23c9
+        1ULL << VIRTIO_NET_F_MRG_RXBUF |
ae23c9
+        1ULL << VIRTIO_F_VERSION_1;
ae23c9
 }
ae23c9
 
ae23c9
 static void
ae23c9
 vubr_queue_set_started(VuDev *dev, int qidx, bool started)
ae23c9
 {
ae23c9
+    VubrDev *vubr = container_of(dev, VubrDev, vudev);
ae23c9
     VuVirtq *vq = vu_get_queue(dev, qidx);
ae23c9
 
ae23c9
+    if (started && vubr->notifier.fd >= 0) {
ae23c9
+        vu_set_queue_host_notifier(dev, vq, vubr->notifier.fd,
ae23c9
+                                   getpagesize(),
ae23c9
+                                   qidx * getpagesize());
ae23c9
+    }
ae23c9
+
ae23c9
     if (qidx % 2 == 1) {
ae23c9
         vu_set_queue_handler(dev, vq, started ? vubr_handle_tx : NULL);
ae23c9
     }
ae23c9
@@ -522,6 +536,8 @@ vubr_new(const char *path, bool client)
ae23c9
         vubr_die("socket");
ae23c9
     }
ae23c9
 
ae23c9
+    dev->notifier.fd = -1;
ae23c9
+
ae23c9
     un.sun_family = AF_UNIX;
ae23c9
     strcpy(un.sun_path, path);
ae23c9
     len = sizeof(un.sun_family) + strlen(path);
ae23c9
@@ -559,6 +575,73 @@ vubr_new(const char *path, bool client)
ae23c9
     return dev;
ae23c9
 }
ae23c9
 
ae23c9
+static void *notifier_thread(void *arg)
ae23c9
+{
ae23c9
+    VuDev *dev = (VuDev *)arg;
ae23c9
+    VubrDev *vubr = container_of(dev, VubrDev, vudev);
ae23c9
+    int pagesize = getpagesize();
ae23c9
+    int qidx;
ae23c9
+
ae23c9
+    while (true) {
ae23c9
+        for (qidx = 0; qidx < VHOST_MAX_NR_VIRTQUEUE; qidx++) {
ae23c9
+            uint16_t *n = vubr->notifier.addr + pagesize * qidx;
ae23c9
+
ae23c9
+            if (*n == qidx) {
ae23c9
+                *n = 0xffff;
ae23c9
+                /* We won't miss notifications if we reset
ae23c9
+                 * the memory first. */
ae23c9
+                smp_mb();
ae23c9
+
ae23c9
+                DPRINT("Got a notification for queue%d via host notifier.\n",
ae23c9
+                       qidx);
ae23c9
+
ae23c9
+                if (qidx % 2 == 1) {
ae23c9
+                    vubr_handle_tx(dev, qidx);
ae23c9
+                }
ae23c9
+            }
ae23c9
+            usleep(1000);
ae23c9
+        }
ae23c9
+    }
ae23c9
+
ae23c9
+    return NULL;
ae23c9
+}
ae23c9
+
ae23c9
+static void
ae23c9
+vubr_host_notifier_setup(VubrDev *dev)
ae23c9
+{
ae23c9
+    char template[] = "/tmp/vubr-XXXXXX";
ae23c9
+    pthread_t thread;
ae23c9
+    size_t length;
ae23c9
+    void *addr;
ae23c9
+    int fd;
ae23c9
+
ae23c9
+    length = getpagesize() * VHOST_MAX_NR_VIRTQUEUE;
ae23c9
+
ae23c9
+    fd = mkstemp(template);
ae23c9
+    if (fd < 0) {
ae23c9
+        vubr_die("mkstemp()");
ae23c9
+    }
ae23c9
+
ae23c9
+    if (posix_fallocate(fd, 0, length) != 0) {
ae23c9
+        vubr_die("posix_fallocate()");
ae23c9
+    }
ae23c9
+
ae23c9
+    addr = mmap(NULL, length, PROT_READ | PROT_WRITE, MAP_SHARED, fd, 0);
ae23c9
+    if (addr == MAP_FAILED) {
ae23c9
+        vubr_die("mmap()");
ae23c9
+    }
ae23c9
+
ae23c9
+    memset(addr, 0xff, length);
ae23c9
+
ae23c9
+    if (pthread_create(&thread, NULL, notifier_thread, &dev->vudev) != 0) {
ae23c9
+        vubr_die("pthread_create()");
ae23c9
+    }
ae23c9
+
ae23c9
+    dev->notifier.fd = fd;
ae23c9
+    dev->notifier.addr = addr;
ae23c9
+    dev->notifier.thread = thread;
ae23c9
+}
ae23c9
+
ae23c9
 static void
ae23c9
 vubr_set_host(struct sockaddr_in *saddr, const char *host)
ae23c9
 {
ae23c9
@@ -673,8 +756,9 @@ main(int argc, char *argv[])
ae23c9
     VubrDev *dev;
ae23c9
     int opt;
ae23c9
     bool client = false;
ae23c9
+    bool host_notifier = false;
ae23c9
 
ae23c9
-    while ((opt = getopt(argc, argv, "l:r:u:c")) != -1) {
ae23c9
+    while ((opt = getopt(argc, argv, "l:r:u:cH")) != -1) {
ae23c9
 
ae23c9
         switch (opt) {
ae23c9
         case 'l':
ae23c9
@@ -693,6 +777,9 @@ main(int argc, char *argv[])
ae23c9
         case 'c':
ae23c9
             client = true;
ae23c9
             break;
ae23c9
+        case 'H':
ae23c9
+            host_notifier = true;
ae23c9
+            break;
ae23c9
         default:
ae23c9
             goto out;
ae23c9
         }
ae23c9
@@ -708,6 +795,10 @@ main(int argc, char *argv[])
ae23c9
         return 1;
ae23c9
     }
ae23c9
 
ae23c9
+    if (host_notifier) {
ae23c9
+        vubr_host_notifier_setup(dev);
ae23c9
+    }
ae23c9
+
ae23c9
     vubr_backend_udp_setup(dev, lhost, lport, rhost, rport);
ae23c9
     vubr_run(dev);
ae23c9
 
ae23c9
@@ -717,7 +808,7 @@ main(int argc, char *argv[])
ae23c9
 
ae23c9
 out:
ae23c9
     fprintf(stderr, "Usage: %s ", argv[0]);
ae23c9
-    fprintf(stderr, "[-c] [-u ud_socket_path] [-l lhost:lport] [-r rhost:rport]\n");
ae23c9
+    fprintf(stderr, "[-c] [-H] [-u ud_socket_path] [-l lhost:lport] [-r rhost:rport]\n");
ae23c9
     fprintf(stderr, "\t-u path to unix doman socket. default: %s\n",
ae23c9
             DEFAULT_UD_SOCKET);
ae23c9
     fprintf(stderr, "\t-l local host and port. default: %s:%s\n",
ae23c9
@@ -725,6 +816,7 @@ out:
ae23c9
     fprintf(stderr, "\t-r remote host and port. default: %s:%s\n",
ae23c9
             DEFAULT_RHOST, DEFAULT_RPORT);
ae23c9
     fprintf(stderr, "\t-c client mode\n");
ae23c9
+    fprintf(stderr, "\t-H use host notifier\n");
ae23c9
 
ae23c9
     return 1;
ae23c9
 }
ae23c9
-- 
ae23c9
1.8.3.1
ae23c9