1
The following changes since commit 13356edb87506c148b163b8c7eb0695647d00c2a:
1
The following changes since commit 352998df1c53b366413690d95b35f76d0721ebed:
2
2
3
Merge tag 'block-pull-request' of https://gitlab.com/stefanha/qemu into staging (2023-01-24 09:45:33 +0000)
3
Merge tag 'i2c-20220314' of https://github.com/philmd/qemu into staging (2022-03-14 14:39:33 +0000)
4
4
5
are available in the git repository at:
5
are available in the git repository at:
6
6
7
https://github.com/jasowang/qemu.git tags/net-pull-request
7
https://github.com/jasowang/qemu.git tags/net-pull-request
8
8
9
for you to fetch changes up to 2bd492bca521ee8594f1d5db8dc9aac126fc4f85:
9
for you to fetch changes up to 12a195fa343aae2ead1301ce04727bd0ae25eb15:
10
10
11
vdpa: fix VHOST_BACKEND_F_IOTLB_ASID flag check (2023-02-02 14:16:48 +0800)
11
vdpa: Expose VHOST_F_LOG_ALL on SVQ (2022-03-15 13:57:44 +0800)
12
12
13
----------------------------------------------------------------
13
----------------------------------------------------------------
14
14
15
Changes since V2:
16
- fix 32bit build errros
17
15
----------------------------------------------------------------
18
----------------------------------------------------------------
16
Christian Svensson (1):
19
Eugenio Pérez (14):
17
net: Increase L2TPv3 buffer to fit jumboframes
20
vhost: Add VhostShadowVirtqueue
21
vhost: Add Shadow VirtQueue kick forwarding capabilities
22
vhost: Add Shadow VirtQueue call forwarding capabilities
23
vhost: Add vhost_svq_valid_features to shadow vq
24
virtio: Add vhost_svq_get_vring_addr
25
vdpa: adapt vhost_ops callbacks to svq
26
vhost: Shadow virtqueue buffers forwarding
27
util: Add iova_tree_alloc_map
28
util: add iova_tree_find_iova
29
vhost: Add VhostIOVATree
30
vdpa: Add custom IOTLB translations to SVQ
31
vdpa: Adapt vhost_vdpa_get_vring_base to SVQ
32
vdpa: Never set log_base addr if SVQ is enabled
33
vdpa: Expose VHOST_F_LOG_ALL on SVQ
18
34
19
Eugenio Pérez (1):
35
Jason Wang (1):
20
vdpa: fix VHOST_BACKEND_F_IOTLB_ASID flag check
36
virtio-net: fix map leaking on error during receive
21
37
22
Fiona Ebner (1):
38
hw/net/virtio-net.c | 1 +
23
hw/net/vmxnet3: allow VMXNET3_MAX_MTU itself as a value
39
hw/virtio/meson.build | 2 +-
24
40
hw/virtio/vhost-iova-tree.c | 110 +++++++
25
Joelle van Dyne (1):
41
hw/virtio/vhost-iova-tree.h | 27 ++
26
vmnet: stop recieving events when VM is stopped
42
hw/virtio/vhost-shadow-virtqueue.c | 636 +++++++++++++++++++++++++++++++++++++
27
43
hw/virtio/vhost-shadow-virtqueue.h | 87 +++++
28
Laurent Vivier (2):
44
hw/virtio/vhost-vdpa.c | 522 +++++++++++++++++++++++++++++-
29
tests/qtest: netdev: test stream and dgram backends
45
include/hw/virtio/vhost-vdpa.h | 8 +
30
net: stream: add a new option to automatically reconnect
46
include/qemu/iova-tree.h | 38 ++-
31
47
util/iova-tree.c | 170 ++++++++++
32
Qiang Liu (2):
48
10 files changed, 1584 insertions(+), 17 deletions(-)
33
hw/net/lan9118: log [read|write]b when mode_16bit is enabled rather than abort
49
create mode 100644 hw/virtio/vhost-iova-tree.c
34
hw/net/can/xlnx-zynqmp-can: fix assertion failures in transfer_fifo()
50
create mode 100644 hw/virtio/vhost-iova-tree.h
35
51
create mode 100644 hw/virtio/vhost-shadow-virtqueue.c
36
Thomas Huth (3):
52
create mode 100644 hw/virtio/vhost-shadow-virtqueue.h
37
net: Move the code to collect available NIC models to a separate function
38
net: Restore printing of the help text with "-nic help"
39
net: Replace "Supported NIC models" with "Available NIC models"
40
41
hw/net/can/xlnx-zynqmp-can.c | 9 +-
42
hw/net/lan9118.c | 17 +-
43
hw/net/vmxnet3.c | 2 +-
44
hw/pci/pci.c | 29 +--
45
include/net/net.h | 14 ++
46
net/l2tpv3.c | 2 +-
47
net/net.c | 50 +++-
48
net/stream.c | 53 ++++-
49
net/vhost-vdpa.c | 2 +-
50
net/vmnet-common.m | 48 +++-
51
net/vmnet_int.h | 2 +
52
qapi/net.json | 7 +-
53
qemu-options.hx | 6 +-
54
tests/qtest/meson.build | 2 +
55
tests/qtest/netdev-socket.c | 549 +++++++++++++++++++++++++++++++++++++++++++
56
15 files changed, 730 insertions(+), 62 deletions(-)
57
create mode 100644 tests/qtest/netdev-socket.c
58
53
59
54
60
55
diff view generated by jsdifflib
1
From: Qiang Liu <cyruscyliu@gmail.com>
1
Commit bedd7e93d0196 ("virtio-net: fix use after unmap/free for sg")
2
tries to fix the use after free of the sg by caching the virtqueue
3
elements in an array and unmap them at once after receiving the
4
packets, But it forgot to unmap the cached elements on error which
5
will lead to leaking of mapping and other unexpected results.
2
6
3
Check fifos before poping data from and pushing data into it.
7
Fixing this by detaching the cached elements on error. This addresses
8
CVE-2022-26353.
4
9
5
Fixes: 98e5d7a2b726 ("hw/net/can: Introduce Xilinx ZynqMP CAN controller")
10
Reported-by: Victor Tom <vv474172261@gmail.com>
6
Resolves: https://gitlab.com/qemu-project/qemu/-/issues/1425
11
Cc: qemu-stable@nongnu.org
7
Resolves: https://gitlab.com/qemu-project/qemu/-/issues/1427
12
Fixes: CVE-2022-26353
8
Reported-by: Qiang Liu <cyruscyliu@gmail.com>
13
Fixes: bedd7e93d0196 ("virtio-net: fix use after unmap/free for sg")
9
Signed-off-by: Qiang Liu <cyruscyliu@gmail.com>
14
Reviewed-by: Michael S. Tsirkin <mst@redhat.com>
10
Signed-off-by: Jason Wang <jasowang@redhat.com>
15
Signed-off-by: Jason Wang <jasowang@redhat.com>
11
---
16
---
12
hw/net/can/xlnx-zynqmp-can.c | 9 ++++++++-
17
hw/net/virtio-net.c | 1 +
13
1 file changed, 8 insertions(+), 1 deletion(-)
18
1 file changed, 1 insertion(+)
14
19
15
diff --git a/hw/net/can/xlnx-zynqmp-can.c b/hw/net/can/xlnx-zynqmp-can.c
20
diff --git a/hw/net/virtio-net.c b/hw/net/virtio-net.c
16
index XXXXXXX..XXXXXXX 100644
21
index XXXXXXX..XXXXXXX 100644
17
--- a/hw/net/can/xlnx-zynqmp-can.c
22
--- a/hw/net/virtio-net.c
18
+++ b/hw/net/can/xlnx-zynqmp-can.c
23
+++ b/hw/net/virtio-net.c
19
@@ -XXX,XX +XXX,XX @@ static void transfer_fifo(XlnxZynqMPCANState *s, Fifo32 *fifo)
24
@@ -XXX,XX +XXX,XX @@ static ssize_t virtio_net_receive_rcu(NetClientState *nc, const uint8_t *buf,
25
26
err:
27
for (j = 0; j < i; j++) {
28
+ virtqueue_detach_element(q->rx_vq, elems[j], lens[j]);
29
g_free(elems[j]);
20
}
30
}
21
31
22
while (!fifo32_is_empty(fifo)) {
23
+ if (fifo32_num_used(fifo) < (4 * CAN_FRAME_SIZE)) {
24
+ g_autofree char *path = object_get_canonical_path(OBJECT(s));
25
+ qemu_log_mask(LOG_GUEST_ERROR, "%s: data left in the fifo is not"
26
+ " enough for transfer.\n", path);
27
+ break;
28
+ }
29
for (i = 0; i < CAN_FRAME_SIZE; i++) {
30
data[i] = fifo32_pop(fifo);
31
}
32
@@ -XXX,XX +XXX,XX @@ static void transfer_fifo(XlnxZynqMPCANState *s, Fifo32 *fifo)
33
* acknowledged. The XlnxZynqMPCAN core receives any message
34
* that it transmits.
35
*/
36
- if (fifo32_is_full(&s->rx_fifo)) {
37
+ if (fifo32_is_full(&s->rx_fifo) ||
38
+ (fifo32_num_free(&s->rx_fifo) < (4 * CAN_FRAME_SIZE))) {
39
ARRAY_FIELD_DP32(s->regs, INTERRUPT_STATUS_REGISTER, RXOFLW, 1);
40
} else {
41
for (i = 0; i < CAN_FRAME_SIZE; i++) {
42
--
32
--
43
2.7.4
33
2.7.4
diff view generated by jsdifflib
1
From: Laurent Vivier <lvivier@redhat.com>
1
From: Eugenio Pérez <eperezma@redhat.com>
2
2
3
Signed-off-by: Laurent Vivier <lvivier@redhat.com>
3
Vhost shadow virtqueue (SVQ) is an intermediate jump for virtqueue
4
notifications and buffers, allowing qemu to track them. While qemu is
5
forwarding the buffers and virtqueue changes, it is able to commit the
6
memory it's being dirtied, the same way regular qemu's VirtIO devices
7
do.
8
9
This commit only exposes basic SVQ allocation and free. Next patches of
10
the series add functionality like notifications and buffers forwarding.
11
12
Signed-off-by: Eugenio Pérez <eperezma@redhat.com>
4
Acked-by: Michael S. Tsirkin <mst@redhat.com>
13
Acked-by: Michael S. Tsirkin <mst@redhat.com>
5
Acked-by: Thomas Huth <thuth@redhat.com>
6
Signed-off-by: Jason Wang <jasowang@redhat.com>
14
Signed-off-by: Jason Wang <jasowang@redhat.com>
7
---
15
---
8
tests/qtest/meson.build | 2 +
16
hw/virtio/meson.build | 2 +-
9
tests/qtest/netdev-socket.c | 448 ++++++++++++++++++++++++++++++++++++++++++++
17
hw/virtio/vhost-shadow-virtqueue.c | 62 ++++++++++++++++++++++++++++++++++++++
10
2 files changed, 450 insertions(+)
18
hw/virtio/vhost-shadow-virtqueue.h | 28 +++++++++++++++++
11
create mode 100644 tests/qtest/netdev-socket.c
19
3 files changed, 91 insertions(+), 1 deletion(-)
20
create mode 100644 hw/virtio/vhost-shadow-virtqueue.c
21
create mode 100644 hw/virtio/vhost-shadow-virtqueue.h
12
22
13
diff --git a/tests/qtest/meson.build b/tests/qtest/meson.build
23
diff --git a/hw/virtio/meson.build b/hw/virtio/meson.build
14
index XXXXXXX..XXXXXXX 100644
24
index XXXXXXX..XXXXXXX 100644
15
--- a/tests/qtest/meson.build
25
--- a/hw/virtio/meson.build
16
+++ b/tests/qtest/meson.build
26
+++ b/hw/virtio/meson.build
17
@@ -XXX,XX +XXX,XX @@ qtests_generic = [
27
@@ -XXX,XX +XXX,XX @@ softmmu_ss.add(when: 'CONFIG_ALL', if_true: files('vhost-stub.c'))
18
'test-hmp',
28
19
'qos-test',
29
virtio_ss = ss.source_set()
20
'readconfig-test',
30
virtio_ss.add(files('virtio.c'))
21
+ 'netdev-socket',
31
-virtio_ss.add(when: 'CONFIG_VHOST', if_true: files('vhost.c', 'vhost-backend.c'))
22
]
32
+virtio_ss.add(when: 'CONFIG_VHOST', if_true: files('vhost.c', 'vhost-backend.c', 'vhost-shadow-virtqueue.c'))
23
if config_host.has_key('CONFIG_MODULES')
33
virtio_ss.add(when: 'CONFIG_VHOST_USER', if_true: files('vhost-user.c'))
24
qtests_generic += [ 'modules-test' ]
34
virtio_ss.add(when: 'CONFIG_VHOST_VDPA', if_true: files('vhost-vdpa.c'))
25
@@ -XXX,XX +XXX,XX @@ qtests = {
35
virtio_ss.add(when: 'CONFIG_VIRTIO_BALLOON', if_true: files('virtio-balloon.c'))
26
'tpm-tis-device-swtpm-test': [io, tpmemu_files, 'tpm-tis-util.c'],
36
diff --git a/hw/virtio/vhost-shadow-virtqueue.c b/hw/virtio/vhost-shadow-virtqueue.c
27
'tpm-tis-device-test': [io, tpmemu_files, 'tpm-tis-util.c'],
28
'vmgenid-test': files('boot-sector.c', 'acpi-utils.c'),
29
+ 'netdev-socket': files('netdev-socket.c', '../unit/socket-helpers.c'),
30
}
31
32
gvnc = dependency('gvnc-1.0', required: false)
33
diff --git a/tests/qtest/netdev-socket.c b/tests/qtest/netdev-socket.c
34
new file mode 100644
37
new file mode 100644
35
index XXXXXXX..XXXXXXX
38
index XXXXXXX..XXXXXXX
36
--- /dev/null
39
--- /dev/null
37
+++ b/tests/qtest/netdev-socket.c
40
+++ b/hw/virtio/vhost-shadow-virtqueue.c
38
@@ -XXX,XX +XXX,XX @@
41
@@ -XXX,XX +XXX,XX @@
39
+/*
42
+/*
40
+ * QTest testcase for netdev stream and dgram
43
+ * vhost shadow virtqueue
41
+ *
44
+ *
42
+ * Copyright (c) 2022 Red Hat, Inc.
45
+ * SPDX-FileCopyrightText: Red Hat, Inc. 2021
46
+ * SPDX-FileContributor: Author: Eugenio Pérez <eperezma@redhat.com>
43
+ *
47
+ *
44
+ * SPDX-License-Identifier: GPL-2.0-or-later
48
+ * SPDX-License-Identifier: GPL-2.0-or-later
45
+ */
49
+ */
46
+
50
+
47
+#include "qemu/osdep.h"
51
+#include "qemu/osdep.h"
48
+#include "qemu/sockets.h"
52
+#include "hw/virtio/vhost-shadow-virtqueue.h"
49
+#include <glib/gstdio.h>
50
+#include "../unit/socket-helpers.h"
51
+#include "libqtest.h"
52
+
53
+
53
+#define CONNECTION_TIMEOUT 5
54
+#include "qemu/error-report.h"
54
+
55
+
55
+#define EXPECT_STATE(q, e, t) \
56
+/**
56
+do { \
57
+ * Creates vhost shadow virtqueue, and instructs the vhost device to use the
57
+ char *resp = NULL; \
58
+ * shadow methods and file descriptors.
58
+ g_test_timer_start(); \
59
+ *
59
+ do { \
60
+ * Returns the new virtqueue or NULL.
60
+ g_free(resp); \
61
+ *
61
+ resp = qtest_hmp(q, "info network"); \
62
+ * In case of error, reason is reported through error_report.
62
+ if (t) { \
63
+ */
63
+ strrchr(resp, t)[0] = 0; \
64
+VhostShadowVirtqueue *vhost_svq_new(void)
64
+ } \
65
+{
65
+ if (g_str_equal(resp, e)) { \
66
+ g_autofree VhostShadowVirtqueue *svq = g_new0(VhostShadowVirtqueue, 1);
66
+ break; \
67
+ int r;
67
+ } \
68
+ } while (g_test_timer_elapsed() < CONNECTION_TIMEOUT); \
69
+ g_assert_cmpstr(resp, ==, e); \
70
+ g_free(resp); \
71
+} while (0)
72
+
68
+
73
+static gchar *tmpdir;
69
+ r = event_notifier_init(&svq->hdev_kick, 0);
74
+
70
+ if (r != 0) {
75
+static int inet_get_free_port_socket_ipv4(int sock)
71
+ error_report("Couldn't create kick event notifier: %s (%d)",
76
+{
72
+ g_strerror(errno), errno);
77
+ struct sockaddr_in addr;
73
+ goto err_init_hdev_kick;
78
+ socklen_t len;
79
+
80
+ memset(&addr, 0, sizeof(addr));
81
+ addr.sin_family = AF_INET;
82
+ addr.sin_addr.s_addr = INADDR_ANY;
83
+ addr.sin_port = 0;
84
+ if (bind(sock, (struct sockaddr *)&addr, sizeof(addr)) < 0) {
85
+ return -1;
86
+ }
74
+ }
87
+
75
+
88
+ len = sizeof(addr);
76
+ r = event_notifier_init(&svq->hdev_call, 0);
89
+ if (getsockname(sock, (struct sockaddr *)&addr, &len) < 0) {
77
+ if (r != 0) {
90
+ return -1;
78
+ error_report("Couldn't create call event notifier: %s (%d)",
79
+ g_strerror(errno), errno);
80
+ goto err_init_hdev_call;
91
+ }
81
+ }
92
+
82
+
93
+ return ntohs(addr.sin_port);
83
+ return g_steal_pointer(&svq);
84
+
85
+err_init_hdev_call:
86
+ event_notifier_cleanup(&svq->hdev_kick);
87
+
88
+err_init_hdev_kick:
89
+ return NULL;
94
+}
90
+}
95
+
91
+
96
+static int inet_get_free_port_socket_ipv6(int sock)
92
+/**
93
+ * Free the resources of the shadow virtqueue.
94
+ *
95
+ * @pvq: gpointer to SVQ so it can be used by autofree functions.
96
+ */
97
+void vhost_svq_free(gpointer pvq)
97
+{
98
+{
98
+ struct sockaddr_in6 addr;
99
+ VhostShadowVirtqueue *vq = pvq;
99
+ socklen_t len;
100
+ event_notifier_cleanup(&vq->hdev_kick);
101
+ event_notifier_cleanup(&vq->hdev_call);
102
+ g_free(vq);
103
+}
104
diff --git a/hw/virtio/vhost-shadow-virtqueue.h b/hw/virtio/vhost-shadow-virtqueue.h
105
new file mode 100644
106
index XXXXXXX..XXXXXXX
107
--- /dev/null
108
+++ b/hw/virtio/vhost-shadow-virtqueue.h
109
@@ -XXX,XX +XXX,XX @@
110
+/*
111
+ * vhost shadow virtqueue
112
+ *
113
+ * SPDX-FileCopyrightText: Red Hat, Inc. 2021
114
+ * SPDX-FileContributor: Author: Eugenio Pérez <eperezma@redhat.com>
115
+ *
116
+ * SPDX-License-Identifier: GPL-2.0-or-later
117
+ */
100
+
118
+
101
+ memset(&addr, 0, sizeof(addr));
119
+#ifndef VHOST_SHADOW_VIRTQUEUE_H
102
+ addr.sin6_family = AF_INET6;
120
+#define VHOST_SHADOW_VIRTQUEUE_H
103
+ addr.sin6_addr = in6addr_any;
104
+ addr.sin6_port = 0;
105
+ if (bind(sock, (struct sockaddr *)&addr, sizeof(addr)) < 0) {
106
+ return -1;
107
+ }
108
+
121
+
109
+ len = sizeof(addr);
122
+#include "qemu/event_notifier.h"
110
+ if (getsockname(sock, (struct sockaddr *)&addr, &len) < 0) {
111
+ return -1;
112
+ }
113
+
123
+
114
+ return ntohs(addr.sin6_port);
124
+/* Shadow virtqueue to relay notifications */
115
+}
125
+typedef struct VhostShadowVirtqueue {
126
+ /* Shadow kick notifier, sent to vhost */
127
+ EventNotifier hdev_kick;
128
+ /* Shadow call notifier, sent to vhost */
129
+ EventNotifier hdev_call;
130
+} VhostShadowVirtqueue;
116
+
131
+
117
+static int inet_get_free_port_multiple(int nb, int *port, bool ipv6)
132
+VhostShadowVirtqueue *vhost_svq_new(void);
118
+{
119
+ int sock[nb];
120
+ int i;
121
+
133
+
122
+ for (i = 0; i < nb; i++) {
134
+void vhost_svq_free(gpointer vq);
123
+ sock[i] = socket(ipv6 ? AF_INET6 : AF_INET, SOCK_STREAM, 0);
135
+G_DEFINE_AUTOPTR_CLEANUP_FUNC(VhostShadowVirtqueue, vhost_svq_free);
124
+ if (sock[i] < 0) {
125
+ break;
126
+ }
127
+ port[i] = ipv6 ? inet_get_free_port_socket_ipv6(sock[i]) :
128
+ inet_get_free_port_socket_ipv4(sock[i]);
129
+ if (port[i] == -1) {
130
+ break;
131
+ }
132
+ }
133
+
136
+
134
+ nb = i;
135
+ for (i = 0; i < nb; i++) {
136
+ closesocket(sock[i]);
137
+ }
138
+
139
+ return nb;
140
+}
141
+
142
+static int inet_get_free_port(bool ipv6)
143
+{
144
+ int nb, port;
145
+
146
+ nb = inet_get_free_port_multiple(1, &port, ipv6);
147
+ g_assert_cmpint(nb, ==, 1);
148
+
149
+ return port;
150
+}
151
+
152
+static void test_stream_inet_ipv4(void)
153
+{
154
+ QTestState *qts0, *qts1;
155
+ char *expect;
156
+ int port;
157
+
158
+ port = inet_get_free_port(false);
159
+ qts0 = qtest_initf("-nodefaults -M none "
160
+ "-netdev stream,id=st0,server=true,addr.type=inet,"
161
+ "addr.ipv4=on,addr.ipv6=off,"
162
+ "addr.host=127.0.0.1,addr.port=%d", port);
163
+
164
+ EXPECT_STATE(qts0, "st0: index=0,type=stream,\r\n", 0);
165
+
166
+ qts1 = qtest_initf("-nodefaults -M none "
167
+ "-netdev stream,server=false,id=st0,addr.type=inet,"
168
+ "addr.ipv4=on,addr.ipv6=off,"
169
+ "addr.host=127.0.0.1,addr.port=%d", port);
170
+
171
+ expect = g_strdup_printf("st0: index=0,type=stream,tcp:127.0.0.1:%d\r\n",
172
+ port);
173
+ EXPECT_STATE(qts1, expect, 0);
174
+ g_free(expect);
175
+
176
+ /* the port is unknown, check only the address */
177
+ EXPECT_STATE(qts0, "st0: index=0,type=stream,tcp:127.0.0.1", ':');
178
+
179
+ qtest_quit(qts1);
180
+ qtest_quit(qts0);
181
+}
182
+
183
+static void test_stream_inet_ipv6(void)
184
+{
185
+ QTestState *qts0, *qts1;
186
+ char *expect;
187
+ int port;
188
+
189
+ port = inet_get_free_port(true);
190
+ qts0 = qtest_initf("-nodefaults -M none "
191
+ "-netdev stream,id=st0,server=true,addr.type=inet,"
192
+ "addr.ipv4=off,addr.ipv6=on,"
193
+ "addr.host=::1,addr.port=%d", port);
194
+
195
+ EXPECT_STATE(qts0, "st0: index=0,type=stream,\r\n", 0);
196
+
197
+ qts1 = qtest_initf("-nodefaults -M none "
198
+ "-netdev stream,server=false,id=st0,addr.type=inet,"
199
+ "addr.ipv4=off,addr.ipv6=on,"
200
+ "addr.host=::1,addr.port=%d", port);
201
+
202
+ expect = g_strdup_printf("st0: index=0,type=stream,tcp:::1:%d\r\n",
203
+ port);
204
+ EXPECT_STATE(qts1, expect, 0);
205
+ g_free(expect);
206
+
207
+ /* the port is unknown, check only the address */
208
+ EXPECT_STATE(qts0, "st0: index=0,type=stream,tcp:::1", ':');
209
+
210
+ qtest_quit(qts1);
211
+ qtest_quit(qts0);
212
+}
213
+
214
+static void test_stream_unix(void)
215
+{
216
+ QTestState *qts0, *qts1;
217
+ char *expect;
218
+ gchar *path;
219
+
220
+ path = g_strconcat(tmpdir, "/stream_unix", NULL);
221
+
222
+ qts0 = qtest_initf("-nodefaults -M none "
223
+ "-netdev stream,id=st0,server=true,"
224
+ "addr.type=unix,addr.path=%s,",
225
+ path);
226
+
227
+ EXPECT_STATE(qts0, "st0: index=0,type=stream,\r\n", 0);
228
+
229
+ qts1 = qtest_initf("-nodefaults -M none "
230
+ "-netdev stream,id=st0,server=false,"
231
+ "addr.type=unix,addr.path=%s",
232
+ path);
233
+
234
+ expect = g_strdup_printf("st0: index=0,type=stream,unix:%s\r\n", path);
235
+ EXPECT_STATE(qts1, expect, 0);
236
+ EXPECT_STATE(qts0, expect, 0);
237
+ g_free(expect);
238
+ g_free(path);
239
+
240
+ qtest_quit(qts1);
241
+ qtest_quit(qts0);
242
+}
243
+
244
+#ifdef CONFIG_LINUX
245
+static void test_stream_unix_abstract(void)
246
+{
247
+ QTestState *qts0, *qts1;
248
+ char *expect;
249
+ gchar *path;
250
+
251
+ path = g_strconcat(tmpdir, "/stream_unix_abstract", NULL);
252
+
253
+ qts0 = qtest_initf("-nodefaults -M none "
254
+ "-netdev stream,id=st0,server=true,"
255
+ "addr.type=unix,addr.path=%s,"
256
+ "addr.abstract=on",
257
+ path);
258
+
259
+ EXPECT_STATE(qts0, "st0: index=0,type=stream,\r\n", 0);
260
+
261
+ qts1 = qtest_initf("-nodefaults -M none "
262
+ "-netdev stream,id=st0,server=false,"
263
+ "addr.type=unix,addr.path=%s,addr.abstract=on",
264
+ path);
265
+
266
+ expect = g_strdup_printf("st0: index=0,type=stream,unix:%s\r\n", path);
267
+ EXPECT_STATE(qts1, expect, 0);
268
+ EXPECT_STATE(qts0, expect, 0);
269
+ g_free(expect);
270
+ g_free(path);
271
+
272
+ qtest_quit(qts1);
273
+ qtest_quit(qts0);
274
+}
275
+#endif
137
+#endif
276
+
277
+#ifndef _WIN32
278
+static void test_stream_fd(void)
279
+{
280
+ QTestState *qts0, *qts1;
281
+ int sock[2];
282
+ int ret;
283
+
284
+ ret = socketpair(AF_LOCAL, SOCK_STREAM, 0, sock);
285
+ g_assert_true(ret == 0);
286
+
287
+ qts0 = qtest_initf("-nodefaults -M none "
288
+ "-netdev stream,id=st0,addr.type=fd,addr.str=%d",
289
+ sock[0]);
290
+
291
+ EXPECT_STATE(qts0, "st0: index=0,type=stream,unix:\r\n", 0);
292
+
293
+ qts1 = qtest_initf("-nodefaults -M none "
294
+ "-netdev stream,id=st0,addr.type=fd,addr.str=%d",
295
+ sock[1]);
296
+
297
+ EXPECT_STATE(qts1, "st0: index=0,type=stream,unix:\r\n", 0);
298
+ EXPECT_STATE(qts0, "st0: index=0,type=stream,unix:\r\n", 0);
299
+
300
+ qtest_quit(qts1);
301
+ qtest_quit(qts0);
302
+
303
+ closesocket(sock[0]);
304
+ closesocket(sock[1]);
305
+}
306
+#endif
307
+
308
+static void test_dgram_inet(void)
309
+{
310
+ QTestState *qts0, *qts1;
311
+ char *expect;
312
+ int port[2];
313
+ int nb;
314
+
315
+ nb = inet_get_free_port_multiple(2, port, false);
316
+ g_assert_cmpint(nb, ==, 2);
317
+
318
+ qts0 = qtest_initf("-nodefaults -M none "
319
+ "-netdev dgram,id=st0,"
320
+ "local.type=inet,local.host=127.0.0.1,local.port=%d,"
321
+ "remote.type=inet,remote.host=127.0.0.1,remote.port=%d",
322
+ port[0], port[1]);
323
+
324
+ expect = g_strdup_printf("st0: index=0,type=dgram,"
325
+ "udp=127.0.0.1:%d/127.0.0.1:%d\r\n",
326
+ port[0], port[1]);
327
+ EXPECT_STATE(qts0, expect, 0);
328
+ g_free(expect);
329
+
330
+ qts1 = qtest_initf("-nodefaults -M none "
331
+ "-netdev dgram,id=st0,"
332
+ "local.type=inet,local.host=127.0.0.1,local.port=%d,"
333
+ "remote.type=inet,remote.host=127.0.0.1,remote.port=%d",
334
+ port[1], port[0]);
335
+
336
+ expect = g_strdup_printf("st0: index=0,type=dgram,"
337
+ "udp=127.0.0.1:%d/127.0.0.1:%d\r\n",
338
+ port[1], port[0]);
339
+ EXPECT_STATE(qts1, expect, 0);
340
+ g_free(expect);
341
+
342
+ qtest_quit(qts1);
343
+ qtest_quit(qts0);
344
+}
345
+
346
+#ifndef _WIN32
347
+static void test_dgram_mcast(void)
348
+{
349
+ QTestState *qts;
350
+
351
+ qts = qtest_initf("-nodefaults -M none "
352
+ "-netdev dgram,id=st0,"
353
+ "remote.type=inet,remote.host=230.0.0.1,remote.port=1234");
354
+
355
+ EXPECT_STATE(qts, "st0: index=0,type=dgram,mcast=230.0.0.1:1234\r\n", 0);
356
+
357
+ qtest_quit(qts);
358
+}
359
+
360
+static void test_dgram_unix(void)
361
+{
362
+ QTestState *qts0, *qts1;
363
+ char *expect;
364
+ gchar *path0, *path1;
365
+
366
+ path0 = g_strconcat(tmpdir, "/dgram_unix0", NULL);
367
+ path1 = g_strconcat(tmpdir, "/dgram_unix1", NULL);
368
+
369
+ qts0 = qtest_initf("-nodefaults -M none "
370
+ "-netdev dgram,id=st0,local.type=unix,local.path=%s,"
371
+ "remote.type=unix,remote.path=%s",
372
+ path0, path1);
373
+
374
+ expect = g_strdup_printf("st0: index=0,type=dgram,udp=%s:%s\r\n",
375
+ path0, path1);
376
+ EXPECT_STATE(qts0, expect, 0);
377
+ g_free(expect);
378
+
379
+ qts1 = qtest_initf("-nodefaults -M none "
380
+ "-netdev dgram,id=st0,local.type=unix,local.path=%s,"
381
+ "remote.type=unix,remote.path=%s",
382
+ path1, path0);
383
+
384
+
385
+ expect = g_strdup_printf("st0: index=0,type=dgram,udp=%s:%s\r\n",
386
+ path1, path0);
387
+ EXPECT_STATE(qts1, expect, 0);
388
+ g_free(expect);
389
+
390
+ unlink(path0);
391
+ g_free(path0);
392
+ unlink(path1);
393
+ g_free(path1);
394
+
395
+ qtest_quit(qts1);
396
+ qtest_quit(qts0);
397
+}
398
+
399
+static void test_dgram_fd(void)
400
+{
401
+ QTestState *qts0, *qts1;
402
+ char *expect;
403
+ int ret;
404
+ int sv[2];
405
+
406
+ ret = socketpair(PF_UNIX, SOCK_DGRAM, 0, sv);
407
+ g_assert_cmpint(ret, !=, -1);
408
+
409
+ qts0 = qtest_initf("-nodefaults -M none "
410
+ "-netdev dgram,id=st0,local.type=fd,local.str=%d",
411
+ sv[0]);
412
+
413
+ expect = g_strdup_printf("st0: index=0,type=dgram,fd=%d unix\r\n", sv[0]);
414
+ EXPECT_STATE(qts0, expect, 0);
415
+ g_free(expect);
416
+
417
+ qts1 = qtest_initf("-nodefaults -M none "
418
+ "-netdev dgram,id=st0,local.type=fd,local.str=%d",
419
+ sv[1]);
420
+
421
+
422
+ expect = g_strdup_printf("st0: index=0,type=dgram,fd=%d unix\r\n", sv[1]);
423
+ EXPECT_STATE(qts1, expect, 0);
424
+ g_free(expect);
425
+
426
+ qtest_quit(qts1);
427
+ qtest_quit(qts0);
428
+
429
+ closesocket(sv[0]);
430
+ closesocket(sv[1]);
431
+}
432
+#endif
433
+
434
+int main(int argc, char **argv)
435
+{
436
+ int ret;
437
+ bool has_ipv4, has_ipv6, has_afunix;
438
+ g_autoptr(GError) err = NULL;
439
+
440
+ socket_init();
441
+ g_test_init(&argc, &argv, NULL);
442
+
443
+ if (socket_check_protocol_support(&has_ipv4, &has_ipv6) < 0) {
444
+ g_error("socket_check_protocol_support() failed\n");
445
+ }
446
+
447
+ tmpdir = g_dir_make_tmp("netdev-socket.XXXXXX", &err);
448
+ if (tmpdir == NULL) {
449
+ g_error("Can't create temporary directory in %s: %s",
450
+ g_get_tmp_dir(), err->message);
451
+ }
452
+
453
+ if (has_ipv4) {
454
+ qtest_add_func("/netdev/stream/inet/ipv4", test_stream_inet_ipv4);
455
+ qtest_add_func("/netdev/dgram/inet", test_dgram_inet);
456
+#ifndef _WIN32
457
+ qtest_add_func("/netdev/dgram/mcast", test_dgram_mcast);
458
+#endif
459
+ }
460
+ if (has_ipv6) {
461
+ qtest_add_func("/netdev/stream/inet/ipv6", test_stream_inet_ipv6);
462
+ }
463
+
464
+ socket_check_afunix_support(&has_afunix);
465
+ if (has_afunix) {
466
+#ifndef _WIN32
467
+ qtest_add_func("/netdev/dgram/unix", test_dgram_unix);
468
+#endif
469
+ qtest_add_func("/netdev/stream/unix", test_stream_unix);
470
+#ifdef CONFIG_LINUX
471
+ qtest_add_func("/netdev/stream/unix/abstract",
472
+ test_stream_unix_abstract);
473
+#endif
474
+#ifndef _WIN32
475
+ qtest_add_func("/netdev/stream/fd", test_stream_fd);
476
+ qtest_add_func("/netdev/dgram/fd", test_dgram_fd);
477
+#endif
478
+ }
479
+
480
+ ret = g_test_run();
481
+
482
+ g_rmdir(tmpdir);
483
+ g_free(tmpdir);
484
+
485
+ return ret;
486
+}
487
--
138
--
488
2.7.4
139
2.7.4
140
141
diff view generated by jsdifflib
New patch
1
1
From: Eugenio Pérez <eperezma@redhat.com>
2
3
At this mode no buffer forwarding will be performed in SVQ mode: Qemu
4
will just forward the guest's kicks to the device.
5
6
Host memory notifiers regions are left out for simplicity, and they will
7
not be addressed in this series.
8
9
Signed-off-by: Eugenio Pérez <eperezma@redhat.com>
10
Acked-by: Michael S. Tsirkin <mst@redhat.com>
11
Signed-off-by: Jason Wang <jasowang@redhat.com>
12
---
13
hw/virtio/vhost-shadow-virtqueue.c | 55 ++++++++++++++
14
hw/virtio/vhost-shadow-virtqueue.h | 14 ++++
15
hw/virtio/vhost-vdpa.c | 144 ++++++++++++++++++++++++++++++++++++-
16
include/hw/virtio/vhost-vdpa.h | 4 ++
17
4 files changed, 215 insertions(+), 2 deletions(-)
18
19
diff --git a/hw/virtio/vhost-shadow-virtqueue.c b/hw/virtio/vhost-shadow-virtqueue.c
20
index XXXXXXX..XXXXXXX 100644
21
--- a/hw/virtio/vhost-shadow-virtqueue.c
22
+++ b/hw/virtio/vhost-shadow-virtqueue.c
23
@@ -XXX,XX +XXX,XX @@
24
#include "hw/virtio/vhost-shadow-virtqueue.h"
25
26
#include "qemu/error-report.h"
27
+#include "qemu/main-loop.h"
28
+#include "linux-headers/linux/vhost.h"
29
+
30
+/**
31
+ * Forward guest notifications.
32
+ *
33
+ * @n: guest kick event notifier, the one that guest set to notify svq.
34
+ */
35
+static void vhost_handle_guest_kick(EventNotifier *n)
36
+{
37
+ VhostShadowVirtqueue *svq = container_of(n, VhostShadowVirtqueue, svq_kick);
38
+ event_notifier_test_and_clear(n);
39
+ event_notifier_set(&svq->hdev_kick);
40
+}
41
+
42
+/**
43
+ * Set a new file descriptor for the guest to kick the SVQ and notify for avail
44
+ *
45
+ * @svq: The svq
46
+ * @svq_kick_fd: The svq kick fd
47
+ *
48
+ * Note that the SVQ will never close the old file descriptor.
49
+ */
50
+void vhost_svq_set_svq_kick_fd(VhostShadowVirtqueue *svq, int svq_kick_fd)
51
+{
52
+ EventNotifier *svq_kick = &svq->svq_kick;
53
+ bool poll_stop = VHOST_FILE_UNBIND != event_notifier_get_fd(svq_kick);
54
+ bool poll_start = svq_kick_fd != VHOST_FILE_UNBIND;
55
+
56
+ if (poll_stop) {
57
+ event_notifier_set_handler(svq_kick, NULL);
58
+ }
59
+
60
+ /*
61
+ * event_notifier_set_handler already checks for guest's notifications if
62
+ * they arrive at the new file descriptor in the switch, so there is no
63
+ * need to explicitly check for them.
64
+ */
65
+ if (poll_start) {
66
+ event_notifier_init_fd(svq_kick, svq_kick_fd);
67
+ event_notifier_set(svq_kick);
68
+ event_notifier_set_handler(svq_kick, vhost_handle_guest_kick);
69
+ }
70
+}
71
+
72
+/**
73
+ * Stop the shadow virtqueue operation.
74
+ * @svq: Shadow Virtqueue
75
+ */
76
+void vhost_svq_stop(VhostShadowVirtqueue *svq)
77
+{
78
+ event_notifier_set_handler(&svq->svq_kick, NULL);
79
+}
80
81
/**
82
* Creates vhost shadow virtqueue, and instructs the vhost device to use the
83
@@ -XXX,XX +XXX,XX @@ VhostShadowVirtqueue *vhost_svq_new(void)
84
goto err_init_hdev_call;
85
}
86
87
+ event_notifier_init_fd(&svq->svq_kick, VHOST_FILE_UNBIND);
88
return g_steal_pointer(&svq);
89
90
err_init_hdev_call:
91
@@ -XXX,XX +XXX,XX @@ err_init_hdev_kick:
92
void vhost_svq_free(gpointer pvq)
93
{
94
VhostShadowVirtqueue *vq = pvq;
95
+ vhost_svq_stop(vq);
96
event_notifier_cleanup(&vq->hdev_kick);
97
event_notifier_cleanup(&vq->hdev_call);
98
g_free(vq);
99
diff --git a/hw/virtio/vhost-shadow-virtqueue.h b/hw/virtio/vhost-shadow-virtqueue.h
100
index XXXXXXX..XXXXXXX 100644
101
--- a/hw/virtio/vhost-shadow-virtqueue.h
102
+++ b/hw/virtio/vhost-shadow-virtqueue.h
103
@@ -XXX,XX +XXX,XX @@ typedef struct VhostShadowVirtqueue {
104
EventNotifier hdev_kick;
105
/* Shadow call notifier, sent to vhost */
106
EventNotifier hdev_call;
107
+
108
+ /*
109
+ * Borrowed virtqueue's guest to host notifier. To borrow it in this event
110
+ * notifier allows to recover the VhostShadowVirtqueue from the event loop
111
+ * easily. If we use the VirtQueue's one, we don't have an easy way to
112
+ * retrieve VhostShadowVirtqueue.
113
+ *
114
+ * So shadow virtqueue must not clean it, or we would lose VirtQueue one.
115
+ */
116
+ EventNotifier svq_kick;
117
} VhostShadowVirtqueue;
118
119
+void vhost_svq_set_svq_kick_fd(VhostShadowVirtqueue *svq, int svq_kick_fd);
120
+
121
+void vhost_svq_stop(VhostShadowVirtqueue *svq);
122
+
123
VhostShadowVirtqueue *vhost_svq_new(void);
124
125
void vhost_svq_free(gpointer vq);
126
diff --git a/hw/virtio/vhost-vdpa.c b/hw/virtio/vhost-vdpa.c
127
index XXXXXXX..XXXXXXX 100644
128
--- a/hw/virtio/vhost-vdpa.c
129
+++ b/hw/virtio/vhost-vdpa.c
130
@@ -XXX,XX +XXX,XX @@
131
#include "hw/virtio/vhost.h"
132
#include "hw/virtio/vhost-backend.h"
133
#include "hw/virtio/virtio-net.h"
134
+#include "hw/virtio/vhost-shadow-virtqueue.h"
135
#include "hw/virtio/vhost-vdpa.h"
136
#include "exec/address-spaces.h"
137
#include "qemu/main-loop.h"
138
#include "cpu.h"
139
#include "trace.h"
140
#include "qemu-common.h"
141
+#include "qapi/error.h"
142
143
/*
144
* Return one past the end of the end of section. Be careful with uint64_t
145
@@ -XXX,XX +XXX,XX @@ static bool vhost_vdpa_one_time_request(struct vhost_dev *dev)
146
return v->index != 0;
147
}
148
149
+static int vhost_vdpa_init_svq(struct vhost_dev *hdev, struct vhost_vdpa *v,
150
+ Error **errp)
151
+{
152
+ g_autoptr(GPtrArray) shadow_vqs = NULL;
153
+
154
+ if (!v->shadow_vqs_enabled) {
155
+ return 0;
156
+ }
157
+
158
+ shadow_vqs = g_ptr_array_new_full(hdev->nvqs, vhost_svq_free);
159
+ for (unsigned n = 0; n < hdev->nvqs; ++n) {
160
+ g_autoptr(VhostShadowVirtqueue) svq = vhost_svq_new();
161
+
162
+ if (unlikely(!svq)) {
163
+ error_setg(errp, "Cannot create svq %u", n);
164
+ return -1;
165
+ }
166
+ g_ptr_array_add(shadow_vqs, g_steal_pointer(&svq));
167
+ }
168
+
169
+ v->shadow_vqs = g_steal_pointer(&shadow_vqs);
170
+ return 0;
171
+}
172
+
173
static int vhost_vdpa_init(struct vhost_dev *dev, void *opaque, Error **errp)
174
{
175
struct vhost_vdpa *v;
176
@@ -XXX,XX +XXX,XX @@ static int vhost_vdpa_init(struct vhost_dev *dev, void *opaque, Error **errp)
177
dev->opaque = opaque ;
178
v->listener = vhost_vdpa_memory_listener;
179
v->msg_type = VHOST_IOTLB_MSG_V2;
180
+ ret = vhost_vdpa_init_svq(dev, v, errp);
181
+ if (ret) {
182
+ goto err;
183
+ }
184
185
vhost_vdpa_get_iova_range(v);
186
187
@@ -XXX,XX +XXX,XX @@ static int vhost_vdpa_init(struct vhost_dev *dev, void *opaque, Error **errp)
188
VIRTIO_CONFIG_S_DRIVER);
189
190
return 0;
191
+
192
+err:
193
+ ram_block_discard_disable(false);
194
+ return ret;
195
}
196
197
static void vhost_vdpa_host_notifier_uninit(struct vhost_dev *dev,
198
@@ -XXX,XX +XXX,XX @@ static void vhost_vdpa_host_notifiers_uninit(struct vhost_dev *dev, int n)
199
200
static void vhost_vdpa_host_notifiers_init(struct vhost_dev *dev)
201
{
202
+ struct vhost_vdpa *v = dev->opaque;
203
int i;
204
205
+ if (v->shadow_vqs_enabled) {
206
+ /* FIXME SVQ is not compatible with host notifiers mr */
207
+ return;
208
+ }
209
+
210
for (i = dev->vq_index; i < dev->vq_index + dev->nvqs; i++) {
211
if (vhost_vdpa_host_notifier_init(dev, i)) {
212
goto err;
213
@@ -XXX,XX +XXX,XX @@ err:
214
return;
215
}
216
217
+static void vhost_vdpa_svq_cleanup(struct vhost_dev *dev)
218
+{
219
+ struct vhost_vdpa *v = dev->opaque;
220
+ size_t idx;
221
+
222
+ if (!v->shadow_vqs) {
223
+ return;
224
+ }
225
+
226
+ for (idx = 0; idx < v->shadow_vqs->len; ++idx) {
227
+ vhost_svq_stop(g_ptr_array_index(v->shadow_vqs, idx));
228
+ }
229
+ g_ptr_array_free(v->shadow_vqs, true);
230
+}
231
+
232
static int vhost_vdpa_cleanup(struct vhost_dev *dev)
233
{
234
struct vhost_vdpa *v;
235
@@ -XXX,XX +XXX,XX @@ static int vhost_vdpa_cleanup(struct vhost_dev *dev)
236
trace_vhost_vdpa_cleanup(dev, v);
237
vhost_vdpa_host_notifiers_uninit(dev, dev->nvqs);
238
memory_listener_unregister(&v->listener);
239
+ vhost_vdpa_svq_cleanup(dev);
240
241
dev->opaque = NULL;
242
ram_block_discard_disable(false);
243
@@ -XXX,XX +XXX,XX @@ static int vhost_vdpa_get_device_id(struct vhost_dev *dev,
244
return ret;
245
}
246
247
+static void vhost_vdpa_reset_svq(struct vhost_vdpa *v)
248
+{
249
+ if (!v->shadow_vqs_enabled) {
250
+ return;
251
+ }
252
+
253
+ for (unsigned i = 0; i < v->shadow_vqs->len; ++i) {
254
+ VhostShadowVirtqueue *svq = g_ptr_array_index(v->shadow_vqs, i);
255
+ vhost_svq_stop(svq);
256
+ }
257
+}
258
+
259
static int vhost_vdpa_reset_device(struct vhost_dev *dev)
260
{
261
+ struct vhost_vdpa *v = dev->opaque;
262
int ret;
263
uint8_t status = 0;
264
265
+ vhost_vdpa_reset_svq(v);
266
+
267
ret = vhost_vdpa_call(dev, VHOST_VDPA_SET_STATUS, &status);
268
trace_vhost_vdpa_reset_device(dev, status);
269
return ret;
270
@@ -XXX,XX +XXX,XX @@ static int vhost_vdpa_get_config(struct vhost_dev *dev, uint8_t *config,
271
return ret;
272
}
273
274
+static int vhost_vdpa_set_vring_dev_kick(struct vhost_dev *dev,
275
+ struct vhost_vring_file *file)
276
+{
277
+ trace_vhost_vdpa_set_vring_kick(dev, file->index, file->fd);
278
+ return vhost_vdpa_call(dev, VHOST_SET_VRING_KICK, file);
279
+}
280
+
281
+/**
282
+ * Set the shadow virtqueue descriptors to the device
283
+ *
284
+ * @dev: The vhost device model
285
+ * @svq: The shadow virtqueue
286
+ * @idx: The index of the virtqueue in the vhost device
287
+ * @errp: Error
288
+ */
289
+static bool vhost_vdpa_svq_setup(struct vhost_dev *dev,
290
+ VhostShadowVirtqueue *svq, unsigned idx,
291
+ Error **errp)
292
+{
293
+ struct vhost_vring_file file = {
294
+ .index = dev->vq_index + idx,
295
+ };
296
+ const EventNotifier *event_notifier = &svq->hdev_kick;
297
+ int r;
298
+
299
+ file.fd = event_notifier_get_fd(event_notifier);
300
+ r = vhost_vdpa_set_vring_dev_kick(dev, &file);
301
+ if (unlikely(r != 0)) {
302
+ error_setg_errno(errp, -r, "Can't set device kick fd");
303
+ }
304
+
305
+ return r == 0;
306
+}
307
+
308
+static bool vhost_vdpa_svqs_start(struct vhost_dev *dev)
309
+{
310
+ struct vhost_vdpa *v = dev->opaque;
311
+ Error *err = NULL;
312
+ unsigned i;
313
+
314
+ if (!v->shadow_vqs) {
315
+ return true;
316
+ }
317
+
318
+ for (i = 0; i < v->shadow_vqs->len; ++i) {
319
+ VhostShadowVirtqueue *svq = g_ptr_array_index(v->shadow_vqs, i);
320
+ bool ok = vhost_vdpa_svq_setup(dev, svq, i, &err);
321
+ if (unlikely(!ok)) {
322
+ error_reportf_err(err, "Cannot setup SVQ %u: ", i);
323
+ return false;
324
+ }
325
+ }
326
+
327
+ return true;
328
+}
329
+
330
static int vhost_vdpa_dev_start(struct vhost_dev *dev, bool started)
331
{
332
struct vhost_vdpa *v = dev->opaque;
333
+ bool ok;
334
trace_vhost_vdpa_dev_start(dev, started);
335
336
if (started) {
337
vhost_vdpa_host_notifiers_init(dev);
338
+ ok = vhost_vdpa_svqs_start(dev);
339
+ if (unlikely(!ok)) {
340
+ return -1;
341
+ }
342
vhost_vdpa_set_vring_ready(dev);
343
} else {
344
vhost_vdpa_host_notifiers_uninit(dev, dev->nvqs);
345
@@ -XXX,XX +XXX,XX @@ static int vhost_vdpa_get_vring_base(struct vhost_dev *dev,
346
static int vhost_vdpa_set_vring_kick(struct vhost_dev *dev,
347
struct vhost_vring_file *file)
348
{
349
- trace_vhost_vdpa_set_vring_kick(dev, file->index, file->fd);
350
- return vhost_vdpa_call(dev, VHOST_SET_VRING_KICK, file);
351
+ struct vhost_vdpa *v = dev->opaque;
352
+ int vdpa_idx = file->index - dev->vq_index;
353
+
354
+ if (v->shadow_vqs_enabled) {
355
+ VhostShadowVirtqueue *svq = g_ptr_array_index(v->shadow_vqs, vdpa_idx);
356
+ vhost_svq_set_svq_kick_fd(svq, file->fd);
357
+ return 0;
358
+ } else {
359
+ return vhost_vdpa_set_vring_dev_kick(dev, file);
360
+ }
361
}
362
363
static int vhost_vdpa_set_vring_call(struct vhost_dev *dev,
364
diff --git a/include/hw/virtio/vhost-vdpa.h b/include/hw/virtio/vhost-vdpa.h
365
index XXXXXXX..XXXXXXX 100644
366
--- a/include/hw/virtio/vhost-vdpa.h
367
+++ b/include/hw/virtio/vhost-vdpa.h
368
@@ -XXX,XX +XXX,XX @@
369
#ifndef HW_VIRTIO_VHOST_VDPA_H
370
#define HW_VIRTIO_VHOST_VDPA_H
371
372
+#include <gmodule.h>
373
+
374
#include "hw/virtio/virtio.h"
375
#include "standard-headers/linux/vhost_types.h"
376
377
@@ -XXX,XX +XXX,XX @@ typedef struct vhost_vdpa {
378
bool iotlb_batch_begin_sent;
379
MemoryListener listener;
380
struct vhost_vdpa_iova_range iova_range;
381
+ bool shadow_vqs_enabled;
382
+ GPtrArray *shadow_vqs;
383
struct vhost_dev *dev;
384
VhostVDPAHostNotifier notifier[VIRTIO_QUEUE_MAX];
385
} VhostVDPA;
386
--
387
2.7.4
388
389
diff view generated by jsdifflib
1
From: Joelle van Dyne <j@getutm.app>
1
From: Eugenio Pérez <eperezma@redhat.com>
2
2
3
When the VM is stopped using the HMP command "stop", soon the handler will
3
This will make qemu aware of the device used buffers, allowing it to
4
stop reading from the vmnet interface. This causes a flood of
4
write the guest memory with its contents if needed.
5
`VMNET_INTERFACE_PACKETS_AVAILABLE` events to arrive and puts the host CPU
6
at 100%. We fix this by removing the event handler from vmnet when the VM
7
is no longer in a running state and restore it when we return to a running
8
state.
9
5
10
Signed-off-by: Joelle van Dyne <j@getutm.app>
6
Signed-off-by: Eugenio Pérez <eperezma@redhat.com>
7
Acked-by: Michael S. Tsirkin <mst@redhat.com>
11
Signed-off-by: Jason Wang <jasowang@redhat.com>
8
Signed-off-by: Jason Wang <jasowang@redhat.com>
12
---
9
---
13
net/vmnet-common.m | 48 +++++++++++++++++++++++++++++++++++-------------
10
hw/virtio/vhost-shadow-virtqueue.c | 38 ++++++++++++++++++++++++++++++++++++++
14
net/vmnet_int.h | 2 ++
11
hw/virtio/vhost-shadow-virtqueue.h | 4 ++++
15
2 files changed, 37 insertions(+), 13 deletions(-)
12
hw/virtio/vhost-vdpa.c | 31 +++++++++++++++++++++++++++++--
13
3 files changed, 71 insertions(+), 2 deletions(-)
16
14
17
diff --git a/net/vmnet-common.m b/net/vmnet-common.m
15
diff --git a/hw/virtio/vhost-shadow-virtqueue.c b/hw/virtio/vhost-shadow-virtqueue.c
18
index XXXXXXX..XXXXXXX 100644
16
index XXXXXXX..XXXXXXX 100644
19
--- a/net/vmnet-common.m
17
--- a/hw/virtio/vhost-shadow-virtqueue.c
20
+++ b/net/vmnet-common.m
18
+++ b/hw/virtio/vhost-shadow-virtqueue.c
21
@@ -XXX,XX +XXX,XX @@
19
@@ -XXX,XX +XXX,XX @@ static void vhost_handle_guest_kick(EventNotifier *n)
22
#include "clients.h"
23
#include "qemu/error-report.h"
24
#include "qapi/error.h"
25
+#include "sysemu/runstate.h"
26
27
#include <vmnet/vmnet.h>
28
#include <dispatch/dispatch.h>
29
@@ -XXX,XX +XXX,XX @@ static void vmnet_bufs_init(VmnetState *s)
30
}
31
}
20
}
32
21
22
/**
23
+ * Forward vhost notifications
24
+ *
25
+ * @n: hdev call event notifier, the one that device set to notify svq.
26
+ */
27
+static void vhost_svq_handle_call(EventNotifier *n)
28
+{
29
+ VhostShadowVirtqueue *svq = container_of(n, VhostShadowVirtqueue,
30
+ hdev_call);
31
+ event_notifier_test_and_clear(n);
32
+ event_notifier_set(&svq->svq_call);
33
+}
34
+
33
+/**
35
+/**
34
+ * Called on state change to un-register/re-register handlers
36
+ * Set the call notifier for the SVQ to call the guest
37
+ *
38
+ * @svq: Shadow virtqueue
39
+ * @call_fd: call notifier
40
+ *
41
+ * Called on BQL context.
35
+ */
42
+ */
36
+static void vmnet_vm_state_change_cb(void *opaque, bool running, RunState state)
43
+void vhost_svq_set_svq_call_fd(VhostShadowVirtqueue *svq, int call_fd)
37
+{
44
+{
38
+ VmnetState *s = opaque;
45
+ if (call_fd == VHOST_FILE_UNBIND) {
39
+
46
+ /*
40
+ if (running) {
47
+ * Fail event_notifier_set if called handling device call.
41
+ vmnet_interface_set_event_callback(
48
+ *
42
+ s->vmnet_if,
49
+ * SVQ still needs device notifications, since it needs to keep
43
+ VMNET_INTERFACE_PACKETS_AVAILABLE,
50
+ * forwarding used buffers even with the unbind.
44
+ s->if_queue,
51
+ */
45
+ ^(interface_event_t event_id, xpc_object_t event) {
52
+ memset(&svq->svq_call, 0, sizeof(svq->svq_call));
46
+ assert(event_id == VMNET_INTERFACE_PACKETS_AVAILABLE);
47
+ /*
48
+ * This function is being called from a non qemu thread, so
49
+ * we only schedule a BH, and do the rest of the io completion
50
+ * handling from vmnet_send_bh() which runs in a qemu context.
51
+ */
52
+ qemu_bh_schedule(s->send_bh);
53
+ });
54
+ } else {
53
+ } else {
55
+ vmnet_interface_set_event_callback(
54
+ event_notifier_init_fd(&svq->svq_call, call_fd);
56
+ s->vmnet_if,
57
+ VMNET_INTERFACE_PACKETS_AVAILABLE,
58
+ NULL,
59
+ NULL);
60
+ }
55
+ }
61
+}
56
+}
62
63
int vmnet_if_create(NetClientState *nc,
64
xpc_object_t if_desc,
65
@@ -XXX,XX +XXX,XX @@ int vmnet_if_create(NetClientState *nc,
66
s->packets_send_current_pos = 0;
67
s->packets_send_end_pos = 0;
68
69
- vmnet_interface_set_event_callback(
70
- s->vmnet_if,
71
- VMNET_INTERFACE_PACKETS_AVAILABLE,
72
- s->if_queue,
73
- ^(interface_event_t event_id, xpc_object_t event) {
74
- assert(event_id == VMNET_INTERFACE_PACKETS_AVAILABLE);
75
- /*
76
- * This function is being called from a non qemu thread, so
77
- * we only schedule a BH, and do the rest of the io completion
78
- * handling from vmnet_send_bh() which runs in a qemu context.
79
- */
80
- qemu_bh_schedule(s->send_bh);
81
- });
82
+ vmnet_vm_state_change_cb(s, 1, RUN_STATE_RUNNING);
83
+
57
+
84
+ s->change = qemu_add_vm_change_state_handler(vmnet_vm_state_change_cb, s);
58
+/**
85
59
* Set a new file descriptor for the guest to kick the SVQ and notify for avail
86
return 0;
60
*
61
* @svq: The svq
62
@@ -XXX,XX +XXX,XX @@ VhostShadowVirtqueue *vhost_svq_new(void)
63
}
64
65
event_notifier_init_fd(&svq->svq_kick, VHOST_FILE_UNBIND);
66
+ event_notifier_set_handler(&svq->hdev_call, vhost_svq_handle_call);
67
return g_steal_pointer(&svq);
68
69
err_init_hdev_call:
70
@@ -XXX,XX +XXX,XX @@ void vhost_svq_free(gpointer pvq)
71
VhostShadowVirtqueue *vq = pvq;
72
vhost_svq_stop(vq);
73
event_notifier_cleanup(&vq->hdev_kick);
74
+ event_notifier_set_handler(&vq->hdev_call, NULL);
75
event_notifier_cleanup(&vq->hdev_call);
76
g_free(vq);
87
}
77
}
88
@@ -XXX,XX +XXX,XX @@ void vmnet_cleanup_common(NetClientState *nc)
78
diff --git a/hw/virtio/vhost-shadow-virtqueue.h b/hw/virtio/vhost-shadow-virtqueue.h
89
return;
79
index XXXXXXX..XXXXXXX 100644
80
--- a/hw/virtio/vhost-shadow-virtqueue.h
81
+++ b/hw/virtio/vhost-shadow-virtqueue.h
82
@@ -XXX,XX +XXX,XX @@ typedef struct VhostShadowVirtqueue {
83
* So shadow virtqueue must not clean it, or we would lose VirtQueue one.
84
*/
85
EventNotifier svq_kick;
86
+
87
+ /* Guest's call notifier, where the SVQ calls guest. */
88
+ EventNotifier svq_call;
89
} VhostShadowVirtqueue;
90
91
void vhost_svq_set_svq_kick_fd(VhostShadowVirtqueue *svq, int svq_kick_fd);
92
+void vhost_svq_set_svq_call_fd(VhostShadowVirtqueue *svq, int call_fd);
93
94
void vhost_svq_stop(VhostShadowVirtqueue *svq);
95
96
diff --git a/hw/virtio/vhost-vdpa.c b/hw/virtio/vhost-vdpa.c
97
index XXXXXXX..XXXXXXX 100644
98
--- a/hw/virtio/vhost-vdpa.c
99
+++ b/hw/virtio/vhost-vdpa.c
100
@@ -XXX,XX +XXX,XX @@ static int vhost_vdpa_set_vring_dev_kick(struct vhost_dev *dev,
101
return vhost_vdpa_call(dev, VHOST_SET_VRING_KICK, file);
102
}
103
104
+static int vhost_vdpa_set_vring_dev_call(struct vhost_dev *dev,
105
+ struct vhost_vring_file *file)
106
+{
107
+ trace_vhost_vdpa_set_vring_call(dev, file->index, file->fd);
108
+ return vhost_vdpa_call(dev, VHOST_SET_VRING_CALL, file);
109
+}
110
+
111
/**
112
* Set the shadow virtqueue descriptors to the device
113
*
114
@@ -XXX,XX +XXX,XX @@ static int vhost_vdpa_set_vring_dev_kick(struct vhost_dev *dev,
115
* @svq: The shadow virtqueue
116
* @idx: The index of the virtqueue in the vhost device
117
* @errp: Error
118
+ *
119
+ * Note that this function does not rewind kick file descriptor if cannot set
120
+ * call one.
121
*/
122
static bool vhost_vdpa_svq_setup(struct vhost_dev *dev,
123
VhostShadowVirtqueue *svq, unsigned idx,
124
@@ -XXX,XX +XXX,XX @@ static bool vhost_vdpa_svq_setup(struct vhost_dev *dev,
125
r = vhost_vdpa_set_vring_dev_kick(dev, &file);
126
if (unlikely(r != 0)) {
127
error_setg_errno(errp, -r, "Can't set device kick fd");
128
+ return false;
129
+ }
130
+
131
+ event_notifier = &svq->hdev_call;
132
+ file.fd = event_notifier_get_fd(event_notifier);
133
+ r = vhost_vdpa_set_vring_dev_call(dev, &file);
134
+ if (unlikely(r != 0)) {
135
+ error_setg_errno(errp, -r, "Can't set device call fd");
90
}
136
}
91
137
92
+ vmnet_vm_state_change_cb(s, 0, RUN_STATE_SHUTDOWN);
138
return r == 0;
93
+ qemu_del_vm_change_state_handler(s->change);
139
@@ -XXX,XX +XXX,XX @@ static int vhost_vdpa_set_vring_kick(struct vhost_dev *dev,
94
if_stopped_sem = dispatch_semaphore_create(0);
140
static int vhost_vdpa_set_vring_call(struct vhost_dev *dev,
95
vmnet_stop_interface(
141
struct vhost_vring_file *file)
96
s->vmnet_if,
142
{
97
diff --git a/net/vmnet_int.h b/net/vmnet_int.h
143
- trace_vhost_vdpa_set_vring_call(dev, file->index, file->fd);
98
index XXXXXXX..XXXXXXX 100644
144
- return vhost_vdpa_call(dev, VHOST_SET_VRING_CALL, file);
99
--- a/net/vmnet_int.h
145
+ struct vhost_vdpa *v = dev->opaque;
100
+++ b/net/vmnet_int.h
101
@@ -XXX,XX +XXX,XX @@ typedef struct VmnetState {
102
int packets_send_end_pos;
103
104
struct iovec iov_buf[VMNET_PACKETS_LIMIT];
105
+
146
+
106
+ VMChangeStateEntry *change;
147
+ if (v->shadow_vqs_enabled) {
107
} VmnetState;
148
+ int vdpa_idx = file->index - dev->vq_index;
108
149
+ VhostShadowVirtqueue *svq = g_ptr_array_index(v->shadow_vqs, vdpa_idx);
109
const char *vmnet_status_map_str(vmnet_return_t status);
150
+
151
+ vhost_svq_set_svq_call_fd(svq, file->fd);
152
+ return 0;
153
+ } else {
154
+ return vhost_vdpa_set_vring_dev_call(dev, file);
155
+ }
156
}
157
158
static int vhost_vdpa_get_features(struct vhost_dev *dev,
110
--
159
--
111
2.7.4
160
2.7.4
161
162
diff view generated by jsdifflib
New patch
1
From: Eugenio Pérez <eperezma@redhat.com>
1
2
3
This allows SVQ to negotiate features with the guest and the device. For
4
the device, SVQ is a driver. While this function bypasses all
5
non-transport features, it needs to disable the features that SVQ does
6
not support when forwarding buffers. This includes packed vq layout,
7
indirect descriptors or event idx.
8
9
Future changes can add support to offer more features to the guest,
10
since the use of VirtQueue gives this for free. This is left out at the
11
moment for simplicity.
12
13
Signed-off-by: Eugenio Pérez <eperezma@redhat.com>
14
Acked-by: Michael S. Tsirkin <mst@redhat.com>
15
Signed-off-by: Jason Wang <jasowang@redhat.com>
16
---
17
hw/virtio/vhost-shadow-virtqueue.c | 44 ++++++++++++++++++++++++++++++++++++++
18
hw/virtio/vhost-shadow-virtqueue.h | 2 ++
19
hw/virtio/vhost-vdpa.c | 15 +++++++++++++
20
3 files changed, 61 insertions(+)
21
22
diff --git a/hw/virtio/vhost-shadow-virtqueue.c b/hw/virtio/vhost-shadow-virtqueue.c
23
index XXXXXXX..XXXXXXX 100644
24
--- a/hw/virtio/vhost-shadow-virtqueue.c
25
+++ b/hw/virtio/vhost-shadow-virtqueue.c
26
@@ -XXX,XX +XXX,XX @@
27
#include "hw/virtio/vhost-shadow-virtqueue.h"
28
29
#include "qemu/error-report.h"
30
+#include "qapi/error.h"
31
#include "qemu/main-loop.h"
32
#include "linux-headers/linux/vhost.h"
33
34
/**
35
+ * Validate the transport device features that both guests can use with the SVQ
36
+ * and SVQs can use with the device.
37
+ *
38
+ * @dev_features: The features
39
+ * @errp: Error pointer
40
+ */
41
+bool vhost_svq_valid_features(uint64_t features, Error **errp)
42
+{
43
+ bool ok = true;
44
+ uint64_t svq_features = features;
45
+
46
+ for (uint64_t b = VIRTIO_TRANSPORT_F_START; b <= VIRTIO_TRANSPORT_F_END;
47
+ ++b) {
48
+ switch (b) {
49
+ case VIRTIO_F_ANY_LAYOUT:
50
+ continue;
51
+
52
+ case VIRTIO_F_ACCESS_PLATFORM:
53
+ /* SVQ trust in the host's IOMMU to translate addresses */
54
+ case VIRTIO_F_VERSION_1:
55
+ /* SVQ trust that the guest vring is little endian */
56
+ if (!(svq_features & BIT_ULL(b))) {
57
+ svq_features |= BIT_ULL(b);
58
+ ok = false;
59
+ }
60
+ continue;
61
+
62
+ default:
63
+ if (svq_features & BIT_ULL(b)) {
64
+ svq_features &= ~BIT_ULL(b);
65
+ ok = false;
66
+ }
67
+ }
68
+ }
69
+
70
+ if (!ok) {
71
+ error_setg(errp, "SVQ Invalid device feature flags, offer: 0x%"PRIx64
72
+ ", ok: 0x%"PRIx64, features, svq_features);
73
+ }
74
+ return ok;
75
+}
76
+
77
+/**
78
* Forward guest notifications.
79
*
80
* @n: guest kick event notifier, the one that guest set to notify svq.
81
diff --git a/hw/virtio/vhost-shadow-virtqueue.h b/hw/virtio/vhost-shadow-virtqueue.h
82
index XXXXXXX..XXXXXXX 100644
83
--- a/hw/virtio/vhost-shadow-virtqueue.h
84
+++ b/hw/virtio/vhost-shadow-virtqueue.h
85
@@ -XXX,XX +XXX,XX @@ typedef struct VhostShadowVirtqueue {
86
EventNotifier svq_call;
87
} VhostShadowVirtqueue;
88
89
+bool vhost_svq_valid_features(uint64_t features, Error **errp);
90
+
91
void vhost_svq_set_svq_kick_fd(VhostShadowVirtqueue *svq, int svq_kick_fd);
92
void vhost_svq_set_svq_call_fd(VhostShadowVirtqueue *svq, int call_fd);
93
94
diff --git a/hw/virtio/vhost-vdpa.c b/hw/virtio/vhost-vdpa.c
95
index XXXXXXX..XXXXXXX 100644
96
--- a/hw/virtio/vhost-vdpa.c
97
+++ b/hw/virtio/vhost-vdpa.c
98
@@ -XXX,XX +XXX,XX @@ static int vhost_vdpa_init_svq(struct vhost_dev *hdev, struct vhost_vdpa *v,
99
Error **errp)
100
{
101
g_autoptr(GPtrArray) shadow_vqs = NULL;
102
+ uint64_t dev_features, svq_features;
103
+ int r;
104
+ bool ok;
105
106
if (!v->shadow_vqs_enabled) {
107
return 0;
108
}
109
110
+ r = hdev->vhost_ops->vhost_get_features(hdev, &dev_features);
111
+ if (r != 0) {
112
+ error_setg_errno(errp, -r, "Can't get vdpa device features");
113
+ return r;
114
+ }
115
+
116
+ svq_features = dev_features;
117
+ ok = vhost_svq_valid_features(svq_features, errp);
118
+ if (unlikely(!ok)) {
119
+ return -1;
120
+ }
121
+
122
shadow_vqs = g_ptr_array_new_full(hdev->nvqs, vhost_svq_free);
123
for (unsigned n = 0; n < hdev->nvqs; ++n) {
124
g_autoptr(VhostShadowVirtqueue) svq = vhost_svq_new();
125
--
126
2.7.4
127
128
diff view generated by jsdifflib
New patch
1
From: Eugenio Pérez <eperezma@redhat.com>
1
2
3
It reports the shadow virtqueue address from qemu virtual address space.
4
5
Since this will be different from the guest's vaddr, but the device can
6
access it, SVQ takes special care about its alignment & lack of garbage
7
data. It assumes that IOMMU will work in host_page_size ranges for that.
8
9
Signed-off-by: Eugenio Pérez <eperezma@redhat.com>
10
Acked-by: Michael S. Tsirkin <mst@redhat.com>
11
Signed-off-by: Jason Wang <jasowang@redhat.com>
12
---
13
hw/virtio/vhost-shadow-virtqueue.c | 29 +++++++++++++++++++++++++++++
14
hw/virtio/vhost-shadow-virtqueue.h | 9 +++++++++
15
2 files changed, 38 insertions(+)
16
17
diff --git a/hw/virtio/vhost-shadow-virtqueue.c b/hw/virtio/vhost-shadow-virtqueue.c
18
index XXXXXXX..XXXXXXX 100644
19
--- a/hw/virtio/vhost-shadow-virtqueue.c
20
+++ b/hw/virtio/vhost-shadow-virtqueue.c
21
@@ -XXX,XX +XXX,XX @@ void vhost_svq_set_svq_call_fd(VhostShadowVirtqueue *svq, int call_fd)
22
}
23
24
/**
25
+ * Get the shadow vq vring address.
26
+ * @svq: Shadow virtqueue
27
+ * @addr: Destination to store address
28
+ */
29
+void vhost_svq_get_vring_addr(const VhostShadowVirtqueue *svq,
30
+ struct vhost_vring_addr *addr)
31
+{
32
+ addr->desc_user_addr = (uint64_t)(intptr_t)svq->vring.desc;
33
+ addr->avail_user_addr = (uint64_t)(intptr_t)svq->vring.avail;
34
+ addr->used_user_addr = (uint64_t)(intptr_t)svq->vring.used;
35
+}
36
+
37
+size_t vhost_svq_driver_area_size(const VhostShadowVirtqueue *svq)
38
+{
39
+ size_t desc_size = sizeof(vring_desc_t) * svq->vring.num;
40
+ size_t avail_size = offsetof(vring_avail_t, ring) +
41
+ sizeof(uint16_t) * svq->vring.num;
42
+
43
+ return ROUND_UP(desc_size + avail_size, qemu_real_host_page_size);
44
+}
45
+
46
+size_t vhost_svq_device_area_size(const VhostShadowVirtqueue *svq)
47
+{
48
+ size_t used_size = offsetof(vring_used_t, ring) +
49
+ sizeof(vring_used_elem_t) * svq->vring.num;
50
+ return ROUND_UP(used_size, qemu_real_host_page_size);
51
+}
52
+
53
+/**
54
* Set a new file descriptor for the guest to kick the SVQ and notify for avail
55
*
56
* @svq: The svq
57
diff --git a/hw/virtio/vhost-shadow-virtqueue.h b/hw/virtio/vhost-shadow-virtqueue.h
58
index XXXXXXX..XXXXXXX 100644
59
--- a/hw/virtio/vhost-shadow-virtqueue.h
60
+++ b/hw/virtio/vhost-shadow-virtqueue.h
61
@@ -XXX,XX +XXX,XX @@
62
#define VHOST_SHADOW_VIRTQUEUE_H
63
64
#include "qemu/event_notifier.h"
65
+#include "hw/virtio/virtio.h"
66
+#include "standard-headers/linux/vhost_types.h"
67
68
/* Shadow virtqueue to relay notifications */
69
typedef struct VhostShadowVirtqueue {
70
+ /* Shadow vring */
71
+ struct vring vring;
72
+
73
/* Shadow kick notifier, sent to vhost */
74
EventNotifier hdev_kick;
75
/* Shadow call notifier, sent to vhost */
76
@@ -XXX,XX +XXX,XX @@ bool vhost_svq_valid_features(uint64_t features, Error **errp);
77
78
void vhost_svq_set_svq_kick_fd(VhostShadowVirtqueue *svq, int svq_kick_fd);
79
void vhost_svq_set_svq_call_fd(VhostShadowVirtqueue *svq, int call_fd);
80
+void vhost_svq_get_vring_addr(const VhostShadowVirtqueue *svq,
81
+ struct vhost_vring_addr *addr);
82
+size_t vhost_svq_driver_area_size(const VhostShadowVirtqueue *svq);
83
+size_t vhost_svq_device_area_size(const VhostShadowVirtqueue *svq);
84
85
void vhost_svq_stop(VhostShadowVirtqueue *svq);
86
87
--
88
2.7.4
89
90
diff view generated by jsdifflib
New patch
1
From: Eugenio Pérez <eperezma@redhat.com>
1
2
3
First half of the buffers forwarding part, preparing vhost-vdpa
4
callbacks to SVQ to offer it. QEMU cannot enable it at this moment, so
5
this is effectively dead code at the moment, but it helps to reduce
6
patch size.
7
8
Signed-off-by: Eugenio Pérez <eperezma@redhat.com>
9
Acked-by: Michael S. Tsirkin <mst@redhat.com>
10
Signed-off-by: Jason Wang <jasowang@redhat.com>
11
---
12
hw/virtio/vhost-vdpa.c | 48 +++++++++++++++++++++++++++++++++++++++++-------
13
1 file changed, 41 insertions(+), 7 deletions(-)
14
15
diff --git a/hw/virtio/vhost-vdpa.c b/hw/virtio/vhost-vdpa.c
16
index XXXXXXX..XXXXXXX 100644
17
--- a/hw/virtio/vhost-vdpa.c
18
+++ b/hw/virtio/vhost-vdpa.c
19
@@ -XXX,XX +XXX,XX @@ static int vhost_vdpa_get_config(struct vhost_dev *dev, uint8_t *config,
20
return ret;
21
}
22
23
+static int vhost_vdpa_set_dev_vring_base(struct vhost_dev *dev,
24
+ struct vhost_vring_state *ring)
25
+{
26
+ trace_vhost_vdpa_set_vring_base(dev, ring->index, ring->num);
27
+ return vhost_vdpa_call(dev, VHOST_SET_VRING_BASE, ring);
28
+}
29
+
30
static int vhost_vdpa_set_vring_dev_kick(struct vhost_dev *dev,
31
struct vhost_vring_file *file)
32
{
33
@@ -XXX,XX +XXX,XX @@ static int vhost_vdpa_set_vring_dev_call(struct vhost_dev *dev,
34
return vhost_vdpa_call(dev, VHOST_SET_VRING_CALL, file);
35
}
36
37
+static int vhost_vdpa_set_vring_dev_addr(struct vhost_dev *dev,
38
+ struct vhost_vring_addr *addr)
39
+{
40
+ trace_vhost_vdpa_set_vring_addr(dev, addr->index, addr->flags,
41
+ addr->desc_user_addr, addr->used_user_addr,
42
+ addr->avail_user_addr,
43
+ addr->log_guest_addr);
44
+
45
+ return vhost_vdpa_call(dev, VHOST_SET_VRING_ADDR, addr);
46
+
47
+}
48
+
49
/**
50
* Set the shadow virtqueue descriptors to the device
51
*
52
@@ -XXX,XX +XXX,XX @@ static int vhost_vdpa_set_log_base(struct vhost_dev *dev, uint64_t base,
53
static int vhost_vdpa_set_vring_addr(struct vhost_dev *dev,
54
struct vhost_vring_addr *addr)
55
{
56
- trace_vhost_vdpa_set_vring_addr(dev, addr->index, addr->flags,
57
- addr->desc_user_addr, addr->used_user_addr,
58
- addr->avail_user_addr,
59
- addr->log_guest_addr);
60
- return vhost_vdpa_call(dev, VHOST_SET_VRING_ADDR, addr);
61
+ struct vhost_vdpa *v = dev->opaque;
62
+
63
+ if (v->shadow_vqs_enabled) {
64
+ /*
65
+ * Device vring addr was set at device start. SVQ base is handled by
66
+ * VirtQueue code.
67
+ */
68
+ return 0;
69
+ }
70
+
71
+ return vhost_vdpa_set_vring_dev_addr(dev, addr);
72
}
73
74
static int vhost_vdpa_set_vring_num(struct vhost_dev *dev,
75
@@ -XXX,XX +XXX,XX @@ static int vhost_vdpa_set_vring_num(struct vhost_dev *dev,
76
static int vhost_vdpa_set_vring_base(struct vhost_dev *dev,
77
struct vhost_vring_state *ring)
78
{
79
- trace_vhost_vdpa_set_vring_base(dev, ring->index, ring->num);
80
- return vhost_vdpa_call(dev, VHOST_SET_VRING_BASE, ring);
81
+ struct vhost_vdpa *v = dev->opaque;
82
+
83
+ if (v->shadow_vqs_enabled) {
84
+ /*
85
+ * Device vring base was set at device start. SVQ base is handled by
86
+ * VirtQueue code.
87
+ */
88
+ return 0;
89
+ }
90
+
91
+ return vhost_vdpa_set_dev_vring_base(dev, ring);
92
}
93
94
static int vhost_vdpa_get_vring_base(struct vhost_dev *dev,
95
--
96
2.7.4
97
98
diff view generated by jsdifflib
1
From: Qiang Liu <cyruscyliu@gmail.com>
1
From: Eugenio Pérez <eperezma@redhat.com>
2
2
3
This patch replaces hw_error to guest error log for [read|write]b
3
Initial version of shadow virtqueue that actually forward buffers. There
4
accesses when mode_16bit is enabled. This avoids aborting qemu.
4
is no iommu support at the moment, and that will be addressed in future
5
5
patches of this series. Since all vhost-vdpa devices use forced IOMMU,
6
Fixes: 1248f8d4cbc3 ("hw/lan9118: Add basic 16-bit mode support.")
6
this means that SVQ is not usable at this point of the series on any
7
Resolves: https://gitlab.com/qemu-project/qemu/-/issues/1433
7
device.
8
Reported-by: Qiang Liu <cyruscyliu@gmail.com>
8
9
Reviewed-by: Philippe Mathieu-Daudé <philmd@linaro.org>
9
For simplicity it only supports modern devices, that expects vring
10
Signed-off-by: Qiang Liu <cyruscyliu@gmail.com>
10
in little endian, with split ring and no event idx or indirect
11
Suggested-by: Philippe Mathieu-Daudé <philmd@linaro.org>
11
descriptors. Support for them will not be added in this series.
12
13
It reuses the VirtQueue code for the device part. The driver part is
14
based on Linux's virtio_ring driver, but with stripped functionality
15
and optimizations so it's easier to review.
16
17
However, forwarding buffers have some particular pieces: One of the most
18
unexpected ones is that a guest's buffer can expand through more than
19
one descriptor in SVQ. While this is handled gracefully by qemu's
20
emulated virtio devices, it may cause unexpected SVQ queue full. This
21
patch also solves it by checking for this condition at both guest's
22
kicks and device's calls. The code may be more elegant in the future if
23
SVQ code runs in its own iocontext.
24
25
Signed-off-by: Eugenio Pérez <eperezma@redhat.com>
26
Acked-by: Michael S. Tsirkin <mst@redhat.com>
12
Signed-off-by: Jason Wang <jasowang@redhat.com>
27
Signed-off-by: Jason Wang <jasowang@redhat.com>
13
---
28
---
14
hw/net/lan9118.c | 17 ++++++++---------
29
hw/virtio/vhost-shadow-virtqueue.c | 352 ++++++++++++++++++++++++++++++++++++-
15
1 file changed, 8 insertions(+), 9 deletions(-)
30
hw/virtio/vhost-shadow-virtqueue.h | 26 +++
16
31
hw/virtio/vhost-vdpa.c | 155 +++++++++++++++-
17
diff --git a/hw/net/lan9118.c b/hw/net/lan9118.c
32
3 files changed, 522 insertions(+), 11 deletions(-)
33
34
diff --git a/hw/virtio/vhost-shadow-virtqueue.c b/hw/virtio/vhost-shadow-virtqueue.c
18
index XXXXXXX..XXXXXXX 100644
35
index XXXXXXX..XXXXXXX 100644
19
--- a/hw/net/lan9118.c
36
--- a/hw/virtio/vhost-shadow-virtqueue.c
20
+++ b/hw/net/lan9118.c
37
+++ b/hw/virtio/vhost-shadow-virtqueue.c
21
@@ -XXX,XX +XXX,XX @@
38
@@ -XXX,XX +XXX,XX @@
22
#include "migration/vmstate.h"
39
#include "qemu/error-report.h"
23
#include "net/net.h"
40
#include "qapi/error.h"
24
#include "net/eth.h"
41
#include "qemu/main-loop.h"
25
-#include "hw/hw.h"
42
+#include "qemu/log.h"
26
#include "hw/irq.h"
43
+#include "qemu/memalign.h"
27
#include "hw/net/lan9118.h"
44
#include "linux-headers/linux/vhost.h"
28
#include "hw/ptimer.h"
45
29
@@ -XXX,XX +XXX,XX @@
46
/**
30
#ifdef DEBUG_LAN9118
47
@@ -XXX,XX +XXX,XX @@ bool vhost_svq_valid_features(uint64_t features, Error **errp)
31
#define DPRINTF(fmt, ...) \
48
}
32
do { printf("lan9118: " fmt , ## __VA_ARGS__); } while (0)
49
33
-#define BADF(fmt, ...) \
50
/**
34
-do { hw_error("lan9118: error: " fmt , ## __VA_ARGS__);} while (0)
51
- * Forward guest notifications.
35
#else
52
+ * Number of descriptors that the SVQ can make available from the guest.
36
#define DPRINTF(fmt, ...) do {} while(0)
53
+ *
37
-#define BADF(fmt, ...) \
54
+ * @svq: The svq
38
-do { fprintf(stderr, "lan9118: error: " fmt , ## __VA_ARGS__);} while (0)
55
+ */
39
#endif
56
+static uint16_t vhost_svq_available_slots(const VhostShadowVirtqueue *svq)
40
57
+{
41
/* The tx and rx fifo ports are a range of aliased 32-bit registers */
58
+ return svq->vring.num - (svq->shadow_avail_idx - svq->shadow_used_idx);
42
@@ -XXX,XX +XXX,XX @@ static uint32_t do_phy_read(lan9118_state *s, int reg)
59
+}
43
case 30: /* Interrupt mask */
60
+
44
return s->phy_int_mask;
61
+static void vhost_vring_write_descs(VhostShadowVirtqueue *svq,
45
default:
62
+ const struct iovec *iovec, size_t num,
46
- BADF("PHY read reg %d\n", reg);
63
+ bool more_descs, bool write)
64
+{
65
+ uint16_t i = svq->free_head, last = svq->free_head;
66
+ unsigned n;
67
+ uint16_t flags = write ? cpu_to_le16(VRING_DESC_F_WRITE) : 0;
68
+ vring_desc_t *descs = svq->vring.desc;
69
+
70
+ if (num == 0) {
71
+ return;
72
+ }
73
+
74
+ for (n = 0; n < num; n++) {
75
+ if (more_descs || (n + 1 < num)) {
76
+ descs[i].flags = flags | cpu_to_le16(VRING_DESC_F_NEXT);
77
+ } else {
78
+ descs[i].flags = flags;
79
+ }
80
+ descs[i].addr = cpu_to_le64((hwaddr)(intptr_t)iovec[n].iov_base);
81
+ descs[i].len = cpu_to_le32(iovec[n].iov_len);
82
+
83
+ last = i;
84
+ i = cpu_to_le16(descs[i].next);
85
+ }
86
+
87
+ svq->free_head = le16_to_cpu(descs[last].next);
88
+}
89
+
90
+static bool vhost_svq_add_split(VhostShadowVirtqueue *svq,
91
+ VirtQueueElement *elem, unsigned *head)
92
+{
93
+ unsigned avail_idx;
94
+ vring_avail_t *avail = svq->vring.avail;
95
+
96
+ *head = svq->free_head;
97
+
98
+ /* We need some descriptors here */
99
+ if (unlikely(!elem->out_num && !elem->in_num)) {
47
+ qemu_log_mask(LOG_GUEST_ERROR,
100
+ qemu_log_mask(LOG_GUEST_ERROR,
48
+ "do_phy_read: PHY read reg %d\n", reg);
101
+ "Guest provided element with no descriptors");
49
return 0;
102
+ return false;
103
+ }
104
+
105
+ vhost_vring_write_descs(svq, elem->out_sg, elem->out_num, elem->in_num > 0,
106
+ false);
107
+ vhost_vring_write_descs(svq, elem->in_sg, elem->in_num, false, true);
108
+
109
+ /*
110
+ * Put the entry in the available array (but don't update avail->idx until
111
+ * they do sync).
112
+ */
113
+ avail_idx = svq->shadow_avail_idx & (svq->vring.num - 1);
114
+ avail->ring[avail_idx] = cpu_to_le16(*head);
115
+ svq->shadow_avail_idx++;
116
+
117
+ /* Update the avail index after write the descriptor */
118
+ smp_wmb();
119
+ avail->idx = cpu_to_le16(svq->shadow_avail_idx);
120
+
121
+ return true;
122
+}
123
+
124
+static bool vhost_svq_add(VhostShadowVirtqueue *svq, VirtQueueElement *elem)
125
+{
126
+ unsigned qemu_head;
127
+ bool ok = vhost_svq_add_split(svq, elem, &qemu_head);
128
+ if (unlikely(!ok)) {
129
+ return false;
130
+ }
131
+
132
+ svq->ring_id_maps[qemu_head] = elem;
133
+ return true;
134
+}
135
+
136
+static void vhost_svq_kick(VhostShadowVirtqueue *svq)
137
+{
138
+ /*
139
+ * We need to expose the available array entries before checking the used
140
+ * flags
141
+ */
142
+ smp_mb();
143
+ if (svq->vring.used->flags & VRING_USED_F_NO_NOTIFY) {
144
+ return;
145
+ }
146
+
147
+ event_notifier_set(&svq->hdev_kick);
148
+}
149
+
150
+/**
151
+ * Forward available buffers.
152
+ *
153
+ * @svq: Shadow VirtQueue
154
+ *
155
+ * Note that this function does not guarantee that all guest's available
156
+ * buffers are available to the device in SVQ avail ring. The guest may have
157
+ * exposed a GPA / GIOVA contiguous buffer, but it may not be contiguous in
158
+ * qemu vaddr.
159
+ *
160
+ * If that happens, guest's kick notifications will be disabled until the
161
+ * device uses some buffers.
162
+ */
163
+static void vhost_handle_guest_kick(VhostShadowVirtqueue *svq)
164
+{
165
+ /* Clear event notifier */
166
+ event_notifier_test_and_clear(&svq->svq_kick);
167
+
168
+ /* Forward to the device as many available buffers as possible */
169
+ do {
170
+ virtio_queue_set_notification(svq->vq, false);
171
+
172
+ while (true) {
173
+ VirtQueueElement *elem;
174
+ bool ok;
175
+
176
+ if (svq->next_guest_avail_elem) {
177
+ elem = g_steal_pointer(&svq->next_guest_avail_elem);
178
+ } else {
179
+ elem = virtqueue_pop(svq->vq, sizeof(*elem));
180
+ }
181
+
182
+ if (!elem) {
183
+ break;
184
+ }
185
+
186
+ if (elem->out_num + elem->in_num > vhost_svq_available_slots(svq)) {
187
+ /*
188
+ * This condition is possible since a contiguous buffer in GPA
189
+ * does not imply a contiguous buffer in qemu's VA
190
+ * scatter-gather segments. If that happens, the buffer exposed
191
+ * to the device needs to be a chain of descriptors at this
192
+ * moment.
193
+ *
194
+ * SVQ cannot hold more available buffers if we are here:
195
+ * queue the current guest descriptor and ignore further kicks
196
+ * until some elements are used.
197
+ */
198
+ svq->next_guest_avail_elem = elem;
199
+ return;
200
+ }
201
+
202
+ ok = vhost_svq_add(svq, elem);
203
+ if (unlikely(!ok)) {
204
+ /* VQ is broken, just return and ignore any other kicks */
205
+ return;
206
+ }
207
+ vhost_svq_kick(svq);
208
+ }
209
+
210
+ virtio_queue_set_notification(svq->vq, true);
211
+ } while (!virtio_queue_empty(svq->vq));
212
+}
213
+
214
+/**
215
+ * Handle guest's kick.
216
*
217
* @n: guest kick event notifier, the one that guest set to notify svq.
218
*/
219
-static void vhost_handle_guest_kick(EventNotifier *n)
220
+static void vhost_handle_guest_kick_notifier(EventNotifier *n)
221
{
222
VhostShadowVirtqueue *svq = container_of(n, VhostShadowVirtqueue, svq_kick);
223
event_notifier_test_and_clear(n);
224
- event_notifier_set(&svq->hdev_kick);
225
+ vhost_handle_guest_kick(svq);
226
+}
227
+
228
+static bool vhost_svq_more_used(VhostShadowVirtqueue *svq)
229
+{
230
+ if (svq->last_used_idx != svq->shadow_used_idx) {
231
+ return true;
232
+ }
233
+
234
+ svq->shadow_used_idx = cpu_to_le16(svq->vring.used->idx);
235
+
236
+ return svq->last_used_idx != svq->shadow_used_idx;
237
}
238
239
/**
240
- * Forward vhost notifications
241
+ * Enable vhost device calls after disable them.
242
+ *
243
+ * @svq: The svq
244
+ *
245
+ * It returns false if there are pending used buffers from the vhost device,
246
+ * avoiding the possible races between SVQ checking for more work and enabling
247
+ * callbacks. True if SVQ used vring has no more pending buffers.
248
+ */
249
+static bool vhost_svq_enable_notification(VhostShadowVirtqueue *svq)
250
+{
251
+ svq->vring.avail->flags &= ~cpu_to_le16(VRING_AVAIL_F_NO_INTERRUPT);
252
+ /* Make sure the flag is written before the read of used_idx */
253
+ smp_mb();
254
+ return !vhost_svq_more_used(svq);
255
+}
256
+
257
+static void vhost_svq_disable_notification(VhostShadowVirtqueue *svq)
258
+{
259
+ svq->vring.avail->flags |= cpu_to_le16(VRING_AVAIL_F_NO_INTERRUPT);
260
+}
261
+
262
+static VirtQueueElement *vhost_svq_get_buf(VhostShadowVirtqueue *svq,
263
+ uint32_t *len)
264
+{
265
+ vring_desc_t *descs = svq->vring.desc;
266
+ const vring_used_t *used = svq->vring.used;
267
+ vring_used_elem_t used_elem;
268
+ uint16_t last_used;
269
+
270
+ if (!vhost_svq_more_used(svq)) {
271
+ return NULL;
272
+ }
273
+
274
+ /* Only get used array entries after they have been exposed by dev */
275
+ smp_rmb();
276
+ last_used = svq->last_used_idx & (svq->vring.num - 1);
277
+ used_elem.id = le32_to_cpu(used->ring[last_used].id);
278
+ used_elem.len = le32_to_cpu(used->ring[last_used].len);
279
+
280
+ svq->last_used_idx++;
281
+ if (unlikely(used_elem.id >= svq->vring.num)) {
282
+ qemu_log_mask(LOG_GUEST_ERROR, "Device %s says index %u is used",
283
+ svq->vdev->name, used_elem.id);
284
+ return NULL;
285
+ }
286
+
287
+ if (unlikely(!svq->ring_id_maps[used_elem.id])) {
288
+ qemu_log_mask(LOG_GUEST_ERROR,
289
+ "Device %s says index %u is used, but it was not available",
290
+ svq->vdev->name, used_elem.id);
291
+ return NULL;
292
+ }
293
+
294
+ descs[used_elem.id].next = svq->free_head;
295
+ svq->free_head = used_elem.id;
296
+
297
+ *len = used_elem.len;
298
+ return g_steal_pointer(&svq->ring_id_maps[used_elem.id]);
299
+}
300
+
301
+static void vhost_svq_flush(VhostShadowVirtqueue *svq,
302
+ bool check_for_avail_queue)
303
+{
304
+ VirtQueue *vq = svq->vq;
305
+
306
+ /* Forward as many used buffers as possible. */
307
+ do {
308
+ unsigned i = 0;
309
+
310
+ vhost_svq_disable_notification(svq);
311
+ while (true) {
312
+ uint32_t len;
313
+ g_autofree VirtQueueElement *elem = vhost_svq_get_buf(svq, &len);
314
+ if (!elem) {
315
+ break;
316
+ }
317
+
318
+ if (unlikely(i >= svq->vring.num)) {
319
+ qemu_log_mask(LOG_GUEST_ERROR,
320
+ "More than %u used buffers obtained in a %u size SVQ",
321
+ i, svq->vring.num);
322
+ virtqueue_fill(vq, elem, len, i);
323
+ virtqueue_flush(vq, i);
324
+ return;
325
+ }
326
+ virtqueue_fill(vq, elem, len, i++);
327
+ }
328
+
329
+ virtqueue_flush(vq, i);
330
+ event_notifier_set(&svq->svq_call);
331
+
332
+ if (check_for_avail_queue && svq->next_guest_avail_elem) {
333
+ /*
334
+ * Avail ring was full when vhost_svq_flush was called, so it's a
335
+ * good moment to make more descriptors available if possible.
336
+ */
337
+ vhost_handle_guest_kick(svq);
338
+ }
339
+ } while (!vhost_svq_enable_notification(svq));
340
+}
341
+
342
+/**
343
+ * Forward used buffers.
344
*
345
* @n: hdev call event notifier, the one that device set to notify svq.
346
+ *
347
+ * Note that we are not making any buffers available in the loop, there is no
348
+ * way that it runs more than virtqueue size times.
349
*/
350
static void vhost_svq_handle_call(EventNotifier *n)
351
{
352
VhostShadowVirtqueue *svq = container_of(n, VhostShadowVirtqueue,
353
hdev_call);
354
event_notifier_test_and_clear(n);
355
- event_notifier_set(&svq->svq_call);
356
+ vhost_svq_flush(svq, true);
357
}
358
359
/**
360
@@ -XXX,XX +XXX,XX @@ void vhost_svq_set_svq_kick_fd(VhostShadowVirtqueue *svq, int svq_kick_fd)
361
if (poll_start) {
362
event_notifier_init_fd(svq_kick, svq_kick_fd);
363
event_notifier_set(svq_kick);
364
- event_notifier_set_handler(svq_kick, vhost_handle_guest_kick);
365
+ event_notifier_set_handler(svq_kick, vhost_handle_guest_kick_notifier);
366
+ }
367
+}
368
+
369
+/**
370
+ * Start the shadow virtqueue operation.
371
+ *
372
+ * @svq: Shadow Virtqueue
373
+ * @vdev: VirtIO device
374
+ * @vq: Virtqueue to shadow
375
+ */
376
+void vhost_svq_start(VhostShadowVirtqueue *svq, VirtIODevice *vdev,
377
+ VirtQueue *vq)
378
+{
379
+ size_t desc_size, driver_size, device_size;
380
+
381
+ svq->next_guest_avail_elem = NULL;
382
+ svq->shadow_avail_idx = 0;
383
+ svq->shadow_used_idx = 0;
384
+ svq->last_used_idx = 0;
385
+ svq->vdev = vdev;
386
+ svq->vq = vq;
387
+
388
+ svq->vring.num = virtio_queue_get_num(vdev, virtio_get_queue_index(vq));
389
+ driver_size = vhost_svq_driver_area_size(svq);
390
+ device_size = vhost_svq_device_area_size(svq);
391
+ svq->vring.desc = qemu_memalign(qemu_real_host_page_size, driver_size);
392
+ desc_size = sizeof(vring_desc_t) * svq->vring.num;
393
+ svq->vring.avail = (void *)((char *)svq->vring.desc + desc_size);
394
+ memset(svq->vring.desc, 0, driver_size);
395
+ svq->vring.used = qemu_memalign(qemu_real_host_page_size, device_size);
396
+ memset(svq->vring.used, 0, device_size);
397
+ svq->ring_id_maps = g_new0(VirtQueueElement *, svq->vring.num);
398
+ for (unsigned i = 0; i < svq->vring.num - 1; i++) {
399
+ svq->vring.desc[i].next = cpu_to_le16(i + 1);
50
}
400
}
51
}
401
}
52
@@ -XXX,XX +XXX,XX @@ static void do_phy_write(lan9118_state *s, int reg, uint32_t val)
402
53
phy_update_irq(s);
403
@@ -XXX,XX +XXX,XX @@ void vhost_svq_set_svq_kick_fd(VhostShadowVirtqueue *svq, int svq_kick_fd)
54
break;
404
void vhost_svq_stop(VhostShadowVirtqueue *svq)
55
default:
405
{
56
- BADF("PHY write reg %d = 0x%04x\n", reg, val);
406
event_notifier_set_handler(&svq->svq_kick, NULL);
57
+ qemu_log_mask(LOG_GUEST_ERROR,
407
+ g_autofree VirtQueueElement *next_avail_elem = NULL;
58
+ "do_phy_write: PHY write reg %d = 0x%04x\n", reg, val);
408
+
409
+ if (!svq->vq) {
410
+ return;
411
+ }
412
+
413
+ /* Send all pending used descriptors to guest */
414
+ vhost_svq_flush(svq, false);
415
+
416
+ for (unsigned i = 0; i < svq->vring.num; ++i) {
417
+ g_autofree VirtQueueElement *elem = NULL;
418
+ elem = g_steal_pointer(&svq->ring_id_maps[i]);
419
+ if (elem) {
420
+ virtqueue_detach_element(svq->vq, elem, 0);
421
+ }
422
+ }
423
+
424
+ next_avail_elem = g_steal_pointer(&svq->next_guest_avail_elem);
425
+ if (next_avail_elem) {
426
+ virtqueue_detach_element(svq->vq, next_avail_elem, 0);
427
+ }
428
+ svq->vq = NULL;
429
+ g_free(svq->ring_id_maps);
430
+ qemu_vfree(svq->vring.desc);
431
+ qemu_vfree(svq->vring.used);
432
}
433
434
/**
435
diff --git a/hw/virtio/vhost-shadow-virtqueue.h b/hw/virtio/vhost-shadow-virtqueue.h
436
index XXXXXXX..XXXXXXX 100644
437
--- a/hw/virtio/vhost-shadow-virtqueue.h
438
+++ b/hw/virtio/vhost-shadow-virtqueue.h
439
@@ -XXX,XX +XXX,XX @@ typedef struct VhostShadowVirtqueue {
440
441
/* Guest's call notifier, where the SVQ calls guest. */
442
EventNotifier svq_call;
443
+
444
+ /* Virtio queue shadowing */
445
+ VirtQueue *vq;
446
+
447
+ /* Virtio device */
448
+ VirtIODevice *vdev;
449
+
450
+ /* Map for use the guest's descriptors */
451
+ VirtQueueElement **ring_id_maps;
452
+
453
+ /* Next VirtQueue element that guest made available */
454
+ VirtQueueElement *next_guest_avail_elem;
455
+
456
+ /* Next head to expose to the device */
457
+ uint16_t shadow_avail_idx;
458
+
459
+ /* Next free descriptor */
460
+ uint16_t free_head;
461
+
462
+ /* Last seen used idx */
463
+ uint16_t shadow_used_idx;
464
+
465
+ /* Next head to consume from the device */
466
+ uint16_t last_used_idx;
467
} VhostShadowVirtqueue;
468
469
bool vhost_svq_valid_features(uint64_t features, Error **errp);
470
@@ -XXX,XX +XXX,XX @@ void vhost_svq_get_vring_addr(const VhostShadowVirtqueue *svq,
471
size_t vhost_svq_driver_area_size(const VhostShadowVirtqueue *svq);
472
size_t vhost_svq_device_area_size(const VhostShadowVirtqueue *svq);
473
474
+void vhost_svq_start(VhostShadowVirtqueue *svq, VirtIODevice *vdev,
475
+ VirtQueue *vq);
476
void vhost_svq_stop(VhostShadowVirtqueue *svq);
477
478
VhostShadowVirtqueue *vhost_svq_new(void);
479
diff --git a/hw/virtio/vhost-vdpa.c b/hw/virtio/vhost-vdpa.c
480
index XXXXXXX..XXXXXXX 100644
481
--- a/hw/virtio/vhost-vdpa.c
482
+++ b/hw/virtio/vhost-vdpa.c
483
@@ -XXX,XX +XXX,XX @@ static int vhost_vdpa_set_vring_dev_addr(struct vhost_dev *dev,
484
* Note that this function does not rewind kick file descriptor if cannot set
485
* call one.
486
*/
487
-static bool vhost_vdpa_svq_setup(struct vhost_dev *dev,
488
- VhostShadowVirtqueue *svq, unsigned idx,
489
- Error **errp)
490
+static int vhost_vdpa_svq_set_fds(struct vhost_dev *dev,
491
+ VhostShadowVirtqueue *svq, unsigned idx,
492
+ Error **errp)
493
{
494
struct vhost_vring_file file = {
495
.index = dev->vq_index + idx,
496
@@ -XXX,XX +XXX,XX @@ static bool vhost_vdpa_svq_setup(struct vhost_dev *dev,
497
r = vhost_vdpa_set_vring_dev_kick(dev, &file);
498
if (unlikely(r != 0)) {
499
error_setg_errno(errp, -r, "Can't set device kick fd");
500
- return false;
501
+ return r;
59
}
502
}
503
504
event_notifier = &svq->hdev_call;
505
@@ -XXX,XX +XXX,XX @@ static bool vhost_vdpa_svq_setup(struct vhost_dev *dev,
506
error_setg_errno(errp, -r, "Can't set device call fd");
507
}
508
509
+ return r;
510
+}
511
+
512
+/**
513
+ * Unmap a SVQ area in the device
514
+ */
515
+static bool vhost_vdpa_svq_unmap_ring(struct vhost_vdpa *v, hwaddr iova,
516
+ hwaddr size)
517
+{
518
+ int r;
519
+
520
+ size = ROUND_UP(size, qemu_real_host_page_size);
521
+ r = vhost_vdpa_dma_unmap(v, iova, size);
522
+ return r == 0;
523
+}
524
+
525
+static bool vhost_vdpa_svq_unmap_rings(struct vhost_dev *dev,
526
+ const VhostShadowVirtqueue *svq)
527
+{
528
+ struct vhost_vdpa *v = dev->opaque;
529
+ struct vhost_vring_addr svq_addr;
530
+ size_t device_size = vhost_svq_device_area_size(svq);
531
+ size_t driver_size = vhost_svq_driver_area_size(svq);
532
+ bool ok;
533
+
534
+ vhost_svq_get_vring_addr(svq, &svq_addr);
535
+
536
+ ok = vhost_vdpa_svq_unmap_ring(v, svq_addr.desc_user_addr, driver_size);
537
+ if (unlikely(!ok)) {
538
+ return false;
539
+ }
540
+
541
+ return vhost_vdpa_svq_unmap_ring(v, svq_addr.used_user_addr, device_size);
542
+}
543
+
544
+/**
545
+ * Map the shadow virtqueue rings in the device
546
+ *
547
+ * @dev: The vhost device
548
+ * @svq: The shadow virtqueue
549
+ * @addr: Assigned IOVA addresses
550
+ * @errp: Error pointer
551
+ */
552
+static bool vhost_vdpa_svq_map_rings(struct vhost_dev *dev,
553
+ const VhostShadowVirtqueue *svq,
554
+ struct vhost_vring_addr *addr,
555
+ Error **errp)
556
+{
557
+ struct vhost_vdpa *v = dev->opaque;
558
+ size_t device_size = vhost_svq_device_area_size(svq);
559
+ size_t driver_size = vhost_svq_driver_area_size(svq);
560
+ int r;
561
+
562
+ ERRP_GUARD();
563
+ vhost_svq_get_vring_addr(svq, addr);
564
+
565
+ r = vhost_vdpa_dma_map(v, addr->desc_user_addr, driver_size,
566
+ (void *)(uintptr_t)addr->desc_user_addr, true);
567
+ if (unlikely(r != 0)) {
568
+ error_setg_errno(errp, -r, "Cannot create vq driver region: ");
569
+ return false;
570
+ }
571
+
572
+ r = vhost_vdpa_dma_map(v, addr->used_user_addr, device_size,
573
+ (void *)(intptr_t)addr->used_user_addr, false);
574
+ if (unlikely(r != 0)) {
575
+ error_setg_errno(errp, -r, "Cannot create vq device region: ");
576
+ }
577
+
578
+ return r == 0;
579
+}
580
+
581
+static bool vhost_vdpa_svq_setup(struct vhost_dev *dev,
582
+ VhostShadowVirtqueue *svq, unsigned idx,
583
+ Error **errp)
584
+{
585
+ uint16_t vq_index = dev->vq_index + idx;
586
+ struct vhost_vring_state s = {
587
+ .index = vq_index,
588
+ };
589
+ int r;
590
+
591
+ r = vhost_vdpa_set_dev_vring_base(dev, &s);
592
+ if (unlikely(r)) {
593
+ error_setg_errno(errp, -r, "Cannot set vring base");
594
+ return false;
595
+ }
596
+
597
+ r = vhost_vdpa_svq_set_fds(dev, svq, idx, errp);
598
return r == 0;
60
}
599
}
61
600
62
@@ -XXX,XX +XXX,XX @@ static void lan9118_16bit_mode_write(void *opaque, hwaddr offset,
601
@@ -XXX,XX +XXX,XX @@ static bool vhost_vdpa_svqs_start(struct vhost_dev *dev)
63
return;
64
}
602
}
65
603
66
- hw_error("lan9118_write: Bad size 0x%x\n", size);
604
for (i = 0; i < v->shadow_vqs->len; ++i) {
67
+ qemu_log_mask(LOG_GUEST_ERROR,
605
+ VirtQueue *vq = virtio_get_queue(dev->vdev, dev->vq_index + i);
68
+ "lan9118_16bit_mode_write: Bad size 0x%x\n", size);
606
VhostShadowVirtqueue *svq = g_ptr_array_index(v->shadow_vqs, i);
69
}
607
+ struct vhost_vring_addr addr = {
70
608
+ .index = i,
71
static uint64_t lan9118_readl(void *opaque, hwaddr offset,
609
+ };
72
@@ -XXX,XX +XXX,XX @@ static uint64_t lan9118_16bit_mode_read(void *opaque, hwaddr offset,
610
+ int r;
73
return lan9118_readl(opaque, offset, size);
611
bool ok = vhost_vdpa_svq_setup(dev, svq, i, &err);
612
if (unlikely(!ok)) {
613
- error_reportf_err(err, "Cannot setup SVQ %u: ", i);
614
+ goto err;
615
+ }
616
+
617
+ vhost_svq_start(svq, dev->vdev, vq);
618
+ ok = vhost_vdpa_svq_map_rings(dev, svq, &addr, &err);
619
+ if (unlikely(!ok)) {
620
+ goto err_map;
621
+ }
622
+
623
+ /* Override vring GPA set by vhost subsystem */
624
+ r = vhost_vdpa_set_vring_dev_addr(dev, &addr);
625
+ if (unlikely(r != 0)) {
626
+ error_setg_errno(&err, -r, "Cannot set device address");
627
+ goto err_set_addr;
628
+ }
629
+ }
630
+
631
+ return true;
632
+
633
+err_set_addr:
634
+ vhost_vdpa_svq_unmap_rings(dev, g_ptr_array_index(v->shadow_vqs, i));
635
+
636
+err_map:
637
+ vhost_svq_stop(g_ptr_array_index(v->shadow_vqs, i));
638
+
639
+err:
640
+ error_reportf_err(err, "Cannot setup SVQ %u: ", i);
641
+ for (unsigned j = 0; j < i; ++j) {
642
+ VhostShadowVirtqueue *svq = g_ptr_array_index(v->shadow_vqs, j);
643
+ vhost_vdpa_svq_unmap_rings(dev, svq);
644
+ vhost_svq_stop(svq);
645
+ }
646
+
647
+ return false;
648
+}
649
+
650
+static bool vhost_vdpa_svqs_stop(struct vhost_dev *dev)
651
+{
652
+ struct vhost_vdpa *v = dev->opaque;
653
+
654
+ if (!v->shadow_vqs) {
655
+ return true;
656
+ }
657
+
658
+ for (unsigned i = 0; i < v->shadow_vqs->len; ++i) {
659
+ VhostShadowVirtqueue *svq = g_ptr_array_index(v->shadow_vqs, i);
660
+ bool ok = vhost_vdpa_svq_unmap_rings(dev, svq);
661
+ if (unlikely(!ok)) {
662
return false;
663
}
74
}
664
}
75
665
@@ -XXX,XX +XXX,XX @@ static int vhost_vdpa_dev_start(struct vhost_dev *dev, bool started)
76
- hw_error("lan9118_read: Bad size 0x%x\n", size);
666
}
77
+ qemu_log_mask(LOG_GUEST_ERROR,
667
vhost_vdpa_set_vring_ready(dev);
78
+ "lan9118_16bit_mode_read: Bad size 0x%x\n", size);
668
} else {
79
return 0;
669
+ ok = vhost_vdpa_svqs_stop(dev);
80
}
670
+ if (unlikely(!ok)) {
671
+ return -1;
672
+ }
673
vhost_vdpa_host_notifiers_uninit(dev, dev->nvqs);
674
}
81
675
82
--
676
--
83
2.7.4
677
2.7.4
84
678
85
679
diff view generated by jsdifflib
1
From: Christian Svensson <blue@cmd.nu>
1
From: Eugenio Pérez <eperezma@redhat.com>
2
2
3
Increase the allocated buffer size to fit larger packets.
3
This iova tree function allows it to look for a hole in allocated
4
Given that jumboframes can commonly be up to 9000 bytes the closest suitable
4
regions and return a totally new translation for a given translated
5
value seems to be 16 KiB.
5
address.
6
6
7
Tested by running qemu towards a Linux L2TPv3 endpoint and pushing
7
It's usage is mainly to allow devices to access qemu address space,
8
jumboframe traffic through the interfaces.
8
remapping guest's one into a new iova space where qemu can add chunks of
9
9
addresses.
10
Signed-off-by: Christian Svensson <blue@cmd.nu>
10
11
Signed-off-by: Eugenio Pérez <eperezma@redhat.com>
12
Reviewed-by: Peter Xu <peterx@redhat.com>
13
Acked-by: Michael S. Tsirkin <mst@redhat.com>
11
Signed-off-by: Jason Wang <jasowang@redhat.com>
14
Signed-off-by: Jason Wang <jasowang@redhat.com>
12
---
15
---
13
net/l2tpv3.c | 2 +-
16
include/qemu/iova-tree.h | 18 +++++++
14
1 file changed, 1 insertion(+), 1 deletion(-)
17
util/iova-tree.c | 136 +++++++++++++++++++++++++++++++++++++++++++++++
15
18
2 files changed, 154 insertions(+)
16
diff --git a/net/l2tpv3.c b/net/l2tpv3.c
19
20
diff --git a/include/qemu/iova-tree.h b/include/qemu/iova-tree.h
17
index XXXXXXX..XXXXXXX 100644
21
index XXXXXXX..XXXXXXX 100644
18
--- a/net/l2tpv3.c
22
--- a/include/qemu/iova-tree.h
19
+++ b/net/l2tpv3.c
23
+++ b/include/qemu/iova-tree.h
20
@@ -XXX,XX +XXX,XX @@
24
@@ -XXX,XX +XXX,XX @@
21
*/
25
#define IOVA_OK (0)
22
26
#define IOVA_ERR_INVALID (-1) /* Invalid parameters */
23
#define BUFFER_ALIGN sysconf(_SC_PAGESIZE)
27
#define IOVA_ERR_OVERLAP (-2) /* IOVA range overlapped */
24
-#define BUFFER_SIZE 2048
28
+#define IOVA_ERR_NOMEM (-3) /* Cannot allocate */
25
+#define BUFFER_SIZE 16384
29
26
#define IOVSIZE 2
30
typedef struct IOVATree IOVATree;
27
#define MAX_L2TPV3_MSGCNT 64
31
typedef struct DMAMap {
28
#define MAX_L2TPV3_IOVCNT (MAX_L2TPV3_MSGCNT * IOVSIZE)
32
@@ -XXX,XX +XXX,XX @@ const DMAMap *iova_tree_find_address(const IOVATree *tree, hwaddr iova);
33
void iova_tree_foreach(IOVATree *tree, iova_tree_iterator iterator);
34
35
/**
36
+ * iova_tree_alloc_map:
37
+ *
38
+ * @tree: the iova tree to allocate from
39
+ * @map: the new map (as translated addr & size) to allocate in the iova region
40
+ * @iova_begin: the minimum address of the allocation
41
+ * @iova_end: the maximum addressable direction of the allocation
42
+ *
43
+ * Allocates a new region of a given size, between iova_min and iova_max.
44
+ *
45
+ * Return: Same as iova_tree_insert, but cannot overlap and can return error if
46
+ * iova tree is out of free contiguous range. The caller gets the assigned iova
47
+ * in map->iova.
48
+ */
49
+int iova_tree_alloc_map(IOVATree *tree, DMAMap *map, hwaddr iova_begin,
50
+ hwaddr iova_end);
51
+
52
+/**
53
* iova_tree_destroy:
54
*
55
* @tree: the iova tree to destroy
56
diff --git a/util/iova-tree.c b/util/iova-tree.c
57
index XXXXXXX..XXXXXXX 100644
58
--- a/util/iova-tree.c
59
+++ b/util/iova-tree.c
60
@@ -XXX,XX +XXX,XX @@ struct IOVATree {
61
GTree *tree;
62
};
63
64
+/* Args to pass to iova_tree_alloc foreach function. */
65
+struct IOVATreeAllocArgs {
66
+ /* Size of the desired allocation */
67
+ size_t new_size;
68
+
69
+ /* The minimum address allowed in the allocation */
70
+ hwaddr iova_begin;
71
+
72
+ /* Map at the left of the hole, can be NULL if "this" is first one */
73
+ const DMAMap *prev;
74
+
75
+ /* Map at the right of the hole, can be NULL if "prev" is the last one */
76
+ const DMAMap *this;
77
+
78
+ /* If found, we fill in the IOVA here */
79
+ hwaddr iova_result;
80
+
81
+ /* Whether have we found a valid IOVA */
82
+ bool iova_found;
83
+};
84
+
85
+/**
86
+ * Iterate args to the next hole
87
+ *
88
+ * @args: The alloc arguments
89
+ * @next: The next mapping in the tree. Can be NULL to signal the last one
90
+ */
91
+static void iova_tree_alloc_args_iterate(struct IOVATreeAllocArgs *args,
92
+ const DMAMap *next)
93
+{
94
+ args->prev = args->this;
95
+ args->this = next;
96
+}
97
+
98
static int iova_tree_compare(gconstpointer a, gconstpointer b, gpointer data)
99
{
100
const DMAMap *m1 = a, *m2 = b;
101
@@ -XXX,XX +XXX,XX @@ int iova_tree_remove(IOVATree *tree, const DMAMap *map)
102
return IOVA_OK;
103
}
104
105
+/**
106
+ * Try to find an unallocated IOVA range between prev and this elements.
107
+ *
108
+ * @args: Arguments to allocation
109
+ *
110
+ * Cases:
111
+ *
112
+ * (1) !prev, !this: No entries allocated, always succeed
113
+ *
114
+ * (2) !prev, this: We're iterating at the 1st element.
115
+ *
116
+ * (3) prev, !this: We're iterating at the last element.
117
+ *
118
+ * (4) prev, this: this is the most common case, we'll try to find a hole
119
+ * between "prev" and "this" mapping.
120
+ *
121
+ * Note that this function assumes the last valid iova is HWADDR_MAX, but it
122
+ * searches linearly so it's easy to discard the result if it's not the case.
123
+ */
124
+static void iova_tree_alloc_map_in_hole(struct IOVATreeAllocArgs *args)
125
+{
126
+ const DMAMap *prev = args->prev, *this = args->this;
127
+ uint64_t hole_start, hole_last;
128
+
129
+ if (this && this->iova + this->size < args->iova_begin) {
130
+ return;
131
+ }
132
+
133
+ hole_start = MAX(prev ? prev->iova + prev->size + 1 : 0, args->iova_begin);
134
+ hole_last = this ? this->iova : HWADDR_MAX;
135
+
136
+ if (hole_last - hole_start > args->new_size) {
137
+ args->iova_result = hole_start;
138
+ args->iova_found = true;
139
+ }
140
+}
141
+
142
+/**
143
+ * Foreach dma node in the tree, compare if there is a hole with its previous
144
+ * node (or minimum iova address allowed) and the node.
145
+ *
146
+ * @key: Node iterating
147
+ * @value: Node iterating
148
+ * @pargs: Struct to communicate with the outside world
149
+ *
150
+ * Return: false to keep iterating, true if needs break.
151
+ */
152
+static gboolean iova_tree_alloc_traverse(gpointer key, gpointer value,
153
+ gpointer pargs)
154
+{
155
+ struct IOVATreeAllocArgs *args = pargs;
156
+ DMAMap *node = value;
157
+
158
+ assert(key == value);
159
+
160
+ iova_tree_alloc_args_iterate(args, node);
161
+ iova_tree_alloc_map_in_hole(args);
162
+ return args->iova_found;
163
+}
164
+
165
+int iova_tree_alloc_map(IOVATree *tree, DMAMap *map, hwaddr iova_begin,
166
+ hwaddr iova_last)
167
+{
168
+ struct IOVATreeAllocArgs args = {
169
+ .new_size = map->size,
170
+ .iova_begin = iova_begin,
171
+ };
172
+
173
+ if (unlikely(iova_last < iova_begin)) {
174
+ return IOVA_ERR_INVALID;
175
+ }
176
+
177
+ /*
178
+ * Find a valid hole for the mapping
179
+ *
180
+ * Assuming low iova_begin, so no need to do a binary search to
181
+ * locate the first node.
182
+ *
183
+ * TODO: Replace all this with g_tree_node_first/next/last when available
184
+ * (from glib since 2.68). To do it with g_tree_foreach complicates the
185
+ * code a lot.
186
+ *
187
+ */
188
+ g_tree_foreach(tree->tree, iova_tree_alloc_traverse, &args);
189
+ if (!args.iova_found) {
190
+ /*
191
+ * Either tree is empty or the last hole is still not checked.
192
+ * g_tree_foreach does not compare (last, iova_last] range, so we check
193
+ * it here.
194
+ */
195
+ iova_tree_alloc_args_iterate(&args, NULL);
196
+ iova_tree_alloc_map_in_hole(&args);
197
+ }
198
+
199
+ if (!args.iova_found || args.iova_result + map->size > iova_last) {
200
+ return IOVA_ERR_NOMEM;
201
+ }
202
+
203
+ map->iova = args.iova_result;
204
+ return iova_tree_insert(tree, map);
205
+}
206
+
207
void iova_tree_destroy(IOVATree *tree)
208
{
209
g_tree_destroy(tree->tree);
29
--
210
--
30
2.7.4
211
2.7.4
212
213
diff view generated by jsdifflib
1
From: Thomas Huth <thuth@redhat.com>
1
From: Eugenio Pérez <eperezma@redhat.com>
2
2
3
The code that collects the available NIC models is not really specific
3
This function does the reverse operation of iova_tree_find: To look for
4
to PCI anymore and will be required in the next patch, too, so let's
4
a mapping that match a translated address so we can do the reverse.
5
move this into a new separate function in net.c instead.
6
5
7
Signed-off-by: Thomas Huth <thuth@redhat.com>
6
This have linear complexity instead of logarithmic, but it supports
7
overlapping HVA. Future developments could reduce it.
8
9
Signed-off-by: Eugenio Pérez <eperezma@redhat.com>
10
Acked-by: Michael S. Tsirkin <mst@redhat.com>
8
Signed-off-by: Jason Wang <jasowang@redhat.com>
11
Signed-off-by: Jason Wang <jasowang@redhat.com>
9
---
12
---
10
hw/pci/pci.c | 29 +----------------------------
13
include/qemu/iova-tree.h | 20 +++++++++++++++++++-
11
include/net/net.h | 14 ++++++++++++++
14
util/iova-tree.c | 34 ++++++++++++++++++++++++++++++++++
12
net/net.c | 34 ++++++++++++++++++++++++++++++++++
15
2 files changed, 53 insertions(+), 1 deletion(-)
13
3 files changed, 49 insertions(+), 28 deletions(-)
14
16
15
diff --git a/hw/pci/pci.c b/hw/pci/pci.c
17
diff --git a/include/qemu/iova-tree.h b/include/qemu/iova-tree.h
16
index XXXXXXX..XXXXXXX 100644
18
index XXXXXXX..XXXXXXX 100644
17
--- a/hw/pci/pci.c
19
--- a/include/qemu/iova-tree.h
18
+++ b/hw/pci/pci.c
20
+++ b/include/qemu/iova-tree.h
19
@@ -XXX,XX +XXX,XX @@ PCIDevice *pci_nic_init_nofail(NICInfo *nd, PCIBus *rootbus,
21
@@ -XXX,XX +XXX,XX @@ int iova_tree_remove(IOVATree *tree, const DMAMap *map);
20
const char *default_devaddr)
22
* @tree: the iova tree to search from
21
{
23
* @map: the mapping to search
22
const char *devaddr = nd->devaddr ? nd->devaddr : default_devaddr;
24
*
23
- GSList *list;
25
- * Search for a mapping in the iova tree that overlaps with the
24
GPtrArray *pci_nic_models;
26
+ * Search for a mapping in the iova tree that iova overlaps with the
25
PCIBus *bus;
27
* mapping range specified. Only the first found mapping will be
26
PCIDevice *pci_dev;
28
* returned.
27
@@ -XXX,XX +XXX,XX @@ PCIDevice *pci_nic_init_nofail(NICInfo *nd, PCIBus *rootbus,
29
*
28
nd->model = g_strdup("virtio-net-pci");
30
@@ -XXX,XX +XXX,XX @@ int iova_tree_remove(IOVATree *tree, const DMAMap *map);
29
}
31
const DMAMap *iova_tree_find(const IOVATree *tree, const DMAMap *map);
30
32
31
- list = object_class_get_list_sorted(TYPE_PCI_DEVICE, false);
33
/**
32
- pci_nic_models = g_ptr_array_new();
34
+ * iova_tree_find_iova:
33
- while (list) {
35
+ *
34
- DeviceClass *dc = OBJECT_CLASS_CHECK(DeviceClass, list->data,
36
+ * @tree: the iova tree to search from
35
- TYPE_DEVICE);
37
+ * @map: the mapping to search
36
- GSList *next;
38
+ *
37
- if (test_bit(DEVICE_CATEGORY_NETWORK, dc->categories) &&
39
+ * Search for a mapping in the iova tree that translated_addr overlaps with the
38
- dc->user_creatable) {
40
+ * mapping range specified. Only the first found mapping will be
39
- const char *name = object_class_get_name(list->data);
41
+ * returned.
40
- /*
42
+ *
41
- * A network device might also be something else than a NIC, see
43
+ * Return: DMAMap pointer if found, or NULL if not found. Note that
42
- * e.g. the "rocker" device. Thus we have to look for the "netdev"
44
+ * the returned DMAMap pointer is maintained internally. User should
43
- * property, too. Unfortunately, some devices like virtio-net only
45
+ * only read the content but never modify or free the content. Also,
44
- * create this property during instance_init, so we have to create
46
+ * user is responsible to make sure the pointer is valid (say, no
45
- * a temporary instance here to be able to check it.
47
+ * concurrent deletion in progress).
46
- */
48
+ */
47
- Object *obj = object_new_with_class(OBJECT_CLASS(dc));
49
+const DMAMap *iova_tree_find_iova(const IOVATree *tree, const DMAMap *map);
48
- if (object_property_find(obj, "netdev")) {
50
+
49
- g_ptr_array_add(pci_nic_models, (gpointer)name);
51
+/**
50
- }
52
* iova_tree_find_address:
51
- object_unref(obj);
53
*
52
- }
54
* @tree: the iova tree to search from
53
- next = list->next;
55
diff --git a/util/iova-tree.c b/util/iova-tree.c
54
- g_slist_free_1(list);
55
- list = next;
56
- }
57
- g_ptr_array_add(pci_nic_models, NULL);
58
+ pci_nic_models = qemu_get_nic_models(TYPE_PCI_DEVICE);
59
60
if (qemu_show_nic_models(nd->model, (const char **)pci_nic_models->pdata)) {
61
exit(0);
62
diff --git a/include/net/net.h b/include/net/net.h
63
index XXXXXXX..XXXXXXX 100644
56
index XXXXXXX..XXXXXXX 100644
64
--- a/include/net/net.h
57
--- a/util/iova-tree.c
65
+++ b/include/net/net.h
58
+++ b/util/iova-tree.c
66
@@ -XXX,XX +XXX,XX @@ void net_socket_rs_init(SocketReadState *rs,
59
@@ -XXX,XX +XXX,XX @@ struct IOVATreeAllocArgs {
67
bool vnet_hdr);
60
bool iova_found;
68
NetClientState *qemu_get_peer(NetClientState *nc, int queue_index);
61
};
69
62
70
+/**
63
+typedef struct IOVATreeFindIOVAArgs {
71
+ * qemu_get_nic_models:
64
+ const DMAMap *needle;
72
+ * @device_type: Defines which devices should be taken into consideration
65
+ const DMAMap *result;
73
+ * (e.g. TYPE_DEVICE for all devices, or TYPE_PCI_DEVICE for PCI)
66
+} IOVATreeFindIOVAArgs;
74
+ *
75
+ * Get an array of pointers to names of NIC devices that are available in
76
+ * the QEMU binary. The array is terminated with a NULL pointer entry.
77
+ * The caller is responsible for freeing the memory when it is not required
78
+ * anymore, e.g. with g_ptr_array_free(..., true).
79
+ *
80
+ * Returns: Pointer to the array that contains the pointers to the names.
81
+ */
82
+GPtrArray *qemu_get_nic_models(const char *device_type);
83
+
67
+
84
/* NIC info */
68
/**
85
69
* Iterate args to the next hole
86
#define MAX_NICS 8
70
*
87
diff --git a/net/net.c b/net/net.c
71
@@ -XXX,XX +XXX,XX @@ const DMAMap *iova_tree_find(const IOVATree *tree, const DMAMap *map)
88
index XXXXXXX..XXXXXXX 100644
72
return g_tree_lookup(tree->tree, map);
89
--- a/net/net.c
90
+++ b/net/net.c
91
@@ -XXX,XX +XXX,XX @@ static int nic_get_free_idx(void)
92
return -1;
93
}
73
}
94
74
95
+GPtrArray *qemu_get_nic_models(const char *device_type)
75
+static gboolean iova_tree_find_address_iterator(gpointer key, gpointer value,
76
+ gpointer data)
96
+{
77
+{
97
+ GPtrArray *nic_models = g_ptr_array_new();
78
+ const DMAMap *map = key;
98
+ GSList *list = object_class_get_list_sorted(device_type, false);
79
+ IOVATreeFindIOVAArgs *args = data;
80
+ const DMAMap *needle;
99
+
81
+
100
+ while (list) {
82
+ g_assert(key == value);
101
+ DeviceClass *dc = OBJECT_CLASS_CHECK(DeviceClass, list->data,
83
+
102
+ TYPE_DEVICE);
84
+ needle = args->needle;
103
+ GSList *next;
85
+ if (map->translated_addr + map->size < needle->translated_addr ||
104
+ if (test_bit(DEVICE_CATEGORY_NETWORK, dc->categories) &&
86
+ needle->translated_addr + needle->size < map->translated_addr) {
105
+ dc->user_creatable) {
87
+ return false;
106
+ const char *name = object_class_get_name(list->data);
107
+ /*
108
+ * A network device might also be something else than a NIC, see
109
+ * e.g. the "rocker" device. Thus we have to look for the "netdev"
110
+ * property, too. Unfortunately, some devices like virtio-net only
111
+ * create this property during instance_init, so we have to create
112
+ * a temporary instance here to be able to check it.
113
+ */
114
+ Object *obj = object_new_with_class(OBJECT_CLASS(dc));
115
+ if (object_property_find(obj, "netdev")) {
116
+ g_ptr_array_add(nic_models, (gpointer)name);
117
+ }
118
+ object_unref(obj);
119
+ }
120
+ next = list->next;
121
+ g_slist_free_1(list);
122
+ list = next;
123
+ }
88
+ }
124
+ g_ptr_array_add(nic_models, NULL);
125
+
89
+
126
+ return nic_models;
90
+ args->result = map;
91
+ return true;
127
+}
92
+}
128
+
93
+
129
int qemu_show_nic_models(const char *arg, const char *const *models)
94
+const DMAMap *iova_tree_find_iova(const IOVATree *tree, const DMAMap *map)
95
+{
96
+ IOVATreeFindIOVAArgs args = {
97
+ .needle = map,
98
+ };
99
+
100
+ g_tree_foreach(tree->tree, iova_tree_find_address_iterator, &args);
101
+ return args.result;
102
+}
103
+
104
const DMAMap *iova_tree_find_address(const IOVATree *tree, hwaddr iova)
130
{
105
{
131
int i;
106
const DMAMap map = { .iova = iova, .size = 0 };
132
--
107
--
133
2.7.4
108
2.7.4
109
110
diff view generated by jsdifflib
1
From: Laurent Vivier <lvivier@redhat.com>
1
From: Eugenio Pérez <eperezma@redhat.com>
2
2
3
In stream mode, if the server shuts down there is currently
3
This tree is able to look for a translated address from an IOVA address.
4
no way to reconnect the client to a new server without removing
5
the NIC device and the netdev backend (or to reboot).
6
4
7
This patch introduces a reconnect option that specifies a delay
5
At first glance it is similar to util/iova-tree. However, SVQ working on
8
to try to reconnect with the same parameters.
6
devices with limited IOVA space need more capabilities, like allocating
7
IOVA chunks or performing reverse translations (qemu addresses to iova).
9
8
10
Add a new test in qtest to test the reconnect option and the
9
The allocation capability, as "assign a free IOVA address to this chunk
11
connect/disconnect events.
10
of memory in qemu's address space" allows shadow virtqueue to create a
11
new address space that is not restricted by guest's addressable one, so
12
we can allocate shadow vqs vrings outside of it.
12
13
13
Signed-off-by: Laurent Vivier <lvivier@redhat.com>
14
It duplicates the tree so it can search efficiently in both directions,
15
and it will signal overlap if iova or the translated address is present
16
in any tree.
17
18
Signed-off-by: Eugenio Pérez <eperezma@redhat.com>
19
Acked-by: Michael S. Tsirkin <mst@redhat.com>
14
Signed-off-by: Jason Wang <jasowang@redhat.com>
20
Signed-off-by: Jason Wang <jasowang@redhat.com>
15
---
21
---
16
net/stream.c | 53 ++++++++++++++++++++++-
22
hw/virtio/meson.build | 2 +-
17
qapi/net.json | 7 ++-
23
hw/virtio/vhost-iova-tree.c | 110 ++++++++++++++++++++++++++++++++++++++++++++
18
qemu-options.hx | 6 +--
24
hw/virtio/vhost-iova-tree.h | 27 +++++++++++
19
tests/qtest/netdev-socket.c | 101 ++++++++++++++++++++++++++++++++++++++++++++
25
3 files changed, 138 insertions(+), 1 deletion(-)
20
4 files changed, 162 insertions(+), 5 deletions(-)
26
create mode 100644 hw/virtio/vhost-iova-tree.c
27
create mode 100644 hw/virtio/vhost-iova-tree.h
21
28
22
diff --git a/net/stream.c b/net/stream.c
29
diff --git a/hw/virtio/meson.build b/hw/virtio/meson.build
23
index XXXXXXX..XXXXXXX 100644
30
index XXXXXXX..XXXXXXX 100644
24
--- a/net/stream.c
31
--- a/hw/virtio/meson.build
25
+++ b/net/stream.c
32
+++ b/hw/virtio/meson.build
33
@@ -XXX,XX +XXX,XX @@ softmmu_ss.add(when: 'CONFIG_ALL', if_true: files('vhost-stub.c'))
34
35
virtio_ss = ss.source_set()
36
virtio_ss.add(files('virtio.c'))
37
-virtio_ss.add(when: 'CONFIG_VHOST', if_true: files('vhost.c', 'vhost-backend.c', 'vhost-shadow-virtqueue.c'))
38
+virtio_ss.add(when: 'CONFIG_VHOST', if_true: files('vhost.c', 'vhost-backend.c', 'vhost-shadow-virtqueue.c', 'vhost-iova-tree.c'))
39
virtio_ss.add(when: 'CONFIG_VHOST_USER', if_true: files('vhost-user.c'))
40
virtio_ss.add(when: 'CONFIG_VHOST_VDPA', if_true: files('vhost-vdpa.c'))
41
virtio_ss.add(when: 'CONFIG_VIRTIO_BALLOON', if_true: files('virtio-balloon.c'))
42
diff --git a/hw/virtio/vhost-iova-tree.c b/hw/virtio/vhost-iova-tree.c
43
new file mode 100644
44
index XXXXXXX..XXXXXXX
45
--- /dev/null
46
+++ b/hw/virtio/vhost-iova-tree.c
26
@@ -XXX,XX +XXX,XX @@
47
@@ -XXX,XX +XXX,XX @@
27
#include "io/channel-socket.h"
48
+/*
28
#include "io/net-listener.h"
49
+ * vhost software live migration iova tree
29
#include "qapi/qapi-events-net.h"
50
+ *
30
+#include "qapi/qapi-visit-sockets.h"
51
+ * SPDX-FileCopyrightText: Red Hat, Inc. 2021
31
+#include "qapi/clone-visitor.h"
52
+ * SPDX-FileContributor: Author: Eugenio Pérez <eperezma@redhat.com>
32
53
+ *
33
typedef struct NetStreamState {
54
+ * SPDX-License-Identifier: GPL-2.0-or-later
34
NetClientState nc;
55
+ */
35
@@ -XXX,XX +XXX,XX @@ typedef struct NetStreamState {
56
+
36
guint ioc_write_tag;
57
+#include "qemu/osdep.h"
37
SocketReadState rs;
58
+#include "qemu/iova-tree.h"
38
unsigned int send_index; /* number of bytes sent*/
59
+#include "vhost-iova-tree.h"
39
+ uint32_t reconnect;
60
+
40
+ guint timer_tag;
61
+#define iova_min_addr qemu_real_host_page_size
41
+ SocketAddress *addr;
62
+
42
} NetStreamState;
63
+/**
43
64
+ * VhostIOVATree, able to:
44
static void net_stream_listen(QIONetListener *listener,
65
+ * - Translate iova address
45
QIOChannelSocket *cioc,
66
+ * - Reverse translate iova address (from translated to iova)
46
void *opaque);
67
+ * - Allocate IOVA regions for translated range (linear operation)
47
+static void net_stream_arm_reconnect(NetStreamState *s);
68
+ */
48
69
+struct VhostIOVATree {
49
static gboolean net_stream_writable(QIOChannel *ioc,
70
+ /* First addressable iova address in the device */
50
GIOCondition condition,
71
+ uint64_t iova_first;
51
@@ -XXX,XX +XXX,XX @@ static gboolean net_stream_send(QIOChannel *ioc,
72
+
52
qemu_set_info_str(&s->nc, "%s", "");
73
+ /* Last addressable iova address in the device */
53
74
+ uint64_t iova_last;
54
qapi_event_send_netdev_stream_disconnected(s->nc.name);
75
+
55
+ net_stream_arm_reconnect(s);
76
+ /* IOVA address to qemu memory maps. */
56
77
+ IOVATree *iova_taddr_map;
57
return G_SOURCE_REMOVE;
78
+};
58
}
79
+
59
@@ -XXX,XX +XXX,XX @@ static gboolean net_stream_send(QIOChannel *ioc,
80
+/**
60
static void net_stream_cleanup(NetClientState *nc)
81
+ * Create a new IOVA tree
61
{
82
+ *
62
NetStreamState *s = DO_UPCAST(NetStreamState, nc, nc);
83
+ * Returns the new IOVA tree
63
+ if (s->timer_tag) {
84
+ */
64
+ g_source_remove(s->timer_tag);
85
+VhostIOVATree *vhost_iova_tree_new(hwaddr iova_first, hwaddr iova_last)
65
+ s->timer_tag = 0;
86
+{
66
+ }
87
+ VhostIOVATree *tree = g_new(VhostIOVATree, 1);
67
+ if (s->addr) {
88
+
68
+ qapi_free_SocketAddress(s->addr);
89
+ /* Some devices do not like 0 addresses */
69
+ s->addr = NULL;
90
+ tree->iova_first = MAX(iova_first, iova_min_addr);
70
+ }
91
+ tree->iova_last = iova_last;
71
if (s->ioc) {
92
+
72
if (QIO_CHANNEL_SOCKET(s->ioc)->fd != -1) {
93
+ tree->iova_taddr_map = iova_tree_new();
73
if (s->ioc_read_tag) {
94
+ return tree;
74
@@ -XXX,XX +XXX,XX @@ static void net_stream_client_connected(QIOTask *task, gpointer opaque)
75
error:
76
object_unref(OBJECT(s->ioc));
77
s->ioc = NULL;
78
+ net_stream_arm_reconnect(s);
79
+}
95
+}
80
+
96
+
81
+static gboolean net_stream_reconnect(gpointer data)
97
+/**
98
+ * Delete an iova tree
99
+ */
100
+void vhost_iova_tree_delete(VhostIOVATree *iova_tree)
82
+{
101
+{
83
+ NetStreamState *s = data;
102
+ iova_tree_destroy(iova_tree->iova_taddr_map);
84
+ QIOChannelSocket *sioc;
103
+ g_free(iova_tree);
85
+
86
+ s->timer_tag = 0;
87
+
88
+ sioc = qio_channel_socket_new();
89
+ s->ioc = QIO_CHANNEL(sioc);
90
+ qio_channel_socket_connect_async(sioc, s->addr,
91
+ net_stream_client_connected, s,
92
+ NULL, NULL);
93
+ return G_SOURCE_REMOVE;
94
+}
104
+}
95
+
105
+
96
+static void net_stream_arm_reconnect(NetStreamState *s)
106
+/**
107
+ * Find the IOVA address stored from a memory address
108
+ *
109
+ * @tree: The iova tree
110
+ * @map: The map with the memory address
111
+ *
112
+ * Return the stored mapping, or NULL if not found.
113
+ */
114
+const DMAMap *vhost_iova_tree_find_iova(const VhostIOVATree *tree,
115
+ const DMAMap *map)
97
+{
116
+{
98
+ if (s->reconnect && s->timer_tag == 0) {
117
+ return iova_tree_find_iova(tree->iova_taddr_map, map);
99
+ s->timer_tag = g_timeout_add_seconds(s->reconnect,
100
+ net_stream_reconnect, s);
101
+ }
102
}
103
104
static int net_stream_client_init(NetClientState *peer,
105
const char *model,
106
const char *name,
107
SocketAddress *addr,
108
+ uint32_t reconnect,
109
Error **errp)
110
{
111
NetStreamState *s;
112
@@ -XXX,XX +XXX,XX @@ static int net_stream_client_init(NetClientState *peer,
113
s->ioc = QIO_CHANNEL(sioc);
114
s->nc.link_down = true;
115
116
+ s->reconnect = reconnect;
117
+ if (reconnect) {
118
+ s->addr = QAPI_CLONE(SocketAddress, addr);
119
+ }
120
qio_channel_socket_connect_async(sioc, addr,
121
net_stream_client_connected, s,
122
NULL, NULL);
123
@@ -XXX,XX +XXX,XX @@ int net_init_stream(const Netdev *netdev, const char *name,
124
sock = &netdev->u.stream;
125
126
if (!sock->has_server || !sock->server) {
127
- return net_stream_client_init(peer, "stream", name, sock->addr, errp);
128
+ return net_stream_client_init(peer, "stream", name, sock->addr,
129
+ sock->has_reconnect ? sock->reconnect : 0,
130
+ errp);
131
+ }
132
+ if (sock->has_reconnect) {
133
+ error_setg(errp, "'reconnect' option is incompatible with "
134
+ "socket in server mode");
135
+ return -1;
136
}
137
return net_stream_server_init(peer, "stream", name, sock->addr, errp);
138
}
139
diff --git a/qapi/net.json b/qapi/net.json
140
index XXXXXXX..XXXXXXX 100644
141
--- a/qapi/net.json
142
+++ b/qapi/net.json
143
@@ -XXX,XX +XXX,XX @@
144
# @addr: socket address to listen on (server=true)
145
# or connect to (server=false)
146
# @server: create server socket (default: false)
147
+# @reconnect: For a client socket, if a socket is disconnected,
148
+# then attempt a reconnect after the given number of seconds.
149
+# Setting this to zero disables this function. (default: 0)
150
+# (since 8.0)
151
#
152
# Only SocketAddress types 'unix', 'inet' and 'fd' are supported.
153
#
154
@@ -XXX,XX +XXX,XX @@
155
{ 'struct': 'NetdevStreamOptions',
156
'data': {
157
'addr': 'SocketAddress',
158
- '*server': 'bool' } }
159
+ '*server': 'bool',
160
+ '*reconnect': 'uint32' } }
161
162
##
163
# @NetdevDgramOptions:
164
diff --git a/qemu-options.hx b/qemu-options.hx
165
index XXXXXXX..XXXXXXX 100644
166
--- a/qemu-options.hx
167
+++ b/qemu-options.hx
168
@@ -XXX,XX +XXX,XX @@ DEF("netdev", HAS_ARG, QEMU_OPTION_netdev,
169
"-netdev socket,id=str[,fd=h][,udp=host:port][,localaddr=host:port]\n"
170
" configure a network backend to connect to another network\n"
171
" using an UDP tunnel\n"
172
- "-netdev stream,id=str[,server=on|off],addr.type=inet,addr.host=host,addr.port=port[,to=maxport][,numeric=on|off][,keep-alive=on|off][,mptcp=on|off][,addr.ipv4=on|off][,addr.ipv6=on|off]\n"
173
- "-netdev stream,id=str[,server=on|off],addr.type=unix,addr.path=path[,abstract=on|off][,tight=on|off]\n"
174
- "-netdev stream,id=str[,server=on|off],addr.type=fd,addr.str=file-descriptor\n"
175
+ "-netdev stream,id=str[,server=on|off],addr.type=inet,addr.host=host,addr.port=port[,to=maxport][,numeric=on|off][,keep-alive=on|off][,mptcp=on|off][,addr.ipv4=on|off][,addr.ipv6=on|off][,reconnect=seconds]\n"
176
+ "-netdev stream,id=str[,server=on|off],addr.type=unix,addr.path=path[,abstract=on|off][,tight=on|off][,reconnect=seconds]\n"
177
+ "-netdev stream,id=str[,server=on|off],addr.type=fd,addr.str=file-descriptor[,reconnect=seconds]\n"
178
" configure a network backend to connect to another network\n"
179
" using a socket connection in stream mode.\n"
180
"-netdev dgram,id=str,remote.type=inet,remote.host=maddr,remote.port=port[,local.type=inet,local.host=addr]\n"
181
diff --git a/tests/qtest/netdev-socket.c b/tests/qtest/netdev-socket.c
182
index XXXXXXX..XXXXXXX 100644
183
--- a/tests/qtest/netdev-socket.c
184
+++ b/tests/qtest/netdev-socket.c
185
@@ -XXX,XX +XXX,XX @@
186
#include <glib/gstdio.h>
187
#include "../unit/socket-helpers.h"
188
#include "libqtest.h"
189
+#include "qapi/qmp/qstring.h"
190
+#include "qemu/sockets.h"
191
+#include "qapi/qobject-input-visitor.h"
192
+#include "qapi/qapi-visit-sockets.h"
193
194
#define CONNECTION_TIMEOUT 5
195
196
@@ -XXX,XX +XXX,XX @@ static void test_stream_inet_ipv4(void)
197
qtest_quit(qts0);
198
}
199
200
+static void wait_stream_connected(QTestState *qts, const char *id,
201
+ SocketAddress **addr)
202
+{
203
+ QDict *resp, *data;
204
+ QString *qstr;
205
+ QObject *obj;
206
+ Visitor *v = NULL;
207
+
208
+ resp = qtest_qmp_eventwait_ref(qts, "NETDEV_STREAM_CONNECTED");
209
+ g_assert_nonnull(resp);
210
+ data = qdict_get_qdict(resp, "data");
211
+ g_assert_nonnull(data);
212
+
213
+ qstr = qobject_to(QString, qdict_get(data, "netdev-id"));
214
+ g_assert_nonnull(data);
215
+
216
+ g_assert(!strcmp(qstring_get_str(qstr), id));
217
+
218
+ obj = qdict_get(data, "addr");
219
+
220
+ v = qobject_input_visitor_new(obj);
221
+ visit_type_SocketAddress(v, NULL, addr, NULL);
222
+ visit_free(v);
223
+ qobject_unref(resp);
224
+}
118
+}
225
+
119
+
226
+static void wait_stream_disconnected(QTestState *qts, const char *id)
120
+/**
121
+ * Allocate a new mapping
122
+ *
123
+ * @tree: The iova tree
124
+ * @map: The iova map
125
+ *
126
+ * Returns:
127
+ * - IOVA_OK if the map fits in the container
128
+ * - IOVA_ERR_INVALID if the map does not make sense (like size overflow)
129
+ * - IOVA_ERR_NOMEM if tree cannot allocate more space.
130
+ *
131
+ * It returns assignated iova in map->iova if return value is VHOST_DMA_MAP_OK.
132
+ */
133
+int vhost_iova_tree_map_alloc(VhostIOVATree *tree, DMAMap *map)
227
+{
134
+{
228
+ QDict *resp, *data;
135
+ /* Some vhost devices do not like addr 0. Skip first page */
229
+ QString *qstr;
136
+ hwaddr iova_first = tree->iova_first ?: qemu_real_host_page_size;
230
+
137
+
231
+ resp = qtest_qmp_eventwait_ref(qts, "NETDEV_STREAM_DISCONNECTED");
138
+ if (map->translated_addr + map->size < map->translated_addr ||
232
+ g_assert_nonnull(resp);
139
+ map->perm == IOMMU_NONE) {
233
+ data = qdict_get_qdict(resp, "data");
140
+ return IOVA_ERR_INVALID;
234
+ g_assert_nonnull(data);
141
+ }
235
+
142
+
236
+ qstr = qobject_to(QString, qdict_get(data, "netdev-id"));
143
+ /* Allocate a node in IOVA address */
237
+ g_assert_nonnull(data);
144
+ return iova_tree_alloc_map(tree->iova_taddr_map, map, iova_first,
238
+
145
+ tree->iova_last);
239
+ g_assert(!strcmp(qstring_get_str(qstr), id));
240
+ qobject_unref(resp);
241
+}
146
+}
242
+
147
+
243
+static void test_stream_inet_reconnect(void)
148
+/**
149
+ * Remove existing mappings from iova tree
150
+ *
151
+ * @iova_tree: The vhost iova tree
152
+ * @map: The map to remove
153
+ */
154
+void vhost_iova_tree_remove(VhostIOVATree *iova_tree, const DMAMap *map)
244
+{
155
+{
245
+ QTestState *qts0, *qts1;
156
+ iova_tree_remove(iova_tree->iova_taddr_map, map);
246
+ int port;
157
+}
247
+ SocketAddress *addr;
158
diff --git a/hw/virtio/vhost-iova-tree.h b/hw/virtio/vhost-iova-tree.h
159
new file mode 100644
160
index XXXXXXX..XXXXXXX
161
--- /dev/null
162
+++ b/hw/virtio/vhost-iova-tree.h
163
@@ -XXX,XX +XXX,XX @@
164
+/*
165
+ * vhost software live migration iova tree
166
+ *
167
+ * SPDX-FileCopyrightText: Red Hat, Inc. 2021
168
+ * SPDX-FileContributor: Author: Eugenio Pérez <eperezma@redhat.com>
169
+ *
170
+ * SPDX-License-Identifier: GPL-2.0-or-later
171
+ */
248
+
172
+
249
+ port = inet_get_free_port(false);
173
+#ifndef HW_VIRTIO_VHOST_IOVA_TREE_H
250
+ qts0 = qtest_initf("-nodefaults -M none "
174
+#define HW_VIRTIO_VHOST_IOVA_TREE_H
251
+ "-netdev stream,id=st0,server=true,addr.type=inet,"
252
+ "addr.ipv4=on,addr.ipv6=off,"
253
+ "addr.host=127.0.0.1,addr.port=%d", port);
254
+
175
+
255
+ EXPECT_STATE(qts0, "st0: index=0,type=stream,\r\n", 0);
176
+#include "qemu/iova-tree.h"
177
+#include "exec/memory.h"
256
+
178
+
257
+ qts1 = qtest_initf("-nodefaults -M none "
179
+typedef struct VhostIOVATree VhostIOVATree;
258
+ "-netdev stream,server=false,id=st0,addr.type=inet,"
259
+ "addr.ipv4=on,addr.ipv6=off,reconnect=1,"
260
+ "addr.host=127.0.0.1,addr.port=%d", port);
261
+
180
+
262
+ wait_stream_connected(qts0, "st0", &addr);
181
+VhostIOVATree *vhost_iova_tree_new(uint64_t iova_first, uint64_t iova_last);
263
+ g_assert_cmpint(addr->type, ==, SOCKET_ADDRESS_TYPE_INET);
182
+void vhost_iova_tree_delete(VhostIOVATree *iova_tree);
264
+ g_assert_cmpstr(addr->u.inet.host, ==, "127.0.0.1");
183
+G_DEFINE_AUTOPTR_CLEANUP_FUNC(VhostIOVATree, vhost_iova_tree_delete);
265
+ qapi_free_SocketAddress(addr);
266
+
184
+
267
+ /* kill server */
185
+const DMAMap *vhost_iova_tree_find_iova(const VhostIOVATree *iova_tree,
268
+ qtest_quit(qts0);
186
+ const DMAMap *map);
187
+int vhost_iova_tree_map_alloc(VhostIOVATree *iova_tree, DMAMap *map);
188
+void vhost_iova_tree_remove(VhostIOVATree *iova_tree, const DMAMap *map);
269
+
189
+
270
+ /* check client has been disconnected */
190
+#endif
271
+ wait_stream_disconnected(qts1, "st0");
272
+
273
+ /* restart server */
274
+ qts0 = qtest_initf("-nodefaults -M none "
275
+ "-netdev stream,id=st0,server=true,addr.type=inet,"
276
+ "addr.ipv4=on,addr.ipv6=off,"
277
+ "addr.host=127.0.0.1,addr.port=%d", port);
278
+
279
+ /* wait connection events*/
280
+ wait_stream_connected(qts0, "st0", &addr);
281
+ g_assert_cmpint(addr->type, ==, SOCKET_ADDRESS_TYPE_INET);
282
+ g_assert_cmpstr(addr->u.inet.host, ==, "127.0.0.1");
283
+ qapi_free_SocketAddress(addr);
284
+
285
+ wait_stream_connected(qts1, "st0", &addr);
286
+ g_assert_cmpint(addr->type, ==, SOCKET_ADDRESS_TYPE_INET);
287
+ g_assert_cmpstr(addr->u.inet.host, ==, "127.0.0.1");
288
+ g_assert_cmpint(atoi(addr->u.inet.port), ==, port);
289
+ qapi_free_SocketAddress(addr);
290
+
291
+ qtest_quit(qts1);
292
+ qtest_quit(qts0);
293
+}
294
+
295
static void test_stream_inet_ipv6(void)
296
{
297
QTestState *qts0, *qts1;
298
@@ -XXX,XX +XXX,XX @@ int main(int argc, char **argv)
299
#ifndef _WIN32
300
qtest_add_func("/netdev/dgram/mcast", test_dgram_mcast);
301
#endif
302
+ qtest_add_func("/netdev/stream/inet/reconnect",
303
+ test_stream_inet_reconnect);
304
}
305
if (has_ipv6) {
306
qtest_add_func("/netdev/stream/inet/ipv6", test_stream_inet_ipv6);
307
--
191
--
308
2.7.4
192
2.7.4
193
194
diff view generated by jsdifflib
1
From: Thomas Huth <thuth@redhat.com>
1
From: Eugenio Pérez <eperezma@redhat.com>
2
2
3
Running QEMU with "-nic help" used to work in QEMU 5.2 and earlier versions
3
Use translations added in VhostIOVATree in SVQ.
4
(it showed the available netdev backends), but this feature got broken during
4
5
some refactoring in version 6.0. Let's restore the old behavior, and while
5
Only introduce usage here, not allocation and deallocation. As with
6
we're at it, let's also print the available NIC models here now since this
6
previous patches, we use the dead code paths of shadow_vqs_enabled to
7
option can be used to configure both, netdev backend and model in one go.
7
avoid commiting too many changes at once. These are impossible to take
8
8
at the moment.
9
Fixes: ad6f932fe8 ("net: do not exit on "netdev_add help" monitor command")
9
10
Signed-off-by: Thomas Huth <thuth@redhat.com>
10
Signed-off-by: Eugenio Pérez <eperezma@redhat.com>
11
Acked-by: Michael S. Tsirkin <mst@redhat.com>
11
Signed-off-by: Jason Wang <jasowang@redhat.com>
12
Signed-off-by: Jason Wang <jasowang@redhat.com>
12
---
13
---
13
net/net.c | 14 ++++++++++++--
14
hw/virtio/vhost-shadow-virtqueue.c | 86 +++++++++++++++++++++++---
14
1 file changed, 12 insertions(+), 2 deletions(-)
15
hw/virtio/vhost-shadow-virtqueue.h | 6 +-
15
16
hw/virtio/vhost-vdpa.c | 122 +++++++++++++++++++++++++++++++------
16
diff --git a/net/net.c b/net/net.c
17
include/hw/virtio/vhost-vdpa.h | 3 +
18
4 files changed, 187 insertions(+), 30 deletions(-)
19
20
diff --git a/hw/virtio/vhost-shadow-virtqueue.c b/hw/virtio/vhost-shadow-virtqueue.c
17
index XXXXXXX..XXXXXXX 100644
21
index XXXXXXX..XXXXXXX 100644
18
--- a/net/net.c
22
--- a/hw/virtio/vhost-shadow-virtqueue.c
19
+++ b/net/net.c
23
+++ b/hw/virtio/vhost-shadow-virtqueue.c
20
@@ -XXX,XX +XXX,XX @@ static int net_param_nic(void *dummy, QemuOpts *opts, Error **errp)
24
@@ -XXX,XX +XXX,XX @@ static uint16_t vhost_svq_available_slots(const VhostShadowVirtqueue *svq)
21
const char *type;
25
return svq->vring.num - (svq->shadow_avail_idx - svq->shadow_used_idx);
22
26
}
23
type = qemu_opt_get(opts, "type");
27
24
- if (type && g_str_equal(type, "none")) {
28
-static void vhost_vring_write_descs(VhostShadowVirtqueue *svq,
25
- return 0; /* Nothing to do, default_net is cleared in vl.c */
29
+/**
26
+ if (type) {
30
+ * Translate addresses between the qemu's virtual address and the SVQ IOVA
27
+ if (g_str_equal(type, "none")) {
31
+ *
28
+ return 0; /* Nothing to do, default_net is cleared in vl.c */
32
+ * @svq: Shadow VirtQueue
33
+ * @vaddr: Translated IOVA addresses
34
+ * @iovec: Source qemu's VA addresses
35
+ * @num: Length of iovec and minimum length of vaddr
36
+ */
37
+static bool vhost_svq_translate_addr(const VhostShadowVirtqueue *svq,
38
+ hwaddr *addrs, const struct iovec *iovec,
39
+ size_t num)
40
+{
41
+ if (num == 0) {
42
+ return true;
43
+ }
44
+
45
+ for (size_t i = 0; i < num; ++i) {
46
+ DMAMap needle = {
47
+ .translated_addr = (hwaddr)(uintptr_t)iovec[i].iov_base,
48
+ .size = iovec[i].iov_len,
49
+ };
50
+ Int128 needle_last, map_last;
51
+ size_t off;
52
+
53
+ const DMAMap *map = vhost_iova_tree_find_iova(svq->iova_tree, &needle);
54
+ /*
55
+ * Map cannot be NULL since iova map contains all guest space and
56
+ * qemu already has a physical address mapped
57
+ */
58
+ if (unlikely(!map)) {
59
+ qemu_log_mask(LOG_GUEST_ERROR,
60
+ "Invalid address 0x%"HWADDR_PRIx" given by guest",
61
+ needle.translated_addr);
62
+ return false;
29
+ }
63
+ }
30
+ if (is_help_option(type)) {
64
+
31
+ GPtrArray *nic_models = qemu_get_nic_models(TYPE_DEVICE);
65
+ off = needle.translated_addr - map->translated_addr;
32
+ show_netdevs();
66
+ addrs[i] = map->iova + off;
33
+ printf("\n");
67
+
34
+ qemu_show_nic_models(type, (const char **)nic_models->pdata);
68
+ needle_last = int128_add(int128_make64(needle.translated_addr),
35
+ g_ptr_array_free(nic_models, true);
69
+ int128_make64(iovec[i].iov_len));
36
+ exit(0);
70
+ map_last = int128_make64(map->translated_addr + map->size);
71
+ if (unlikely(int128_gt(needle_last, map_last))) {
72
+ qemu_log_mask(LOG_GUEST_ERROR,
73
+ "Guest buffer expands over iova range");
74
+ return false;
37
+ }
75
+ }
76
+ }
77
+
78
+ return true;
79
+}
80
+
81
+static void vhost_vring_write_descs(VhostShadowVirtqueue *svq, hwaddr *sg,
82
const struct iovec *iovec, size_t num,
83
bool more_descs, bool write)
84
{
85
@@ -XXX,XX +XXX,XX @@ static void vhost_vring_write_descs(VhostShadowVirtqueue *svq,
86
} else {
87
descs[i].flags = flags;
88
}
89
- descs[i].addr = cpu_to_le64((hwaddr)(intptr_t)iovec[n].iov_base);
90
+ descs[i].addr = cpu_to_le64(sg[n]);
91
descs[i].len = cpu_to_le32(iovec[n].iov_len);
92
93
last = i;
94
@@ -XXX,XX +XXX,XX @@ static bool vhost_svq_add_split(VhostShadowVirtqueue *svq,
95
{
96
unsigned avail_idx;
97
vring_avail_t *avail = svq->vring.avail;
98
+ bool ok;
99
+ g_autofree hwaddr *sgs = g_new(hwaddr, MAX(elem->out_num, elem->in_num));
100
101
*head = svq->free_head;
102
103
@@ -XXX,XX +XXX,XX @@ static bool vhost_svq_add_split(VhostShadowVirtqueue *svq,
104
return false;
38
}
105
}
39
106
40
idx = nic_get_free_idx();
107
- vhost_vring_write_descs(svq, elem->out_sg, elem->out_num, elem->in_num > 0,
108
- false);
109
- vhost_vring_write_descs(svq, elem->in_sg, elem->in_num, false, true);
110
+ ok = vhost_svq_translate_addr(svq, sgs, elem->out_sg, elem->out_num);
111
+ if (unlikely(!ok)) {
112
+ return false;
113
+ }
114
+ vhost_vring_write_descs(svq, sgs, elem->out_sg, elem->out_num,
115
+ elem->in_num > 0, false);
116
+
117
+
118
+ ok = vhost_svq_translate_addr(svq, sgs, elem->in_sg, elem->in_num);
119
+ if (unlikely(!ok)) {
120
+ return false;
121
+ }
122
+
123
+ vhost_vring_write_descs(svq, sgs, elem->in_sg, elem->in_num, false, true);
124
125
/*
126
* Put the entry in the available array (but don't update avail->idx until
127
@@ -XXX,XX +XXX,XX @@ void vhost_svq_set_svq_call_fd(VhostShadowVirtqueue *svq, int call_fd)
128
void vhost_svq_get_vring_addr(const VhostShadowVirtqueue *svq,
129
struct vhost_vring_addr *addr)
130
{
131
- addr->desc_user_addr = (uint64_t)(intptr_t)svq->vring.desc;
132
- addr->avail_user_addr = (uint64_t)(intptr_t)svq->vring.avail;
133
- addr->used_user_addr = (uint64_t)(intptr_t)svq->vring.used;
134
+ addr->desc_user_addr = (uint64_t)(uintptr_t)svq->vring.desc;
135
+ addr->avail_user_addr = (uint64_t)(uintptr_t)svq->vring.avail;
136
+ addr->used_user_addr = (uint64_t)(uintptr_t)svq->vring.used;
137
}
138
139
size_t vhost_svq_driver_area_size(const VhostShadowVirtqueue *svq)
140
@@ -XXX,XX +XXX,XX @@ void vhost_svq_stop(VhostShadowVirtqueue *svq)
141
* Creates vhost shadow virtqueue, and instructs the vhost device to use the
142
* shadow methods and file descriptors.
143
*
144
+ * @iova_tree: Tree to perform descriptors translations
145
+ *
146
* Returns the new virtqueue or NULL.
147
*
148
* In case of error, reason is reported through error_report.
149
*/
150
-VhostShadowVirtqueue *vhost_svq_new(void)
151
+VhostShadowVirtqueue *vhost_svq_new(VhostIOVATree *iova_tree)
152
{
153
g_autofree VhostShadowVirtqueue *svq = g_new0(VhostShadowVirtqueue, 1);
154
int r;
155
@@ -XXX,XX +XXX,XX @@ VhostShadowVirtqueue *vhost_svq_new(void)
156
157
event_notifier_init_fd(&svq->svq_kick, VHOST_FILE_UNBIND);
158
event_notifier_set_handler(&svq->hdev_call, vhost_svq_handle_call);
159
+ svq->iova_tree = iova_tree;
160
return g_steal_pointer(&svq);
161
162
err_init_hdev_call:
163
diff --git a/hw/virtio/vhost-shadow-virtqueue.h b/hw/virtio/vhost-shadow-virtqueue.h
164
index XXXXXXX..XXXXXXX 100644
165
--- a/hw/virtio/vhost-shadow-virtqueue.h
166
+++ b/hw/virtio/vhost-shadow-virtqueue.h
167
@@ -XXX,XX +XXX,XX @@
168
#include "qemu/event_notifier.h"
169
#include "hw/virtio/virtio.h"
170
#include "standard-headers/linux/vhost_types.h"
171
+#include "hw/virtio/vhost-iova-tree.h"
172
173
/* Shadow virtqueue to relay notifications */
174
typedef struct VhostShadowVirtqueue {
175
@@ -XXX,XX +XXX,XX @@ typedef struct VhostShadowVirtqueue {
176
/* Virtio device */
177
VirtIODevice *vdev;
178
179
+ /* IOVA mapping */
180
+ VhostIOVATree *iova_tree;
181
+
182
/* Map for use the guest's descriptors */
183
VirtQueueElement **ring_id_maps;
184
185
@@ -XXX,XX +XXX,XX @@ void vhost_svq_start(VhostShadowVirtqueue *svq, VirtIODevice *vdev,
186
VirtQueue *vq);
187
void vhost_svq_stop(VhostShadowVirtqueue *svq);
188
189
-VhostShadowVirtqueue *vhost_svq_new(void);
190
+VhostShadowVirtqueue *vhost_svq_new(VhostIOVATree *iova_tree);
191
192
void vhost_svq_free(gpointer vq);
193
G_DEFINE_AUTOPTR_CLEANUP_FUNC(VhostShadowVirtqueue, vhost_svq_free);
194
diff --git a/hw/virtio/vhost-vdpa.c b/hw/virtio/vhost-vdpa.c
195
index XXXXXXX..XXXXXXX 100644
196
--- a/hw/virtio/vhost-vdpa.c
197
+++ b/hw/virtio/vhost-vdpa.c
198
@@ -XXX,XX +XXX,XX @@ static void vhost_vdpa_listener_region_add(MemoryListener *listener,
199
vaddr, section->readonly);
200
201
llsize = int128_sub(llend, int128_make64(iova));
202
+ if (v->shadow_vqs_enabled) {
203
+ DMAMap mem_region = {
204
+ .translated_addr = (hwaddr)(uintptr_t)vaddr,
205
+ .size = int128_get64(llsize) - 1,
206
+ .perm = IOMMU_ACCESS_FLAG(true, section->readonly),
207
+ };
208
+
209
+ int r = vhost_iova_tree_map_alloc(v->iova_tree, &mem_region);
210
+ if (unlikely(r != IOVA_OK)) {
211
+ error_report("Can't allocate a mapping (%d)", r);
212
+ goto fail;
213
+ }
214
+
215
+ iova = mem_region.iova;
216
+ }
217
218
vhost_vdpa_iotlb_batch_begin_once(v);
219
ret = vhost_vdpa_dma_map(v, iova, int128_get64(llsize),
220
@@ -XXX,XX +XXX,XX @@ static void vhost_vdpa_listener_region_del(MemoryListener *listener,
221
222
llsize = int128_sub(llend, int128_make64(iova));
223
224
+ if (v->shadow_vqs_enabled) {
225
+ const DMAMap *result;
226
+ const void *vaddr = memory_region_get_ram_ptr(section->mr) +
227
+ section->offset_within_region +
228
+ (iova - section->offset_within_address_space);
229
+ DMAMap mem_region = {
230
+ .translated_addr = (hwaddr)(uintptr_t)vaddr,
231
+ .size = int128_get64(llsize) - 1,
232
+ };
233
+
234
+ result = vhost_iova_tree_find_iova(v->iova_tree, &mem_region);
235
+ iova = result->iova;
236
+ vhost_iova_tree_remove(v->iova_tree, &mem_region);
237
+ }
238
vhost_vdpa_iotlb_batch_begin_once(v);
239
ret = vhost_vdpa_dma_unmap(v, iova, int128_get64(llsize));
240
if (ret) {
241
@@ -XXX,XX +XXX,XX @@ static int vhost_vdpa_init_svq(struct vhost_dev *hdev, struct vhost_vdpa *v,
242
243
shadow_vqs = g_ptr_array_new_full(hdev->nvqs, vhost_svq_free);
244
for (unsigned n = 0; n < hdev->nvqs; ++n) {
245
- g_autoptr(VhostShadowVirtqueue) svq = vhost_svq_new();
246
+ g_autoptr(VhostShadowVirtqueue) svq = vhost_svq_new(v->iova_tree);
247
248
if (unlikely(!svq)) {
249
error_setg(errp, "Cannot create svq %u", n);
250
@@ -XXX,XX +XXX,XX @@ static int vhost_vdpa_svq_set_fds(struct vhost_dev *dev,
251
/**
252
* Unmap a SVQ area in the device
253
*/
254
-static bool vhost_vdpa_svq_unmap_ring(struct vhost_vdpa *v, hwaddr iova,
255
- hwaddr size)
256
+static bool vhost_vdpa_svq_unmap_ring(struct vhost_vdpa *v,
257
+ const DMAMap *needle)
258
{
259
+ const DMAMap *result = vhost_iova_tree_find_iova(v->iova_tree, needle);
260
+ hwaddr size;
261
int r;
262
263
- size = ROUND_UP(size, qemu_real_host_page_size);
264
- r = vhost_vdpa_dma_unmap(v, iova, size);
265
+ if (unlikely(!result)) {
266
+ error_report("Unable to find SVQ address to unmap");
267
+ return false;
268
+ }
269
+
270
+ size = ROUND_UP(result->size, qemu_real_host_page_size);
271
+ r = vhost_vdpa_dma_unmap(v, result->iova, size);
272
return r == 0;
273
}
274
275
static bool vhost_vdpa_svq_unmap_rings(struct vhost_dev *dev,
276
const VhostShadowVirtqueue *svq)
277
{
278
+ DMAMap needle = {};
279
struct vhost_vdpa *v = dev->opaque;
280
struct vhost_vring_addr svq_addr;
281
- size_t device_size = vhost_svq_device_area_size(svq);
282
- size_t driver_size = vhost_svq_driver_area_size(svq);
283
bool ok;
284
285
vhost_svq_get_vring_addr(svq, &svq_addr);
286
287
- ok = vhost_vdpa_svq_unmap_ring(v, svq_addr.desc_user_addr, driver_size);
288
+ needle.translated_addr = svq_addr.desc_user_addr;
289
+ ok = vhost_vdpa_svq_unmap_ring(v, &needle);
290
if (unlikely(!ok)) {
291
return false;
292
}
293
294
- return vhost_vdpa_svq_unmap_ring(v, svq_addr.used_user_addr, device_size);
295
+ needle.translated_addr = svq_addr.used_user_addr;
296
+ return vhost_vdpa_svq_unmap_ring(v, &needle);
297
+}
298
+
299
+/**
300
+ * Map the SVQ area in the device
301
+ *
302
+ * @v: Vhost-vdpa device
303
+ * @needle: The area to search iova
304
+ * @errorp: Error pointer
305
+ */
306
+static bool vhost_vdpa_svq_map_ring(struct vhost_vdpa *v, DMAMap *needle,
307
+ Error **errp)
308
+{
309
+ int r;
310
+
311
+ r = vhost_iova_tree_map_alloc(v->iova_tree, needle);
312
+ if (unlikely(r != IOVA_OK)) {
313
+ error_setg(errp, "Cannot allocate iova (%d)", r);
314
+ return false;
315
+ }
316
+
317
+ r = vhost_vdpa_dma_map(v, needle->iova, needle->size + 1,
318
+ (void *)(uintptr_t)needle->translated_addr,
319
+ needle->perm == IOMMU_RO);
320
+ if (unlikely(r != 0)) {
321
+ error_setg_errno(errp, -r, "Cannot map region to device");
322
+ vhost_iova_tree_remove(v->iova_tree, needle);
323
+ }
324
+
325
+ return r == 0;
326
}
327
328
/**
329
@@ -XXX,XX +XXX,XX @@ static bool vhost_vdpa_svq_map_rings(struct vhost_dev *dev,
330
struct vhost_vring_addr *addr,
331
Error **errp)
332
{
333
+ DMAMap device_region, driver_region;
334
+ struct vhost_vring_addr svq_addr;
335
struct vhost_vdpa *v = dev->opaque;
336
size_t device_size = vhost_svq_device_area_size(svq);
337
size_t driver_size = vhost_svq_driver_area_size(svq);
338
- int r;
339
+ size_t avail_offset;
340
+ bool ok;
341
342
ERRP_GUARD();
343
- vhost_svq_get_vring_addr(svq, addr);
344
+ vhost_svq_get_vring_addr(svq, &svq_addr);
345
346
- r = vhost_vdpa_dma_map(v, addr->desc_user_addr, driver_size,
347
- (void *)(uintptr_t)addr->desc_user_addr, true);
348
- if (unlikely(r != 0)) {
349
- error_setg_errno(errp, -r, "Cannot create vq driver region: ");
350
+ driver_region = (DMAMap) {
351
+ .translated_addr = svq_addr.desc_user_addr,
352
+ .size = driver_size - 1,
353
+ .perm = IOMMU_RO,
354
+ };
355
+ ok = vhost_vdpa_svq_map_ring(v, &driver_region, errp);
356
+ if (unlikely(!ok)) {
357
+ error_prepend(errp, "Cannot create vq driver region: ");
358
return false;
359
}
360
+ addr->desc_user_addr = driver_region.iova;
361
+ avail_offset = svq_addr.avail_user_addr - svq_addr.desc_user_addr;
362
+ addr->avail_user_addr = driver_region.iova + avail_offset;
363
364
- r = vhost_vdpa_dma_map(v, addr->used_user_addr, device_size,
365
- (void *)(intptr_t)addr->used_user_addr, false);
366
- if (unlikely(r != 0)) {
367
- error_setg_errno(errp, -r, "Cannot create vq device region: ");
368
+ device_region = (DMAMap) {
369
+ .translated_addr = svq_addr.used_user_addr,
370
+ .size = device_size - 1,
371
+ .perm = IOMMU_RW,
372
+ };
373
+ ok = vhost_vdpa_svq_map_ring(v, &device_region, errp);
374
+ if (unlikely(!ok)) {
375
+ error_prepend(errp, "Cannot create vq device region: ");
376
+ vhost_vdpa_svq_unmap_ring(v, &driver_region);
377
}
378
+ addr->used_user_addr = device_region.iova;
379
380
- return r == 0;
381
+ return ok;
382
}
383
384
static bool vhost_vdpa_svq_setup(struct vhost_dev *dev,
385
diff --git a/include/hw/virtio/vhost-vdpa.h b/include/hw/virtio/vhost-vdpa.h
386
index XXXXXXX..XXXXXXX 100644
387
--- a/include/hw/virtio/vhost-vdpa.h
388
+++ b/include/hw/virtio/vhost-vdpa.h
389
@@ -XXX,XX +XXX,XX @@
390
391
#include <gmodule.h>
392
393
+#include "hw/virtio/vhost-iova-tree.h"
394
#include "hw/virtio/virtio.h"
395
#include "standard-headers/linux/vhost_types.h"
396
397
@@ -XXX,XX +XXX,XX @@ typedef struct vhost_vdpa {
398
MemoryListener listener;
399
struct vhost_vdpa_iova_range iova_range;
400
bool shadow_vqs_enabled;
401
+ /* IOVA mapping used by the Shadow Virtqueue */
402
+ VhostIOVATree *iova_tree;
403
GPtrArray *shadow_vqs;
404
struct vhost_dev *dev;
405
VhostVDPAHostNotifier notifier[VIRTIO_QUEUE_MAX];
41
--
406
--
42
2.7.4
407
2.7.4
408
409
diff view generated by jsdifflib
1
From: Fiona Ebner <f.ebner@proxmox.com>
1
From: Eugenio Pérez <eperezma@redhat.com>
2
2
3
Currently, VMXNET3_MAX_MTU itself (being 9000) is not considered a
3
This is needed to achieve migration, so the destination can restore its
4
valid value for the MTU, but a guest running ESXi 7.0 might try to
4
index.
5
set it and fail the assert [0].
6
5
7
In the Linux kernel, dev->max_mtu itself is a valid value for the MTU
6
Setting base as last used idx, so destination will see as available all
8
and for the vmxnet3 driver it's 9000, so a guest running Linux will
7
the entries that the device did not use, including the in-flight
9
also fail the assert when trying to set an MTU of 9000.
8
processing ones.
10
9
11
VMXNET3_MAX_MTU and s->mtu don't seem to be used in relation to buffer
10
This is ok for networking, but other kinds of devices might have
12
allocations/accesses, so allowing the upper limit itself as a value
11
problems with these retransmissions.
13
should be fine.
14
12
15
[0]: https://forum.proxmox.com/threads/114011/
13
Signed-off-by: Eugenio Pérez <eperezma@redhat.com>
16
14
Acked-by: Michael S. Tsirkin <mst@redhat.com>
17
Fixes: d05dcd94ae ("net: vmxnet3: validate configuration values during activate (CVE-2021-20203)")
18
Signed-off-by: Fiona Ebner <f.ebner@proxmox.com>
19
Signed-off-by: Jason Wang <jasowang@redhat.com>
15
Signed-off-by: Jason Wang <jasowang@redhat.com>
20
---
16
---
21
hw/net/vmxnet3.c | 2 +-
17
hw/virtio/vhost-vdpa.c | 17 +++++++++++++++++
22
1 file changed, 1 insertion(+), 1 deletion(-)
18
1 file changed, 17 insertions(+)
23
19
24
diff --git a/hw/net/vmxnet3.c b/hw/net/vmxnet3.c
20
diff --git a/hw/virtio/vhost-vdpa.c b/hw/virtio/vhost-vdpa.c
25
index XXXXXXX..XXXXXXX 100644
21
index XXXXXXX..XXXXXXX 100644
26
--- a/hw/net/vmxnet3.c
22
--- a/hw/virtio/vhost-vdpa.c
27
+++ b/hw/net/vmxnet3.c
23
+++ b/hw/virtio/vhost-vdpa.c
28
@@ -XXX,XX +XXX,XX @@ static void vmxnet3_activate_device(VMXNET3State *s)
24
@@ -XXX,XX +XXX,XX @@ static int vhost_vdpa_set_vring_base(struct vhost_dev *dev,
29
vmxnet3_setup_rx_filtering(s);
25
static int vhost_vdpa_get_vring_base(struct vhost_dev *dev,
30
/* Cache fields from shared memory */
26
struct vhost_vring_state *ring)
31
s->mtu = VMXNET3_READ_DRV_SHARED32(d, s->drv_shmem, devRead.misc.mtu);
27
{
32
- assert(VMXNET3_MIN_MTU <= s->mtu && s->mtu < VMXNET3_MAX_MTU);
28
+ struct vhost_vdpa *v = dev->opaque;
33
+ assert(VMXNET3_MIN_MTU <= s->mtu && s->mtu <= VMXNET3_MAX_MTU);
29
int ret;
34
VMW_CFPRN("MTU is %u", s->mtu);
30
35
31
+ if (v->shadow_vqs_enabled) {
36
s->max_rx_frags =
32
+ VhostShadowVirtqueue *svq = g_ptr_array_index(v->shadow_vqs,
33
+ ring->index);
34
+
35
+ /*
36
+ * Setting base as last used idx, so destination will see as available
37
+ * all the entries that the device did not use, including the in-flight
38
+ * processing ones.
39
+ *
40
+ * TODO: This is ok for networking, but other kinds of devices might
41
+ * have problems with these retransmissions.
42
+ */
43
+ ring->num = svq->last_used_idx;
44
+ return 0;
45
+ }
46
+
47
ret = vhost_vdpa_call(dev, VHOST_GET_VRING_BASE, ring);
48
trace_vhost_vdpa_get_vring_base(dev, ring->index, ring->num);
49
return ret;
37
--
50
--
38
2.7.4
51
2.7.4
52
53
diff view generated by jsdifflib
1
From: Eugenio Pérez <eperezma@redhat.com>
1
From: Eugenio Pérez <eperezma@redhat.com>
2
2
3
VHOST_BACKEND_F_IOTLB_ASID is the feature bit, not the bitmask. Since
3
Setting the log address would make the device start reporting invalid
4
the device under test also provided VHOST_BACKEND_F_IOTLB_MSG_V2 and
4
dirty memory because the SVQ vrings are located in qemu's memory.
5
VHOST_BACKEND_F_IOTLB_BATCH, this went unnoticed.
6
5
7
Fixes: c1a1008685 ("vdpa: always start CVQ in SVQ mode if possible")
8
Signed-off-by: Eugenio Pérez <eperezma@redhat.com>
6
Signed-off-by: Eugenio Pérez <eperezma@redhat.com>
9
Reviewed-by: Michael S. Tsirkin <mst@redhat.com>
7
Acked-by: Michael S. Tsirkin <mst@redhat.com>
10
Acked-by: Jason Wang <jasowang@redhat.com>
11
Signed-off-by: Jason Wang <jasowang@redhat.com>
8
Signed-off-by: Jason Wang <jasowang@redhat.com>
12
---
9
---
13
net/vhost-vdpa.c | 2 +-
10
hw/virtio/vhost-vdpa.c | 3 ++-
14
1 file changed, 1 insertion(+), 1 deletion(-)
11
1 file changed, 2 insertions(+), 1 deletion(-)
15
12
16
diff --git a/net/vhost-vdpa.c b/net/vhost-vdpa.c
13
diff --git a/hw/virtio/vhost-vdpa.c b/hw/virtio/vhost-vdpa.c
17
index XXXXXXX..XXXXXXX 100644
14
index XXXXXXX..XXXXXXX 100644
18
--- a/net/vhost-vdpa.c
15
--- a/hw/virtio/vhost-vdpa.c
19
+++ b/net/vhost-vdpa.c
16
+++ b/hw/virtio/vhost-vdpa.c
20
@@ -XXX,XX +XXX,XX @@ static int vhost_vdpa_net_cvq_start(NetClientState *nc)
17
@@ -XXX,XX +XXX,XX @@ static int vhost_vdpa_dev_start(struct vhost_dev *dev, bool started)
21
g_strerror(errno), errno);
18
static int vhost_vdpa_set_log_base(struct vhost_dev *dev, uint64_t base,
22
return -1;
19
struct vhost_log *log)
23
}
20
{
24
- if (!(backend_features & VHOST_BACKEND_F_IOTLB_ASID) ||
21
- if (vhost_vdpa_one_time_request(dev)) {
25
+ if (!(backend_features & BIT_ULL(VHOST_BACKEND_F_IOTLB_ASID)) ||
22
+ struct vhost_vdpa *v = dev->opaque;
26
!vhost_vdpa_net_valid_svq_features(v->dev->features, NULL)) {
23
+ if (v->shadow_vqs_enabled || vhost_vdpa_one_time_request(dev)) {
27
return 0;
24
return 0;
28
}
25
}
26
29
--
27
--
30
2.7.4
28
2.7.4
31
29
32
30
diff view generated by jsdifflib
1
From: Thomas Huth <thuth@redhat.com>
1
From: Eugenio Pérez <eperezma@redhat.com>
2
2
3
Just because a NIC model is compiled into the QEMU binary does not
3
SVQ is able to log the dirty bits by itself, so let's use it to not
4
necessary mean that it can be used with each and every machine.
4
block migration.
5
So let's rather talk about "available" models instead of "supported"
6
models, just to avoid confusion.
7
5
8
Reviewed-by: Claudio Fontana <cfontana@suse.de>
6
Also, ignore set and clear of VHOST_F_LOG_ALL on set_features if SVQ is
9
Signed-off-by: Thomas Huth <thuth@redhat.com>
7
enabled. Even if the device supports it, the reports would be nonsense
8
because SVQ memory is in the qemu region.
9
10
The log region is still allocated. Future changes might skip that, but
11
this series is already long enough.
12
13
Signed-off-by: Eugenio Pérez <eperezma@redhat.com>
14
Acked-by: Michael S. Tsirkin <mst@redhat.com>
10
Signed-off-by: Jason Wang <jasowang@redhat.com>
15
Signed-off-by: Jason Wang <jasowang@redhat.com>
11
---
16
---
12
net/net.c | 2 +-
17
hw/virtio/vhost-vdpa.c | 39 +++++++++++++++++++++++++++++++++++----
13
1 file changed, 1 insertion(+), 1 deletion(-)
18
include/hw/virtio/vhost-vdpa.h | 1 +
19
2 files changed, 36 insertions(+), 4 deletions(-)
14
20
15
diff --git a/net/net.c b/net/net.c
21
diff --git a/hw/virtio/vhost-vdpa.c b/hw/virtio/vhost-vdpa.c
16
index XXXXXXX..XXXXXXX 100644
22
index XXXXXXX..XXXXXXX 100644
17
--- a/net/net.c
23
--- a/hw/virtio/vhost-vdpa.c
18
+++ b/net/net.c
24
+++ b/hw/virtio/vhost-vdpa.c
19
@@ -XXX,XX +XXX,XX @@ int qemu_show_nic_models(const char *arg, const char *const *models)
25
@@ -XXX,XX +XXX,XX @@ static bool vhost_vdpa_one_time_request(struct vhost_dev *dev)
26
return v->index != 0;
27
}
28
29
+static int vhost_vdpa_get_dev_features(struct vhost_dev *dev,
30
+ uint64_t *features)
31
+{
32
+ int ret;
33
+
34
+ ret = vhost_vdpa_call(dev, VHOST_GET_FEATURES, features);
35
+ trace_vhost_vdpa_get_features(dev, *features);
36
+ return ret;
37
+}
38
+
39
static int vhost_vdpa_init_svq(struct vhost_dev *hdev, struct vhost_vdpa *v,
40
Error **errp)
41
{
42
@@ -XXX,XX +XXX,XX @@ static int vhost_vdpa_init_svq(struct vhost_dev *hdev, struct vhost_vdpa *v,
20
return 0;
43
return 0;
21
}
44
}
22
45
23
- printf("Supported NIC models:\n");
46
- r = hdev->vhost_ops->vhost_get_features(hdev, &dev_features);
24
+ printf("Available NIC models:\n");
47
+ r = vhost_vdpa_get_dev_features(hdev, &dev_features);
25
for (i = 0 ; models[i]; i++) {
48
if (r != 0) {
26
printf("%s\n", models[i]);
49
error_setg_errno(errp, -r, "Can't get vdpa device features");
50
return r;
51
@@ -XXX,XX +XXX,XX @@ static int vhost_vdpa_set_mem_table(struct vhost_dev *dev,
52
static int vhost_vdpa_set_features(struct vhost_dev *dev,
53
uint64_t features)
54
{
55
+ struct vhost_vdpa *v = dev->opaque;
56
int ret;
57
58
if (vhost_vdpa_one_time_request(dev)) {
59
return 0;
27
}
60
}
61
62
+ if (v->shadow_vqs_enabled) {
63
+ if ((v->acked_features ^ features) == BIT_ULL(VHOST_F_LOG_ALL)) {
64
+ /*
65
+ * QEMU is just trying to enable or disable logging. SVQ handles
66
+ * this sepparately, so no need to forward this.
67
+ */
68
+ v->acked_features = features;
69
+ return 0;
70
+ }
71
+
72
+ v->acked_features = features;
73
+
74
+ /* We must not ack _F_LOG if SVQ is enabled */
75
+ features &= ~BIT_ULL(VHOST_F_LOG_ALL);
76
+ }
77
+
78
trace_vhost_vdpa_set_features(dev, features);
79
ret = vhost_vdpa_call(dev, VHOST_SET_FEATURES, &features);
80
if (ret) {
81
@@ -XXX,XX +XXX,XX @@ static int vhost_vdpa_set_vring_call(struct vhost_dev *dev,
82
static int vhost_vdpa_get_features(struct vhost_dev *dev,
83
uint64_t *features)
84
{
85
- int ret;
86
+ struct vhost_vdpa *v = dev->opaque;
87
+ int ret = vhost_vdpa_get_dev_features(dev, features);
88
+
89
+ if (ret == 0 && v->shadow_vqs_enabled) {
90
+ /* Add SVQ logging capabilities */
91
+ *features |= BIT_ULL(VHOST_F_LOG_ALL);
92
+ }
93
94
- ret = vhost_vdpa_call(dev, VHOST_GET_FEATURES, features);
95
- trace_vhost_vdpa_get_features(dev, *features);
96
return ret;
97
}
98
99
diff --git a/include/hw/virtio/vhost-vdpa.h b/include/hw/virtio/vhost-vdpa.h
100
index XXXXXXX..XXXXXXX 100644
101
--- a/include/hw/virtio/vhost-vdpa.h
102
+++ b/include/hw/virtio/vhost-vdpa.h
103
@@ -XXX,XX +XXX,XX @@ typedef struct vhost_vdpa {
104
bool iotlb_batch_begin_sent;
105
MemoryListener listener;
106
struct vhost_vdpa_iova_range iova_range;
107
+ uint64_t acked_features;
108
bool shadow_vqs_enabled;
109
/* IOVA mapping used by the Shadow Virtqueue */
110
VhostIOVATree *iova_tree;
28
--
111
--
29
2.7.4
112
2.7.4
113
114
diff view generated by jsdifflib