TPEPerformance/Meetings/2017-03-13: Difference between revisions

From MozillaWiki
Jump to navigation Jump to search
No edit summary
(→‎kanru: Use [ps-radar])
 
(12 intermediate revisions by the same user not shown)
Line 5: Line 5:


<small>[[TPEPerformance/Meetings/{{#time: Y-m-d | {{SUBPAGENAME}} -1 week}}|&laquo; previous week]] | [[TPEPerformance|index]] | [[TPEPerformance/Meetings/{{#time: Y-m-d | {{SUBPAGENAME}} +1 week}}|next week &raquo;]]</small>
<small>[[TPEPerformance/Meetings/{{#time: Y-m-d | {{SUBPAGENAME}} -1 week}}|&laquo; previous week]] | [[TPEPerformance|index]] | [[TPEPerformance/Meetings/{{#time: Y-m-d | {{SUBPAGENAME}} +1 week}}|next week &raquo;]]</small>
= Attendees =
* kanru, thinker, cervantes, ting, wpan, bchien


= Action Items =
= Action Items =
== New action items ==
== New action items ==
* Update BuildDisplayList milestones. +thinker, bchien
* Find reviewer for {{bug|1024669}} +kanru, bchien, cyu
== Action items from last meeting ==
== Action items from last meeting ==


Line 14: Line 20:
=== Web Painting - Improve DisplayList Memory Locality ===
=== Web Painting - Improve DisplayList Memory Locality ===
'''Milestones'''
'''Milestones'''
* {{Inprogress|Make sure consensus and tech design with Quantum Render team before 2/22}}
* {{Done|Make sure consensus and tech design with Quantum Render team before 2/22}}
** QR will not touch BuildDisplayList
** We will improve the BuildDisplayList perf
 
=== Remove Sync IPC Everywhere ===
=== Remove Sync IPC Everywhere ===
'''Milestones'''
'''Milestones'''
* {{Inprogress|Refactor found/profiled sync IPC to old async IPC {{Bug|1331674}}}}
* {{Inprogress|Refactor found/profiled sync IPC to old async IPC {{Bug|1331674}}}}
** PScreenManager under review
** Will work on Spellchecker bug
* {{Done|Block/Disallow new sync IPC been added {{bug|1336919}}}}
* {{Done|Block/Disallow new sync IPC been added {{bug|1336919}}}}
* {{Inprogress|Easy use async IPC (build async infrastructure, refactor as new interface) {{bug|1313200}}}}
* {{Inprogress|Easy use async IPC (build async infrastructure, refactor as new interface) {{bug|1313200}}}}
* Enforce all sync IPC to use new async IPC
* Enforce all sync IPC to use new async IPC
=== Quantum Telemetry ===
=== Quantum Telemetry ===
'''Milestones'''
'''Milestones'''
* {{Inprogress|Implement telemetry probe to measure event to display latency {{bug|1313465}}}}
* {{Inprogress|Implement telemetry probe to measure event to display latency {{bug|1313465}}}}
** Reviewer agreed to put telemetry code in PresShell. Need to update patch
* Define more telemetry probe
* Define more telemetry probe
=== TaskTracer integration to Gecko Profiler ===
=== TaskTracer integration to Gecko Profiler ===
'''Milestones'''
'''Milestones'''
* {{Inprogress|Default compiled into nightly binary}}
* {{Inprogress|Default compiled into nightly binary}}
** Verify runtime overhead with talos test
** Verify runtime overhead with talos test
** Patches almost ready. Trying to build tasktracer by default on try.
* Dump gecko profiler and tasktrace data from runtime memory to gecko profiler file
* Dump gecko profiler and tasktrace data from runtime memory to gecko profiler file
* {{Done|Add-on supports to support task tracker}}
* {{Done|Add-on supports to support task tracker}}
== Uptime ==
== Uptime ==
=== Crash Client (Breakpad) Improvement ===
=== Crash Client (Breakpad) Improvement ===
'''Milestones'''
'''Milestones'''
* {{Inprogress|Heap memory in minidumps {{bug|1286802}}}}
* {{Inprogress|Heap memory in minidumps {{bug|1286802}}}}
** Windows port finished. Cleaning up patch for review.
* {{Inprogress|Include thread names in minidumps {{bug|1024669}}}}
* {{Inprogress|Include thread names in minidumps {{bug|1024669}}}}
* Include thread names in minidumps web interface
* Include thread names in minidumps web interface
* Include JS Stacks in minidumps {{bug|1295918}}
* Include JS Stacks in minidumps {{bug|1295918}}
* Include JS Stacks in web interface
* Include JS Stacks in web interface
=== Windows Dynamic Analysis Improvement ===
=== Windows Dynamic Analysis Improvement ===
'''Milestones'''
'''Milestones'''
* {{Done|Make ASan build for Windows}}
* {{Done|Make ASan build for Windows}}
* {{Inprogress|Enable Treeherder Tier 3, and fix crash bugs (ETA: Begin of March)}}
* {{Inprogress|Enable Treeherder Tier 3, and fix crash bugs (ETA: Begin of March)}}
** Updated clang on tooltool for building both ASan and clang-plugin
** Cannot login loaner machine.
** Send patches for review.
* Enable Treeherder Tier 2, and fix crash bugs
* Enable Treeherder Tier 2, and fix crash bugs
* Enable Treeherder Tier 1, and fix crash bugs (Exit on green light)
* Enable Treeherder Tier 1, and fix crash bugs (Exit on green light)
Line 77: Line 98:
= This Week Priorities =
= This Week Priorities =
== thinker ==
== thinker ==
* Fix tasktracer mutex lock order patch
* Try maintain dirty region on frame tree
** Need to discuss with CJ
* Take a look at antialias text wrong width bug
== wcpan ==
== wcpan ==
* To land input latency telemetry patch
* To clarify the user story of input latency telemetry
== kanru ==
== kanru ==
* Spellchecker sync ipc bug
* Sort out backlogs
** [ps-radar]
* Review patch for JSPlugins
== cervantes ==
== cervantes ==
* To land TlsAlloc patch
* Clean up heap-in-mindump patch for windows for review
== ting ==
== ting ==
* Figure how to connect to windows loaner machine to debug timeout issue
* Patches for review
* Windows debugger sharing


= Incoming =
= Incoming =

Latest revision as of 06:42, 13 March 2017


« previous week | index | next week »

Attendees

  • kanru, thinker, cervantes, ting, wpan, bchien

Action Items

New action items

  • Update BuildDisplayList milestones. +thinker, bchien
  • Find reviewer for bug 1024669 +kanru, bchien, cyu

Action items from last meeting

Project Status

Quantum Flow

Web Painting - Improve DisplayList Memory Locality

Milestones

  • [DONE] Make sure consensus and tech design with Quantum Render team before 2/22
    • QR will not touch BuildDisplayList
    • We will improve the BuildDisplayList perf

Remove Sync IPC Everywhere

Milestones

  • [IN PROGRESS] Refactor found/profiled sync IPC to old async IPC bug 1331674
    • PScreenManager under review
    • Will work on Spellchecker bug
  • [DONE] Block/Disallow new sync IPC been added bug 1336919
  • [IN PROGRESS] Easy use async IPC (build async infrastructure, refactor as new interface) bug 1313200
  • Enforce all sync IPC to use new async IPC

Quantum Telemetry

Milestones

  • [IN PROGRESS] Implement telemetry probe to measure event to display latency bug 1313465
    • Reviewer agreed to put telemetry code in PresShell. Need to update patch
  • Define more telemetry probe

TaskTracer integration to Gecko Profiler

Milestones

  • [IN PROGRESS] Default compiled into nightly binary
    • Verify runtime overhead with talos test
    • Patches almost ready. Trying to build tasktracer by default on try.
  • Dump gecko profiler and tasktrace data from runtime memory to gecko profiler file
  • [DONE] Add-on supports to support task tracker

Uptime

Crash Client (Breakpad) Improvement

Milestones

  • [IN PROGRESS] Heap memory in minidumps bug 1286802
    • Windows port finished. Cleaning up patch for review.
  • [IN PROGRESS] Include thread names in minidumps bug 1024669
  • Include thread names in minidumps web interface
  • Include JS Stacks in minidumps bug 1295918
  • Include JS Stacks in web interface

Windows Dynamic Analysis Improvement

Milestones

  • [DONE] Make ASan build for Windows
  • [IN PROGRESS] Enable Treeherder Tier 3, and fix crash bugs (ETA: Begin of March)
    • Updated clang on tooltool for building both ASan and clang-plugin
    • Cannot login loaner machine.
    • Send patches for review.
  • Enable Treeherder Tier 2, and fix crash bugs
  • Enable Treeherder Tier 1, and fix crash bugs (Exit on green light)
  • Handover to sheriff and release engineering

Previous Week

thinker

  • bug 1331173 NEW:: Bigger Header segment capacity for TaskTracer
    • bug 1331173 is landing
  • bug 1346151 NEW:: Fix building time warning messages of TaskTracer
    • bug 1346151 is landing

cervantes

  • bug 1320134 NEW:: Crash in xul.dll@0x4d768 | xul.dll@0x4d739 | xul.dll@0x4d739 | xul.dll@0x4d739 | xul.dll@0x4d739 | xul.dll@0x4d739 | xul.dll@0x4d739 | xul.dll@0x4d739 | xul.dll@0x4d739 | xul.dll@0x4d739 | xul.dll@0x4d739 | xul.dll@0x4d739 | xul.dll@0x4d739 | xul.dll@0...
    • bug 1320134 - update the patch and retest on try
  • bug 1286802 NEW:: Add some heap areas to the minidump
    • Working on bug 1286802
  • bug 1346151 NEW:: Fix building time warning messages of TaskTracer
    • Reviewed bug bug 1346151

ting

  • bug 1340699 RESOLVED::FIXED Move TestAUSReadStrings from 'make check' to moz.build / xpcshell
    • bug 1340699 ni'd assignee for the test binary actually is not included in the package
  • bug 1333003 ASSIGNED:: Add a test platform for win64 ASan
    • bug 1333003 submitted current patches for reviewing, timed out on treeherder don't seem to be related
    • bug 1333003 trace mozbuild to understand how can i make asan runtime dll everywhere a binary is generated

wcpan

  • bug 1310127 RESOLVED::FIXED Use MOZ_MUST_USE in netwerk/protocol/http
    • bug 1310127 - rebase to central
  • bug 1341531 NEW:: Measure input event to event dispatcher (handler)
    • bug 1341531 - request review

This Week Priorities

thinker

  • Fix tasktracer mutex lock order patch
  • Try maintain dirty region on frame tree
    • Need to discuss with CJ
  • Take a look at antialias text wrong width bug

wcpan

  • To land input latency telemetry patch
  • To clarify the user story of input latency telemetry

kanru

  • Spellchecker sync ipc bug
  • Sort out backlogs
    • [ps-radar]
  • Review patch for JSPlugins

cervantes

  • To land TlsAlloc patch
  • Clean up heap-in-mindump patch for windows for review

ting

  • Figure how to connect to windows loaner machine to debug timeout issue
  • Patches for review
  • Windows debugger sharing

Incoming

Retrieved from "https://wiki.allizom.org/index.php?title=TPEPerformance/Meetings/2017-03-13&oldid=1165412"