Re: [PATCH 3/4] iommu: add qcom_iommu

2017-05-11 Thread Sricharan R
Hi, On 5/11/2017 10:20 PM, Rob Clark wrote: > On Thu, May 11, 2017 at 11:08 AM, Sricharan R > wrote: >> Hi Rob, >> >> >> >>> +static irqreturn_t qcom_iommu_fault(int irq, void *dev) >>> +{ >>> + struct qcom_iommu_ctx *ctx = dev; >>> + u32 fsr, fsynr; >>> +

Re: [PATCH] iova: allow allocation of lowest domain PFN

2017-05-11 Thread Aaron Sierra
- Original Message - > From: "Robin Murphy" > To: "Aaron Sierra" , "Joerg Roedel" > Cc: "iommu" , "Nate Watterson" > > Sent: Thursday, May 11, 2017 12:39:01 PM >

Re: [PATCH] iova: allow allocation of lowest domain PFN

2017-05-11 Thread Aaron Sierra
- Original Message - > From: "Robin Murphy" > To: "Aaron Sierra" , "Joerg Roedel" > Cc: "iommu" , "Nate Watterson" > > Sent: Thursday, May 11, 2017 12:39:01 PM >

Re: [PATCH] iova: allow allocation of lowest domain PFN

2017-05-11 Thread Robin Murphy
Hi Aaron, On 11/05/17 14:36, Aaron Sierra wrote: > I found it was impossible to allocate all of the address space > defined by an IOVA domain. For example, a trivial domain with 2 pages > would cover PFNs 0 (start) and 1 (limit), but attempting to allocate > more than one total PFN would fail

Re: [PATCH 3/4] iommu: add qcom_iommu

2017-05-11 Thread Rob Clark
On Thu, May 11, 2017 at 11:08 AM, Sricharan R wrote: > Hi Rob, > > > >> +static irqreturn_t qcom_iommu_fault(int irq, void *dev) >> +{ >> + struct qcom_iommu_ctx *ctx = dev; >> + u32 fsr, fsynr; >> + unsigned long iova; >> + >> + fsr = iommu_readl(ctx,

[PATCH 4.9 044/103] x86/pci-calgary: Fix iommu_free() comparison of unsigned expression >= 0

2017-05-11 Thread Greg Kroah-Hartman
4.9-stable review patch. If anyone has any objections, please let me know. -- From: Nikola Pajkovsky commit 68dee8e2f2cacc54d038394e70d22411dee89da2 upstream. commit 8fd524b355da ("x86: Kill bad_dma_address variable") has killed bad_dma_address variable

[PATCH] iova: allow allocation of lowest domain PFN

2017-05-11 Thread Aaron Sierra
I found it was impossible to allocate all of the address space defined by an IOVA domain. For example, a trivial domain with 2 pages would cover PFNs 0 (start) and 1 (limit), but attempting to allocate more than one total PFN would fail because the start PFN could never be allocated. This adds a

[PATCH 3.18 12/39] x86/pci-calgary: Fix iommu_free() comparison of unsigned expression >= 0

2017-05-11 Thread Greg Kroah-Hartman
3.18-stable review patch. If anyone has any objections, please let me know. -- From: Nikola Pajkovsky commit 68dee8e2f2cacc54d038394e70d22411dee89da2 upstream. commit 8fd524b355da ("x86: Kill bad_dma_address variable") has killed bad_dma_address variable

[PATCH] iommu/mediatek: include linux/dma-mapping.h

2017-05-11 Thread Arnd Bergmann
The mediatek iommu driver relied on an implicit include of dma-mapping.h, but for some reason that is no longer there in 4.12-rc1: drivers/iommu/mtk_iommu_v1.c: In function 'mtk_iommu_domain_finalise': drivers/iommu/mtk_iommu_v1.c:233:16: error: implicit declaration of function

Re: [PATCH 1/2] PCI: Save properties required to handle FLR for replay purposes.

2017-05-11 Thread Jean-Philippe Brucker
Hi, On 10/05/17 19:39, Ashok Raj wrote: > From: CQ Tang > > Requires: https://patchwork.kernel.org/patch/9593891 Since your series is likely to go in much earlier than my SVM mess, maybe you could carry that PCI patch along with it? Or I could resend it on its own if you

Re: [v5 1/4] ACPICA: IORT: Add Cavium ThunderX2 SMMUv3 model definition.

2017-05-11 Thread Will Deacon
On Thu, May 11, 2017 at 02:26:02AM +0200, Rafael J. Wysocki wrote: > On Wednesday, May 10, 2017 05:01:55 PM Geetha sowjanya wrote: > > From: Linu Cherian > > > > Add SMMUv3 model definition for ThunderX2. > > > > Signed-off-by: Linu Cherian > >

Re: [PATCH] iommu: remove unnecessary code

2017-05-11 Thread David Woodhouse
On Wed, 2017-05-10 at 22:49 -0500, Gustavo A. R. Silva wrote: > did_old is an unsigned variable and, greater-than-or-equal-to-zero > comparison of an unsigned variable is always true. > > Addresses-Coverity-ID: 1398477 > Signed-off-by: Gustavo A. R. Silva So... why do

[PATCH] iommu: remove unnecessary code

2017-05-11 Thread Gustavo A. R. Silva
did_old is an unsigned variable and, greater-than-or-equal-to-zero comparison of an unsigned variable is always true. Addresses-Coverity-ID: 1398477 Signed-off-by: Gustavo A. R. Silva --- drivers/iommu/intel-iommu.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)