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

Issue 760566 link

Starred by 5 users

Issue metadata

Status: WontFix
Owner:
Closed: Sep 2017
Cc:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 2
Type: Bug-Regression



Sign in to add a comment

9.9%-15.4% regression in loading.desktop at 497781:497974

Project Member Reported by pmeenan@chromium.org, Aug 30 2017

Issue description

See the link to graphs below.
 
Project Member

Comment 1 by 42576172...@developer.gserviceaccount.com, Aug 30 2017

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

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


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

chromium-rel-mac-retina
chromium-rel-mac11
Project Member

Comment 3 by 42576172...@developer.gserviceaccount.com, Aug 30 2017

Cc: rsesek@chromium.org
Owner: rsesek@chromium.org
Status: Assigned (was: Untriaged)

=== Auto-CCing suspected CL author rsesek@chromium.org ===

Hi rsesek@chromium.org, the bisect results pointed to your CL, please take a look at the
results.


=== BISECT JOB RESULTS ===
Perf regression found with culprit

Suspected Commit
  Author : Robert Sesek
  Commit : 5cd8eef498e28e1aacb9109645c001bbbfe1bc26
  Date   : Mon Aug 28 19:51:25 2017
  Subject: [Mac] A potential fix for WaitableEvent::WaitMany on macOS 10.11.

Bisect Details
  Configuration: mac_retina_perf_bisect
  Benchmark    : loading.desktop
  Metric       : timeToFirstContentfulPaint_avg/pcv1-cold/AllRecipes
  Change       : 7.71% | 645.443833334 -> 695.234499998

Revision             Result                  N
chromium@497780      645.444 +- 17.9329      6      good
chromium@497839      637.792 +- 9.57121      6      good
chromium@497847      647.113 +- 61.5009      9      good
chromium@497849      638.954 +- 21.6986      6      good
chromium@497850      704.911 +- 40.5145      6      bad       <--
chromium@497851      703.221 +- 42.721       6      bad
chromium@497854      706.438 +- 68.0718      6      bad
chromium@497869      690.487 +- 18.3236      6      bad
chromium@497898      695.234 +- 46.2787      6      bad

To Run This Test
  src/tools/perf/run_benchmark -v --browser=release --output-format=chartjson --upload-results --pageset-repeat=1 --also-run-disabled-tests --story-filter=AllRecipes loading.desktop

More information on addressing performance regressions:
  http://g.co/ChromePerformanceRegressions

Debug information about this bisect:
  https://chromeperf.appspot.com/buildbucket_job_status/8969831206896429728


For feedback, file a bug with component Speed>Bisection
Project Member

Comment 4 by 42576172...@developer.gserviceaccount.com, Aug 30 2017

Cc: dalecur...@chromium.org
 Issue 760631  has been merged into this issue.
Cc: nhiroki@chromium.org
 Issue 761217  has been merged into this issue.
Status: WontFix (was: Assigned)
Nothing to be done here. That CL is a workaround for a kernel panic issue ( bug 756102 ). It's faster on 10.12.
Cc: briander...@chromium.org
 Issue 762215  has been merged into this issue.
 Issue 762175  has been merged into this issue.
 Issue 762275  has been merged into this issue.
 Issue 762292  has been merged into this issue.
Cc: liberato@google.com
 Issue 757333  has been merged into this issue.
Cc: wolenetz@chromium.org xhwang@google.com
 Issue 763486  has been merged into this issue.

Sign in to add a comment