From 7093558b311b223004845d0a422eb88bed15b418 Mon Sep 17 00:00:00 2001 From: Chad Versace Date: Wed, 9 Jan 2013 11:40:29 -0800 Subject: [PATCH] i965: Quote the PRM on a HorzStride subtlety Reviewed-by: Ian Romanick Signed-off-by: Chad Versace --- src/mesa/drivers/dri/i965/brw_eu_emit.c | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/src/mesa/drivers/dri/i965/brw_eu_emit.c b/src/mesa/drivers/dri/i965/brw_eu_emit.c index fecbff15d65..b34754a9cfc 100644 --- a/src/mesa/drivers/dri/i965/brw_eu_emit.c +++ b/src/mesa/drivers/dri/i965/brw_eu_emit.c @@ -126,7 +126,10 @@ brw_set_dest(struct brw_compile *p, struct brw_instruction *insn, else { insn->bits1.da16.dest_subreg_nr = dest.subnr / 16; insn->bits1.da16.dest_writemask = dest.dw1.bits.writemask; - /* even ignored in da16, still need to set as '01' */ + /* From the Ivybridge PRM, Vol 4, Part 3, Section 5.2.4.1: + * Although Dst.HorzStride is a don't care for Align16, HW needs + * this to be programmed as "01". + */ insn->bits1.da16.dest_horiz_stride = 1; } } -- 2.30.2