On Mon, 2003-02-03 at 16:02, Keith Whitwell wrote:
> >
> > -#define COMMIT_RING() do { \
> > - RADEON_WRITE( RADEON_CP_RB_WPTR, dev_priv->ring.tail ); \
> > +#define COMMIT_RING() do { \
> > + /* read from PCI bus to ensure correct posting */ \
> > + RADEON_READ( RADEON_CP_RB_WPTR ); \
> > + RADEON_WRITE( RADEON_CP_RB_WPTR, dev_priv->ring.tail ); \
> > + RADEON_READ( RADEON_CP_RB_WPTR ); \
> > } while (0)
>
> Ouch. Put a conditional around that at least, so that not everybody suffers...
If it makes a difference on AGP here, it's for the good. It may well be
necessary as well.
--
Earthling Michel Dänzer (MrCooper)/ Debian GNU/Linux (powerpc) developer
XFree86 and DRI project member / CS student, Free Software enthusiast
-------------------------------------------------------
This SF.NET email is sponsored by:
SourceForge Enterprise Edition + IBM + LinuxWorld = Something 2 See!
http://www.vasoftware.com
_______________________________________________
Dri-devel mailing list
[EMAIL PROTECTED]
https://lists.sourceforge.net/lists/listinfo/dri-devel
