Mesa (master): pan/decode: Remove SHORT_SLIDE indirection
GitLab Mirror
gitlab-mirror at kemper.freedesktop.org
Fri Jan 24 19:15:47 UTC 2020
Module: Mesa
Branch: master
Commit: 58aa2b8cfc62044bfa35ee2f9e50559af0e6c7e0
URL: http://cgit.freedesktop.org/mesa/mesa/commit/?id=58aa2b8cfc62044bfa35ee2f9e50559af0e6c7e0
Author: Alyssa Rosenzweig <alyssa.rosenzweig at collabora.com>
Date: Fri Jan 24 08:25:08 2020 -0500
pan/decode: Remove SHORT_SLIDE indirection
../src/panfrost/pandecode/decode.c: In function ‘pandecode_compute_fbd’:
../src/panfrost/pandecode/decode.c:789:35: warning: taking address of packed member of ‘struct mali_compute_fbd’ may result in an unaligned pointer value [-Waddress-of-packed-member]
789 | pandecode_u32_slide(num, s->unknown ## num, ARRAY_SIZE(s->unknown ## num))
| ~^~~~~~~~~
../src/panfrost/pandecode/decode.c:800:9: note: in expansion of macro ‘SHORT_SLIDE’
800 | SHORT_SLIDE(1);
| ^~~~~~~~~~~
Signed-off-by: Alyssa Rosenzweig <alyssa.rosenzweig at collabora.com>
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/merge_requests/3543>
---
src/panfrost/pandecode/decode.c | 21 ++++++---------------
1 file changed, 6 insertions(+), 15 deletions(-)
diff --git a/src/panfrost/pandecode/decode.c b/src/panfrost/pandecode/decode.c
index 80ac64440f5..74a123b2897 100644
--- a/src/panfrost/pandecode/decode.c
+++ b/src/panfrost/pandecode/decode.c
@@ -774,20 +774,6 @@ pandecode_sfbd(uint64_t gpu_va, int job_no, bool is_fragment, unsigned gpu_id)
return info;
}
-static void
-pandecode_u32_slide(unsigned name, const u32 *slide, unsigned count)
-{
- pandecode_log(".unknown%d = {", name);
-
- for (int i = 0; i < count; ++i)
- pandecode_log_cont("%X, ", slide[i]);
-
- pandecode_log("},\n");
-}
-
-#define SHORT_SLIDE(num) \
- pandecode_u32_slide(num, s->unknown ## num, ARRAY_SIZE(s->unknown ## num))
-
static void
pandecode_compute_fbd(uint64_t gpu_va, int job_no)
{
@@ -797,7 +783,12 @@ pandecode_compute_fbd(uint64_t gpu_va, int job_no)
pandecode_log("struct mali_compute_fbd framebuffer_%"PRIx64"_%d = {\n", gpu_va, job_no);
pandecode_indent++;
- SHORT_SLIDE(1);
+ pandecode_log(".unknown1 = {");
+
+ for (int i = 0; i < ARRAY_SIZE(s->unknown1); ++i)
+ pandecode_log_cont("%X, ", s->unknown1[i]);
+
+ pandecode_log("},\n");
pandecode_indent--;
pandecode_log_cont("},\n");
More information about the mesa-commit
mailing list