http://bugs.freedesktop.org/show_bug.cgi?id=10211





--- Comment #3 from Nicolai Hähnle <[EMAIL PROTECTED]>  2008-07-28 02:29:07 PST 
---
I'm always suspicious about changes in this area that are based on hearsay.
Note that those comments in r300_reg.h are way older than AMD's spec releases;
they were written in a time when we had zero knowledge as to what these
registers really mean.

From what I understand, 0xA (which is FREE|FLUSH, according to specs)
corresponds to a cache invalidation. This is necessary if the destination
buffer was changed by someone else since the last 3D operation, e.g. if the
destination buffer has moved or if software fallbacks were run.

On the other hand, 0x2 (which is a simple FLUSH) should be sufficient to make
sure any writes done by the 3D operation are written out to memory.

So when you think about it, this patch might fix something (I doubt it as long
as I don't see concrete evidence for it), but it does so in a very hacky way
that is completely unrelated to what's really going on.

My conclusion: "not relevant" or "should be replaced by something else based on
serious analysis of what we're doing". Then again, maybe I'm completely off
base and Jerome Glisse knows more on this topic.


-- 
Configure bugmail: http://bugs.freedesktop.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.
-------------------------------------------------------------------------
This SF.Net email is sponsored by the Moblin Your Move Developer's challenge
Build the coolest Linux based applications with Moblin SDK & win great prizes
Grand prize is a trip for two to an Open Source event anywhere in the world
http://moblin-contest.org/redirect.php?banner_id=100&url=/
--
_______________________________________________
Dri-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/dri-devel

Reply via email to