1386596 - 1.51 - 4.76% a11yr / tp5o (windows7-32) regression on push e9ec92fd4f47d69cec602e81bb283d045b6bd87a (Sat Jul 15 2017)

Status: RESOLVED FIXED

(Core :: CSS Parsing and Computation, defect)

Description

[Ionuț Goldan [:igoldan]]

Talos has detected a Firefox performance regression from push:

https://hg.mozilla.org/integration/autoland/pushloghtml?changeset=e9ec92fd4f47d69cec602e81bb283d045b6bd87a

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

Regressions:

  5%  a11yr summary windows7-32 opt e10s     737.29 -> 772.42
  2%  tp5o summary windows7-32 opt e10s      338.11 -> 343.22


You can find links to graphs and comparison views for each of the above tests at: https://treeherder.mozilla.org/perf.html#/alerts?id=8485

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 Talos jobs in a pushlog format.

To learn more about the regressing test(s), please see: https://wiki.mozilla.org/Buildbot/Talos/Tests

For information on reproducing and debugging the regression, either on try or locally, see: https://wiki.mozilla.org/Buildbot/Talos/Running

*** Please let us know your plans within 3 business days, or the offending patch(es) will be backed out! ***

Our wiki page outlines the common responses and expectations: https://wiki.mozilla.org/Buildbot/Talos/RegressionBugsHandling

Comment 1

[Ionuț Goldan [:igoldan]]

:emilio These regression alerts triggered rather late. I filed this bug for tracking reasons. Treat it like you treat the other related bug 1381844.

Comment 2

[Emilio Cobos Álvarez [:emilio]]

Yeah, the right fix for those is coming. If there's still a regression, I can still downgrade the hottest ones to debug only.

Comment 3

[Ryan VanderMeulen [:RyanVM]]

AFAICT, these regressions are still present.

Comment 4

[Liz Henry (:lizzard) (privacy/fingerprinting team)]

Emilio, any luck here? Do you still see the perf regression in 56? Thanks.

Comment 5

[Emilio Cobos Álvarez [:emilio]]

Should be fixed given we moved the assertions back to debug-only in bug 1381844, and it's the same push the one that caused it.