Firefox Startup Performance – Feb 11, 2010
Posted: February 12, 2010 Filed under: firefox, mozilla | Tags: firefox, mozilla, Performance, startup 2 CommentsVarious minor updates this week:
- Static build: Ted did the first pass of review. Getting close. Will land on the Places branch once binary tests build, to see perf impact.
- Ben’s fastload cache replacement is still waiting on first-review from Ben Smedberg.
- Neil took up bug 354048 to not rebuild toolbars at startup, should reduce the DOM activity some at least. The patch is in review by Dao and Mano.
- Marco has nearly completed bug 542943, removing the bookmark redirect hash, which means less SQL at startup and less memory usage by the bookmarks service.
- I put a patch up on bug 545516 that fixes some bugs and cleans up the layout of the Performance Snapshot. Will get it landed next week.
- Started working with John O’Duinn and Mike Morgan to get a team together to work on the graph server.
Yay for better graph server!
It seems that a lot of performance work is stuck in waiting for reviews. Patches that remove deaad code, unused functions, decom, etc are for example:
bug 405407 Merge nsDiskCacheStreamIO and nsDiskCacheStreamOutput
bug 537223 Cleanup of gfxColor.h
bug 331032 Decom nsICacheEntryInfo, replace by ExpirationTime attribute to nsICachingChannel
bug 230675 ‘decom’ of nsICacheVisitor.idl: saves 10% / 150K from nkcache_s.lib
bug 386937 ConvertPixel doesn’t work as nsColor and gfx_color are not compatible…
To really address performance, start kicking the reviewers to focus on performance patches that are proven to work and proven to improve performance.