[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20260121130315.GE1134360@nvidia.com>
Date: Wed, 21 Jan 2026 09:03:15 -0400
From: Jason Gunthorpe <jgg@...dia.com>
To: Jonathan Cameron <jonathan.cameron@...wei.com>
Cc: "Tian, Kevin" <kevin.tian@...el.com>,
Nicolin Chen <nicolinc@...dia.com>,
"will@...nel.org" <will@...nel.org>,
"robin.murphy@....com" <robin.murphy@....com>,
"bhelgaas@...gle.com" <bhelgaas@...gle.com>,
"Williams, Dan J" <dan.j.williams@...el.com>,
"joro@...tes.org" <joro@...tes.org>,
"praan@...gle.com" <praan@...gle.com>,
"baolu.lu@...ux.intel.com" <baolu.lu@...ux.intel.com>,
"miko.lenczewski@....com" <miko.lenczewski@....com>,
"linux-arm-kernel@...ts.infradead.org" <linux-arm-kernel@...ts.infradead.org>,
"iommu@...ts.linux.dev" <iommu@...ts.linux.dev>,
"linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
"linux-pci@...r.kernel.org" <linux-pci@...r.kernel.org>,
linux-cxl@...r.kernel.org
Subject: Re: [PATCH RFCv1 1/3] PCI: Allow ATS to be always on for CXL.cache
capable devices
On Wed, Jan 21, 2026 at 10:03:07AM +0000, Jonathan Cameron wrote:
> On Wed, 21 Jan 2026 08:01:36 +0000
> "Tian, Kevin" <kevin.tian@...el.com> wrote:
>
> > +Dan. I recalled an offline discussion in which he raised concern on
> > having the kernel blindly enable ATS for cxl.cache device instead of
> > creating a knob for admin to configure from userspace (in case
> > security is viewed more important than functionality, upon allowing
> > DMA to read data out of CPU caches)...
> >
>
> +CC Linux-cxl
A cxl.cache device supporting ATS will automatically enable ATS today
if the kernel option to enable translation is set.
Even if the device is marked untrusted by the PCI layer (eg an
external port).
Yes this is effectively a security issue, but it is not really a CXL
specific problem.
We might perfer to not enable ATS for untrusted devices and then fail to
load drivers for "ats always on" cases.
Or maybe we can enable one of the ATS security features someday,
though I wonder if those work for CXL..
Jason
Powered by blists - more mailing lists