drm/nouveau/core/memory: distinguish between coherent/non-coherent targets

Signed-off-by: Ben Skeggs <bskeggs@redhat.com>

diff --git a/drivers/gpu/drm/nouveau/include/nvkm/core/memory.h b/drivers/gpu/drm/nouveau/include/nvkm/core/memory.h
index 9363b839a9da005f7760ae8733e0856deae83cb5..33ca6769266a5d880228c0c99261244c4f69f4d1 100644 (file)
--- a/drivers/gpu/drm/nouveau/include/nvkm/core/memory.h
+++ b/drivers/gpu/drm/nouveau/include/nvkm/core/memory.h
@@ -6,9 +6,10 @@ struct nvkm_vma;
 struct nvkm_vm;
 
 enum nvkm_memory_target {
-       NVKM_MEM_TARGET_INST,
-       NVKM_MEM_TARGET_VRAM,
-       NVKM_MEM_TARGET_HOST,
+       NVKM_MEM_TARGET_INST, /* instance memory */
+       NVKM_MEM_TARGET_VRAM, /* video memory */
+       NVKM_MEM_TARGET_HOST, /* coherent system memory */
+       NVKM_MEM_TARGET_NCOH, /* non-coherent system memory */
 };
 
 struct nvkm_memory {

diff --git a/drivers/gpu/drm/nouveau/nvkm/engine/fifo/gf100.c b/drivers/gpu/drm/nouveau/nvkm/engine/fifo/gf100.c
index ec68ea9747d523a696d0028f127af0bef41475ac..8772a995329a60f5e05be264b32dc4c39af5d11c 100644 (file)
--- a/drivers/gpu/drm/nouveau/nvkm/engine/fifo/gf100.c
+++ b/drivers/gpu/drm/nouveau/nvkm/engine/fifo/gf100.c
@@ -68,7 +68,14 @@ gf100_fifo_runlist_commit(struct gf100_fifo *fifo)
        }
        nvkm_done(cur);
 
-       target = (nvkm_memory_target(cur) == NVKM_MEM_TARGET_HOST) ? 0x3 : 0x0;
+       switch (nvkm_memory_target(cur)) {
+       case NVKM_MEM_TARGET_VRAM: target = 0; break;
+       case NVKM_MEM_TARGET_NCOH: target = 3; break;
+       default:
+               mutex_unlock(&subdev->mutex);
+               WARN_ON(1);
+               return;
+       }
 
        nvkm_wr32(device, 0x002270, (nvkm_memory_addr(cur) >> 12) |
                                    (target << 28));

diff --git a/drivers/gpu/drm/nouveau/nvkm/engine/fifo/gk104.c b/drivers/gpu/drm/nouveau/nvkm/engine/fifo/gk104.c
index 38c0910722c02c23c3ccf598f1839f732affcb24..73ae4ecc2e9a887c9298cf6aa818436df71010dc 100644 (file)
--- a/drivers/gpu/drm/nouveau/nvkm/engine/fifo/gk104.c
+++ b/drivers/gpu/drm/nouveau/nvkm/engine/fifo/gk104.c
@@ -83,10 +83,13 @@ gk104_fifo_runlist_commit(struct gk104_fifo *fifo, int runl)
        }
        nvkm_done(mem);
 
-       if (nvkm_memory_target(mem) == NVKM_MEM_TARGET_VRAM)
-               target = 0;
-       else
-               target = 3;
+       switch (nvkm_memory_target(mem)) {
+       case NVKM_MEM_TARGET_VRAM: target = 0; break;
+       case NVKM_MEM_TARGET_NCOH: target = 3; break;
+       default:
+               WARN_ON(1);
+               return;
+       }
 
        nvkm_wr32(device, 0x002270, (nvkm_memory_addr(mem) >> 12) |
                                    (target << 28));

diff --git a/drivers/gpu/drm/nouveau/nvkm/falcon/v1.c b/drivers/gpu/drm/nouveau/nvkm/falcon/v1.c
index 24a295858287e8d7cde0e50b4077c07d1bb08f87..b537f111f39c81cce70ec2049b134160e2586ec1 100644 (file)
--- a/drivers/gpu/drm/nouveau/nvkm/falcon/v1.c
+++ b/drivers/gpu/drm/nouveau/nvkm/falcon/v1.c
@@ -136,10 +136,13 @@ nvkm_falcon_v1_bind_context(struct nvkm_falcon *falcon, struct nvkm_gpuobj *ctx)
        nvkm_falcon_wr32(falcon, 0xe00 + 4 * FALCON_DMAIDX_PHYS_SYS_NCOH, 0x6);
 
        /* Set context */
-       if (nvkm_memory_target(ctx->memory) == NVKM_MEM_TARGET_VRAM)
-               inst_loc = 0x0; /* FB */
-       else
-               inst_loc = 0x3; /* Non-coherent sysmem */
+       switch (nvkm_memory_target(ctx->memory)) {
+       case NVKM_MEM_TARGET_VRAM: inst_loc = 0; break;
+       case NVKM_MEM_TARGET_NCOH: inst_loc = 3; break;
+       default:
+               WARN_ON(1);
+               return;
+       }
 
        /* Enable context */
        nvkm_falcon_mask(falcon, 0x048, 0x1, 0x1);

diff --git a/drivers/gpu/drm/nouveau/nvkm/subdev/instmem/gk20a.c b/drivers/gpu/drm/nouveau/nvkm/subdev/instmem/gk20a.c
index a94238374a3924f62416cd5035030f201046bd9b..9dec58ec3d9f41512a504aef0b246a6f5470cd33 100644 (file)
--- a/drivers/gpu/drm/nouveau/nvkm/subdev/instmem/gk20a.c
+++ b/drivers/gpu/drm/nouveau/nvkm/subdev/instmem/gk20a.c
@@ -116,7 +116,7 @@ struct gk20a_instmem {
 static enum nvkm_memory_target
 gk20a_instobj_target(struct nvkm_memory *memory)
 {
-       return NVKM_MEM_TARGET_HOST;
+       return NVKM_MEM_TARGET_NCOH;
 }
 
 static u64