Permaorange leak in pdf.js browser-chrome tests when Gecko 55 merges to beta on 2017-06-12 due to bug 1348062

RESOLVED FIXED in Firefox 55

Status

()

P2
normal
RESOLVED FIXED
2 years ago
a year ago

People

(Reporter: philor, Assigned: mayhemer)

Tracking

({memory-leak, regression})

55 Branch
mozilla55
memory-leak, regression
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(firefox-esr45 unaffected, firefox-esr52 unaffected, firefox53 unaffected, firefox54 unaffected, firefox55+ fixed)

Details

(Reporter)

Description

2 years ago
In six weeks, when trunk merges to beta, the browser-chrome chunk that runs the pdf.js tests is going to have a permaleak like https://treeherder.mozilla.org/logviewer.html#?job_id=95033251&repo=try or https://treeherder.mozilla.org/logviewer.html#?job_id=95033462&repo=try on ASan, from bug 1348062.

Should be possible to reproduce locally by just changing /config/milestone.txt from 55a1 to 55, but if it's some more subtle part of the central-to-beta changes, the stack of patches in https://treeherder.mozilla.org/#/jobs?repo=try&revision=bfaaa622d12692fedd9853e2e7ca051a19c7f285 should give you a full beta simulation on try.
(Reporter)

Updated

2 years ago
Flags: needinfo?(honzab.moz)
(Reporter)

Comment 1

2 years ago
[Tracking Requested - why for this release]: merge bustage, closed tree, delayed b1, tearing of hair and rending of garments.
tracking-firefox55: --- → ?
Tracking 55+ for this permaorange.
tracking-firefox55: ? → +
(Assignee)

Comment 3

2 years ago
I *think* this comes from the core of how the throttling (interdicted in bug 1312754 ) is currently implemented.  I have plans to rewrite it a bit, specifically in relation to this bug - remove the parts that I suspect could cause this leak.  Bug 1362071.
Flags: needinfo?(honzab.moz)
(Assignee)

Updated

2 years ago
Depends on: 1362071
(Assignee)

Comment 4

2 years ago
If this needs to be fixed quickly, then we may want to back the offending patch (bug 1348062) out.  Otherwise see comment 3, bug 1362071 should be fixed in approx 2 weeks.
Flags: needinfo?(philringnalda)
(Reporter)

Comment 5

2 years ago
Still have 5 weeks until the merge (and yeah, I'm carrying a backout of bug 1348062 around in my merge simulation stack, to keep it rebased and handy just in case).
Flags: needinfo?(philringnalda)
(Assignee)

Comment 6

2 years ago
(In reply to Phil Ringnalda (:philor) from comment #5)
> Still have 5 weeks until the merge (and yeah, I'm carrying a backout of bug
> 1348062 around in my merge simulation stack, to keep it rebased and handy
> just in case).

Thanks.  I will back bug 1348062 out should bug 1362071 be delayed.
Assignee: nobody → honzab.moz

Updated

2 years ago
Priority: -- → P2
(Assignee)

Updated

a year ago
No longer depends on: 1362071
(Assignee)

Updated

a year ago
Depends on: 1365306
Honza, with the soft code freeze coming up, do you think think 1362071 is in shape to land in time?
Flags: needinfo?(honzab.moz)
(Assignee)

Comment 8

a year ago
(In reply to Mike Taylor [:miketaylr] from comment #7)
> Honza, with the soft code freeze coming up, do you think think 1362071 is in
> shape to land in time?

The interesting bug is bug 1365306.  And yes, I'm about to r? and land it soon.
Flags: needinfo?(honzab.moz)
(Assignee)

Comment 9

a year ago
So, the probable fix has landed, can you please verify this bug is now fixed?
Flags: needinfo?(philringnalda)
(Reporter)

Comment 10

a year ago
Indeed, fixed by bug 1365306.
Status: NEW → RESOLVED
Last Resolved: a year ago
status-firefox55: affected → fixed
Flags: needinfo?(philringnalda)
Resolution: --- → FIXED
Target Milestone: --- → mozilla55
You need to log in before you can comment on or make changes to this bug.