panfrost/decode: Clip mali_compute_fbd at 64-bytes
authorAlyssa Rosenzweig <alyssa.rosenzweig@collabora.com>
Wed, 19 Jun 2019 16:24:01 +0000 (09:24 -0700)
committerAlyssa Rosenzweig <alyssa.rosenzweig@collabora.com>
Thu, 20 Jun 2019 14:48:04 +0000 (07:48 -0700)
Looking at internal evidence (later fields including a literal other
compute job inception-style, seeming memory corruption, no clear
function, and the field after this being a pointer to *itself*), it
looks like this is really a much smaller descriptor.

Signed-off-by: Alyssa Rosenzweig <alyssa.rosenzweig@collabora.com>
src/gallium/drivers/panfrost/include/panfrost-job.h
src/gallium/drivers/panfrost/pandecode/decode.c

index aaf6de72bc6e46d5b4d8cc656b6d5147804bad7b..6da86148cd75ea3d5710ada6d2c987516abfba60 100644 (file)
@@ -1411,15 +1411,6 @@ struct mali_single_framebuffer {
 
 struct mali_compute_fbd {
         u32 unknown1[16];
-        mali_ptr unknown2;
-        mali_ptr unknown3;
-        u32 unknown4[16];
-        u32 unknown5[18];
-        mali_ptr unknown6;
-        u32 unknown7[4];
-        mali_ptr unknown8;
-        mali_ptr unknown9;
-        u32 unknown10[16];
 } __attribute__((packed));
 
 /* Format bits for the render target flags */
index c402ff91cbc461c061e59d7bf910d3e74387d79b..8adcb5a7d4470d42b6d463a5028af91d6c910ae8 100644 (file)
@@ -563,15 +563,6 @@ pandecode_compute_fbd(uint64_t gpu_va, int job_no)
         pandecode_indent++;
 
         SHORT_SLIDE(1);
-        MEMORY_PROP(s, unknown2);
-        MEMORY_PROP(s, unknown3);
-        SHORT_SLIDE(4);
-        SHORT_SLIDE(5);
-        MEMORY_PROP(s, unknown6);
-        SHORT_SLIDE(7);
-        MEMORY_PROP(s, unknown8);
-        MEMORY_PROP(s, unknown9);
-        SHORT_SLIDE(10);
 
         pandecode_indent--;
         printf("},\n");