On 14/02/15 08:03, Yong Wu wrote:
On Tue, 2015-02-10 at 12:07 +0000, Robin Murphy wrote:
On 10/02/15 04:39, Yingjoe Chen wrote:
On Fri, 2015-02-06 at 14:55 +0000, Robin Murphy wrote
<...>
diff --git a/arch/arm64/include/asm/dma-mapping.h
b/arch/arm64/include/asm/dma-mapping.h
index 6932bb5..c1b271f 100644
--- a/arch/arm64/include/asm/dma-mapping.h
+++ b/arch/arm64/include/asm/dma-mapping.h
@@ -62,13 +62,30 @@ static inline bool is_device_dma_coherent(struct device
*dev)
#include <asm-generic/dma-mapping-common.h>
+#ifdef CONFIG_IOMMU_DMA
+static inline struct iommu_dma_domain *get_dma_domain(struct device *dev)
+{
+ return dev->archdata.dma_domain;
+}
+
+static inline void set_dma_domain(struct device *dev,
+ struct iommu_dma_domain *dma_domain)
+{
+ dev->archdata.dma_domain = dma_domain;
+}
+#endif
+
static inline dma_addr_t phys_to_dma(struct device *dev, phys_addr_t paddr)
{
+ if (WARN_ON(dev && get_dma_domain(dev)))
+ return DMA_ERROR_CODE;
return (dma_addr_t)paddr;
}
Hi Robin,
Build fail if CONFIG_IOMMU_DMA is not enabled.
In file included from ../include/linux/dma-mapping.h:82:0,
from ../arch/arm64/kernel/asm-offsets.c:23:
../arch/arm64/include/asm/dma-mapping.h: In function 'phys_to_dma':
../arch/arm64/include/asm/dma-mapping.h:81:2: error: implicit declaration of
function 'get_dma_domain' [-Werror=implicit-function-declaration]
if (WARN_ON(dev && get_dma_domain(dev)))
^
Joe.C
Bah, how did I manage to make such a half-finished mess of the includes?
Current fixup diff below.
Thanks,
Robin.
Dear Robin,
We have test this patch on our mt8173, it also could work well.
Tested-by: Yong Wu <yong...@mediatek.com>
And I have 2 questiones about how to use this iommu.
1)if We call "arch_setup_dma_ops" to create a iommu domain, then is
this function "bus_set_iommu" who also need "struct iommu_ops *"
necessary to be called or not in your design?
I've intentionally bypassed bus_set_iommu() because it doesn't
necessarily work well for platform devices - I've got 7 separate IOMMUs
here and I need to know which is which, so the notion of "the IOMMU for
the platform bus" doesn't make a whole lot of sense. Thus
iommu_dma_create_domain() creates the iommu_domain directly using the
iommu_ops provided, because...
2)int (*domain_init)(struct iommu_domain *domain);
About this function, it will alloc pagetable for the iommu
domain. And We expect the pagetable memory is uncacheable, so try to
call "dma_alloc_coherent", unfortunately the "struct device *" can't be
passed into this function. so is it possible if adding a parameter in
this function.
...one of the ideas of the new of_iommu_configure framework is that
iommu_ops structures can represent individual IOMMU devices if
necessary. The ops->priv pointer was included for that purpose, but
isn't very clean so Will has plans to remove it again - it's easy enough
to achieve the same effect by having the driver embed the ops in its
private instance data instead. I've done that with the ARM SMMU driver
which has a similar issue of needing hardware details at domain_init()
time (no patches ready yet but I have an iommu/dev branch on top of the
iommu/dma branch with some current work-in-progress bits)
Thanks,
Robin.
_______________________________________________
iommu mailing list
iommu@lists.linux-foundation.org
https://lists.linuxfoundation.org/mailman/listinfo/iommu
