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

Issue 758921 link

Starred by 6 users

Issue metadata

Status: Fixed
Owner:
Closed: Aug 2017
Cc:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 2
Type: Bug-Regression



Sign in to add a comment

226.8% regression in thread_times.key_silk_cases at 496668:496950

Project Member Reported by kraynov@chromium.org, Aug 25 2017

Issue description

See the link to graphs below.
 
Project Member

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

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

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


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

android-nexus7v2
Project Member

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

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

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

Hi bcwhite@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 : Brian White
  Commit : 7662ff8cb53121cd4d8916007d430b36efd45895
  Date   : Thu Aug 24 00:03:13 2017
  Subject: Validate histograms before and after each task on Android.

Bisect Details
  Configuration: android_nexus7_perf_bisect
  Benchmark    : thread_times.key_silk_cases
  Metric       : thread_other_cpu_time_per_frame/http___mobile-news.sandbox.google.com_news_pt0?swipe
  Change       : 160.12% | 0.904445743146 -> 2.35265010684

Revision             Result                     N
chromium@496667      0.904446 +- 0.34436        6      good
chromium@496809      1.14733 +- 1.12556         9      good
chromium@496845      0.980377 +- 0.391796       6      good
chromium@496863      0.928337 +- 0.474311       6      good
chromium@496872      0.876611 +- 0.0959793      6      good
chromium@496876      0.982091 +- 0.414371       6      good
chromium@496877      0.978626 +- 0.450857       6      good
chromium@496878      2.62885 +- 1.12421         6      bad       <--
chromium@496880      3.04048 +- 0.53917         6      bad
chromium@496950      2.35265 +- 2.02868         6      bad

To Run This Test
  src/tools/perf/run_benchmark -v --browser=android-chromium --output-format=chartjson --upload-results --pageset-repeat=1 --also-run-disabled-tests --story-filter=http...mobile.news.sandbox.google.com.news.pt0.swipe thread_times.key_silk_cases

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

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


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

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

 Issue 758952  has been merged into this issue.
Status: Started (was: Assigned)
This is expected.  The CL (https://chromium-review.googlesource.com/619272) is part of an important investigation into a core trampler (#736675 and #744734).  It will be reverted as soon as a Canary can be built to gather information from the field.

Project Member

Comment 6 by 42576172...@developer.gserviceaccount.com, Aug 25 2017

 Issue 758955  has been merged into this issue.
Project Member

Comment 7 by 42576172...@developer.gserviceaccount.com, Aug 25 2017

 Issue 758958  has been merged into this issue.
Project Member

Comment 8 by 42576172...@developer.gserviceaccount.com, Aug 25 2017

 Issue 758960  has been merged into this issue.
Project Member

Comment 9 by 42576172...@developer.gserviceaccount.com, Aug 25 2017

 Issue 758961  has been merged into this issue.
Project Member

Comment 10 by 42576172...@developer.gserviceaccount.com, Aug 25 2017

 Issue 758967  has been merged into this issue.
Project Member

Comment 11 by 42576172...@developer.gserviceaccount.com, Aug 25 2017

 Issue 758973  has been merged into this issue.
Project Member

Comment 12 by 42576172...@developer.gserviceaccount.com, Aug 25 2017

 Issue 758974  has been merged into this issue.
Project Member

Comment 13 by 42576172...@developer.gserviceaccount.com, Aug 25 2017

 Issue 758976  has been merged into this issue.
Project Member

Comment 14 by 42576172...@developer.gserviceaccount.com, Aug 25 2017

 Issue 759048  has been merged into this issue.
Project Member

Comment 15 by 42576172...@developer.gserviceaccount.com, Aug 25 2017

 Issue 758992  has been merged into this issue.
Project Member

Comment 16 by 42576172...@developer.gserviceaccount.com, Aug 25 2017

Cc: chcunningham@chromium.org
 Issue 759101  has been merged into this issue.
Project Member

Comment 17 by 42576172...@developer.gserviceaccount.com, Aug 26 2017

 Issue 759104  has been merged into this issue.
Project Member

Comment 18 by 42576172...@developer.gserviceaccount.com, Aug 26 2017

 Issue 759030  has been merged into this issue.
Project Member

Comment 19 by 42576172...@developer.gserviceaccount.com, Aug 26 2017

Issue 758887 has been merged into this issue.
Project Member

Comment 20 by 42576172...@developer.gserviceaccount.com, Aug 26 2017

Issue 758908 has been merged into this issue.
Status: Fixed (was: Started)
The offending CL has been reverted (according to plan).
https://chromium-review.googlesource.com/c/chromium/src/+/636165

Project Member

Comment 22 by 42576172...@developer.gserviceaccount.com, Aug 26 2017

Issue 758959 has been merged into this issue.
Project Member

Comment 23 by 42576172...@developer.gserviceaccount.com, Aug 26 2017

Issue 758945 has been merged into this issue.
Project Member

Comment 24 by 42576172...@developer.gserviceaccount.com, Aug 28 2017

 Issue 759556  has been merged into this issue.
Project Member

Comment 25 by 42576172...@developer.gserviceaccount.com, Aug 28 2017

 Issue 759558  has been merged into this issue.
Project Member

Comment 26 by 42576172...@developer.gserviceaccount.com, Aug 28 2017

Cc: tebbi@chromium.org
 Issue 759467  has been merged into this issue.
Project Member

Comment 27 by 42576172...@developer.gserviceaccount.com, Aug 29 2017

Cc: pmeenan@chromium.org
 Issue 760172  has been merged into this issue.
Project Member

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

Cc: u...@chromium.org
 Issue 760517  has been merged into this issue.
Project Member

Comment 29 by 42576172...@developer.gserviceaccount.com, Aug 31 2017

 Issue 760680  has been merged into this issue.
Project Member

Comment 30 by 42576172...@developer.gserviceaccount.com, Sep 21 2017

Cc: mvanouwe...@chromium.org erikc...@chromium.org
 Issue 759609  has been merged into this issue.
 Issue 759645  has been merged into this issue.

Comment 32 by u...@chromium.org, Oct 11 2017

Cc: -u...@chromium.org

Sign in to add a comment