Mesa (master): i965/disasm: Don' t disassemble the URB complete field on Broadwell.

Kenneth Graunke kwg at kemper.freedesktop.org
Mon Jul 21 18:27:28 UTC 2014


Module: Mesa
Branch: master
Commit: caf8c07dd40b26ae3f4b8f1bf4ff26bd226a3b7b
URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=caf8c07dd40b26ae3f4b8f1bf4ff26bd226a3b7b

Author: Kenneth Graunke <kenneth at whitecape.org>
Date:   Thu Jul 17 15:55:05 2014 -0700

i965/disasm: Don't disassemble the URB complete field on Broadwell.

It doesn't exist, so attempting to read it will trigger generation
assertions in the brw_inst API.

Signed-off-by: Kenneth Graunke <kenneth at whitecape.org>
Reviewed-by: Matt Turner <mattst88 at gmail.com>

---

 src/mesa/drivers/dri/i965/brw_disasm.c |    6 ++++--
 1 file changed, 4 insertions(+), 2 deletions(-)

diff --git a/src/mesa/drivers/dri/i965/brw_disasm.c b/src/mesa/drivers/dri/i965/brw_disasm.c
index d7362c1..f66865f 100644
--- a/src/mesa/drivers/dri/i965/brw_disasm.c
+++ b/src/mesa/drivers/dri/i965/brw_disasm.c
@@ -1410,8 +1410,10 @@ brw_disassemble_inst(FILE *file, struct brw_context *brw, brw_inst *inst,
                err |= control(file, "urb used", urb_used,
                               brw_inst_urb_used(brw, inst), &space);
             }
-            err |= control(file, "urb complete", urb_complete,
-                           brw_inst_urb_complete(brw, inst), &space);
+            if (brw->gen < 8) {
+               err |= control(file, "urb complete", urb_complete,
+                              brw_inst_urb_complete(brw, inst), &space);
+            }
             break;
          case BRW_SFID_THREAD_SPAWNER:
             break;




More information about the mesa-commit mailing list