On Tue, 2010-08-17 at 14:26 +0300, Yonit Halperin wrote: > This patch replaces the DirectDraw dependent surfaces allocation with mspace > malloc. > (bugzilla #29254). > > Future work on this issue can include: > 1) Tuning mspace for surfaces. > 2) Separating the deallocations of DEVRAM resources from those of surfaces. > Currently, there is only one release_ring.
I'll take a look at this. > 3) compacting the VRAM. Also, related is http://bugs.freedesktop.org/show_bug.cgi?id=29053 After talking to you on irc about this and looking at the code in detail i think we need to move surfaces_info to DevRes so its shared between all pdevs for the device. This is not a problem since surface ids are not reused between pdevs (due to the global surfaces_used array). > However, when real DirectDraw Qxl support does land, the previous > method can be used again (Though separating the deallocations of > DEVRAM resources from those of surfaces will be still a good idea). If we can make our allocator work well, I don't see what using the directdraw allocator really gives us though. -- =-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-= Alexander Larsson Red Hat, Inc [email protected] [email protected] He's an otherworldly devious senator gone bad. She's a brilliant bisexual stripper who inherited a spooky stately manor from her late maiden aunt. They fight crime! _______________________________________________ Spice-devel mailing list [email protected] http://lists.freedesktop.org/mailman/listinfo/spice-devel
