From 3a1b7efce84d1a2ca1842a5456694268ed0fd1e0 Mon Sep 17 00:00:00 2001 From: Topi Pohjolainen Date: Mon, 11 Sep 2017 14:12:15 +0300 Subject: [PATCH] intel/blorp/hiz: Always set sample number Reviewed-by: Chad Versace Reviewed-by: Kenneth Graunke Signed-off-by: Topi Pohjolainen --- src/intel/blorp/blorp_genX_exec.h | 11 +++++++++++ 1 file changed, 11 insertions(+) diff --git a/src/intel/blorp/blorp_genX_exec.h b/src/intel/blorp/blorp_genX_exec.h index 5f9a8ab4a51..53892620986 100644 --- a/src/intel/blorp/blorp_genX_exec.h +++ b/src/intel/blorp/blorp_genX_exec.h @@ -1454,6 +1454,17 @@ blorp_emit_gen8_hiz_op(struct blorp_batch *batch, if (params->stencil.enabled) assert(params->hiz_op == BLORP_HIZ_OP_DEPTH_CLEAR); + /* From the BDW PRM Volume 2, 3DSTATE_WM_HZ_OP: + * + * 3DSTATE_MULTISAMPLE packet must be used prior to this packet to change + * the Number of Multisamples. This packet must not be used to change + * Number of Multisamples in a rendering sequence. + * + * Since HIZ may be the first thing in a batch buffer, play safe and always + * emit 3DSTATE_MULTISAMPLE. + */ + blorp_emit_3dstate_multisample(batch, params); + /* If we can't alter the depth stencil config and multiple layers are * involved, the HiZ op will fail. This is because the op requires that a * new config is emitted for each additional layer. -- 2.30.2