drm/etnaviv: wire up iova handling in new cmdbuf abstraction
authorLucas Stach <l.stach@pengutronix.de>
Mon, 16 Jan 2017 15:52:44 +0000 (16:52 +0100)
committerLucas Stach <l.stach@pengutronix.de>
Thu, 2 Feb 2017 09:30:20 +0000 (10:30 +0100)
Don't call the IOMMU directly, but go through the new cmdbuf abstraction.

Signed-off-by: Lucas Stach <l.stach@pengutronix.de>
Reviewed-by: Christian Gmeiner <christian.gmeiner@gmail.com>
drivers/gpu/drm/etnaviv/etnaviv_buffer.c
drivers/gpu/drm/etnaviv/etnaviv_cmdbuf.c
drivers/gpu/drm/etnaviv/etnaviv_cmdbuf.h
drivers/gpu/drm/etnaviv/etnaviv_dump.c
drivers/gpu/drm/etnaviv/etnaviv_gpu.c

index 9813b9b33553575c1975735ecd8518ce18db9684..ed9588f36bc9b4a6214eca1e2954f520a0ce8ef4 100644 (file)
@@ -126,7 +126,7 @@ static void etnaviv_buffer_dump(struct etnaviv_gpu *gpu,
        u32 *ptr = buf->vaddr + off;
 
        dev_info(gpu->dev, "virt %p phys 0x%08x free 0x%08x\n",
-                       ptr, etnaviv_iommu_get_cmdbuf_va(gpu, buf) + off, size - len * 4 - off);
+                       ptr, etnaviv_cmdbuf_get_va(buf) + off, size - len * 4 - off);
 
        print_hex_dump(KERN_INFO, "cmd ", DUMP_PREFIX_OFFSET, 16, 4,
                        ptr, len * 4, 0);
@@ -159,7 +159,7 @@ static u32 etnaviv_buffer_reserve(struct etnaviv_gpu *gpu,
        if (buffer->user_size + cmd_dwords * sizeof(u64) > buffer->size)
                buffer->user_size = 0;
 
-       return etnaviv_iommu_get_cmdbuf_va(gpu, buffer) + buffer->user_size;
+       return etnaviv_cmdbuf_get_va(buffer) + buffer->user_size;
 }
 
 u16 etnaviv_buffer_init(struct etnaviv_gpu *gpu)
@@ -170,7 +170,7 @@ u16 etnaviv_buffer_init(struct etnaviv_gpu *gpu)
        buffer->user_size = 0;
 
        CMD_WAIT(buffer);
