Kristian Høgsberg wrote: > On 6/14/07, Thomas Hellström <[EMAIL PROTECTED]> wrote: > >> Kristian Høgsberg wrote: > > ... > >> > True. And if we bump libdrm major version, we can drop the hash table >> > and skip lists too. With DRI interface changes, I moved the hash >> > table implementation into libGL, the only place it's used. >> > >> > Kristian >> Aargh. :) >> It's used in the XvMC drivers for DRI drawable tracking (At least VIA), >> although one can argue whether it really belongs in libdrm. > > > Can't you just use a regular X resource class there? We're tracking > DRI drawables in AIGLX too, but we call CreateNewResourceType to > create the resource and then track them using AddResource and > LookupIDByType. > Hmm, These are X Server side functions, right?
I was referring to the XvMC client libs. They are DRI clients just like Mesa, and use the DRI side client protocol in the same way, only for video instead of 3D. Currently there is no client side library for the dri protocol, so at least the via library is duplicating (with small modifications) the DRI client side protocol code Mesa is using. Currently i810 XvMC is using libdrm but not the DRI protocol. Via XvMC is using both. And there are or will probably be some closed source variants out there as well. So we should not assume that Mesa is the only DRI- or libdrm client. If libdrm is changed we need to update those clients as well. One way to make this cleaner is probably to separate out the DRI client protocol and drawable tracking into a separate libdriclient.so which is linked to by any client requiring DRI functionality. That would mean (referring to via XvMC file naming here), xf86dri.c, xf86dri.h, xf86dristr.h and perhaps driDrawable.c and the corresponding Mesa files. /Thomas > Kristian ------------------------------------------------------------------------- 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
