Re: Perf regression coming (and going away) on Linux Release

3 views
Skip to first unread message

Primiano Tucci

unread,
Mar 10, 2016, 2:43:11 PM3/10/16
to Perf Sheriffs
tracing@ -> perf-sheriffs@
Apologies if you received this twice, I confused the lists, this was meant for sheriffs.

On Thu, Mar 10, 2016 at 7:33 PM Primiano Tucci <prim...@chromium.org> wrote:
TL;DR
If you are not perf-sheriffing you can stop reading here.
crrev.com/1675143004 (#380196) caused a perf regression on linux-release.
crrev.com/1777363002 should fix it

As speculated in the commit message, crrev.com/1675143004 ended up causing a visible across-the board perf regression (example) due to crbug.com/593344
I was hoping that would have not been visible (the local benchmarks I tried before landing were noisy), but that was not the case.
Not sure if the GASP alert system realized that already, I think it will do that soon. The regression is linux-only.
I have a cure pending review in, that should fix it crrev.com/1777363002.

So, if you are sheriffing and see an across-the-board perf regression on Linux around #380196, (I guess all page cyclers might be affected) that's me and should go away once crrev.com/1777363002 lands.
Apology for the inconvenience, I tried to be as much proactive as possible but... perf benchmarks are hard.

Ryan Schoen

unread,
Mar 10, 2016, 2:53:15 PM3/10/16
to Primiano Tucci, rmci...@chromium.org, pic...@chromium.org, Perf Sheriffs
+regression sheriffs for today and tomorrow

--
You received this message because you are subscribed to the Google Groups "Perf Sheriffs" group.
To unsubscribe from this group and stop receiving emails from it, send an email to perf-sheriff...@chromium.org.

Reply all
Reply to author
Forward
0 new messages