[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[PULL V3 15/15] vdpa: Expose VHOST_F_LOG_ALL on SVQ
From: |
Jason Wang |
Subject: |
[PULL V3 15/15] vdpa: Expose VHOST_F_LOG_ALL on SVQ |
Date: |
Tue, 15 Mar 2022 14:14:20 +0800 |
From: Eugenio Pérez <eperezma@redhat.com>
SVQ is able to log the dirty bits by itself, so let's use it to not
block migration.
Also, ignore set and clear of VHOST_F_LOG_ALL on set_features if SVQ is
enabled. Even if the device supports it, the reports would be nonsense
because SVQ memory is in the qemu region.
The log region is still allocated. Future changes might skip that, but
this series is already long enough.
Signed-off-by: Eugenio Pérez <eperezma@redhat.com>
Acked-by: Michael S. Tsirkin <mst@redhat.com>
Signed-off-by: Jason Wang <jasowang@redhat.com>
---
hw/virtio/vhost-vdpa.c | 39 +++++++++++++++++++++++++++++++++++----
include/hw/virtio/vhost-vdpa.h | 1 +
2 files changed, 36 insertions(+), 4 deletions(-)
diff --git a/hw/virtio/vhost-vdpa.c b/hw/virtio/vhost-vdpa.c
index 89d59f8..c5ed7a3 100644
--- a/hw/virtio/vhost-vdpa.c
+++ b/hw/virtio/vhost-vdpa.c
@@ -373,6 +373,16 @@ static bool vhost_vdpa_one_time_request(struct vhost_dev
*dev)
return v->index != 0;
}
+static int vhost_vdpa_get_dev_features(struct vhost_dev *dev,
+ uint64_t *features)
+{
+ int ret;
+
+ ret = vhost_vdpa_call(dev, VHOST_GET_FEATURES, features);
+ trace_vhost_vdpa_get_features(dev, *features);
+ return ret;
+}
+
static int vhost_vdpa_init_svq(struct vhost_dev *hdev, struct vhost_vdpa *v,
Error **errp)
{
@@ -385,7 +395,7 @@ static int vhost_vdpa_init_svq(struct vhost_dev *hdev,
struct vhost_vdpa *v,
return 0;
}
- r = hdev->vhost_ops->vhost_get_features(hdev, &dev_features);
+ r = vhost_vdpa_get_dev_features(hdev, &dev_features);
if (r != 0) {
error_setg_errno(errp, -r, "Can't get vdpa device features");
return r;
@@ -610,12 +620,29 @@ static int vhost_vdpa_set_mem_table(struct vhost_dev *dev,
static int vhost_vdpa_set_features(struct vhost_dev *dev,
uint64_t features)
{
+ struct vhost_vdpa *v = dev->opaque;
int ret;
if (vhost_vdpa_one_time_request(dev)) {
return 0;
}
+ if (v->shadow_vqs_enabled) {
+ if ((v->acked_features ^ features) == BIT_ULL(VHOST_F_LOG_ALL)) {
+ /*
+ * QEMU is just trying to enable or disable logging. SVQ handles
+ * this sepparately, so no need to forward this.
+ */
+ v->acked_features = features;
+ return 0;
+ }
+
+ v->acked_features = features;
+
+ /* We must not ack _F_LOG if SVQ is enabled */
+ features &= ~BIT_ULL(VHOST_F_LOG_ALL);
+ }
+
trace_vhost_vdpa_set_features(dev, features);
ret = vhost_vdpa_call(dev, VHOST_SET_FEATURES, &features);
if (ret) {
@@ -1200,10 +1227,14 @@ static int vhost_vdpa_set_vring_call(struct vhost_dev
*dev,
static int vhost_vdpa_get_features(struct vhost_dev *dev,
uint64_t *features)
{
- int ret;
+ struct vhost_vdpa *v = dev->opaque;
+ int ret = vhost_vdpa_get_dev_features(dev, features);
+
+ if (ret == 0 && v->shadow_vqs_enabled) {
+ /* Add SVQ logging capabilities */
+ *features |= BIT_ULL(VHOST_F_LOG_ALL);
+ }
- ret = vhost_vdpa_call(dev, VHOST_GET_FEATURES, features);
- trace_vhost_vdpa_get_features(dev, *features);
return ret;
}
diff --git a/include/hw/virtio/vhost-vdpa.h b/include/hw/virtio/vhost-vdpa.h
index ee8e939..a29dbb3 100644
--- a/include/hw/virtio/vhost-vdpa.h
+++ b/include/hw/virtio/vhost-vdpa.h
@@ -30,6 +30,7 @@ typedef struct vhost_vdpa {
bool iotlb_batch_begin_sent;
MemoryListener listener;
struct vhost_vdpa_iova_range iova_range;
+ uint64_t acked_features;
bool shadow_vqs_enabled;
/* IOVA mapping used by the Shadow Virtqueue */
VhostIOVATree *iova_tree;
--
2.7.4
- [PULL V3 05/15] vhost: Add vhost_svq_valid_features to shadow vq, (continued)
- [PULL V3 05/15] vhost: Add vhost_svq_valid_features to shadow vq, Jason Wang, 2022/03/15
- [PULL V3 06/15] virtio: Add vhost_svq_get_vring_addr, Jason Wang, 2022/03/15
- [PULL V3 07/15] vdpa: adapt vhost_ops callbacks to svq, Jason Wang, 2022/03/15
- [PULL V3 08/15] vhost: Shadow virtqueue buffers forwarding, Jason Wang, 2022/03/15
- [PULL V3 09/15] util: Add iova_tree_alloc_map, Jason Wang, 2022/03/15
- [PULL V3 10/15] util: add iova_tree_find_iova, Jason Wang, 2022/03/15
- [PULL V3 12/15] vdpa: Add custom IOTLB translations to SVQ, Jason Wang, 2022/03/15
- [PULL V3 11/15] vhost: Add VhostIOVATree, Jason Wang, 2022/03/15
- [PULL V3 13/15] vdpa: Adapt vhost_vdpa_get_vring_base to SVQ, Jason Wang, 2022/03/15
- [PULL V3 14/15] vdpa: Never set log_base addr if SVQ is enabled, Jason Wang, 2022/03/15
- [PULL V3 15/15] vdpa: Expose VHOST_F_LOG_ALL on SVQ,
Jason Wang <=
- Re: [PULL V3 00/15] Net patches, Peter Maydell, 2022/03/15