Re: [PATCH v2 1/1] swiotlb: rework "fix info leak with DMA_FROM_DEVICE"

2022-03-06 Thread Christoph Hellwig
On Sun, Mar 06, 2022 at 12:30:55PM -0800, Linus Torvalds wrote: > So I would expect that > > (a) READ/WRITE actually fills the whole buffer > > (b) READ/WRITE are the only ones where we care about performance at a > bounce-buffer level > > so it boils down to "do we still do this horrible memc

RE: [PATCH] iommu/iova: Free all CPU rcache for retry when iova alloc failure

2022-03-06 Thread yf.wang--- via iommu
On Fri, 2022-03-04 at 14:03 +, Robin Murphy wrote: > > OK, so either there's a mystery bug where IOVAs somehow get freed on > offline CPUs, or the hotplug notifier isn't working correctly, or > you've > contrived a situation where alloc_iova_fast() is actually racing > against > iova_cpuhp_

RE: [PATCH] iommu/iova: Free all CPU rcache for retry when iova alloc failure

2022-03-06 Thread yf.wang--- via iommu
On Fri, 2022-03-04 at 14:03 +, Robin Murphy wrote: > > OK, so either there's a mystery bug where IOVAs somehow get freed on > offline CPUs, or the hotplug notifier isn't working correctly, or > you've > contrived a situation where alloc_iova_fast() is actually racing > against > iova_cpuhp_

Re: [PATCH v7 01/11] iommu: Add DMA ownership management interfaces

2022-03-06 Thread Lu Baolu
Hi Robin, On 3/4/22 10:10 PM, Robin Murphy wrote: On 2022-03-04 13:55, Eric Auger wrote: Hi Robin, On 3/4/22 1:22 PM, Robin Murphy wrote: On 2022-03-04 10:43, Lu Baolu wrote: Hi Eric, On 2022/3/4 18:34, Eric Auger wrote: I hit a WARN_ON() when unbinding an e1000e driver just after boot: s

RE: [PATCH] iommu/iova: Free all CPU rcache for retry when iova alloc failure

2022-03-06 Thread yf.wang--- via iommu
On 2022-03-04 9:22, John Garry wrote: > On 04/03/2022 04:46, yf.wang--- via iommu wrote: > > * MEDIATEK Confidentiality Notice > > The > > information contained in this e-mail message (including any > > attachments) may be confidential, proprietary, privileged, or

Re: [PATCH v2 1/1] swiotlb: rework "fix info leak with DMA_FROM_DEVICE"

2022-03-06 Thread Linus Torvalds
On Sat, Mar 5, 2022 at 11:58 PM Christoph Hellwig wrote: > > All of thee apply to all I/O. SG_IO is just a vector here as a > particularly badly designed userspace interface that happens to get a > decent test coverage. I phrased that badly. I didn't mean SG_IO in particular, I meant any of the

RE: [PATCH 10/12] swiotlb: add a SWIOTLB_ANY flag to lift the low memory restriction

2022-03-06 Thread Michael Kelley (LINUX) via iommu
From: Dongli Zhang Sent: Friday, March 4, 2022 10:28 AM > > Hi Michael, > > On 3/4/22 10:12 AM, Michael Kelley (LINUX) wrote: > > From: Christoph Hellwig Sent: Tuesday, March 1, 2022 2:53 AM > >> > >> Power SVM wants to allocate a swiotlb buffer that is not restricted to low > >> memory for >