Re: [PATCH 1/1] iommu/vt-d: Unconditionally flush device TLB for pasid table updates

[Baolu Lu]

On 8/20/24 11:02 AM, Lu Baolu wrote:
> The caching mode of an IOMMU is irrelevant to the behavior of the device
> TLB. Previously, commit <304b3bde24b5> ("iommu/vt-d: Remove caching mode
> check before device TLB flush") removed this redundant check in the
> domain unmap path.
>
> Checking the caching mode before flushing the device TLB after a pasid
> table entry is updated is unnecessary and can lead to inconsistent
> behavior.
>
> Extends this consistency by removing the caching mode check in the pasid
> table update path.
>
> Suggested-by: Yi Liu<yi.l.liu@xxxxxxxxx>
> Signed-off-by: Lu Baolu<baolu.lu@xxxxxxxxxxxxxxx>
> ---
>   drivers/iommu/intel/pasid.c | 12 +++---------
>   1 file changed, 3 insertions(+), 9 deletions(-)

Queued for v6.12-rc1.

Thanks,
baolu