On Thu, 2003-11-27 at 15:47, Dmitri Katchalov wrote:
> 
> I've made this quick and dirty patch for Radeon driver only. 
> I understand that it is not perfect as it has to be copied
> into every other driver. A better solution is needed IMHO.

I've posted about http://penguinppc.org/~daenzer/DRI/radeon-agp8x.diff
before, which is a similar patch by Hui Yu. I understand Keith Whitwell
has integrated yet another version on a CVS branch.

> As a side note I'm now getting a whooping 1% improvement with 
> AGP 8x according to ut2003 benchmark. I wonder is there is 
> something else I should look at.

Do you have reason to believe it should be better? :)


> Also AGPFastWrite locks up my machine hard, is this to be expected?

That's why it's an option which defaults to off.


> >         xf86DrvMsg(pScreen->myNum, X_WARNING, 
> >             "[agp] mode x%d not supported in AGP `2.0, forcing 4x mode\n",
> >             info->agpMode);
> >         mode |= RADEON_AGP2_4X_MODE;

This is a bad idea, as higher modes tend to be less stable. The only
better v2 default than the current 1x would be the mode set up by
firmware.


PS: Unified diffs (diff -u) are usually preferred.

-- 
Earthling Michel DÃnzer      |     Debian (powerpc), X and DRI developer
Software libre enthusiast    |   http://svcs.affero.net/rm.php?r=daenzer


-------------------------------------------------------
This SF.net email is sponsored by: SF.net Giveback Program.
Does SourceForge.net help you be more productive?  Does it
help you create better code?  SHARE THE LOVE, and help us help
YOU!  Click Here: http://sourceforge.net/donate/
_______________________________________________
Dri-devel mailing list
[EMAIL PROTECTED]
https://lists.sourceforge.net/lists/listinfo/dri-devel

Reply via email to