Issue metadata
Sign in to add a comment
|
5.4% regression in thread_times.simple_mobile_sites at 407147:407304 |
||||||||||||||||||||
Issue descriptionSee the link to graphs below.
,
Jul 26 2016
Started bisect job https://chromeperf.appspot.com/buildbucket_job_status/9006081045182232048
,
Jul 26 2016
Started bisect job https://chromeperf.appspot.com/buildbucket_job_status/9006081038852594944
,
Jul 26 2016
===== BISECT JOB RESULTS ===== Status: completed ===== SUSPECTED CL(s) ===== Subject : Support early associated interface binding on ChannelMojo Author : rockot Commit description: Changes the associated bindings implementation for ChannelMojo such that remote interfaces can be acquired immediately upon ChannelMojo construction rather than having to wait for connection on the IO thread. Simplifies the Channel bootstrapping process, removing a round-trip Init message (and in fact the entire IPC::mojom::Boostrap interface) since there's no need to actually exchange associated interface handles over the pipe. Instead both sides can assume the other will use a fixed, reserved endpoint ID for their IPC::mojom::Channel interface. This also removes the restriction that associated interfaces must be added to a Channel after Init. Instead the same constraints apply as with AddFilter: an associated interface, like a filter, may be added at any time as long as either Init hasn't been called OR the remote process hasn't been launched. The result of this CL is that any place it's safe to AddFilter, it's also safe to AddAssociatedInterface; and any place it's safe to Send, it's also safe to GetRemoteAssociatedInterface and begin using any such remote interface immediately. Remote interface requests as well as all messages to remote interfaces retain FIFO with respect to any Send calls on the same thread. Local interface request dispatch as well as all messages on locally bound associated interfaces retain FIFO with respect to any OnMessageReceived calls on the same thread. BUG=612500, 619202 Committed: https://crrev.com/e1037f997da9e1d44ca3b09d4ff32f0465673091 Committed: https://crrev.com/508da24622f957a01b076ccd058bfdccc79068a4 Review-Url: https://codereview.chromium.org/2163633003 Cr-Original-Original-Commit-Position: refs/heads/master@{#406720} Cr-Original-Commit-Position: refs/heads/master@{#407050} Cr-Commit-Position: refs/heads/master@{#407264} Commit : 0e4de5f9a519c6cd206448a10eccc7a535e3db64 Date : Fri Jul 22 21:20:12 2016 ===== TESTED REVISIONS ===== Revision Mean Std Dev N Good? chromium@407146 3.08862 0.0251655 5 good chromium@407225 3.06836 0.016063 5 good chromium@407245 3.07203 0.015684 5 good chromium@407255 3.06933 0.0238318 5 good chromium@407260 3.07235 0.0178268 5 good chromium@407263 3.08109 0.0279089 5 good chromium@407264 3.26451 0.0198482 5 bad <-- chromium@407265 3.25591 0.0293786 5 bad chromium@407304 3.25821 0.0147985 5 bad Bisect job ran on: android_nexus5_perf_bisect Bug ID: 631418 Test Command: src/tools/perf/run_benchmark -v --browser=android-chromium --output-format=chartjson --upload-results --also-run-disabled-tests thread_times.simple_mobile_sites Test Metric: thread_IO_cpu_time_per_frame/thread_IO_cpu_time_per_frame Relative Change: 5.49% Score: 99.9 Buildbot stdio: http://build.chromium.org/p/tryserver.chromium.perf/builders/android_nexus5_perf_bisect/builds/3874 Job details: https://chromeperf.appspot.com/buildbucket_job_status/9006081038852594944 Not what you expected? We'll investigate and get back to you! https://chromeperf.appspot.com/bad_bisect?try_job_id=5807569939988480 | 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!
,
Jul 26 2016
===== BISECT JOB RESULTS ===== Status: completed ===== SUSPECTED CL(s) ===== Subject : Support early associated interface binding on ChannelMojo Author : rockot Commit description: Changes the associated bindings implementation for ChannelMojo such that remote interfaces can be acquired immediately upon ChannelMojo construction rather than having to wait for connection on the IO thread. Simplifies the Channel bootstrapping process, removing a round-trip Init message (and in fact the entire IPC::mojom::Boostrap interface) since there's no need to actually exchange associated interface handles over the pipe. Instead both sides can assume the other will use a fixed, reserved endpoint ID for their IPC::mojom::Channel interface. This also removes the restriction that associated interfaces must be added to a Channel after Init. Instead the same constraints apply as with AddFilter: an associated interface, like a filter, may be added at any time as long as either Init hasn't been called OR the remote process hasn't been launched. The result of this CL is that any place it's safe to AddFilter, it's also safe to AddAssociatedInterface; and any place it's safe to Send, it's also safe to GetRemoteAssociatedInterface and begin using any such remote interface immediately. Remote interface requests as well as all messages to remote interfaces retain FIFO with respect to any Send calls on the same thread. Local interface request dispatch as well as all messages on locally bound associated interfaces retain FIFO with respect to any OnMessageReceived calls on the same thread. BUG=612500, 619202 Committed: https://crrev.com/e1037f997da9e1d44ca3b09d4ff32f0465673091 Committed: https://crrev.com/508da24622f957a01b076ccd058bfdccc79068a4 Review-Url: https://codereview.chromium.org/2163633003 Cr-Original-Original-Commit-Position: refs/heads/master@{#406720} Cr-Original-Commit-Position: refs/heads/master@{#407050} Cr-Commit-Position: refs/heads/master@{#407264} Commit : 0e4de5f9a519c6cd206448a10eccc7a535e3db64 Date : Fri Jul 22 21:20:12 2016 ===== TESTED REVISIONS ===== Revision Mean Std Dev N Good? chromium@407146 3.10731 0.017486 5 good chromium@407225 3.08605 0.0103101 5 good chromium@407245 3.06763 0.0251302 5 good chromium@407255 3.056 0.00982202 5 good chromium@407260 3.05711 0.0128596 5 good chromium@407263 3.06645 0.0223365 5 good chromium@407264 3.26559 0.0189133 5 bad <-- chromium@407265 3.28121 0.00894092 5 bad chromium@407304 3.25215 0.0172229 5 bad Bisect job ran on: android_nexus5_perf_bisect Bug ID: 631418 Test Command: src/tools/perf/run_benchmark -v --browser=android-chromium --output-format=chartjson --upload-results --also-run-disabled-tests thread_times.simple_mobile_sites Test Metric: thread_IO_cpu_time_per_frame/thread_IO_cpu_time_per_frame Relative Change: 4.66% Score: 99.9 Buildbot stdio: http://build.chromium.org/p/tryserver.chromium.perf/builders/android_nexus5_perf_bisect/builds/3873 Job details: https://chromeperf.appspot.com/buildbucket_job_status/9006081045182232048 Not what you expected? We'll investigate and get back to you! https://chromeperf.appspot.com/bad_bisect?try_job_id=5846465163821056 | 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! |
|||||||||||||||||||||
►
Sign in to add a comment |
|||||||||||||||||||||
Comment 1 by primiano@chromium.org
, Jul 26 2016