[Mesa-dev] [PATCH] i965: Fix ARB_indirect_parameters logic.

[Kenneth Graunke]

On Monday, October 30, 2017 2:14:24 PM PDT Plamena Manolova wrote:
> This patch modifies the ARB_indirect_parameters logic in
> brw_draw_prims, so that our implementation isn't affected if
> another application attempts to use predicates. Previously we
> were using a predicate with a DELTAS_EQUAL comparison operation
> and relying on the MI_PREDICATE_DATA register being 0. Our code
> to initialize MI_PREDICATE_DATA to 0 was incorrect, so we were
> accidentally using whatever value was written there. Because the
> kernel does not initialize the MI_PREDICATE_DATA register on
> hardware context creation, we might inherit the value from whatever
> context was last running on the GPU (likely another process).
> The Haswell command parser also does not currently allow us to write
> the MI_PREDICATE_DATA register. Rather than fixing this and requiring
> an updated kernel, we switch to a different approach which uses a
> SRCS_EQUAL predicate that makes no assumptions about the states of any
> of the predicate registers.
> 
> Fixes: piglit.spec.arb_indirect_parameters.tf-count-arrays
> Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=103085
> 
> Signed-off-by: Plamena Manolova <plamena.manolova at intel.com>

Reviewed-by: Kenneth Graunke <kenneth at whitecape.org>

and pushed.  Thanks Pam!
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 833 bytes
Desc: This is a digitally signed message part.
URL: <https://lists.freedesktop.org/archives/mesa-dev/attachments/20171030/14fb6b2a/attachment.sig>