From a1af92877d3d91886cf01be9e6c65311960e3baf Mon Sep 17 00:00:00 2001 From: Eric Anholt Date: Sat, 16 Oct 2004 01:58:02 +0000 Subject: [PATCH] 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. --- src/mesa/drivers/dri/r200/r200_texstate.c | 10 ++++++++-- 1 file 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 ); -- 2.30.2