-       CMD_LINK(buffer, 2, etnaviv_iommu_get_cmdbuf_va(gpu, buffer) +
+       CMD_LINK(buffer, 2, etnaviv_cmdbuf_get_va(buffer) +
                 buffer->user_size - 4);
 
        return buffer->user_size / 8;
@@ -262,7 +262,7 @@ void etnaviv_buffer_queue(struct etnaviv_gpu *gpu, unsigned int event,
        if (drm_debug & DRM_UT_DRIVER)
                etnaviv_buffer_dump(gpu, buffer, 0, 0x50);
 
-       link_target = etnaviv_iommu_get_cmdbuf_va(gpu, cmdbuf);
+       link_target = etnaviv_cmdbuf_get_va(cmdbuf);
        link_dwords = cmdbuf->size / 8;
 
        /*
@@ -356,12 +356,13 @@ void etnaviv_buffer_queue(struct etnaviv_gpu *gpu, unsigned int event,
        CMD_LOAD_STATE(buffer, VIVS_GL_EVENT, VIVS_GL_EVENT_EVENT_ID(event) |
                       VIVS_GL_EVENT_FROM_PE);
        CMD_WAIT(buffer);
-       CMD_LINK(buffer, 2, etnaviv_iommu_get_cmdbuf_va(gpu, buffer) +
+       CMD_LINK(buffer, 2, etnaviv_cmdbuf_get_va(buffer) +
                            buffer->user_size - 4);
 
        if (drm_debug & DRM_UT_DRIVER)
                pr_info("stream link to 0x%08x @ 0x%08x %p\n",
-                       return_target, etnaviv_iommu_get_cmdbuf_va(gpu, cmdbuf), cmdbuf->vaddr);
+                       return_target, etnaviv_cmdbuf_get_va(cmdbuf),
+                       cmdbuf->vaddr);
 
        if (drm_debug & DRM_UT_DRIVER) {
                print_hex_dump(KERN_INFO, "cmd ", DUMP_PREFIX_OFFSET, 16, 4,
index 08f01ce63ff6f815fd8317857dcebe435ddd6930..155b1fddca8e6466b678c7c82886ecdfb91dd389 100644 (file)
@@ -52,3 +52,8 @@ void etnaviv_cmdbuf_free(struct etnaviv_cmdbuf *cmdbuf)
                    cmdbuf->paddr);
        kfree(cmdbuf);
 }
+
+u32 etnaviv_cmdbuf_get_va(struct etnaviv_cmdbuf *buf)
+{
+       return etnaviv_iommu_get_cmdbuf_va(buf->gpu, buf);
+}
index 883f03960f95fc5a9e261f0eee2c79c67852ab61..6d64685cd3e7aad50d2e6622180d70533a9b2027 100644 (file)
@@ -43,4 +43,6 @@ struct etnaviv_cmdbuf {
        struct etnaviv_vram_mapping *bo_map[0];
 };
 
+u32 etnaviv_cmdbuf_get_va(struct etnaviv_cmdbuf *buf);
+
 #endif /* __ETNAVIV_CMDBUF_H__ */
index 6a2ff6b77f2a01e99656f932792198730a4b31e3..d019b5e311cc5502e2d273525ea4144f5d0a796d 100644 (file)
@@ -178,12 +178,11 @@ void etnaviv_core_dump(struct etnaviv_gpu *gpu)
        etnaviv_core_dump_mmu(&iter, gpu, mmu_size);
        etnaviv_core_dump_mem(&iter, ETDUMP_BUF_RING, gpu->buffer->vaddr,
                              gpu->buffer->size,
-                             etnaviv_iommu_get_cmdbuf_va(gpu, gpu->buffer));
+                             etnaviv_cmdbuf_get_va(gpu->buffer));
 
        list_for_each_entry(cmd, &gpu->active_cmd_list, node)
                etnaviv_core_dump_mem(&iter, ETDUMP_BUF_CMD, cmd->vaddr,
-                                     cmd->size,
-                                     etnaviv_iommu_get_cmdbuf_va(gpu, cmd));
+                                     cmd->size, etnaviv_cmdbuf_get_va(cmd));
 
        /* Reserve space for the bomap */
        if (n_bomap_pages) {
index ee8aacc289440ecf85c232801da5e040bc808bd4..3a689c3fbe5b687288fb782327ff8a93d7e12139 100644 (file)
@@ -629,7 +629,7 @@ static void etnaviv_gpu_hw_init(struct etnaviv_gpu *gpu)
        prefetch = etnaviv_buffer_init(gpu);
 
        gpu_write(gpu, VIVS_HI_INTR_ENBL, ~0U);
-       etnaviv_gpu_start_fe(gpu, etnaviv_iommu_get_cmdbuf_va(gpu, gpu->buffer),
+       etnaviv_gpu_start_fe(gpu, etnaviv_cmdbuf_get_va(gpu->buffer),
                             prefetch);
 }
 
@@ -703,7 +703,7 @@ int etnaviv_gpu_init(struct etnaviv_gpu *gpu)
        }
 
        if (gpu->mmu->version == ETNAVIV_IOMMU_V1 &&
-           gpu->buffer->paddr - gpu->memory_base > 0x80000000) {
+           etnaviv_cmdbuf_get_va(gpu->buffer) > 0x80000000) {
                ret = -EINVAL;
                dev_err(gpu->dev,
                        "command buffer outside valid memory window\n");