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

Reply via email to