Ok here's an updated patch set that takes into account most of your comments.  
There are still a few open issues though:
  - fence register allocation for pre-965 buffer execution
  - unmap_mapping_range cost (it looked like the function would short-circuit
    if there was nothing to do, but I didn't measure it)
  - pre-965 still needs testing

For the alignment argument we talked about, I ended up making the GTT mapping 
ioctl do both the fake offset creation and alignment setting, based on the 
current value of the tiling_mode.  It's possible that an object could have 
been pinned with a bad alignment for a given tiling mode though, in which case 
I just return -EINVAL.  Not sure if we should try to do something smarter 
given a dumb caller...

Patches are in reply to this message.

Thanks,
Jesse

-------------------------------------------------------------------------
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

Reply via email to