Intermittent e10s leakcheck | tab process: 20056 bytes leaked (AsyncTransactionTrackersHolder, ChildDNSRecord, CipherSuiteChangeObserver, CompositorChild, CondVar, ...)

RESOLVED FIXED in Firefox 38

Status

()

Core
Audio/Video
RESOLVED FIXED
3 years ago
3 years ago

People

(Reporter: Tomcat, Assigned: mccr8)

Tracking

(Blocks: 1 bug, {intermittent-failure, memory-leak})

Trunk
mozilla38
x86
Linux
intermittent-failure, memory-leak
Points:
---

Firefox Tracking Flags

(e10s+, firefox36 unaffected, firefox37 unaffected, firefox38 fixed, firefox-esr31 unaffected)

Details

(URL)

Attachments

(1 attachment)

(Reporter)

Description

3 years ago
Ubuntu VM 12.04 x64 mozilla-inbound debug test mochitest-e10s-3

https://treeherder.mozilla.org/logviewer.html#?job_id=6155303&repo=mozilla-inbound

03:04:57 WARNING - TEST-UNEXPECTED-FAIL | leakcheck | tab process: 20056 bytes leaked (AsyncTransactionTrackersHolder, ChildDNSRecord, CipherSuiteChangeObserver, CompositorChild, CondVar, ...)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Summary: Intermittent leakcheck | tab process: 20056 bytes leaked (AsyncTransactionTrackersHolder, ChildDNSRecord, CipherSuiteChangeObserver, CompositorChild, CondVar, ...) → Intermittent e10s leakcheck | tab process: 20056 bytes leaked (AsyncTransactionTrackersHolder, ChildDNSRecord, CipherSuiteChangeObserver, CompositorChild, CondVar, ...)
Comment hidden (Treeherder Robot)
(Assignee)

Comment 5

3 years ago
The GeckoMediaPluginService leak slowly grows as people add more stuff.  Until that's properly made multiprocess, we need to bump the threshold again.
Assignee: nobody → continuation
Component: General → Video/Audio
(Assignee)

Comment 6

3 years ago
Created attachment 8561488 [details] [diff] [review]
Increase tab process leak threshold a little.
Attachment #8561488 - Flags: review?(erahm)
Comment on attachment 8561488 [details] [diff] [review]
Increase tab process leak threshold a little.

Review of attachment 8561488 [details] [diff] [review]:
-----------------------------------------------------------------

lgtm, presumably there's a bug for fixing this in GMP right?
Attachment #8561488 - Flags: review?(erahm) → review+
(Assignee)

Comment 8

3 years ago
(In reply to Eric Rahm [:erahm] from comment #7)
> lgtm, presumably there's a bug for fixing this in GMP right?

Yeah, sorry, I should have linked it here.  Bug 1057908 is the bug.  The GMP service depends on getting some profile shutdown message to do its teardown, but that isn't sent in the child process, so it just leaks.  When it is run only in the parent process, this shouldn't be a problem any more.
(Assignee)

Comment 9

3 years ago
This really does not need a try run.
Keywords: checkin-needed
(Assignee)

Updated

3 years ago
Keywords: checkin-needed

Updated

3 years ago
tracking-e10s: --- → +
(Assignee)

Updated

3 years ago
Status: NEW → RESOLVED
Last Resolved: 3 years ago
Resolution: --- → FIXED
status-firefox36: --- → unaffected
status-firefox37: --- → unaffected
status-firefox38: --- → fixed
status-firefox-esr31: --- → unaffected
Keywords: leave-open
Target Milestone: --- → mozilla38
(Assignee)

Updated

3 years ago
Blocks: 1051230
You need to log in before you can comment on or make changes to this bug.