From: <th...@tu...> - 2006-05-29 10:28:33
|
Dave Airlie wrote: >>>> New semantics: >>>> The new manager always aligns to 16 bytes, except when it is >>>> bypassed by the SiS fb module. >>>> >>> >>> >> Yes you're right. The core functions support any alignment so the >> constraints are device specific. >> > > Just another question, could this code be used to replace all or parts > of i915_mem.c and radeon_mem.c as well? if yes any chance of doing > that as well? > > Dave. > > Hi, Dave, Hmm, the thought was that this was a temporary solution until we get time to finish the more advanced memory manager in drm-ttm-branch, and the VIA and SiS managers were in desperate need of attention. Both i915 and radeon memory managers could probably be moved over to the new code, but then only using the functionality in drm_mm.c and not drm_sman.c. I'll have a look. However, I'd rather focus on getting the ttm branch in a usable state and start moving drivers over to that functionality, eventually obsoleting the code in drm_sman.c. What's left in the ttm branch is to properly implement fence objects and a scheme to handle small allocations more efficiently. The latter is a bit tricky. /Thomas |