vdpa-net: improve migration downtime through descriptor ASID and persistent IOTLB

[PATCH 15/40] vdpa: add svq_switching and flush_map to header

Posted by Si-Wei Liu 11 months, 3 weeks ago

Will be used in next patches.

Signed-off-by: Si-Wei Liu <si-wei.liu@oracle.com>
---
 include/hw/virtio/vhost-vdpa.h | 6 ++++++
 1 file changed, 6 insertions(+)

diff --git a/include/hw/virtio/vhost-vdpa.h b/include/hw/virtio/vhost-vdpa.h
index 7b8d3bf..0fe0f60 100644
--- a/include/hw/virtio/vhost-vdpa.h
+++ b/include/hw/virtio/vhost-vdpa.h
@@ -72,6 +72,12 @@ typedef struct vhost_vdpa_shared {
     bool shadow_data;
 
     unsigned refcnt;
+
+    /* SVQ switching is in progress? 1: turn on SVQ, -1: turn off SVQ */
+    int svq_switching;
+
+    /* Flush mappings on reset due to shared address space */
+    bool flush_map;
 } VhostVDPAShared;
 
 typedef struct vhost_vdpa {
-- 
1.8.3.1

Re: [PATCH 15/40] vdpa: add svq_switching and flush_map to header

Posted by Jason Wang 10 months, 2 weeks ago

On Fri, Dec 8, 2023 at 2:50 AM Si-Wei Liu <si-wei.liu@oracle.com> wrote:
>
> Will be used in next patches.
>
> Signed-off-by: Si-Wei Liu <si-wei.liu@oracle.com>
> ---
>  include/hw/virtio/vhost-vdpa.h | 6 ++++++
>  1 file changed, 6 insertions(+)
>
> diff --git a/include/hw/virtio/vhost-vdpa.h b/include/hw/virtio/vhost-vdpa.h
> index 7b8d3bf..0fe0f60 100644
> --- a/include/hw/virtio/vhost-vdpa.h
> +++ b/include/hw/virtio/vhost-vdpa.h
> @@ -72,6 +72,12 @@ typedef struct vhost_vdpa_shared {
>      bool shadow_data;
>
>      unsigned refcnt;
> +
> +    /* SVQ switching is in progress? 1: turn on SVQ, -1: turn off SVQ */
> +    int svq_switching;

Nit: just curious about any reason why 0, 1 or true false is not used?

Thanks

> +
> +    /* Flush mappings on reset due to shared address space */
> +    bool flush_map;
>  } VhostVDPAShared;
>
>  typedef struct vhost_vdpa {
> --
> 1.8.3.1
>