pan/decode: Prefer uint64_t to mali_ptr
authorAlyssa Rosenzweig <alyssa.rosenzweig@collabora.com>
Wed, 14 Aug 2019 20:21:21 +0000 (13:21 -0700)
committerAlyssa Rosenzweig <alyssa.rosenzweig@collabora.com>
Wed, 14 Aug 2019 22:09:17 +0000 (15:09 -0700)
This removes an unwanted dependency on panfrost-job.h

Signed-off-by: Alyssa Rosenzweig <alyssa.rosenzweig@collabora.com>
src/panfrost/pandecode/cmdline.c
src/panfrost/pandecode/common.c
src/panfrost/pandecode/decode.h

index 0ac952fa6d5ac6488bf4c856fa7c0c9c42a498ab..256b532f314e32c2570483749df4ce9bfe196200 100644 (file)
@@ -44,7 +44,7 @@ pandecode_read_filename(const char *base, const char *name)
 }
 
 static void
-pandecode_read_memory(const char *base, const char *name, mali_ptr gpu_va)
+pandecode_read_memory(const char *base, const char *name, uint64_t gpu_va)
 {
         FILE *fp = pandecode_read_filename(base, name);
 
@@ -70,7 +70,7 @@ pandecode_read_mmap(const char *base, const char *line)
 {
         assert(strlen(line) < 500);
 
-        mali_ptr addr;
+        uint64_t addr;
         char name[512];
 
         sscanf(line, "MMAP %" PRIx64 " %s", &addr, name);
@@ -80,7 +80,7 @@ pandecode_read_mmap(const char *base, const char *line)
 static void
 pandecode_read_job_submit(const char *base, const char *line)
 {
-        mali_ptr addr;
+        uint64_t addr;
         unsigned core_req;
         unsigned is_bifrost;
 
index 0cefdb899576541bc5f6cf242042406e492a55b8..6154b3cd505eb487dde356063422c67c7ed510dd 100644 (file)
@@ -37,7 +37,7 @@
 static struct pandecode_mapped_memory mmaps;
 
 struct pandecode_mapped_memory *
-pandecode_find_mapped_gpu_mem_containing(mali_ptr addr)
+pandecode_find_mapped_gpu_mem_containing(uint64_t addr)
 {
         list_for_each_entry(struct pandecode_mapped_memory, pos, &mmaps.node, node) {
                 if (addr >= pos->gpu_va && addr < pos->gpu_va + pos->length)
@@ -48,7 +48,7 @@ pandecode_find_mapped_gpu_mem_containing(mali_ptr addr)
 }
 
 void
-pandecode_inject_mmap(mali_ptr gpu_va, void *cpu, unsigned sz, const char *name)
+pandecode_inject_mmap(uint64_t gpu_va, void *cpu, unsigned sz, const char *name)
 {
         struct pandecode_mapped_memory *mapped_mem = NULL;
 
@@ -73,7 +73,7 @@ pandecode_inject_mmap(mali_ptr gpu_va, void *cpu, unsigned sz, const char *name)
 }
 
 char *
-pointer_as_memory_reference(mali_ptr ptr)
+pointer_as_memory_reference(uint64_t ptr)
 {
         struct pandecode_mapped_memory *mapped;
         char *out = malloc(128);
@@ -89,7 +89,7 @@ pointer_as_memory_reference(mali_ptr ptr)
 
         /* Just use the raw address if other options are exhausted */
 
-        snprintf(out, 128, MALI_PTR_FMT, ptr);
+        snprintf(out, 128, "0x%" PRIx64, ptr);
         return out;
 
 }
index 3db0c0d7091e35aeb00d599bb0216ee165824361..293203b1472dd30c63036080d0f2bc0d7baa22f2 100644 (file)
@@ -29,7 +29,7 @@
 #include <stdio.h>
 #include <stdlib.h>
 #include <stddef.h>
-#include <panfrost-job.h>
+#include <inttypes.h>
 #include "util/list.h"
 
 struct pandecode_mapped_memory {
@@ -38,23 +38,23 @@ struct pandecode_mapped_memory {
         size_t length;
 
         void *addr;
-        mali_ptr gpu_va;
+        uint64_t gpu_va;
 
         char name[32];
 };
 
 void pandecode_initialize(void);
 
-char *pointer_as_memory_reference(mali_ptr ptr);
+char *pointer_as_memory_reference(uint64_t ptr);
 
-struct pandecode_mapped_memory *pandecode_find_mapped_gpu_mem_containing(mali_ptr addr);
+struct pandecode_mapped_memory *pandecode_find_mapped_gpu_mem_containing(uint64_t addr);
 
 void
-pandecode_inject_mmap(mali_ptr gpu_va, void *cpu, unsigned sz, const char *name);
+pandecode_inject_mmap(uint64_t gpu_va, void *cpu, unsigned sz, const char *name);
 
 static inline void *
 __pandecode_fetch_gpu_mem(const struct pandecode_mapped_memory *mem,
-                          mali_ptr gpu_va, size_t size,
+                          uint64_t gpu_va, size_t size,
                           int line, const char *filename)
 {
         if (!mem)
@@ -88,6 +88,6 @@ __pandecode_fetch_gpu_mem(const struct pandecode_mapped_memory *mem,
                                       __LINE__, __FILE__)
 
 /* Common entrypoint */
-int pandecode_jc(mali_ptr jc_gpu_va, bool bifrost);
+int pandecode_jc(uint64_t jc_gpu_va, bool bifrost);
 
 #endif /* __MMAP_TRACE_H__ */