Issue metadata
Sign in to add a comment
|
8.4%-16.5% regression in system_health.memory_mobile at 479754:479813 |
||||||||||||||||||||||
Issue descriptionSee the link to graphs below.
,
Jun 21 2017
Started bisect job https://chromeperf.appspot.com/buildbucket_job_status/8976141104775683296
,
Jun 22 2017
=== BISECT JOB RESULTS === Perf regression found with culprit Suspected Commit Author : Marijn Kruisselbrink Commit : 469805742123ec65bdeea766d8960c5abd7f82e8 Date : Thu Jun 15 19:33:41 2017 Subject: If the file to rename doesn't exist, return OK immediately. Bisect Details Configuration: android_webview_arm64_aosp_perf_bisect Benchmark : system_health.memory_mobile Metric : memory:webview:all_processes:reported_by_os:system_memory:java_heap:proportional_resident_size_avg/load_media/load_media_soundcloud Change : 6.96% | 6476458.66667 -> 6025557.33333 Revision Result N chromium@479753 6476459 +- 72446.3 6 good chromium@479778 6529365 +- 46378.6 6 good chromium@479782 6548821 +- 55156.8 6 good chromium@479784 6605483 +- 102877 6 good chromium@479785 6092459 +- 46106.5 6 bad <-- chromium@479791 6068224 +- 26066.8 6 bad chromium@479803 6025557 +- 50450.4 6 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.media.soundcloud system_health.memory_mobile Debug Info https://chromeperf.appspot.com/buildbucket_job_status/8976141104775683296 Is this bisect wrong? https://chromeperf.appspot.com/bad_bisect?try_job_id=5496309944942592 | 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!
,
Jun 22 2017
The graph has a land-revert pattern and also the culprit that the bisect found actually improves the metric. So, I think this is not related to 734700 and probably should be ignored. Please change if you disagree. |
|||||||||||||||||||||||
►
Sign in to add a comment |
|||||||||||||||||||||||
Comment 1 by lanwei@chromium.org
, Jun 21 2017