i965: Defeat the register stride checker in URB reads.
authorKenneth Graunke <kenneth@whitecape.org>
Wed, 8 Jun 2016 23:24:50 +0000 (16:24 -0700)
committerKenneth Graunke <kenneth@whitecape.org>
Tue, 14 Jun 2016 02:36:46 +0000 (19:36 -0700)
commited3ba651f6faa4ea94dde16fa880781090785477
treede23e519faf9b4f91a8ca5e439a8197a2b914fa9
parent9f37df06dafbf54cec6749543cac1baa77d0b5e2
i965: Defeat the register stride checker in URB reads.

Pulling DF inputs from the URB generates messages like:

   send(8)         g23<1>DF        g1<8,8,1>UD
                   urb 3 SIMD8 read mlen 1 rlen 2      { align1 1Q };

which makes the simulator angry:

   "For 64-bit Align1 operation or multiplication of dwords in CHV,
    source horizontal stride must be aligned to qword."

This seems to be documented in the Cherryview PRM, Volume 7, Page 823:

   "When source or destination datatype is 64b or operation is integer
    DWord multiply, regioning in Align1 must follow these rules:

    1. Source and Destination horizontal stride must be aligned to the
       same qword."

Setting the source horizontal stride to QWord is insane, as it's the
message header containing 8 URB handles in a single 32-bit DWord.
Instead, we should whack the destination type to UD, D, or F so that
the register stride checker doesn't notice.  The destination type of
send messages is basically irrelevant anyway.

Cc: "12.0" <mesa-stable@lists.freedesktop.org>
Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=95462
Signed-off-by: Kenneth Graunke <kenneth@whitecape.org>
Reviewed-by: Jordan Justen <jordan.l.justen@intel.com>
src/mesa/drivers/dri/i965/brw_fs_generator.cpp