Issue metadata
Sign in to add a comment
|
13.2%-23.4% regression in v8.browsing_desktop at 512462:512521 |
||||||||||||||||||||
Issue descriptionSee the link to graphs below.
,
Nov 3 2017
Started bisect job https://chromeperf.appspot.com/buildbucket_job_status/8963957293440696304
,
Nov 3 2017
=== Auto-CCing suspected CL author zmo@chromium.org === Hi zmo@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 : Zhenyao Mo Commit : 39d8a7f229e1330e72724895eb4730b7108e47a0 Date : Mon Oct 30 13:45:42 2017 Subject: Revert "One step further into moving GPU feature decisions to GPU process" Bisect Details Configuration: win_x64_perf_bisect Benchmark : v8.browsing_desktop Metric : memory:chrome:renderer_processes:reported_by_chrome:v8:heap:allocated_objects_size_avg/browse_news/browse_news_flipboard Change : 8.92% | 22083156.8764 -> 24053126.9887 Revision Result N chromium@512468 22083157 +- 1882655 6 good chromium@512478 22583901 +- 504019 6 good chromium@512483 22431522 +- 2085880 9 good chromium@512484 22048597 +- 3701889 14 good chromium@512485 23469660 +- 1465144 9 bad <-- chromium@512486 23697201 +- 843769 6 bad chromium@512488 24025639 +- 1535610 6 bad chromium@512507 24053127 +- 2060156 6 bad To Run This Test src/tools/perf/run_benchmark -v --browser=release_x64 --output-format=chartjson --upload-results --pageset-repeat=1 --also-run-disabled-tests --story-filter=browse.news.flipboard v8.browsing_desktop More information on addressing performance regressions: http://g.co/ChromePerformanceRegressions Debug information about this bisect: https://chromeperf.appspot.com/buildbucket_job_status/8963957293440696304 For feedback, file a bug with component Speed>Bisection
,
Nov 3 2017
,
Nov 3 2017
My CL is pure revert of three previously landed CL. It can't be responsible for this regression. Unassign myself.
,
Nov 3 2017
Looking at the graphs, I see an improvement that contains 0728a85617eae06098373349af80c28ce26b60fd Is it not possible that the initial CL improved the metrics and the revert causes a regression?
,
Nov 3 2017
That's possible, but there is nothing I can do. I am still working on relanding my reverted work, but they may not take the same path and may not have the original perf impact.
,
Nov 3 2017
Well, you don't have to have a fix ready. In this case it is enough to just attribute it to the change.
,
Nov 3 2017
Issue 781196 has been merged into this issue.
,
Nov 7 2017
,
Nov 8 2017
Issue 781652 has been merged into this issue. |
|||||||||||||||||||||
►
Sign in to add a comment |
|||||||||||||||||||||
Comment 1 by 42576172...@developer.gserviceaccount.com
, Nov 3 2017