Closed Bug 1481812 Opened Last year Closed Last year

Refactor TelemetrySession to reduce memory usage in content process

Categories

(Toolkit :: Telemetry, defect, P1)

defect

Tracking

()

RESOLVED FIXED
mozilla64
Tracking Status
firefox64 --- fixed

People

(Reporter: janerik, Assigned: janerik)

References

(Blocks 1 open bug)

Details

(Whiteboard: [overhead:84k])

Attachments

(1 file)

As per the proposal in https://docs.google.com/document/d/10m2MS19J1ybCW-ruxlc7B9HeBCJ-6ykRLAeNt0lrrj4/edit# we should try to minimize session usage in content processes.

We might be able to not use it in content processes at all (and initialize the content-process part of GCTelemetry in the controller only)
Blocks: 1475556
Priority: -- → P3
Priority: P3 → P1
Assignee: nobody → jrediger
This avoids loading the remaining parts of TelemetrySession in a content process.
This saves around 10 kb of memory.
Once we move TelemetrySession out of the content process we're on step closer to renaming it to MainPing (bug 1486034)
Pushed by jrediger@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/801c33dcde58
Refactor memory gathering into its own module r=chutten
https://hg.mozilla.org/mozilla-central/rev/801c33dcde58
Status: NEW → RESOLVED
Closed: Last year
Resolution: --- → FIXED
Target Milestone: --- → mozilla64
It looks like this reduced JS memory usage for the AWSY(sb) test by 84KB [1].

[1] https://treeherder.mozilla.org/perf.html#/graphs?timerange=172800&series=autoland,1684884,1,4
Whiteboard: [overhead:84k]
I confirm that:

== Change summary for alert #17162 (as of Fri, 26 Oct 2018 02:50:08 GMT) ==

Improvements:

  2%  Base Content JS osx-10-10 opt stylo     5,276,837.33 -> 5,192,416.00

For up to date results, see: https://treeherder.mozilla.org/perf.html#/alerts?id=17162
You need to log in before you can comment on or make changes to this bug.