https://treeherder.allizom.org/perf.html#/compare?originalProject=mozilla-inbound&originalRevision=151695836c37&newProject=mozilla-inbound&newRevision=592fc90e655a&framework=1&filter=paint%20e10s it looks like we have winxp, win8 ts_paint e10s regressions. the range here is interesting, there was a build failure between: https://treeherder.mozilla.org/#/jobs?repo=mozilla-inbound&fromchange=151695836c37&tochange=a0355cdb6bdb&filter-searchStr=mozilla-inbound%20talos%20other it is either one of bug 1000780 or 948267. I am trying to narrow that down as 948267 has actual builds.
tracking-e10s: --- → -
looking back into this- I finally have a win8 build going for the missing revision. A few pushes later there was bug 1240164 on the same test- that is why this looked suspicious. I should have data by the end of the day.
well, I am unable to push to try and get a build with the revisions missing data, nor can I fill in on treeherder :(
Product: Testing → Core
I have ruled out bug 948267 with a try push (https://treeherder.mozilla.org/#/jobs?repo=try&revision=b9edbbd3e5c1&selectedJob=15894992), this leaves bug 1000780 as the culprit for this regression. From reading the bug, it looks as though it is known there are other performance issues. I know this is 2 weeks later- sort of hard to expect a quick fix, but I would like to confirm this is expected and document that along with any work done to fix this.
Hmm, this isn't really expected, no. The test doesn't make use of Function#bind at all, AFAICT, and even if it did, it'd be highly unlikely to do so in a way that'd hit the perf cliffs mentioned in bug 1000780. Joel, would it be possible to bisect this further? Specifically, testing with rccfd55971698 would be interesting, as the parts up to and including that one are preparatory and don't actually change Function#bind itself. One possibility is that this is simply the time it takes to compile the added self-hosted JS. If that's the case, we can't immediately do anything about it. We do have plans to look into the startup costs associated with that, though, so medium-term, we should see improvements here. (If that really is the cause, that is.)
Flags: needinfo?(till) → needinfo?(jmaher)
have some try pushes: https://firstname.lastname@example.org&fromchange=8ff9643f16cb&tochange=b092c135defa will verify them in an hour or so!
odd, the builds are not working: c:/builds/moz2_slave/try-w64-0000000000000000000000/build/src/dom/workers/WorkerPrivate.cpp(2213) : error C2614: 'mozilla::dom::workers::WorkerPrivateParent<mozilla::dom::workers::WorkerPrivate>' : illegal member initialization: 'mNowBaseTimeHighRes' is not a base or member Return code: 1 tips on how to fix this? I don't see how it is related to the code changed- maybe I retrigger?
Flags: needinfo?(jmaher) → needinfo?(till)
Huh, that's certainly not related, no. A retrigger sounds good, yes.
Flags: needinfo?(till) → needinfo?(jmaher)
got it, I overlooked the obvious that there is a build breakage in the patch right before this set of patches landed, and was backed out right afterwards. Did a new set of pushes: https://email@example.com&fromchange=c25ba7f40040&tochange=d0ca1cf180ca for reference, here is the original commit: https://hg.mozilla.org/integration/mozilla-inbound/pushloghtml?changeset=f29f1d9a3cd3 lets give this ~2 hours for builds and results.
part 5 is the culprit: https://hg.mozilla.org/integration/mozilla-inbound/rev/f29f1d9a3cd3
(In reply to Joel Maher (:jmaher) from comment #9) > part 5 is the culprit: > https://hg.mozilla.org/integration/mozilla-inbound/rev/f29f1d9a3cd3 In that case, I don't think there's much we can do immediately. Presumably, this only affects e10s because the content process is started only when a tab is loaded, so the parsing and bytecode compilation for the self-hosted JS is part of the measured time. As I said before, we do have plans to tackle this, but they're not specific to this regression at all, so I'd argue we should live with this for now.
Thanks :till, lets mark this as wontfix and look for wins in the future!
Status: NEW → RESOLVED
Last Resolved: 2 years ago
Resolution: --- → WONTFIX
You need to log in before you can comment on or make changes to this bug.