i965: Create new program cache bo when clearing the program cache
authorJordan Justen <jordan.l.justen@intel.com>
Fri, 2 Feb 2018 21:03:10 +0000 (13:03 -0800)
committerJordan Justen <jordan.l.justen@intel.com>
Sat, 3 Feb 2018 20:16:58 +0000 (12:16 -0800)
commit83e60ce927142752c57163fcb8b30eca2370d014
tree6967bafc9a66e75f6a0894b99623ada84d16a151
parent589e9db23f18cddedb2001ddcc6dae3d5ae701a3
i965: Create new program cache bo when clearing the program cache

When the disk shader cache CI testing was enabled, we started noticing
occasional failures on deqp test runs. (Mainly SNB, rarely HSW)

Before this change, when we cleared the (in memory) program cache we
reused the same bo. Since the disk shader cache quickly restores
programs, it appears that this would lead to overwrites of the older
program binaries in the in memory program cache that apparently were
still executing in some cases. If these programs were still executing,
this could cause a GPU hang.

This issue is probably not disk shader cache specific, but may have
been hidden due to the compiler taking time to recompile programs
after the cache was cleared.

v2:
 * Don't add `copy` param to brw_cache_new_bo (Ken)
 * Call from brw_program_cache_check_size (Ken)

Cc: Kenneth Graunke <kenneth@whitecape.org>
Cc: mesa-stable@lists.freedesktop.org
Signed-off-by: Jordan Justen <jordan.l.justen@intel.com>
Reviewed-by: Kenneth Graunke <kenneth@whitecape.org>
src/mesa/drivers/dri/i965/brw_program_cache.c