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 /src | |
| 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.
Diffstat (limited to 'src')
| -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 );  | 
