i965: emit BRW_NEW_AUX_STATE on aux state changes
authorIago Toral Quiroga <itoral@igalia.com>
Fri, 15 Sep 2017 07:13:07 +0000 (09:13 +0200)
committerIago Toral Quiroga <itoral@igalia.com>
Mon, 18 Sep 2017 08:47:51 +0000 (10:47 +0200)
commit3d9cb39fd0c5895bd9774413e03c95c5b40bf030
tree679ac5fb05ccfc8dec60b6b105a7510f44012ae0
parent9a8bf42308f437c3895a5bc82024de33713d3cb6
i965: emit BRW_NEW_AUX_STATE on aux state changes

Fixes a regression introduced with b96313c0e1289b296d7, which removed
BRW_NEW_BLORP for a bunch of SURFACE_STATE setup code, including render
targets, on the basis that blorp invalidates binding tables but not
surface states, however, at least on Broadwell, this caused a regression
in a CTS test, which Ken and Jason tracked down to the fact that we
are not uploading new render target surface states after allocating
new CCS_D surfaces for fast clears (which allocation is deferred until
an actual clear occurs).

The reason this only fails in BDW is that on SKL+ we use CCS_E which
is allocated up front so it exists in the initial surface state, the
problem can be reproduced in these platforms too if we use
INTEL_DEBUG=norcb to force the CCS_D path.

This patch, together with the ones preceding it, fixes the regression
by ensuring that we track and flag as dirty all aux state changes.

Credit goes to Jason and Ken for figuring out the reason for the
regression.

Fixes:
KHR-GL45.transform_feedback.draw_xfb_test

Reviewed-by: Jason Ekstrand <jason@jlekstrand.net>
src/mesa/drivers/dri/i965/intel_mipmap_tree.c