Bug 350662 - g-m-m: Patch to improved startup time
Summary: g-m-m: Patch to improved startup time
Status: RESOLVED FIXED
: 370926 (view as bug list)
Alias: None
Product: openSUSE 11.0
Classification: openSUSE
Component: GNOME (show other bugs)
Version: Alpha 3
Hardware: Other openSUSE 10.3
: P5 - None : Normal (vote)
Target Milestone: ---
Assignee: Scott Reeves
QA Contact: E-mail List
URL:
Whiteboard: gnome-performance
Keywords:
Depends on:
Blocks: main-menu-perf
  Show dependency tree / graph
 
Reported: 2007-12-25 22:58 UTC by Magnus Boman
Modified: 2008-04-19 03:23 UTC (History)
3 users (show)

See Also:
Found By: ---
Services Priority:
Business Priority:
Blocker: ---
Marketing QA Status: ---
IT Deployment: ---

Attachments
Patch (1.17 KB, text/x-patch)
2007-12-25 22:58 UTC, Magnus Boman 		Details
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Magnus Boman 2007-12-25 22:58:17 UTC 
Created attachment 188705 [details]
Patch

Attached is a patch which improves the startup time of g-m-m.

tile_table_reload was called once for each section created, and then again for all sections in apply_lockdown_settings. This patch makes sure that tile_table_reload is only called once per section.
A (nice) side-effect of this is that it also doesn't create unnecessary tiles which it later have to delete as they don't fit in the window.

This patch also fixes a bug where the apply_lockdown_settings did not update Recently Used Documents.

kcachegrind shows lr cost...
...Before this patch: 2 844 609 350
...After this patch:    996 026 118
Comment 1 Federico Mena Quintero 2008-04-05 05:04:32 UTC 
This patch kicks total ass.

I submitted it for openSUSE 11.0 along with the fix for bug #230478 (see that bug for the actual patch).
Comment 2 Federico Mena Quintero 2008-04-19 03:23:38 UTC 
*** Bug 370926 has been marked as a duplicate of this bug. ***