[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <ec1c5a6c-53d3-46e5-8f6f-e12ee49dc0b9@linux.intel.com>
Date: Thu, 5 Feb 2026 16:34:33 +0800
From: Baolu Lu <baolu.lu@...ux.intel.com>
To: Viktor Kleen <viktor@...en.org>
Cc: David Woodhouse <dwmw2@...radead.org>, Joerg Roedel <joro@...tes.org>,
Will Deacon <will@...nel.org>, Robin Murphy <robin.murphy@....com>,
Kevin Tian <kevin.tian@...el.com>, Jason Gunthorpe <jgg@...pe.ca>,
iommu@...ts.linux.dev, linux-kernel@...r.kernel.org
Subject: Re: [PATCH] iommu/vt-d: Treat PAGE_SNOOP and PWSNP separately
On 2/3/26 03:21, Viktor Kleen wrote:
> The PASID_FLAG_PAGE_SNOOP and PASID_FLAG_PWSNP constants are identical.
> This will cause the pasid code to always set both or neither of the
> PGSNP and PWSNP bits in PASID table entries. However, PWSNP is a
> reserved bit if SMPWC is not set in the IOMMU's extended capability
> register, even if SC is supported.
>
> This has resulted in DMAR errors when testing the iommufd code on an
> Arrow Lake platform. With this patch, those errors disappear and the
> PASID table entries look correct.
>
> Fixes: 101a2854110fa ("iommu/vt-d: Follow PT_FEAT_DMA_INCOHERENT into the PASID entry")
> Signed-off-by: Viktor Kleen<viktor@...en.org>
> ---
> drivers/iommu/intel/pasid.h | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
Queued for iommu/vt-d.
Thanks,
baolu
Powered by blists - more mailing lists