Issue metadata
Sign in to add a comment
|
56.3% regression in system_health.memory_mobile at 454708:454753 |
||||||||||||||||||||
Issue descriptionSee the link to graphs below.
,
Mar 7 2017
Started bisect job https://chromeperf.appspot.com/buildbucket_job_status/8985767546331003712
,
Mar 7 2017
=== Auto-CCing suspected CL author jfernandez@igalia.com === Hi jfernandez@igalia.com, 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 : jfernandez Commit : a90dba1f2dd094e51229dd2e5e059c6b53cee93a Date : Sat Mar 04 01:36:34 2017 Subject: [table] Stretching tables when needed due to self-alignment properties Bisect Details Configuration: android_webview_arm64_aosp_perf_bisect Benchmark : system_health.memory_mobile Metric : memory:webview:all_processes:reported_by_chrome:cc:effective_size_avg/load_tools/load_tools_weather Change : 58.13% | 17328296.0 -> 27401725.3333 Revision Result N chromium@454707 17328296 +- 0.0 6 good chromium@454730 17328296 +- 0.0 6 good chromium@454742 17328296 +- 0.0 6 good chromium@454743 27401725 +- 1779821 6 bad <-- chromium@454744 27077763 +- 5404.2 6 bad chromium@454745 27076776 +- 0.0 6 bad chromium@454748 27076776 +- 0.0 6 bad chromium@454753 27401725 +- 1779821 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.tools.weather system_health.memory_mobile Debug Info https://chromeperf.appspot.com/buildbucket_job_status/8985767546331003712 Is this bisect wrong? https://chromeperf.appspot.com/bad_bisect?try_job_id=5782027959271424 | 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!
,
Mar 7 2017
I'll take a look ASAP.
,
Mar 7 2017
Sorry, replied with my personal account. I'll take a look ASAP.
,
Mar 9 2017
https://crrev.com/2528253003 has been reverted. Could anybody verify the regression is solved now ?
,
Mar 10 2017
In any case, I think we should merge https://crrev.com/2740063003 in M59 as soon as possible.
,
Mar 30 2017
This issue should be FIXED now, as the patch causing the regression as been reverted. |
|||||||||||||||||||||
►
Sign in to add a comment |
|||||||||||||||||||||
Comment 1 by kraynov@chromium.org
, Mar 7 2017