Issue metadata
Sign in to add a comment
|
2.1%-14.2% regression in media_perftests at 476915:476958 |
||||||||||||||||||||
Issue descriptionThese regressions all appear to be a retrun to a previous average from ~100 revisions prior. Likely some change landed and reverted. Bisects just to be sure.
,
Jun 6 2017
Started bisect job https://chromeperf.appspot.com/buildbucket_job_status/8977508179883889392
,
Jun 7 2017
=== Auto-CCing suspected CL author hans@chromium.org === Hi hans@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 : hans Commit : ba2ff9ad0d643a443575c839e56ae9edf7106643 Date : Mon Jun 05 02:02:06 2017 Subject: Revert of "win: Set is_clang=true by default over the weekend." Bisect Details Configuration: winx64_10_perf_bisect Benchmark : media.tough_video_cases Metric : vm_working_set_delta_size/tulip2.mp4_gpu Change : 1.88% | 23577.3333333 -> 24020.6666667 Revision Result N chromium@476917 23577.3 +- 59.5091 6 good chromium@476920 23493.3 +- 102.554 6 good chromium@476921 23924.0 +- 128.623 6 bad <-- chromium@476922 23934.7 +- 72.5902 6 bad chromium@476923 24003.3 +- 132.353 6 bad chromium@476928 23966.0 +- 76.4722 6 bad chromium@476938 23918.7 +- 50.8068 6 bad chromium@476958 24020.7 +- 124.376 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 media.tough_video_cases Debug Info https://chromeperf.appspot.com/buildbucket_job_status/8977508179883889392 Is this bisect wrong? https://chromeperf.appspot.com/bad_bisect?try_job_id=5781074111627264 | 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 7 2017
Issue 730118 has been merged into this issue.
,
Jun 7 2017
> These regressions all appear to be a retrun to a previous average from ~100 revisions prior. Likely some change landed and reverted. Bisects just to be sure. Yes, looks like this is from the end of the clang on winows experiment last weekend. I'll mark it wontfix since there's nothing to do at the moment, but it's nice to know we'll improve this when we turn on clang for real.
,
Jun 7 2017
Issue 730538 has been merged into this issue.
,
Jun 13 2017
,
Jun 19 2017
Issue 734568 has been merged into this issue.
,
Jun 23 2017
|
|||||||||||||||||||||
►
Sign in to add a comment |
|||||||||||||||||||||
Comment 1 by chcunningham@chromium.org
, Jun 6 2017