Re: [PATCH v8 04/25] iommu/arm-smmu-v3: Move IDR parsing to common functions

On Tue, Sep 22, 2026 at 12:45:20PM -0700, Nicolin Chen wrote:

Also, Jason's suggestion in v7 was hyp_arm_smmu_v3_device, which
looks nicer than ARM_SMMU_OBJ...

I guess the point is to emphasis the substition technique, I don't
mind.

quoted

+	if (coherent && !disable_msipolling &&
+	    smmu->features & ARM_SMMU_FEAT_MSI)
+		smmu->options |= ARM_SMMU_OPT_MSIPOLL;

Will pKVM ever use MSIPOLL?

Not sure I understood the approach in this patch, why was anything
left behind? I imagined everything would be moved?

Maybe a few more remarks in the commit message?

quoted

+	if (smmu->features & ARM_SMMU_FEAT_HYP &&
+	    cpus_have_cap(ARM64_HAS_VIRT_HOST_EXTN))
+		smmu->features |= ARM_SMMU_FEAT_E2H;

Why is ARM64_HAS_VIRT_HOST_EXTN left behind?

+1

And the TT_LE/BE and S1/S2 checks, and ...

If kvm doesn't want to dev_err log that's fine, we should able to
#define that away too shouldn't we? And maybe it does want to log?

This seems pretty close otherwise

Jason