[Mesa-dev] [PATCH] r600g: Take care of bo placement domains.

[Mathias Fröhlich]

Hi,

Thanks for the comments.

On Monday, May 02, 2011 13:14:02 Michel Dänzer wrote:
> > 
> > +       if(bo->domains != domains) {
> > +               return 0;
> > +       }
> 
> I wonder if this test isn't too strict, i.e. if there aren't cases where
> the values aren't identical, but where the BO should still be considered
> compatible, e.g. because one is a subset of the other. But the patch is
> probably an improvement regardless, and this could always be improved
> later on.

From my point of view this is subject to experiment with.
For my test application, this gave slightly better performance than what I 
have tried else.
But the hugest performance gain stems from the initial placement change and 
from the more fine grained usage to placement mapping.
So for now I decided to send this variant ...

Greetings

Mathias