[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Qemu-devel] [PATCH v2 13/16] vhost-pci-slave: add "reset_virtio"
From: |
Wei Wang |
Subject: |
[Qemu-devel] [PATCH v2 13/16] vhost-pci-slave: add "reset_virtio" |
Date: |
Fri, 12 May 2017 16:35:45 +0800 |
After the vhost-pci-net device being hotplugged to the VM, the device
uses the features bits that have been negotiated with the remote virtio
device to negotiate with the driver. If the driver accepts a subset of
the feature bits, it implies that the vhost-pci-net can only suppoort
a subset of the features supported by the remote virtio device. In this
case, the remote virtio_device will be reset, and re-start the vhost-user
protocol.
Add the "reset_virtio" field as an indicator to the slave in this case.
Signed-off-by: Wei Wang <address@hidden>
---
hw/net/vhost-pci-net.c | 11 +++++++++++
hw/virtio/vhost-pci-slave.c | 15 +++++++++++++--
include/hw/virtio/vhost-pci-slave.h | 1 +
3 files changed, 25 insertions(+), 2 deletions(-)
diff --git a/hw/net/vhost-pci-net.c b/hw/net/vhost-pci-net.c
index 1379204..a2dca50 100644
--- a/hw/net/vhost-pci-net.c
+++ b/hw/net/vhost-pci-net.c
@@ -189,8 +189,19 @@ static void vpnet_set_features(VirtIODevice *vdev,
uint64_t features)
*/
static bool need_send;
int ret;
+ VhostPCIDev *vp_dev = get_vhost_pci_dev();
if (need_send) {
+ /*
+ * If the remote negotiated feature bits are not equal to the
+ * feature bits that have been negotiated between the device and
+ * driver, the remote virtio device needs a reset. Set reset_virtio
+ * to indicate to the slave about this case.
+ */
+ if (vp_dev->feature_bits != features) {
+ vp_dev->feature_bits = features;
+ vp_dev->reset_virtio = 1;
+ }
need_send = 0;
ret = vp_slave_send_feature_bits(features);
if (ret < 0) {
diff --git a/hw/virtio/vhost-pci-slave.c b/hw/virtio/vhost-pci-slave.c
index 6cc9c21..a7d3c8d 100644
--- a/hw/virtio/vhost-pci-slave.c
+++ b/hw/virtio/vhost-pci-slave.c
@@ -171,8 +171,15 @@ static void vp_slave_set_device_type(VhostUserMsg *msg)
switch (vp_dev->dev_type) {
case VIRTIO_ID_NET:
- vp_dev->feature_bits |= VHOST_PCI_FEATURE_BITS |
- VHOST_PCI_NET_FEATURE_BITS;
+ /*
+ * The setting of reset_virtio implies that the feature_bits has been
+ * remotely negotiated. So, skip adding the supported features to
+ * feature_bits in this case.
+ */
+ if (!vp_dev->reset_virtio) {
+ vp_dev->feature_bits |= VHOST_PCI_FEATURE_BITS |
+ VHOST_PCI_NET_FEATURE_BITS;
+ }
break;
default:
error_report("%s: device type %d is not supported",
@@ -400,6 +407,9 @@ static int vp_slave_set_vhost_pci(VhostUserMsg *msg)
switch (cmd) {
case VHOST_USER_SET_VHOST_PCI_START:
+ if (vp_dev->reset_virtio) {
+ vp_dev->reset_virtio = 0;
+ }
ret = vp_slave_device_create(vp_dev->dev_type);
if (ret < 0) {
return ret;
@@ -585,6 +595,7 @@ static void vp_dev_init(VhostPCIDev *vp_dev)
vp_dev->vdev = NULL;
QLIST_INIT(&vp_dev->remoteq_list);
vp_dev->remoteq_num = 0;
+ vp_dev->reset_virtio = 0;
}
int vhost_pci_slave_init(QemuOpts *opts)
diff --git a/include/hw/virtio/vhost-pci-slave.h
b/include/hw/virtio/vhost-pci-slave.h
index ab21e70..594917f 100644
--- a/include/hw/virtio/vhost-pci-slave.h
+++ b/include/hw/virtio/vhost-pci-slave.h
@@ -29,6 +29,7 @@ typedef struct RemoteMem {
typedef struct VhostPCIDev {
/* Ponnter to the slave device */
VirtIODevice *vdev;
+ bool reset_virtio;
uint16_t dev_type;
uint64_t feature_bits;
/* Records the end (offset to the BAR) of the last mapped region */
--
2.7.4
- [Qemu-devel] [PATCH v2 07/16] vhost-user: send device id to the slave, (continued)
- [Qemu-devel] [PATCH v2 07/16] vhost-user: send device id to the slave, Wei Wang, 2017/05/12
- [Qemu-devel] [PATCH v2 05/16] vhost-pci-net-pci: add vhost-pci-net-pci, Wei Wang, 2017/05/12
- [Qemu-devel] [PATCH v2 06/16] virtio: add inter-vm notification support, Wei Wang, 2017/05/12
- [Qemu-devel] [PATCH v2 08/16] vhost-user: send guest physical address of virtqueues to the slave, Wei Wang, 2017/05/12
- [Qemu-devel] [PATCH v2 09/16] vhost-user: send VHOST_USER_SET_VHOST_PCI_START/STOP, Wei Wang, 2017/05/12
- [Qemu-devel] [PATCH v2 10/16] vhost-pci-net: send the negotiated feature bits to the master, Wei Wang, 2017/05/12
- [Qemu-devel] [PATCH v2 11/16] vhost-user: add asynchronous read for the vhost-user master, Wei Wang, 2017/05/12
- [Qemu-devel] [PATCH v2 12/16] vhost-user: handling VHOST_USER_SET_FEATURES, Wei Wang, 2017/05/12
- [Qemu-devel] [PATCH v2 13/16] vhost-pci-slave: add "reset_virtio",
Wei Wang <=
- [Qemu-devel] [PATCH v2 14/16] vhost-pci-slave: add support to delete a vhost-pci device, Wei Wang, 2017/05/12
- [Qemu-devel] [PATCH v2 15/16] vhost-pci-net: tell the driver that it is ready to send packets, Wei Wang, 2017/05/12
- [Qemu-devel] [PATCH v2 16/16] vl: enable vhost-pci-slave, Wei Wang, 2017/05/12
- Re: [Qemu-devel] [PATCH v2 00/16] Vhost-pci for inter-VM communication, no-reply, 2017/05/12
- Re: [Qemu-devel] [PATCH v2 00/16] Vhost-pci for inter-VM communication, Jason Wang, 2017/05/16