Hi Jason,

this would be the same as case 3. Is that intentional and if so would you 
combine the cases?
--Michael

-----Ursprüngliche Nachricht-----
Von: mesa-dev [mailto:mesa-dev-boun...@lists.freedesktop.org] Im Auftrag von 
Jason Ekstrand
Gesendet: Dienstag, 22. November 2016 03:59
An: mesa-dev@lists.freedesktop.org
Cc: Jason Ekstrand <jason.ekstr...@intel.com>
Betreff: [Mesa-dev] [PATCH] intel/aubinator: Use the correct length for MEDIA 
commands

---
 src/intel/tools/decoder.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/src/intel/tools/decoder.c b/src/intel/tools/decoder.c index 
6bd02bf..55488eb 100644
--- a/src/intel/tools/decoder.c
+++ b/src/intel/tools/decoder.c
@@ -612,8 +612,8 @@ gen_group_get_length(struct gen_group *group, const 
uint32_t *p)
          return field(h, 0, 7) + 2;
       case 1:
          return 1;
-      case 2:
-         return 2;
+      case 2: /* MEDIA */
+         return field(h, 0, 7) + 2;
       case 3:
          return field(h, 0, 7) + 2;
       }
--
2.5.0.400.gff86faf

_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev

_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev

Reply via email to