New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 844618 link

Starred by 2 users

Issue metadata

Status: WontFix
Owner: ----
Closed: May 2018
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 2
Type: Bug-Regression



Sign in to add a comment

11.4%-18% regression in blink_perf.dom at 558246:558641

Project Member Reported by npm@chromium.org, May 18 2018

Issue description

See the link to graphs below.
 
Project Member

Comment 1 by 42576172...@developer.gserviceaccount.com, May 18 2018

All graphs for this bug:
  https://chromeperf.appspot.com/group_report?bug_id=844618

(For debugging:) Original alerts at time of bug-filing:
  https://chromeperf.appspot.com/group_report?sid=504e002704eb16991848373287995c18b3a396fce4f886dd2b81f76a7c1f288d


Bot(s) for this bug's original alert(s):

android-nexus5
android-one
chromium-rel-mac11-air
chromium-rel-mac11-pro
linux-perf
Project Member

Comment 3 by 42576172...@developer.gserviceaccount.com, May 22 2018

Cc: falken@chromium.org y...@yoav.ws v8-autoroll@chromium.org afdo-chr...@skia-buildbots.google.com.iam.gserviceaccount.com gsat...@chromium.org
Owner: falken@chromium.org
Status: Assigned (was: Untriaged)
📍 Found significant differences after each of 5 commits.
https://pinpoint-dot-chromeperf.appspot.com/job/12ea17b4240000

Roll AFDO from 68.0.3430.0_rc-r1 to 68.0.3431.0_rc-r1 by afdo-chromium-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com
https://chromium.googlesource.com/chromium/src/+/ed75d2932ed7eaf95fafa505007de464316dbb0b

Replace MoveResourceLoaderNonBlocking CHECK with DCHECK by yoav@yoav.ws
https://chromium.googlesource.com/chromium/src/+/cb1d73966b7fff0bd5a28a64514b2ab4ced0ea81

[hash-table] Inline GetHash and GetSimpleHash by gsathya@chromium.org
https://chromium.googlesource.com/v8/v8/+/0e8b67ce7f04115cc56903178f277cba75efbc21

Update V8 to version 6.8.225. by v8-autoroll@chromium.org
https://chromium.googlesource.com/chromium/src/+/7e54a0b17cab5f74d0a79b0069718b76205c97d3

service worker: Enable ServiceWorkerRegistration#updateViaCache by default. by falken@chromium.org
https://chromium.googlesource.com/chromium/src/+/39970a79fb0f27170b039a0eb371ed8c36ece484

Understanding performance regressions:
  http://g.co/ChromePerformanceRegressions

Comment 4 by falken@chromium.org, May 22 2018

Components: Blink>DOM
Owner: ----
Status: Untriaged (was: Assigned)
I don't think this can be my change which only affects service worker updates. The regressed test is single-select-add and textarea-dom which I don't expect to use service workers.

Comment 6 by fergal@chromium.org, May 23 2018

Cc: fergal@chromium.org g...@chromium.org
Components: -Blink>DOM Internals>Skia
Owner: g...@chromium.org
The large jump in the graph occurs at 558621 (https://crrev.com/c/1058846).

The perf benchmark in an android one and the CL is changing the rev of something on Android.

Assigning to gbiv@ as the sheriff.

Comment 7 by g...@chromium.org, May 23 2018

I feel like I've received more bugs for AFDO rolls in random blink bits in the last week than I have since it landed in January. :)

So, for background, AFDO changes how the compiler decides to optimize by feeding samples of how Chrome executes back into the compiler. Since blink benchmarks, AIUI, are pretty sensitive to individual optimization decisions, noise in the profile gathering process can cause swings every now and then. So, this regression might disappear in one of the next few rolls, which happen roughly daily.

That said, my goal at the moment is to find a way to make the blink flakiness with AFDO stop. Even if this fixes itself, I'll try to fold it into that effort.

In this case, I only see three substantial instances of presumably-AFDO-caused-flakiness since we landed AFDO 5ish months ago.

(I'm also unsure if the select-single-add regression is caused by AFDO, since the AFDO rolls near it were in `560068 - 560141`, which was after the drop and `558202 - 558245`, which was a point before the large drop. I'll look into it to confirm.)
Project Member

Comment 8 by 42576172...@developer.gserviceaccount.com, May 28 2018

Cc: hta@chromium.org szuend@google.com bshe@chromium.org u...@chromium.org
Owner: bshe@chromium.org
Status: Assigned (was: Untriaged)
📍 Found significant differences after each of 4 commits.
https://pinpoint-dot-chromeperf.appspot.com/job/14dceafc240000

Add use counter for sorting non-packed JSArrays by szuend@google.com
https://chromium.googlesource.com/chromium/src/+/07c9bc6ed5c27bfedee1d7f52ed7f732c6af9e81

Enable sdpSemantic RTCPeerConnection parameter by hta@chromium.org
https://chromium.googlesource.com/chromium/src/+/02bd2d220545367bbad5f2dd02d78f1d50836cec

Initial handling of bloated V8 heaps near the heap limit. by ulan@chromium.org
https://chromium.googlesource.com/chromium/src/+/fc84822a48a47a20858c43f794b408a156df97d1

Split ContentInputDelegate to PlatformUiInputDelegate and ContentInputDelegate by bshe@chromium.org
https://chromium.googlesource.com/chromium/src/+/41db7d269b73979eaf2bbc24e457319da7c74b15

Understanding performance regressions:
  http://g.co/ChromePerformanceRegressions

Comment 9 by bshe@chromium.org, May 28 2018

Owner: hta@chromium.org
My CL didn't touch blink code and it was a simple refactor. It shouldn't affect blink perf. The other three CLs all changed blink. Not sure which one is more likely. Randomly select hta@. It looks like you enabled a new feature in the list CL above, do you mind to take a look?

Comment 10 by hta@chromium.org, May 28 2018

Owner: u...@chromium.org
The flag flip only exposed the flag - its default value didn't change - so shouldn't have any effect.
Randomly passing the baton to ulan@.

Comment 11 by u...@chromium.org, May 29 2018

Owner: ----
Status: WontFix (was: Assigned)
My CL is behind a flag and is a no-op for bots.

In any case, looks like the graphs have recovered. Closing the issue.

Sign in to add a comment