From: Lionel Landwerlin <llandwerlin@gmail.com>
Date: Mon, 17 Oct 2016 16:11:04 +0000 (-0700)
Subject: intel: aubinator: use different colors to signal batch start/end
X-Git-Url: https://git.libre-soc.org/?a=commitdiff_plain;h=6b17e3a6da82df47056e1c6f8a79af2fab629b2c;p=mesa.git

intel: aubinator: use different colors to signal batch start/end

This makes the stream of commands a bit easier to read.

v2 (Ken): Use bold text on green headers for easier readability;
          swap the green and blue headers so the majority stay blue.

Signed-off-by: Lionel Landwerlin <lionel.g.landwerlin@intel.com>
Signed-off-by: Kenneth Graunke <kenneth@whitecape.org>
Reviewed-by: Lionel Landwerlin <lionel.g.landwerlin@intel.com>
---

diff --git a/src/intel/tools/aubinator.c b/src/intel/tools/aubinator.c
index d716a652806..31c1f8987f2 100644
--- a/src/intel/tools/aubinator.c
+++ b/src/intel/tools/aubinator.c
@@ -50,8 +50,9 @@
 #define AUB_MI_BATCH_BUFFER_END (0x0500 << 16)
 
 #define CSI "\e["
-#define HEADER CSI "37;44m"
-#define NORMAL CSI "0m"
+#define BLUE_HEADER  CSI "0;44m"
+#define GREEN_HEADER CSI "1;42m"
+#define NORMAL       CSI "0m"
 
 /* options */
 
@@ -727,9 +728,13 @@ parse_commands(struct gen_spec *spec, uint32_t *cmds, int size, int engine)
       const char *color, *reset_color = NORMAL;
       uint64_t offset;
 
-      if (option_full_decode)
-         color = HEADER;
-      else
+      if (option_full_decode) {
+         if ((p[0] & 0xffff0000) == AUB_MI_BATCH_BUFFER_START ||
+             (p[0] & 0xffff0000) == AUB_MI_BATCH_BUFFER_END)
+            color = GREEN_HEADER;
+         else
+            color = BLUE_HEADER;
+      } else
          color = NORMAL;
 
       if (option_color == COLOR_NEVER) {