On Wed, Oct 27, 2021 at 12:44:20PM +0200, Eric Auger wrote:
> Signed-off-by: Jean-Philippe Brucker <[email protected]>
> Signed-off-by: Liu, Yi L <[email protected]>
> Signed-off-by: Ashok Raj <[email protected]>
> Signed-off-by: Jacob Pan <[email protected]>
> Signed-off-by: Eric Auger <[email protected]>

This Signed-of-by chain looks dubious, you are the author but the last
one in the chain?

> +int iommu_uapi_attach_pasid_table(struct iommu_domain *domain,
> +                               void __user *uinfo)
> +{

[...]

> +     if (pasid_table_data.format == IOMMU_PASID_FORMAT_SMMUV3 &&
> +         pasid_table_data.argsz <
> +             offsetofend(struct iommu_pasid_table_config, 
> vendor_data.smmuv3))
> +             return -EINVAL;

This check looks like it belongs in driver specific code.

Regards,

        Joerg

_______________________________________________
kvmarm mailing list
[email protected]
https://lists.cs.columbia.edu/mailman/listinfo/kvmarm

Reply via email to