Performance/Snappy/2012-12-13

From MozillaWiki
Jump to: navigation, search

Snappy December 13, 2012 Minutes

Actions

  • lmandel to follow up with Bas about bug 792527
    • pinged bas in the bug (the bug that Bas is working on is bug 509052)

Incoming

  • Next meeting in Jan 10

Projects

Startup & Shutdown

benwa, vladan, rafael, bbondy, yoric

Results
  • bug 818296: We no longer nuke cross compartment wrapper on shutdown. This should save 200ms+ on shutdown but our telemetry bucket prevent use from confirming this accurately :(
  • bug 810101 : Readahead for safebrowsing .sbstore files landed. ~50ms improvement on aklotz's laptop, probably even more in other environments
Todo
  • bug 818257: Investigate gradual 30% regression in startup times over the past year
  • espindola:
    • bug 815709: Shutdown time is read in the main thread.
    • With bug 815709 done, bug 814765: Include late writes in the Telemetry ping.
    • With those fixed, add support to the symbolication server for OS X and have the metrics script symbolicate the stacks.
    • Fix any late writes that show up.
    • Add a shtudown watchdog feature to the extension. Initially will only be used for shutdown to report shutdown above 30 seconds
  • aklotz: bug 815418 - telemetry on what proportion of attempted Firefox start-ups result in 'Firefox is running and not responding' - many of the same issues as bug 815709, going to try to minimize duplication of effort

Tab Strip (tab switch)

jrmuizel, bas, avi

Results
  • bug : Finished Australis theme perf impact, found ~50% regression in tab animation FPS on slow machines. Avi looked at Australis frame paint times, confirms paint times have doubled or tripled during tab animations
  • Draft version of tab switch timing add-on available at https://builder.addons.mozilla.org/package/162308/ (bug 812381) thanks to a new contributor
  • avih:
    • vlad landed bug 731974 which improves animation timing considerably and now it's mostly achieving 60 fps on slow machines.
    • Frame rates on fast machines are not indicative of animation performance.
    • Filed and working on bug 820167: Adding flush+paint processing measurements to the existing frame-rates measurement API (Start/StopFrameTimeRecording - bug 696398)
Todo
  • improve the telemetry probe being used for tab switch timing

Content (GC, CC, DOM storage, flash hang, DOM event/task scheduling)

billm, jcoppeard, smaug, mccr8, aklotz

Results
  • bug 807021: Applied Honza's feedback on LocalStorage off the main thread. Got some performance numbers. Received a review from smaug, will land on Nightly only this week (no uplift)
  • bug 600307: Gave feedback to Honza on his total rewrite of LocalStorage. His version lands in Firefox 21
  • bug 805591: shorlander says he plans to ui-review this today! Thanks vladan and lmandel. QA in progress, still waiting for r+ from bsmedberg
  • bug 715376: Patches posted for review; bz has r+'d most of the interesting ones. Talos numbers indicate that there's minimal performance difference.

Add-on and Web page introduced slowness (add-on manager start-up)

irving

Results
  • Beginning to understand the code and issues
  • Feedback and discussion on bug 699839 for using async sqlite in XPIProvider
Todo
  • Add telemetry to identify particular add-ons causing slow start-up