summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorEric Anholt <[email protected]>2004-10-16 01:58:02 +0000
committerEric Anholt <[email protected]>2004-10-16 01:58:02 +0000
commita1af92877d3d91886cf01be9e6c65311960e3baf (patch)
tree1f10d1791692c814c02000778ab76fbf50e217a5
parent33899b7c351fda77bed6dee5e5d02e31c2f7e0e5 (diff)
Always turn on the TAM_DEBUG3=0x6 workaround on real r200s. It appears that
the current cases for turning it on were insufficient (Bugzilla #1519, 729, 814) and it has no significant performance impact. Performance tested with quake3 in GL_LINEAR mode both with and without anisotropy, with the workaround always on or always off.
-rw-r--r--src/mesa/drivers/dri/r200/r200_texstate.c10
1 files changed, 8 insertions, 2 deletions
diff --git a/src/mesa/drivers/dri/r200/r200_texstate.c b/src/mesa/drivers/dri/r200/r200_texstate.c
index 81aec4f9383..57ee245ee90 100644
--- a/src/mesa/drivers/dri/r200/r200_texstate.c
+++ b/src/mesa/drivers/dri/r200/r200_texstate.c
@@ -1362,8 +1362,14 @@ void r200UpdateTextureState( GLcontext *ctx )
* Texture cache LRU hang workaround -------------
* not needed for r200 derivatives?
*/
- dbg = 0x0;
+ /* While the cases below attempt to only enable the workaround in the
+ * specific cases necessary, they were insufficient. See bugzilla #1519,
+ * #729, #814. Tests with quake3 showed no impact on performance.
+ */
+ dbg = 0x6;
+
+ /*
if (((rmesa->hw.ctx.cmd[CTX_PP_CNTL] & (R200_TEX_0_ENABLE )) &&
((((rmesa->hw.tex[0].cmd[TEX_PP_TXFILTER] & R200_MIN_FILTER_MASK)) &
0x04) == 0)) ||
@@ -1388,7 +1394,7 @@ void r200UpdateTextureState( GLcontext *ctx )
0x04) == 0)))
{
dbg |= 0x04;
- }
+ }*/
if (dbg != rmesa->hw.tam.cmd[TAM_DEBUG3]) {
R200_STATECHANGE( rmesa, tam );