New issue
Advanced search Search tips

Issue 648168 link

Starred by 1 user

Issue metadata

Status: Duplicate
Merged: issue 647609
Owner:
Closed: Sep 2016
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 2
Type: Bug-Regression



Sign in to add a comment

30.2% regression in v8.browsing_desktop_ignition at 418878:418955

Project Member Reported by ulan@google.com, Sep 19 2016

Issue description

See the link to graphs below.
 

Comment 1 by ulan@google.com, Sep 19 2016

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

Original alerts at time of bug-filing:
  https://chromeperf.appspot.com/group_report?keys=agxzfmNocm9tZXBlcmZyFAsSB0Fub21hbHkYgICguaCKvQkM


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

chromium-rel-win7-x64-dual
Project Member

Comment 3 by sheriffbot@chromium.org, Sep 19 2016

Labels: Hotlist-Google
Project Member

Comment 4 by 42576172...@developer.gserviceaccount.com, Sep 20 2016

Mergedinto: 647609
Status: Duplicate (was: Assigned)

===== BISECT JOB RESULTS =====
Status: completed


===== SUSPECTED CL(s) =====
Subject : Revert of [heap] Decouple old generation allocation limit from external memory. (patchset #1 id:1 of https://codereview.chromium.org/2329993002/ )
Author  : ulan
Commit description:
  
Reason for revert:
Regressions in telemetry benchmarks:
 crbug.com/646819 .

Original issue's description:
> [heap] Decouple old generation allocation limit from external memory.
>
> We check for external memory limit in Heap::ReportExternalMemoryPressure.
>
> BUG= chromium:616434 
>
> Committed: https://crrev.com/672d079ccba686019fa1457c83b42c2e692ef88b
> Cr-Commit-Position: refs/heads/master@{#39374}

TBR=hpayer@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG= chromium:616434 

Review-Url: https://codereview.chromium.org/2339033005
Cr-Commit-Position: refs/heads/master@{#39442}
Commit  : eeb4f28aed721c343d43be6e454fafba4c0f7ea6
Date    : Thu Sep 15 10:14:22 2016


===== TESTED REVISIONS =====
Revision                       Mean     Std Dev  N  Good?
chromium@418877                275.225  14.5386  8  good
chromium@418887                284.332  21.487   8  good
chromium@418890                282.719  17.1238  8  good
chromium@418890,v8@f84f45f9ca  290.373  16.2474  8  good
chromium@418890,v8@eeb4f28aed  344.243  22.3365  8  bad    <--
chromium@418890,v8@b37daacd6b  338.457  10.8935  5  bad
chromium@418890,v8@b848716c98  330.746  14.7733  8  bad
chromium@418891                352.197  16.736   5  bad
chromium@418892                335.929  15.0617  8  bad
chromium@418897                325.636  9.93018  5  bad
chromium@418916                323.498  19.2287  5  bad
chromium@418955                346.335  20.5224  5  bad

Bisect job ran on: win_x64_perf_bisect
Bug ID: 648168

Test Command: src/tools/perf/run_benchmark -v --browser=release_x64 --output-format=chartjson --upload-results --also-run-disabled-tests v8.browsing_desktop_ignition
Test Metric: v8-gc-incremental-step_sum/v8-gc-incremental-step_sum
Relative Change: 27.57%
Score: 99.9

Buildbot stdio: http://build.chromium.org/p/tryserver.chromium.perf/builders/win_x64_perf_bisect/builds/1473
Job details: https://chromeperf.appspot.com/buildbucket_job_status/9001105135473765728


Not what you expected? We'll investigate and get back to you!
  https://chromeperf.appspot.com/bad_bisect?try_job_id=5856325601329152

| O O | Visit http://www.chromium.org/developers/speed-infra/perf-bug-faq
|  X  | for more information addressing perf regression bugs. For feedback,
| / \ | file a bug with component Tests>AutoBisect.  Thank you!

Sign in to add a comment