i965/gen7: Add some safety checks for send messages from GRFs.
authorEric Anholt <eric@anholt.net>
Fri, 9 Nov 2012 19:38:14 +0000 (11:38 -0800)
committerEric Anholt <eric@anholt.net>
Wed, 5 Dec 2012 00:27:53 +0000 (16:27 -0800)
src/mesa/drivers/dri/i965/brw_eu_emit.c

index eada9d99e9618d56353edca655d79b41833b3281..72fb5c5ae2c8b8f0af83f96b608c0923c847577f 100644 (file)
@@ -236,11 +236,26 @@ void
 brw_set_src0(struct brw_compile *p, struct brw_instruction *insn,
             struct brw_reg reg)
 {
+   struct brw_context *brw = p->brw;
+   struct intel_context *intel = &brw->intel;
+
    if (reg.type != BRW_ARCHITECTURE_REGISTER_FILE)
       assert(reg.nr < 128);
 
    gen7_convert_mrf_to_grf(p, &reg);
 
+   if (intel->gen >= 6 && (insn->header.opcode == BRW_OPCODE_SEND ||
+                           insn->header.opcode == BRW_OPCODE_SENDC)) {
+      /* Any source modifiers or regions will be ignored, since this just
+       * identifies the MRF/GRF to start reading the message contents from.
+       * Check for some likely failures.
+       */
+      assert(!reg.negate);
+      assert(!reg.abs);
+      assert(reg.address_mode == BRW_ADDRESS_DIRECT);
+      assert(reg.vstride != BRW_VERTICAL_STRIDE_0);
+   }
+
    validate_reg(insn, reg);
 
    insn->bits1.da1.src0_reg_file = reg.file;