1062857 - 3.14% Win7 tp5 regression on Aurora (v.33) August 18 from push f0bdcf487ac4

Status: RESOLVED WONTFIX

(Testing :: Talos, defect)

Description

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

here is a graph showing the regression:
http://graphs.mozilla.org/graph.html#tests=[[255,52,25]]&sel=none&displayrange=90&datatype=running

I did some retriggers:
https://tbpl.mozilla.org/?tree=Mozilla-Aurora&fromchange=74e3bdff5561&tochange=aa20cc59dedf&jobname=Windows%207%2032-bit%20mozilla-aurora%20pgo%20talos%20tp5o

You can see where the tp5o_paint value goes above 208 (and remains there) is from this push:
http://hg.mozilla.org/releases/mozilla-aurora/pushloghtml?fromchange=39a821f3aac9&tochange=f0bdcf487ac4

in that push there are 4 bugs, 1016434 is a backout (there is no corresponding improvement, so I don't suspect it) and the other 3 are suspect.

Comment 1

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

:jandem, :jya, do you have thoughts on the patches you landed and if they would cause a 3% regression in tp5 on windows 7?

Comment 2

[(Away)]

> in that push there are 4 bugs, 1016434 is a backout (there is no
> corresponding improvement, so I don't suspect it) and the other 3 are
> suspect.

The backed-out change (bug 759252) claimed to be good for perf, though. Jaws, would it be plausible that the initial gain got washed out in noise while the backout's regression got noticed? (Maybe combined with whatever changes happened to be riding along)

Comment 3

[Jean-Yves Avenard [:jya]]

my changes had nothing to do with paint. Only changed a value threshold so audio playback would start

Comment 4

[Jared Wein [:jaws] (please needinfo? me)]

(In reply to David Major [:dmajor] from comment #2)
> > in that push there are 4 bugs, 1016434 is a backout (there is no
> > corresponding improvement, so I don't suspect it) and the other 3 are
> > suspect.
> 
> The backed-out change (bug 759252) claimed to be good for perf, though.
> Jaws, would it be plausible that the initial gain got washed out in noise
> while the backout's regression got noticed? (Maybe combined with whatever
> changes happened to be riding along)

Yeah that sounds plausible to me.

Comment 5

[Jan de Mooij [:jandem]]

It *could* be bug 1054359 but I really doubt it. Based on comment 2 and comment 4, bug 759252 seems more likely...

Comment 6

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

:jaws, should we reland on try to see if the backout was the cause.  This is a small regression on a single platform (although an important platform).  If you have other ideas to try out, I would be happy to do that.

Comment 7

[Alessio Placitelli [:Dexter]]

Sorry for the delay. As for bug 1037214, I just changed the way OOM errors are reported in a small portion of code, swapping a |NS_RUNTIMEABORT("OOM")| with error reporting to the script: I'm not touching any paint code (at least directly).

Comment 8

[Jared Wein [:jaws] (please needinfo? me)]

We don't have any plans to reland until OMTA is ready. Given that we just had a patch land and then back out, this seems pretty clear to me.

Comment 9

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

as this is a shipped and small regression, I am going to close this. feel free to reopen if you feel strongly we should keep this open.