diff options
author | Eric Anholt <anholt@FreeBSD.org> | 2004-10-16 01:58:02 +0000 |
---|---|---|
committer | Eric Anholt <anholt@FreeBSD.org> | 2004-10-16 01:58:02 +0000 |
commit | a1af92877d3d91886cf01be9e6c65311960e3baf (patch) | |
tree | 1f10d1791692c814c02000778ab76fbf50e217a5 | |
parent | 33899b7c351fda77bed6dee5e5d02e31c2f7e0e5 (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.c | 10 |
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 81aec4f938..57ee245ee9 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 ); |