From: Joerg Roedel <jroe...@suse.de>

The map and unmap functions of the IOMMU-API changed their
semantics: They do no longer guarantee that the hardware
TLBs are synchronized with the page-table updates they made.

To make conversion easier, new synchronized functions have
been introduced which give these guarantees again until the
code is converted to use the new TLB-flush interface of the
IOMMU-API, which allows certain optimizations.

But for now, just convert this code to use the synchronized
functions so that it will behave as before.

Cc: Thierry Reding <thierry.red...@gmail.com>
Cc: David Airlie <airl...@linux.ie>
Cc: Jonathan Hunter <jonath...@nvidia.com>
Cc: dri-devel@lists.freedesktop.org
Cc: linux-te...@vger.kernel.org
Signed-off-by: Joerg Roedel <jroe...@suse.de>
---
 drivers/gpu/drm/tegra/drm.c | 6 +++---
 drivers/gpu/drm/tegra/gem.c | 6 +++---
 2 files changed, 6 insertions(+), 6 deletions(-)

diff --git a/drivers/gpu/drm/tegra/drm.c b/drivers/gpu/drm/tegra/drm.c
index 518f4b6..bc4528ee 100644
--- a/drivers/gpu/drm/tegra/drm.c
+++ b/drivers/gpu/drm/tegra/drm.c
@@ -1156,8 +1156,8 @@ void *tegra_drm_alloc(struct tegra_drm *tegra, size_t 
size,
        }
 
        *dma = iova_dma_addr(&tegra->carveout.domain, alloc);
-       err = iommu_map(tegra->domain, *dma, virt_to_phys(virt),
-                       size, IOMMU_READ | IOMMU_WRITE);
+       err = iommu_map_sync(tegra->domain, *dma, virt_to_phys(virt),
+                            size, IOMMU_READ | IOMMU_WRITE);
        if (err < 0)
                goto free_iova;
 
@@ -1180,7 +1180,7 @@ void tegra_drm_free(struct tegra_drm *tegra, size_t size, 
void *virt,
                size = PAGE_ALIGN(size);
 
        if (tegra->domain) {
-               iommu_unmap(tegra->domain, dma, size);
+               iommu_unmap_sync(tegra->domain, dma, size);
                free_iova(&tegra->carveout.domain,
                          iova_pfn(&tegra->carveout.domain, dma));
        }
diff --git a/drivers/gpu/drm/tegra/gem.c b/drivers/gpu/drm/tegra/gem.c
index 7a39a35..639bc75 100644
--- a/drivers/gpu/drm/tegra/gem.c
+++ b/drivers/gpu/drm/tegra/gem.c
@@ -135,8 +135,8 @@ static int tegra_bo_iommu_map(struct tegra_drm *tegra, 
struct tegra_bo *bo)
 
        bo->paddr = bo->mm->start;
 
-       err = iommu_map_sg(tegra->domain, bo->paddr, bo->sgt->sgl,
-                          bo->sgt->nents, prot);
+       err = iommu_map_sg_sync(tegra->domain, bo->paddr, bo->sgt->sgl,
+                               bo->sgt->nents, prot);
        if (err < 0) {
                dev_err(tegra->drm->dev, "failed to map buffer: %zd\n", err);
                goto remove;
@@ -162,7 +162,7 @@ static int tegra_bo_iommu_unmap(struct tegra_drm *tegra, 
struct tegra_bo *bo)
                return 0;
 
        mutex_lock(&tegra->mm_lock);
-       iommu_unmap(tegra->domain, bo->paddr, bo->size);
+       iommu_unmap_sync(tegra->domain, bo->paddr, bo->size);
        drm_mm_remove_node(bo->mm);
        mutex_unlock(&tegra->mm_lock);
 
-- 
2.7.4

_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel

Reply via email to