Performance/Status Meetings/2008-February-13: Difference between revisions

From MozillaWiki
Jump to navigation Jump to search
No edit summary
 
(2 intermediate revisions by one other user not shown)
Line 26: Line 26:


== Performance ==
== Performance ==
* Old tinderbox testing machines (non-talos) going away -- {{bug|413695}}.
** Replacing machines with same pageset but running in Talos
* JPEG/image speedups
** remaining: {{bug|412396}} JPEG decoding 10% slower on Mac trunk
* Perf Bugs for FF3:
* Perf Bugs for FF3:
** perf-only: http://tinyurl.com/396fxa
** perf-only: http://tinyurl.com/396fxa
Line 45: Line 39:
** {{bug|311566}} - nsCSSScanner::GatherIdent does lots of string appending
** {{bug|311566}} - nsCSSScanner::GatherIdent does lots of string appending
** {{bug|412796}} - Optimize fastload system (mmap fileIO, endianness, packed structs)
** {{bug|412796}} - Optimize fastload system (mmap fileIO, endianness, packed structs)
** Checked in: {{bug|411579}} - Optimize read file buffer sizes for faster startup times


* Profile Guided Optimization
* Profile Guided Optimization
Line 51: Line 44:
** TODO: Figure out Linux (sayrer)
** TODO: Figure out Linux (sayrer)
** TODO: Fix win32 breakage (cairo; possibly fixed with cairo patch?)
** TODO: Fix win32 breakage (cairo; possibly fixed with cairo patch?)
* ICC partial compilation


* Other
* Other
** {{bug|353962}} -- JS spinlocks vs. full locks (Crashes in Fx2)
** <strike>{{bug|353962}} -- JS spinlocks vs. full locks (Crashes in Fx2)</strike> Fixed!


* On radar (unowned stuff that people have mentioned have mentioned we should look at at some point)
* On radar (unowned stuff that people have mentioned have mentioned we should look at at some point)
** Partial compiles with icc
** Theme optimization
** Theme optimization
** roc's scrollbar issues
** roc's scrollbar issues
Line 68: Line 62:


== Fragmentation ==
== Fragmentation ==
* {{bug|407459}} -- Hook up jemalloc to our build
* jemalloc
** ported to windows, mac and linux.  integrated to build system on mac and linux.  windows work continuing.  Hope to have numbers by end of the week.
** <s>{{bug|407459}} -- jemalloc on Windows</s>
* {{bug|403830}} -- Content arena
** <s>{{bug|417066}} -- jemalloc on Linux</s>
** {{bug|408720}} -- Fallout  
* <s>{{bug|403830}} -- Content arena</s>
** '''{{bug|414894}} We're going to back it out.'''
** <s>{{bug|408720}} -- Fallout</s>
** <s>{{bug|414894}} We're going to back it out.</s>
** New arenas
** New arenas
* Other bugs (should have perf and footprint keywords)
* Other bugs (should have perf and footprint keywords)
** {{bug|403354}} -- Get rid of nsCStringArray ParseString usage [significantly better but doing some additional work on]
** <s>{{bug|407442}} -- nsEventListenerManager allocates a lot [patch]</s>
** {{bug|407442}} -- nsEventListenerManager allocates a lot [patch]
** <s>{{bug|408113}} -- JS reallocate context->stackPool arenas way too often [patch]</s>
** <s>{{bug|408113}} -- JS reallocate context->stackPool arenas way too often [patch]</s>
** <s>{{bug|408301}} -- XPConnect wrappers w/o XPConnect proto don't share JS scopes with their protos. [patch]</s>
** <s>{{bug|408301}} -- XPConnect wrappers w/o XPConnect proto don't share JS scopes with their protos. [patch]</s>
** <s>{{bug|410250}} -- nsID/nsJSID stupid mallocs [patch]</s>
** <s>{{bug|410250}} -- nsID/nsJSID stupid mallocs [patch]</s>
** <s>{{bug|410357}} -- use nsAutoTArray for nsCSSDeclaration::mOrder [patch]</s>
** <s>{{bug|410357}} -- use nsAutoTArray for nsCSSDeclaration::mOrder [patch]</s>
** {{bug|410392}} -- stack allocate gfx objects [patch, needs work]
** {{bug|403354}} -- Get rid of nsCStringArray ParseString usage [patch]
** {{bug|408921}} -- Arenas size+header are too big for size classification [in progress]
** {{bug|408921}} -- Arenas size+header are too big for size classification [patch]




