Re: [PATCH v6 0/5] Define EINVAL as device/domain incompatibility

From: Joerg Roedel
Date: Mon Sep 26 2022 - 10:12:43 EST


Hi Nicolin,

On Fri, Sep 23, 2022 at 12:16:29AM -0700, Nicolin Chen wrote:
> This series is to replace the previous EMEDIUMTYPE patch in a VFIO series:
> https://lore.kernel.org/kvm/Yxnt9uQTmbqul5lf@xxxxxxxxxx/

\o/

> Nicolin Chen (5):
> iommu/amd: Drop unnecessary checks in amd_iommu_attach_device()
> iommu: Add return value rules to attach_dev op and APIs
> iommu: Regulate EINVAL in ->attach_dev callback functions
> iommu: Use EINVAL for incompatible device/domain in ->attach_dev
> iommu: Propagate return value in ->attach_dev callback functions

This looks good to me, but I'd like to have the SMMU people have a look
at patch 4. And I think it is too late for this to make it into 6.1, so
please re-send after 6.1-rc1 is out.

Thanks,

Joerg