[sugar] [PATCH] screenshots hurt

Erik Garrison erik at laptop.org
Thu Sep 4 01:29:14 EDT 2008


Sugar Developers,

In the field and amongst our own developers there have been numerous
complaints about the performance of Sugar.  One simple thing we can do
to improve user experience is to stop taking screenshots on every
tabbing event, every activity focus change event, and every frame
appearance event.

The attached patch to Sugar (most recent git head, 48dd24c6e) does
simply that.  Andrés Ambrois and I have tested the changes in question
on a machine running 656 and on a machine running 2263 (the most recent
joyride I have available-- code in question is identical to the current
sugar git head).  We noted a considerable improvement in the speed and
consistency of activity switching performance.

These qualitative results are consistent with prior user interface
profiling tests by Riccardo Lucchese, which have established that
screenshot functionality occupies just shy of 50% of CPU time during
activity switching [1].

Please apply the patch and test on a *REAL* machine.  Please comment on
the effects.  Note that on recent joyride builds the appearance of the
frame on every tabbing event makes it more difficult to notice the
performance change.

Erik

[1] http://lists.laptop.org/pipermail/devel/2008-July/016722.html

The lines removed by the attached patch were part of the following
commits: 

commit f300006b76ed905e52fb5b5848e09bf34a91216e
Author: Marco Pesenti Gritti <mpg at redhat.com>
Date:   Tue Oct 23 15:19:17 2007 +0200

commit 0f565484a95c083d89c6f40b0d390d4109b5c46f
Author: Benjamin Berg <benjamin at sipsolutions.net>
Date:   Fri Jun 20 16:49:38 2008 +0200

commit 14f1ca9b8491dee343305066facf8e7ae24e381f
Author: Tomeu Vizoso <tomeu at tomeuvizoso.net>
Date:   Wed Apr 2 14:53:52 2008 +0200
-------------- next part --------------
A non-text attachment was scrubbed...
Name: sugar-no_screen_shots.patch
Type: text/x-diff
Size: 2205 bytes
Desc: not available
Url : http://lists.laptop.org/pipermail/sugar/attachments/20080904/1064dea0/attachment.patch 


More information about the Sugar mailing list