<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Tue, Nov 22, 2016 at 1:53 AM, Lionel Landwerlin <span dir="ltr"><<a href="mailto:lionel.g.landwerlin@intel.com" target="_blank">lionel.g.landwerlin@intel.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">On 22/11/16 02:58, Jason Ekstrand wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
---<br>
  src/intel/tools/decoder.c | 4 ++--<br>
  1 file changed, 2 insertions(+), 2 deletions(-)<br>
<br>
diff --git a/src/intel/tools/decoder.c b/src/intel/tools/decoder.c<br>
index 6bd02bf..55488eb 100644<br>
--- a/src/intel/tools/decoder.c<br>
+++ b/src/intel/tools/decoder.c<br>
@@ -612,8 +612,8 @@ gen_group_get_length(struct gen_group *group, const uint32_t *p)<br>
           return field(h, 0, 7) + 2;<br>
        case 1:<br>
           return 1;<br>
-      case 2:<br>
-         return 2;<br>
+      case 2: /* MEDIA */<br>
+         return field(h, 0, 7) + 2;<br>
</blockquote>
<br></span>
The documentation seems to indicate 0..15 as bits for DWord Length.<br>
Am I missing something?<br></blockquote><div><br></div><div>*most* of them use 0..15.  However the GPGPU_WALKER command only uses 0..7.  Since I didn't figure we'd have a lot of commands larger than 255 dwords, I figured just reading 0..7 would be ok.  We could always lookup the "DWord Length" field by name or just special-case GPGPU_WALKER<br><br></div><div>--Jason<br></div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="HOEnZb"><div class="h5">
<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
        case 3:<br>
           return field(h, 0, 7) + 2;<br>
        }<br>
</blockquote>
<br>
<br>
</div></div></blockquote></div><br></div></div>