From: Alyssa Rosenzweig <alyssa@collabora.com>
Date: Mon, 22 Nov 2021 00:03:33 +0000 (-0500)
Subject: pan/va: Only hex dump when verbosely disassembling
X-Git-Tag: upstream/22.3.5~15098
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=72b3f21cd47e2281baf02caac434d9e737a72915;p=platform%2Fupstream%2Fmesa.git

pan/va: Only hex dump when verbosely disassembling

Closer behaviour to Bifrost, making the entrypoints symmetric.

Signed-off-by: Alyssa Rosenzweig <alyssa@collabora.com>
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/14063>
---

diff --git a/src/panfrost/bifrost/valhall/disassemble.h b/src/panfrost/bifrost/valhall/disassemble.h
index 5cd644f..7bde8b3 100644
--- a/src/panfrost/bifrost/valhall/disassemble.h
+++ b/src/panfrost/bifrost/valhall/disassemble.h
@@ -36,7 +36,7 @@ va_print_dest(FILE *fp, uint8_t dest, bool can_mask)
 void va_disasm_instr(FILE *fp, uint64_t instr);
 
 static inline void
-disassemble_valhall(FILE *fp, const uint64_t *code, unsigned size)
+disassemble_valhall(FILE *fp, const uint64_t *code, unsigned size, bool verbose)
 {
    assert((size & 7) == 0);
 
@@ -48,11 +48,13 @@ disassemble_valhall(FILE *fp, const uint64_t *code, unsigned size)
       if (instr == 0)
          return;
 
-      /* Print byte pattern */
-      for (unsigned j = 0; j < 8; ++j)
-         fprintf(fp, "%02x ", (uint8_t) (instr >> (j * 8)));
+      if (verbose) {
+         /* Print byte pattern */
+         for (unsigned j = 0; j < 8; ++j)
+            fprintf(fp, "%02x ", (uint8_t) (instr >> (j * 8)));
 
-      fprintf(fp, "   ");
+         fprintf(fp, "   ");
+      }
 
       va_disasm_instr(fp, instr);
       fprintf(fp, "\n");