| Message ID | 20230923170540.1447301-6-lulu@redhat.com (mailing list archive) |
|---|---|
| State | RFC |
| Headers | show |
| Series | vdpa: Add support for iommufd | expand |
| Context | Check | Description |
|---|---|---|
| netdev/tree_selection | success | Not a local patch |
On Sun, Sep 24, 2023 at 1:06 AM Cindy Lu <lulu@redhat.com> wrote: > > Add new vdpa_config_ops to support iommufd > > Signed-off-by: Cindy Lu <lulu@redhat.com> > --- > include/linux/vdpa.h | 34 +++++++++++++++++++++++++++++++++- > 1 file changed, 33 insertions(+), 1 deletion(-) > > diff --git a/include/linux/vdpa.h b/include/linux/vdpa.h > index 6d0f5e4e82c2..4ada5bd6f90e 100644 > --- a/include/linux/vdpa.h > +++ b/include/linux/vdpa.h > @@ -5,6 +5,7 @@ > #include <linux/kernel.h> > #include <linux/device.h> > #include <linux/interrupt.h> > +#include <linux/iommufd.h> > #include <linux/vhost_iotlb.h> > #include <linux/virtio_net.h> > #include <linux/if_ether.h> > @@ -91,6 +92,12 @@ struct vdpa_device { > struct vdpa_mgmt_dev *mdev; > unsigned int ngroups; > unsigned int nas; > + struct iommufd_access *iommufd_access; > + struct iommufd_device *iommufd_device; > + struct iommufd_ctx *iommufd_ictx; > + unsigned long *vq_bitmap; > + atomic_t iommufd_users; > + bool iommufd_attached; > }; > > /** > @@ -282,6 +289,15 @@ struct vdpa_map_file { > * @iova: iova to be unmapped > * @size: size of the area > * Returns integer: success (0) or error (< 0) > + * @bind_iommufd: use vdpa_iommufd_physical_bind for an IOMMU > + * backed device. > + * otherwise use vdpa_iommufd_emulated_bind > + * @unbind_iommufd: use vdpa_iommufd_physical_unbind for an IOMMU > + * backed device. > + * otherwise, use vdpa_iommufd_emulated_unbind > + * @attach_ioas: use vdpa_iommufd_physical_attach_ioas for an > + * IOMMU backed device. > + * @detach_ioas: Opposite of attach_ioas Those should be marked as mandatory only for parents with specific translations (e.g simulator and mlx5_vdpa). Or anything I missed? Thanks > * @free: Free resources that belongs to vDPA (optional) > * @vdev: vdpa device > */ > @@ -341,6 +357,12 @@ struct vdpa_config_ops { > u64 iova, u64 size); > int (*set_group_asid)(struct vdpa_device *vdev, unsigned int group, > unsigned int asid); > + /* IOMMUFD ops */ > + int (*bind_iommufd)(struct vdpa_device *vdev, struct iommufd_ctx *ictx, > + u32 *out_device_id); > + void (*unbind_iommufd)(struct vdpa_device *vdev); > + int (*attach_ioas)(struct vdpa_device *vdev, u32 *pt_id); > + int (*detach_ioas)(struct vdpa_device *vdev); > > /* Free device resources */ > void (*free)(struct vdpa_device *vdev); > @@ -510,4 +532,14 @@ struct vdpa_mgmt_dev { > int vdpa_mgmtdev_register(struct vdpa_mgmt_dev *mdev); > void vdpa_mgmtdev_unregister(struct vdpa_mgmt_dev *mdev); > > -#endif /* _LINUX_VDPA_H */ > +int vdpa_iommufd_physical_bind(struct vdpa_device *vdpa, > + struct iommufd_ctx *ictx, u32 *out_device_id); > +void vdpa_iommufd_physical_unbind(struct vdpa_device *vdpa); > +int vdpa_iommufd_physical_attach_ioas(struct vdpa_device *vdpa, u32 *pt_id); > +int vdpa_iommufd_emulated_bind(struct vdpa_device *vdpa, > + struct iommufd_ctx *ictx, u32 *out_device_id); > +void vdpa_iommufd_emulated_unbind(struct vdpa_device *vdpa); > +int vdpa_iommufd_emulated_attach_ioas(struct vdpa_device *vdpa, u32 *pt_id); > +int vdpa_iommufd_emulated_detach_ioas(struct vdpa_device *vdpa); > + > +#endif > -- > 2.34.3 >
diff --git a/include/linux/vdpa.h b/include/linux/vdpa.h index 6d0f5e4e82c2..4ada5bd6f90e 100644 --- a/include/linux/vdpa.h +++ b/include/linux/vdpa.h @@ -5,6 +5,7 @@ #include <linux/kernel.h> #include <linux/device.h> #include <linux/interrupt.h> +#include <linux/iommufd.h> #include <linux/vhost_iotlb.h> #include <linux/virtio_net.h> #include <linux/if_ether.h> @@ -91,6 +92,12 @@ struct vdpa_device { struct vdpa_mgmt_dev *mdev; unsigned int ngroups; unsigned int nas; + struct iommufd_access *iommufd_access; + struct iommufd_device *iommufd_device; + struct iommufd_ctx *iommufd_ictx; + unsigned long *vq_bitmap; + atomic_t iommufd_users; + bool iommufd_attached; }; /** @@ -282,6 +289,15 @@ struct vdpa_map_file { * @iova: iova to be unmapped * @size: size of the area * Returns integer: success (0) or error (< 0) + * @bind_iommufd: use vdpa_iommufd_physical_bind for an IOMMU + * backed device. + * otherwise use vdpa_iommufd_emulated_bind + * @unbind_iommufd: use vdpa_iommufd_physical_unbind for an IOMMU + * backed device. + * otherwise, use vdpa_iommufd_emulated_unbind + * @attach_ioas: use vdpa_iommufd_physical_attach_ioas for an + * IOMMU backed device. + * @detach_ioas: Opposite of attach_ioas * @free: Free resources that belongs to vDPA (optional) * @vdev: vdpa device */ @@ -341,6 +357,12 @@ struct vdpa_config_ops { u64 iova, u64 size); int (*set_group_asid)(struct vdpa_device *vdev, unsigned int group, unsigned int asid); + /* IOMMUFD ops */ + int (*bind_iommufd)(struct vdpa_device *vdev, struct iommufd_ctx *ictx, + u32 *out_device_id); + void (*unbind_iommufd)(struct vdpa_device *vdev); + int (*attach_ioas)(struct vdpa_device *vdev, u32 *pt_id); + int (*detach_ioas)(struct vdpa_device *vdev); /* Free device resources */ void (*free)(struct vdpa_device *vdev); @@ -510,4 +532,14 @@ struct vdpa_mgmt_dev { int vdpa_mgmtdev_register(struct vdpa_mgmt_dev *mdev); void vdpa_mgmtdev_unregister(struct vdpa_mgmt_dev *mdev); -#endif /* _LINUX_VDPA_H */ +int vdpa_iommufd_physical_bind(struct vdpa_device *vdpa, + struct iommufd_ctx *ictx, u32 *out_device_id); +void vdpa_iommufd_physical_unbind(struct vdpa_device *vdpa); +int vdpa_iommufd_physical_attach_ioas(struct vdpa_device *vdpa, u32 *pt_id); +int vdpa_iommufd_emulated_bind(struct vdpa_device *vdpa, + struct iommufd_ctx *ictx, u32 *out_device_id); +void vdpa_iommufd_emulated_unbind(struct vdpa_device *vdpa); +int vdpa_iommufd_emulated_attach_ioas(struct vdpa_device *vdpa, u32 *pt_id); +int vdpa_iommufd_emulated_detach_ioas(struct vdpa_device *vdpa); + +#endif
Add new vdpa_config_ops to support iommufd Signed-off-by: Cindy Lu <lulu@redhat.com> --- include/linux/vdpa.h | 34 +++++++++++++++++++++++++++++++++- 1 file changed, 33 insertions(+), 1 deletion(-)