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

Issue 731648 link

Starred by 4 users

Issue metadata

Status: WontFix
Owner:
Last visit > 30 days ago
Closed: Jun 2017
Cc:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 2
Type: Bug-Regression



Sign in to add a comment

3.7% regression in memory.long_running_idle_gmail_tbmv2 at 477620:477751

Project Member Reported by primiano@chromium.org, Jun 9 2017

Issue description

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

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


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

chromium-rel-win7-dual
Cc: hpayer@chromium.org
Owner: hpayer@chromium.org

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

Hi hpayer@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 : hpayer
  Commit : cf8f7bdc9d5ee314a10fe043e421ec95604d4eca
  Date   : Wed Jun 07 13:21:28 2017
  Subject: [heap] Increase memory reducer activation limit for external memory changes.

Bisect Details
  Configuration: win_perf_bisect
  Benchmark    : memory.long_running_idle_gmail_tbmv2
  Metric       : memory:chrome:renderer_processes:reported_by_chrome:v8:effective_size_avg/memory:chrome:renderer_processes:reported_by_chrome:v8:effective_size_avg
  Change       : 1.81% | 60003507.2249 -> 61087124.3564

Revision                           Result                   N
chromium@477619                    60003507 +- 2937092      14      good
chromium@477652                    60206861 +- 1561100      9       good
chromium@477669                    59819674 +- 1396129      6       good
chromium@477677                    59450117 +- 1361260      6       good
chromium@477677,v8@f2cd359ac9      59413575 +- 2358249      9       good
chromium@477677,v8@d2066154c8      59908754 +- 5089685      9       good
chromium@477677,v8@cf8f7bdc9d      60755079 +- 1543666      9       bad       <--
chromium@477678                    60956012 +- 2251198      9       bad
chromium@477679                    60891609 +- 760353       6       bad
chromium@477681                    60623501 +- 973590       9       bad
chromium@477685                    61164371 +- 1093626      6       bad
chromium@477751                    61087124 +- 1906512      9       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 memory.long_running_idle_gmail_tbmv2

Debug Info
  https://chromeperf.appspot.com/buildbucket_job_status/8977268476280012800

Is this bisect wrong?
  https://chromeperf.appspot.com/bad_bisect?try_job_id=4894043508572160


| 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 Speed>Bisection.  Thank you!
Cc: u...@chromium.org ulan@google.com
 Issue 731705  has been merged into this issue.
Project Member

Comment 5 by 42576172...@developer.gserviceaccount.com, Jun 10 2017

 Issue 731061  has been merged into this issue.
Project Member

Comment 6 by 42576172...@developer.gserviceaccount.com, Jun 10 2017

 Issue 731651  has been merged into this issue.

Comment 7 by hpayer@chromium.org, Jun 12 2017

Status: WontFix (was: Untriaged)
This change reduced the memory savings of a CL that landed a few days earlier but was triggering garbage collection too aggressively.
Project Member

Comment 8 by 42576172...@developer.gserviceaccount.com, Jun 12 2017

Cc: alexclarke@chromium.org
 Issue 732298  has been merged into this issue.
Project Member

Comment 9 by 42576172...@developer.gserviceaccount.com, Jun 14 2017

Cc: mlippautz@chromium.org
 Issue 733114  has been merged into this issue.
Project Member

Comment 10 by 42576172...@developer.gserviceaccount.com, Jun 16 2017

 Issue 733114  has been merged into this issue.

Sign in to add a comment