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

Issue 917259 link

Starred by 3 users

Issue metadata

Status: Assigned
Owner:
Last visit > 30 days ago
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 2
Type: Bug-Regression



Sign in to add a comment

23.7% regression in blink_perf.bindings at 616904:616965

Project Member Reported by kouhei@google.com, Dec 21

Issue description

See the link to graphs below.
 
All graphs for this bug:
  https://chromeperf.appspot.com/group_report?bug_id=917259

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


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

mac-10_13_laptop_high_end-perf

blink_perf.bindings - Benchmark documentation link:
  https://bit.ly/blink-perf-benchmarks
📍 Couldn't reproduce a difference.
https://pinpoint-dot-chromeperf.appspot.com/job/14b3d926140000
Cc: carlscab@google.com
Owner: carlscab@google.com
Status: Assigned (was: Untriaged)
📍 Found a significant difference after 1 commit.
https://pinpoint-dot-chromeperf.appspot.com/job/149b765e940000

Add SequenceManager to field trial testing config by carlscab@google.com
https://chromium.googlesource.com/chromium/src/+/84ecbf7c0b1eca668dba1471a50f065c6d558737
post-message: 295.3 → 366 (+70.68)

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

Benchmark documentation link:
  https://bit.ly/blink-perf-benchmarks
Cc: skyos...@chromium.org alexclarke@chromium.org
📍 Couldn't reproduce a difference.
https://pinpoint-dot-chromeperf.appspot.com/job/14fc5bf1940000
This is strange the regression only seems to be on one mac bot, although I note one of the windows bots got more noisy around the same time.  Anyway I instrumented chrome's task posting and ran this test and it seems to do a lot of cross thread immediate task posting.  It's not obvious why that would be worse one the high end mac laptop.

https://chromeperf.appspot.com/report?sid=3d0f4253a54d17811ff5de630a4d60eb9de3b6452769e7c7574a27476585e77c
Hmm I was misreading the graph, the regression is on the low end bot too.
Cc: majidvp@google.com
 Issue 920372  has been merged into this issue.
Owner: alexclarke@chromium.org
Project Member

Comment 13 by 42576172...@developer.gserviceaccount.com, Yesterday (42 hours ago)

Project Member

Comment 14 by 42576172...@developer.gserviceaccount.com, Yesterday (36 hours ago)

Cc: altimin@chromium.org chrishall@chromium.org bashi@chromium.org
📍 Found significant differences after each of 3 commits.
https://pinpoint-dot-chromeperf.appspot.com/job/163c4d2e540000

[scheduler] Remove expensive task blocking logic. by altimin@chromium.org
https://chromium.googlesource.com/chromium/src/+/071306963377eb133a94b80a6bbcce949f99e7b8
post-message: 291.4 → 265.1 (-26.29)

Adding feature flag for 'accessibility language detection'. by chrishall@chromium.org
https://chromium.googlesource.com/chromium/src/+/4a4225f68380d640a87837dd4fa11827192bb4ef
post-message: 275.6 → 270.7 (-4.807)

CacheStorage: Hold a reference to CacheStorage from Cache by bashi@chromium.org
https://chromium.googlesource.com/chromium/src/+/7bc86423ba2bdd56b15a5cf44089043142326dda
post-message: 267 → 275.8 (+8.824)

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

Benchmark documentation link:
  https://bit.ly/blink-perf-benchmarks

Comment 15 by skyos...@chromium.org, Today (20 hours ago)

The mac regression seems to be resolved now by the removal of the expensive task blocking logic (although the bot[1] hasn't run the test since Friday for some reason). I don't have any theories as to why that patch would have had this impact on performance -- ideas welcome :) The regression seems to have been mac-only too [2].

[1] https://chromeperf.appspot.com/group_report?bug_id=917259
[2] https://chromeperf.appspot.com/report?sid=9d2787fac49c8efd202af8ecff80d934976c5c3795dd5d0af5237ce2185abbb9&start_rev=620208&end_rev=624732

Comment 16 by alexclarke@google.com, Today (20 hours ago)

That's unexpected.  The task cost estimators might account for at least some of the improvement.
Project Member

Comment 17 by 42576172...@developer.gserviceaccount.com, Today (17 hours ago)

Project Member

Comment 18 by 42576172...@developer.gserviceaccount.com, Today (15 hours ago)

📍 Couldn't reproduce a difference.
https://pinpoint-dot-chromeperf.appspot.com/job/17ddd21a540000

Sign in to add a comment