Closed Bug 1932110 Opened 3 months ago Closed 3 months ago

0.34 - 0.28% Base Content JS / Base Content JS + 2 more (Linux, OSX, Windows) regression on Thu November 14 2024

Categories

(Core :: JavaScript Engine, defect)

defect

Tracking

()

RESOLVED WONTFIX
Tracking Status
firefox-esr128 --- unaffected
firefox132 --- unaffected
firefox133 --- unaffected
firefox134 --- affected

People

(Reporter: intermittent-bug-filer, Unassigned)

References

(Regression)

Details

(Keywords: perf, perf-alert, regression)

Perfherder has detected a awsy performance regression from push a5cc817a57593d97a2aec6fa489459b10b83e51a. 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)
0.34% Base Content JS linux1804-64-shippable-qr fission 1,517,003.00 -> 1,522,106.00
0.33% Base Content JS windows11-64-2009-shippable-qr fission 1,523,184.00 -> 1,528,160.00
0.31% Base Content JS macosx1015-64-shippable-qr fission 1,517,332.33 -> 1,522,007.33
0.28% Base Content JS linux1804-64-shippable-qr fission 1,517,331.33 -> 1,521,555.33

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 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.

You can run all of these tests on try with ./mach try perf --alert 42719

The following documentation link provides more information about this command.

For more information on performance sheriffing please see our FAQ.

If you have any questions, please do not hesitate to reach out to fbilt@mozilla.com.

Flags: needinfo?(debadree333)

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

Hello i am working on this presently, trying to debug!

Flags: needinfo?(debadree333)

Hello i am still on the investigation its taking a little bit more time but probable cause seems to be the addition of new globals by the proposal.

This is a small regression so it is fine as long as you understand what the cause is, and it is something that is expected and required by the change.

This is an expected and acceptable outcome of landing the original patch. Closing as WONTFIX.

Status: NEW → RESOLVED
Closed: 3 months ago
Resolution: --- → WONTFIX
You need to log in before you can comment on or make changes to this bug.