Jean-Philippe Brucker
2021-Nov-23 15:52 UTC
[PATCH v2 1/5] iommu/virtio: Add definitions for VIRTIO_IOMMU_F_BYPASS_CONFIG
Add definitions for the VIRTIO_IOMMU_F_BYPASS_CONFIG, which supersedes
VIRTIO_IOMMU_F_BYPASS.
Reviewed-by: Kevin Tian <kevin.tian at intel.com>
Signed-off-by: Jean-Philippe Brucker <jean-philippe at linaro.org>
---
include/uapi/linux/virtio_iommu.h | 8 +++++++-
1 file changed, 7 insertions(+), 1 deletion(-)
diff --git a/include/uapi/linux/virtio_iommu.h
b/include/uapi/linux/virtio_iommu.h
index 237e36a280cb..cafd8cf7febf 100644
--- a/include/uapi/linux/virtio_iommu.h
+++ b/include/uapi/linux/virtio_iommu.h
@@ -16,6 +16,7 @@
#define VIRTIO_IOMMU_F_BYPASS 3
#define VIRTIO_IOMMU_F_PROBE 4
#define VIRTIO_IOMMU_F_MMIO 5
+#define VIRTIO_IOMMU_F_BYPASS_CONFIG 6
struct virtio_iommu_range_64 {
__le64 start;
@@ -36,6 +37,8 @@ struct virtio_iommu_config {
struct virtio_iommu_range_32 domain_range;
/* Probe buffer size */
__le32 probe_size;
+ __u8 bypass;
+ __u8 reserved[7];
};
/* Request types */
@@ -66,11 +69,14 @@ struct virtio_iommu_req_tail {
__u8 reserved[3];
};
+#define VIRTIO_IOMMU_ATTACH_F_BYPASS (1 << 0)
+
struct virtio_iommu_req_attach {
struct virtio_iommu_req_head head;
__le32 domain;
__le32 endpoint;
- __u8 reserved[8];
+ __le32 flags;
+ __u8 reserved[4];
struct virtio_iommu_req_tail tail;
};
--
2.33.1
Eric Auger
2021-Nov-27 07:59 UTC
[PATCH v2 1/5] iommu/virtio: Add definitions for VIRTIO_IOMMU_F_BYPASS_CONFIG
Hi Jean, On 11/23/21 4:52 PM, Jean-Philippe Brucker wrote:> Add definitions for the VIRTIO_IOMMU_F_BYPASS_CONFIG, which supersedes > VIRTIO_IOMMU_F_BYPASS. > > Reviewed-by: Kevin Tian <kevin.tian at intel.com> > Signed-off-by: Jean-Philippe Brucker <jean-philippe at linaro.org> > --- > include/uapi/linux/virtio_iommu.h | 8 +++++++- > 1 file changed, 7 insertions(+), 1 deletion(-) > > diff --git a/include/uapi/linux/virtio_iommu.h b/include/uapi/linux/virtio_iommu.h > index 237e36a280cb..cafd8cf7febf 100644 > --- a/include/uapi/linux/virtio_iommu.h > +++ b/include/uapi/linux/virtio_iommu.h > @@ -16,6 +16,7 @@ > #define VIRTIO_IOMMU_F_BYPASS 3 > #define VIRTIO_IOMMU_F_PROBE 4 > #define VIRTIO_IOMMU_F_MMIO 5 > +#define VIRTIO_IOMMU_F_BYPASS_CONFIG 6 > > struct virtio_iommu_range_64 { > __le64 start; > @@ -36,6 +37,8 @@ struct virtio_iommu_config { > struct virtio_iommu_range_32 domain_range; > /* Probe buffer size */ > __le32 probe_size; > + __u8 bypass; > + __u8 reserved[7];in [PATCH v3] virtio-iommu: Rework the bypass feature I see + u8 bypass; + u8 reserved[3]; What was exactly voted? Thanks Eric> }; > > /* Request types */ > @@ -66,11 +69,14 @@ struct virtio_iommu_req_tail { > __u8 reserved[3]; > }; > > +#define VIRTIO_IOMMU_ATTACH_F_BYPASS (1 << 0) > + > struct virtio_iommu_req_attach { > struct virtio_iommu_req_head head; > __le32 domain; > __le32 endpoint; > - __u8 reserved[8]; > + __le32 flags; > + __u8 reserved[4]; > struct virtio_iommu_req_tail tail; > }; >