Closed Bug 1542263 Opened 5 years ago Closed 4 years ago

2.61 - 5.65% Heap Unclassified / Images (linux64-shippable, linux64-shippable-qr, windows10-64-shippable, windows7-32-shippable) regression on push 60c20a0f320cc769a8da229cdf6edfbbf38d3ed3 (Wed Apr 3 2019)


(Core :: CSS Parsing and Computation, defect, P2)




Performance Impact low
Tracking Status
firefox-esr60 --- unaffected
firefox-esr68 --- wontfix
firefox69 --- wontfix
firefox70 --- wontfix
firefox71 --- fix-optional


(Reporter: Bebe, Assigned: emilio)




(4 keywords)


(1 obsolete file)

We have detected an awsy regression from push:

As author of one of the patches included in that push, we need your help to address this regression.


6% Heap Unclassified windows7-32-shippable opt 35,823,294.67 -> 37,848,650.37
5% Images linux64-shippable-qr opt stylo tp6 7,758,852.51 -> 8,117,218.04
5% Heap Unclassified windows10-64-shippable opt stylo tp6 62,115,137.20 -> 64,980,197.55
3% Heap Unclassified linux64-shippable opt stylo tp6 96,206,383.49 -> 98,716,540.38

You can find links to graphs and comparison views for each of the above tests at:

On the page above you can see an alert for each affected platform as well as a link to a graph showing the history of scores for this test. There is also a link to a treeherder page showing the jobs in a pushlog format.

To learn more about the regressing test(s), please see:

Blocks: 1534654
Component: General → CSS Parsing and Computation
Product: Testing → Core
Regressed by: 1535788
Version: Version 3 → unspecified
Assignee: nobody → emilio
Flags: needinfo?(emilio)

I'll land the trivial fix for the unclassified memory now. If there's a memory regression from that I can look into it as well.

Flags: needinfo?(emilio)
Keywords: leave-open
Pushed by
Measure the styleset memory usage from the document, not the pres shell.

Florin, mind letting me know if after that patch there's still a memory usage regression compared to before bug 1535788 landed?

If so, I can look into that as well. Otherwise, that should take care of it.


Flags: needinfo?(fstrugariu)
Backout by
Backed out changeset b88166b3b314 for 1401692.html perma failures a=backout
Pushed by
Measure the styleset memory usage from the document, not the pres shell.
Flags: needinfo?(emilio)

Maybe you can also answer the question in comment 3 now that the fix for the unclassified memory has been in for 3 days?

Flags: needinfo?(igoldan)


Looks like that only partially fixed the regressions:

Flags: needinfo?(fstrugariu)

Removing ni?, as Florin provided the answer.

Flags: needinfo?(igoldan)

Ok, I'll work more on it.

Flags: needinfo?(emilio)
Depends on: 1544535

Just one set of stylesheets is enough. While at it, unify SheetType and Origin.

Comment on attachment 9058367 [details]
Bug 1542263 - Don't keep two list of stylesheets in ServoStyleSet. r=#style

Revision D27564 was moved to bug 1544535. Setting attachment 9058367 [details] to obsolete.

Attachment #9058367 - Attachment is obsolete: true
Depends on: 1544546
Depends on: 1544548
Priority: -- → P2

Does it look better now that the dependent bugs have landed? Or do I need to go dig more?

Flags: needinfo?(igoldan)
Flags: needinfo?(fstrugariu)
Flags: needinfo?(emilio)

(In reply to Emilio Cobos Álvarez (:emilio) from comment #15)

Does it look better now that the dependent bugs have landed? Or do I need to go dig more?

I noticed another round of improvements for the Heap regressions. I'm happy considering them fixed.
However, the Images regression remained unchanged.

Flags: needinfo?(igoldan)

Mkay, let me try to dig what can I do here to improve a bit more...

Flags: needinfo?(emilio)
Flags: needinfo?(fstrugariu)

Not actively looking into this.

Flags: needinfo?(emilio)
Whiteboard: [qf]
Whiteboard: [qf] → [qf:p3:resource]

The leave-open keyword is there and there is no activity for 6 months.
:emilio, maybe it's time to close this bug?

Flags: needinfo?(emilio)

WFM per comment 16. we should look into the images regressions, but there are other bugs open for that like bug 1633410.

Closed: 4 years ago
Flags: needinfo?(emilio)
Resolution: --- → WORKSFORME
Has Regression Range: --- → yes
Performance Impact: --- → P3
Whiteboard: [qf:p3:resource]
You need to log in before you can comment on or make changes to this bug.