Performance/Snappy/2012-12-13

From MozillaWiki
Jump to navigation Jump to search
The printable version is no longer supported and may have rendering errors. Please update your browser bookmarks and please use the default browser print function instead.

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
Retrieved from "https://wiki.mozilla.org/index.php?title=Performance/Snappy/2012-12-13&oldid=494497"