On Wed, Aug 27, 2008 at 11:45 PM, Jerome Glisse <[EMAIL PROTECTED]> wrote: > On Tue, 26 Aug 2008 18:02:24 +1000 > "Dave Airlie" <[EMAIL PROTECTED]> wrote: > >> 4. added clean flag for buffers that get migrated to VRAM immediately. >> - with pixmaps we have no idea of the usage pattern usually at create >> time, >> however if nothing has ever mapped the bo and nothing has ever >> moved it then >> we can optimise the first move to VRAM away and just memset/solid >> fill the VRAM area to 0. > > After quick look i think this optimization add too much code. Maybe a better > policy like only actualy deciding from which pool to allocate the object on > first use of it.
For pixmaps you can't really do this with any good idea. Its not a huge amount of code, its like a fill + the clean flags, I don't think it makes life much harder... > > I still need to read the code, i spoted this VRAM from a quick review of > patch. > But i saw horrible stuff like too many GEM domain for specific things which > sounds very bad. We've only got 3 gem domains for radeon, you sure you looking at the right branch? Dave. ------------------------------------------------------------------------- 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 Dri-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/dri-devel