Bill Currie wrote:
> 
> On Wed, Oct 24, 2001 at 02:59:11PM -0600, Keith Whitwell wrote:
> > Ralf,  I haven't been able to test your patch, but it sounds like it works
> > well for those who've tried it.  If you clean it up to include just the agp
> > texture stuff, I can commit it...
> 
> Index: lib/GL/mesa/src/drv/mga/mgatexmem.c
> ===================================================================
> RCS file: /cvsroot/dri/xc/xc/lib/GL/mesa/src/drv/mga/mgatexmem.c,v
> retrieving revision 1.8
> diff -u -r1.8 mgatexmem.c
> --- lib/GL/mesa/src/drv/mga/mgatexmem.c 2001/05/01 21:39:10     1.8
> +++ lib/GL/mesa/src/drv/mga/mgatexmem.c 2001/10/25 01:21:14
> @@ -129,7 +129,8 @@
> 
>     i--;
>     list[0].prev = MGA_NR_TEX_REGIONS;
> -   list[i].prev = i-1;
> +   if (i)
> +          list[i].prev = i-1;
>     list[i].next = MGA_NR_TEX_REGIONS;
>     list[MGA_NR_TEX_REGIONS].prev = i;
>     list[MGA_NR_TEX_REGIONS].next = 0;
> 
> Note: this is a hand edited patch as the AGP texturing patch is in the diff
> too (so there will be fuzz). /Every/ dri seg I got (once I was able to run
> quakeforge under gdb) involved .prev being 255 (remember: prev is an unsigned
> char). I don't feel I've done enough testing, but with this patch I have had
> niether crash nor texture corruption.

Hmm.  That list should be circular, so this patch isn't correct, but there may
be a bug there for some other reason.  I'll look at it...

Keith

_________________________________________________________
Do You Yahoo!?
Get your free @yahoo.com address at http://mail.yahoo.com


_______________________________________________
Dri-devel mailing list
[EMAIL PROTECTED]
https://lists.sourceforge.net/lists/listinfo/dri-devel

Reply via email to