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.t...@intel.com> > Signed-off-by: Jean-Philippe Brucker <jean-phili...@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; > }; > _______________________________________________ iommu mailing list iommu@lists.linux-foundation.org https://lists.linuxfoundation.org/mailman/listinfo/iommu