gpu: nvgpu: Cleanup GMMU debug printing

Ensure that all debug prints are consistent from chip to chip
and function to function. The following maps letters in the
debug print to their meaning:

  C  Mapping is cachable
  v  Mapping is volatile
  S  Mapping is sparse
  P  Mapping is private (VPR/WPR)
  c  Mapping is coherent
  V  Mapping is valid

JIRA NVGPU-30

Change-Id: Ia890af88677c3e6d3fdd8c4fe266158c35b8afcd
Signed-off-by: Alex Waterman <alexw@nvidia.com>
Reviewed-on: https://git-master/r/1514903
GVS: Gerrit_Virtual_Submit
Reviewed-by: Deepak Nibade <dnibade@nvidia.com>
Tested-by: Deepak Nibade <dnibade@nvidia.com>
Reviewed-by: Vijayakumar Subbu <vsubbu@nvidia.com>

1 files changed, 5 insertions, 3 deletions

diff --git a/drivers/gpu/nvgpu/common/mm/gmmu.c b/drivers/gpu/nvgpu/common/mm/gmmu.c
index dcb53c2a..1f3519aa 100644
--- a/drivers/gpu/nvgpu/common/mm/gmmu.c
+++ b/drivers/gpu/nvgpu/common/mm/gmmu.c
@@ -666,7 +666,7 @@ static int __nvgpu_gmmu_update_page_table(struct vm_gk20a *vm,
                    "vm=%s "
                    "%-5s GPU virt %#-12llx +%#-9llx    phys %#-12llx "
                    "phys offset: %#-4llx;  pgsz: %3dkb perm=%-2s | "
-                   "kind=%#02x APT=%-6s %c%c%c",
+                   "kind=%#02x APT=%-6s %c%c%c%c%c",
                    vm->name,
                    sgt ? "MAP" : "UNMAP",
                    virt_addr,
@@ -677,9 +677,11 @@ static int __nvgpu_gmmu_update_page_table(struct vm_gk20a *vm,
                    nvgpu_gmmu_perm_str(attrs->rw_flag),
                    attrs->kind_v,
                    nvgpu_aperture_str(attrs->aperture),
-                   attrs->cacheable ? 'C' : 'V', /* C = cached, V = volatile. */
+                   attrs->cacheable ? 'C' : 'c', /* C = cached, V = volatile. */
                    attrs->sparse    ? 'S' : '-',
-                   attrs->priv      ? 'P' : '-');
+                   attrs->priv      ? 'P' : '-',
+                   attrs->coherent  ? 'c' : '-',
+                   attrs->valid     ? 'V' : '-');
 
         /*
          * Handle VIDMEM progamming. Currently uses a different scatter list