1489441 - 2.33% displaylist_mutate (windows10-64-qr) regression on push 58de9b24ec33e5fb7de1e55ccd3128d0c3fc5b2c (Wed Sep 5 2018)

Status: VERIFIED FIXED

(Testing :: General, defect)

Description

[Joel Maher ( :jmaher ) (UTC -8)]

Talos has detected a Firefox performance regression from push:

https://hg.mozilla.org/integration/mozilla-inbound/pushloghtml?changeset=58de9b24ec33e5fb7de1e55ccd3128d0c3fc5b2c

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

Regressions:

  2%  displaylist_mutate windows10-64-qr opt e10s stylo     7,428.81 -> 7,601.74


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

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

[Joel Maher ( :jmaher ) (UTC -8)]

:jrmuizel, I see that you updated webrender in bug 1487885, can you confirm this is an expected performance regression?

Comment 2

[Joel Maher ( :jmaher ) (UTC -8)]

:jrmuizel, I don't know if you are working on this, can you give an update on the current state of investigating/fixing these regressions?

Comment 3

[Jeff Muizelaar [:jrmuizel]]

We're keeping this in mind. Since we haven't released WebRender yet I don't think you need to track this and we won't be backing out the change. We'll be tracking WebRender's displaylist_mutate performance as part of the release criteria.

Comment 4

[Ionuț Goldan [:igoldan]]

:jrmuizel I see the current baseline is way lower than the regression we noticed in this bug. I'm closing this bug as resolved.