i965/sklgt4: Implement depth/timestamp write w/a
authorBen Widawsky <ben@bwidawsk.net>
Thu, 26 May 2016 18:04:07 +0000 (11:04 -0700)
committerBen Widawsky <ben@bwidawsk.net>
Thu, 26 May 2016 21:08:17 +0000 (14:08 -0700)
commitddcfc35f62ed3ad83b100beacb5b30394dcd9960
tree8aeba816d339321be3b323005f423a5e3383b893
parentf1fa8b4a1ca73fac4400e13b9aaca8c6b2955d2c
i965/sklgt4: Implement depth/timestamp write w/a

The stated bug describes a scenario in which a post sync write operation for
depth or timestamp can be ignored. There are two workarounds suggested, the
first and easier is to simply do a cs stall when we do these type of writes.
The second option is to do a PIPE_CONTROL flush after the post sync but before
the data is required.

Generally, I believe the data written out is consumed by the application on the
CPU side and so doing the easier of the two is ideal. Furthermore, these queries
aren't tremendously common in the perf sensitive apps I have looked at. However,
there could be cases where a shader stage might directly consume the data, and
as a result option 2 may be desirable.

This patch goes with the easier solution for now.

gen9lp bug_de_id=2137196

By itself, this does *not* fix any of the GT4 hangs we're currently
experiencing.

Cc: Mika Kuoppala <mika.kuoppala@intel.com>
Signed-off-by: Ben Widawsky <ben@bwidawsk.net>
Reviewed-by: Anuj Phogat <anuj.phogat@gmail.com>
src/mesa/drivers/dri/i965/brw_queryobj.c