1853934 - 13.08 - 5.1% perf_reftest coalesce-2.html / perf_reftest coalesce-1.html + 4 more (Windows) regression on Thu September 14 2023

Status: RESOLVED FIXED

(Core :: CSS Parsing and Computation, defect)

Description

[Andrej (:aglavic)]

Perfherder has detected a talos performance regression from push 80b1c6393b00d426ebf22109147720a1bbbf13b1. As author of one of the patches included in that push, we need your help to address this regression.

Regressions:

Ratio	Test	Platform	Options	Absolute values (old vs new)
13%	perf_reftest coalesce-2.html	windows10-64-shippable-qr	e10s fission stylo webrender-sw	21.34 -> 24.13
12%	perf_reftest coalesce-2.html	windows10-64-shippable-qr	e10s fission stylo webrender	21.61 -> 24.24
12%	perf_reftest coalesce-2.html	windows10-64-shippable-qr	e10s fission stylo webrender-sw	21.52 -> 24.04
7%	perf_reftest coalesce-1.html	windows10-64-shippable-qr	e10s fission stylo webrender-sw	58.76 -> 62.60
6%	perf_reftest coalesce-1.html	windows10-64-shippable-qr	e10s fission stylo webrender-sw	59.11 -> 62.93
5%	perf_reftest coalesce-1.html	windows10-64-shippable-qr	e10s fission stylo webrender	59.76 -> 62.80

Details of the alert can be found in the alert summary, including links to graphs and comparisons for each of the affected tests. Please follow our guide to handling regression bugs and let us know your plans within 3 business days, or the offending patch(es) may be backed out in accordance with our regression policy.

If you need the profiling jobs you can trigger them yourself from treeherder job view or ask a sheriff to do that for you.

For more information on performance sheriffing please see our FAQ.

Comment 1

[BugBot [:suhaib / :marco/ :calixte]]

Set release status flags based on info from the regressing bug 1792501

Comment 2

[David Shin[:dshin]]

(Found optimizations based on profiler results, tracked in blocking bug 1854151)

Comment 3

[Donal Meehan [:dmeehan]]

:dshin any follow-up expected here or should this be resolved?
Bug 1854151 has landed and the graphs in the alert summary seem back to the previous values

Comment 4

[David Shin[:dshin]]

Great! Agreed from examining the graph. Thanks!
Closing.