Latest revision as of 18:05, 13 February 2008

« Back to Status Meetings

Gecko 1.9 Performance and Leaks Meeting Details

  • Wednesdays - Firefox 3 - 10:00am Pacific, 1:00pm Eastern
  • Script Conference Room, Building S, Mountain View
  • 650-903-0800 or 650-215-1282 x91 Conf# 217 (US/INTL)
  • 1-800-707-2533 (pin 369) Conf# 217 (US)
  • irc.mozilla.org #perf for backchannel

Leaks

  • Ben Turner fixed a number of leaks causing documents to be kept alive too long.
  • Peterv has tentative patch for session restore leaking restored windows in some cases (such as error pages or pages not finishing loading).
  • 18 P1/P2s still open.

Action Items

  • AI:sicking Look into enabling trace-refcount, without the tracing part in nightlies.
  • AI:sicking Test leaked handles on windows manually using purify.
  • AI:sayrer Run leak buster while looking for trace-malloc leaks. [didn't get this done--sayrer]

Performance

  • Txul platform differences according to mac mini machines
    • mac: slowest on 1.8, slower still on 1.9 (30% regression) Looks even or better than branch
    • windows: fastest on 1.8, now in second place (30% regression) -- improving, still slower
    • linux: middle on 1.8, now fastest on 1.9 (30% improvement)
  • Ts improvements
    • Ts analysis: bug 406810
    • bug 311566 - nsCSSScanner::GatherIdent does lots of string appending
    • bug 412796 - Optimize fastload system (mmap fileIO, endianness, packed structs)
  • Profile Guided Optimization
    • bug 361343 - Win32 turn-on; bug 410994 - PGO Spidermonkey
    • TODO: Figure out Linux (sayrer)
    • TODO: Fix win32 breakage (cairo; possibly fixed with cairo patch?)
  • ICC partial compilation
  • Other
    • bug 353962 -- JS spinlocks vs. full locks (Crashes in Fx2) Fixed!
  • On radar (unowned stuff that people have mentioned have mentioned we should look at at some point)
    • Theme optimization
    • roc's scrollbar issues
    • startup performance (fastload perf, loading libXUL and other files, etc.) (unowned)
    • more work to speed up onSecurityChange, but further work will require SSL work and reducing the number of notifications
    • XPConnect accessor speed (bug 412531) (request for priority+owner by dietrich)

Action Items

  • make faster

Fragmentation

  • jemalloc
  • bug 403830 -- Content arena
  • Other bugs (should have perf and footprint keywords)
    • bug 407442 -- nsEventListenerManager allocates a lot [patch]
    • bug 408113 -- JS reallocate context->stackPool arenas way too often [patch]
    • bug 408301 -- XPConnect wrappers w/o XPConnect proto don't share JS scopes with their protos. [patch]
    • bug 410250 -- nsID/nsJSID stupid mallocs [patch]
    • bug 410357 -- use nsAutoTArray for nsCSSDeclaration::mOrder [patch]
    • bug 403354 -- Get rid of nsCStringArray ParseString usage [patch]
    • bug 408921 -- Arenas size+header are too big for size classification [patch]


Action Items

  • fix things
Retrieved from "https://wiki.mozilla.org/index.php?title=Performance/Status_Meetings/2008-February-13&oldid=83472"