Issue metadata
Sign in to add a comment
|
14.8%-388.2% regression in page_cycler_v2.intl_ko_th_vi at 415576:415662 |
||||||||||||||||||||||
Issue descriptionSee the link to graphs below.
,
Sep 2 2016
Started bisect job https://chromeperf.appspot.com/buildbucket_job_status/9002612770838467920
,
Sep 2 2016
Started bisect job https://chromeperf.appspot.com/buildbucket_job_status/9002612762345239264
,
Sep 2 2016
kouhei@: There are quite a few big regressions in page_cycler_v2 recently, and bisects are always failing to find the cause(s). I am assigning the bugs to you since you seem to have made many (most?) PCv2 changed recently. If I am wrong, could you please help me find the right owner of these bugs?
,
Sep 2 2016
Bisect failed: http://build.chromium.org/p/tryserver.chromium.perf/builders/android_nexus6_perf_bisect/builds/2536 Failure reason: the build has failed. Additional errors: The metric was not found in the test output. Either of the initial "good" or "bad" revisions failed to be tested or built.
,
Sep 2 2016
Bisect failed: http://build.chromium.org/p/tryserver.chromium.perf/builders/android_nexus6_perf_bisect/builds/2535 Failure reason: the build has failed. Additional errors: The metric was not found in the test output. Either of the initial "good" or "bad" revisions failed to be tested or built.
,
Sep 20 2016
,
Sep 29 2016
Started bisect job https://chromeperf.appspot.com/buildbucket_job_status/9000176898201868464
,
Sep 29 2016
ksakamoto: I see this in the revision range, could it be the cause for the change? 2016-08-30 ksakamoto Update FirstMeaningfulPaint to use Blink's implementation
,
Sep 30 2016
Yes my change must be the cause for this; it changes FMP value drastically on some pages.
,
Sep 30 2016
===== BISECT JOB RESULTS ===== Status: completed ===== SUSPECTED CL(s) ===== Subject : Update FirstMeaningfulPaint to use Blink's implementation Author : ksakamoto Commit description: Note to perf sheriffs: this is going to affect timeToFirstMeaningfulPaint values in page_cycler_v2 benchmarks. This updates FirstMeaningfulPaint metric to use trace events logged by Blink's FirstMeaningfulPaintDetector, instead of computing layout significance from FrameView::performLayout trace event. Since Blink implementation counts more layout objects / uses the end time of the paint phase, this patch will change TTFMP value, but it increases accuracy (see https://crbug.com/638124#c8 for details). This requires the new trace event added in https://crrev.com/2276573003/. BUG= chromium:638124 Review-Url: https://codereview.chromium.org/2259723002 Commit : 7bf2ae5c26055537f6c07d88d15d29485bca65b1 Date : Wed Aug 31 01:50:43 2016 ===== TESTED REVISIONS ===== Revision Mean Std Dev N Good? chromium@415621 890.494 17.0437 5 good chromium@415624 894.552 14.4167 5 good chromium@415624,catapult@e403ae0791 892.082 9.4746 5 good chromium@415624,catapult@17bda7618b 901.457 22.2923 5 good chromium@415624,catapult@dccd40a985 890.567 16.3982 5 good chromium@415624,catapult@7bf2ae5c26 1405.94 15.7303 5 bad <-- chromium@415625 1413.39 26.6171 5 bad chromium@415626 1401.95 14.2115 5 bad chromium@415627 1407.98 15.7058 5 bad chromium@415632 1408.49 18.9131 5 bad chromium@415642 1421.21 34.3954 5 bad Bisect job ran on: android_nexus5_perf_bisect Bug ID: 643198 Test Command: src/tools/perf/run_benchmark -v --browser=android-chromium --output-format=chartjson --upload-results --also-run-disabled-tests page_cycler_v2.intl_ko_th_vi Test Metric: timeToFirstMeaningfulPaint_avg/pcv1-warm/http___kenh14.vn_home.chn Relative Change: 59.60% Score: 99.9 Buildbot stdio: http://build.chromium.org/p/tryserver.chromium.perf/builders/android_nexus5_perf_bisect/builds/4162 Job details: https://chromeperf.appspot.com/buildbucket_job_status/9000176898201868464 Not what you expected? We'll investigate and get back to you! https://chromeperf.appspot.com/bad_bisect?try_job_id=5013101609484288 | 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 Tests>AutoBisect. Thank you!
,
Apr 11 2017
Started bisect job https://chromeperf.appspot.com/buildbucket_job_status/8982584513897873696 |
|||||||||||||||||||||||
►
Sign in to add a comment |
|||||||||||||||||||||||
Comment 1 by mustaq@chromium.org
, Sep 1 2016