Hi.

Looking at the irc discussion

http://dri.egore911.de/index.php?page=dri-devel-2007-04-30.log

23:31 <+anholt> <javascript:addUser('<+anholt>');> the userland 
interface of ttm is unacceptably bad. I've got some concerns about 
deadlock with it. The general fencing flags appear to be too limited, 
and extending that appropriately means you don't need DD flags
23:31 <+anholt> <javascript:addUser('<+anholt>');> we want zero-copy 
data upload when appropriate, which doesn't appear to be possible currently

and ...

23:34 < krh> <javascript:addUser('< krh>');> anholt: is it mostly the 
user space interface or are you looking at a deeper rewrite?
23:34 <+anholt> <javascript:addUser('<+anholt>');> krh: everything I've 
encountered in ttm so far has made me want to rewrite it

Eric, As some people are planning to put some work into fixing parts of 
the code up, it would be nice if you could share your thoughts on 
DRI-devel and be a little more specific.

The userspace part indeed needs to be looked at, for many reasons, but 
above all 64-bit padding and future expansion. Support for zero-copy 
data uploads (drm_bo_type_user) is just a matter of implementing it, 
which is a very easy task but requires cache-coherent high-memory 
allowing  PTEs which are not generally available on all architectures 
and texturing from them is a bit slow and prone to hangs on i915.

As for the kernel side, some parts look hairy but there is probably a 
reason why, so don't hesitate to ask. There is a fair amount of 
trial-and-error time put down in the code to figure out what's working 
and what's not. Identifying the parts that need fixing and making a 
common effort to fix them is probably a better way than to do another 
rewrite in the dark to find out later that there is yet another opinion 
on how to do things and we're getting nowhere.

Since you're one of the few who has looked at the code in-depth and 
tried to figure out what it does, your suggestions are very valuable.

Regards,
Thomas






-------------------------------------------------------------------------
This SF.net email is sponsored by DB2 Express
Download DB2 Express C - the FREE version of DB2 express and take
control of your XML. No limits. Just data. Click to get it now.
http://sourceforge.net/powerbar/db2/
--
_______________________________________________
Dri-devel mailing list
Dri-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/dri-devel

Reply via email to