New issue
Advanced search Search tips

Issue 691659 link

Starred by 1 user

Issue metadata

Status: Duplicate
Merged: issue 691618
Owner:
Closed: May 2017
Cc:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 2
Type: Bug-Regression



Sign in to add a comment

3.8% regression in system_health.memory_mobile at 449662:449712

Project Member Reported by majidvp@google.com, Feb 13 2017

Issue description

See the link to graphs below.
 

Comment 1 by majidvp@google.com, Feb 13 2017

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

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


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

android-webview-nexus6
Project Member

Comment 3 by 42576172...@developer.gserviceaccount.com, Feb 14 2017


=== BISECT JOB RESULTS ===
Perf regression found but unable to narrow commit range

Build failures prevented the bisect from narrowing the range further.


Bisect Details
  Configuration: android_webview_nexus6_aosp_perf_bisect
  Benchmark    : system_health.memory_mobile
  Metric       : memory:webview:all_processes:reported_by_os:system_memory:native_heap:proportional_resident_size_avg/load_games/load_games_bubbles
  Change       : 1.15% | 32147728.0 -> 32516083.5556

Suspected Commit Range
  6 commits in range
  https://chromium.googlesource.com/chromium/src/+log/5fb4064595996e5f0ae942d928e3bccc5209bca7..04dc38a467ade5dbac587f40ffeb6bc941dde896


Revision             Result                   N
chromium@449661      32147728 +- 1490233      14       good
chromium@449668      32408921 +- 1288178      14       good
chromium@449669      ---                      ---      build failure
chromium@449670      ---                      ---      build failure
chromium@449671      ---                      ---      build failure
chromium@449672      ---                      ---      build failure
chromium@449673      ---                      ---      build failure
chromium@449674      32691984 +- 1502009      14       bad
chromium@449687      32470572 +- 480543       9        bad
chromium@449712      32516084 +- 452042       9        bad

Please refer to the following doc on diagnosing memory regressions:
  https://chromium.googlesource.com/chromium/src/+/master/docs/memory-infra/memory_benchmarks.md

To Run This Test
  src/tools/perf/run_benchmark -v --browser=android-webview --output-format=chartjson --upload-results --pageset-repeat=1 --also-run-disabled-tests --story-filter=load.games.bubbles system_health.memory_mobile

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

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


| 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!
Labels: Performance-Memory
Project Member

Comment 6 by 42576172...@developer.gserviceaccount.com, Apr 27 2017


=== BISECT JOB RESULTS ===
Bisect failed for unknown reasons

Please contact the team (see below) and report the error.


Bisect Details
  Configuration: android_webview_nexus6_aosp_perf_bisect
  Benchmark    : system_health.memory_mobile
  Metric       : memory:webview:all_processes:reported_by_chrome:malloc:effective_size_avg/load_games/load_games_bubbles


Please refer to the following doc on diagnosing memory regressions:
  https://chromium.googlesource.com/chromium/src/+/master/docs/memory-infra/memory_benchmarks.md

To Run This Test
  src/tools/perf/run_benchmark -v --browser=android-webview --output-format=chartjson --upload-results --pageset-repeat=1 --also-run-disabled-tests --story-filter=load.games.bubbles system_health.memory_mobile

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

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


| 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!
Project Member

Comment 8 by 42576172...@developer.gserviceaccount.com, Apr 27 2017

Cc: fdoray@chromium.org
Owner: fdoray@chromium.org

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

Hi fdoray@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 : fdoray
  Commit : 24e5d1ddee9fa61587b6f0279a25792d0208b9f1
  Date   : Wed Apr 12 19:58:13 2017
  Subject: Revert of Use TaskScheduler instead of WorkerPool in BackgroundTaskRunner.cpp (patchset #1 id:1 of https://codereview.chromium.org/2685673005/ )

Bisect Details
  Configuration: android_webview_nexus6_aosp_perf_bisect
  Benchmark    : system_health.memory_mobile
  Metric       : memory:webview:all_processes:reported_by_chrome:malloc:effective_size_avg/load_games/load_games_bubbles
  Change       : 4.19% | 27059054.0 -> 25727446.4444

Revision             Result                   N
chromium@462278      27059054 +- 658214       6       good
chromium@463825      27034452 +- 1309260      9       good
chromium@464019      27387211 +- 785117       6       good
chromium@464116      27102862 +- 1323437      9       good
chromium@464119      27062275 +- 1100534      6       good
chromium@464120      26870142 +- 912742       6       good
chromium@464121      25883698 +- 1847534      9       bad       <--
chromium@464122      25781847 +- 3803826      14      bad
chromium@464128      25544368 +- 1733797      9       bad
chromium@464140      25635918 +- 2084495      6       bad
chromium@464164      25687049 +- 1578184      9       bad
chromium@464212      25628747 +- 1322643      6       bad
chromium@464599      25875048 +- 4161877      14      bad
chromium@465372      25727446 +- 1356943      9       bad

Please refer to the following doc on diagnosing memory regressions:
  https://chromium.googlesource.com/chromium/src/+/master/docs/memory-infra/memory_benchmarks.md

To Run This Test
  src/tools/perf/run_benchmark -v --browser=android-webview --output-format=chartjson --upload-results --pageset-repeat=1 --also-run-disabled-tests --story-filter=load.games.bubbles system_health.memory_mobile

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

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


| 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!
Mergedinto: 691618
Status: Duplicate (was: Untriaged)

Sign in to add a comment