On 02/14/12 09:10, Yonit Halperin wrote: > RHBZ #790083 > > Signed-off-by: Yonit Halperin <yhalp...@redhat.com>
You are doing two things in one patch: (a) fix the compat mode bug, which also matches the patch description, and (b) skip vram when it is unused (in compat mode). I'd love to see (b) done in a different way: simply walk all surfaces and tag them dirty. Will have the same effect for compat mode (no surfaces used -> nothing tagged dirty) and additionally it will (in native mode) only migrate over the vram areas which are actually filled with surfaces. thanks, Gerd