Issue metadata
Sign in to add a comment
|
371.4% regression in tracing.tracing_with_background_memory_infra at 501705:505034 |
||||||||||||||||||||
Issue descriptionSee the link to graphs below.
,
Oct 1 2017
Started bisect job https://chromeperf.appspot.com/buildbucket_job_status/8966935590799816112
,
Oct 1 2017
=== Auto-CCing suspected CL author lalitm@chromium.org === Hi lalitm@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 : Lalit Maganti Commit : fdc0a261697b6e767f45acb3d61fb5dc005d205a Date : Thu Sep 21 12:30:10 2017 Subject: memory-infra: Enable LevelDB global dumps in background mode Bisect Details Configuration: android_nexus6_perf_bisect Benchmark : tracing.tracing_with_background_memory_infra Metric : MojoLevelDB_memory_dump_cpu_overhead_avg/http___www.yahoo.com_ Change : 414.99% | 0.0101666666667 -> 0.0523571428571 Revision Result N chromium@501704 0.0101667 +- 0.0032914 6 good chromium@503369 0.0123125 +- 0.00717417 8 good chromium@503421 0.0107143 +- 0.00639754 7 good chromium@503422 0.05 +- 0.0193391 6 bad <-- chromium@503423 0.0465 +- 0.0150333 5 bad chromium@503425 0.0485833 +- 0.0100602 6 bad chromium@503428 0.0512857 +- 0.011808 7 bad chromium@503434 0.0561667 +- 0.0361778 6 bad chromium@503447 0.0476 +- 0.0131605 5 bad chromium@503473 0.056 +- 0.0168523 5 bad chromium@503578 0.0495 +- 0.0108397 7 bad chromium@503786 0.13605 +- 0.585129 10 bad chromium@504202 0.0522 +- 0.0132212 5 bad chromium@505034 0.0523571 +- 0.020527 7 bad To Run This Test src/tools/perf/run_benchmark -v --browser=android-chromium --output-format=chartjson --upload-results --pageset-repeat=1 --also-run-disabled-tests --story-filter=http...www.yahoo.com. tracing.tracing_with_background_memory_infra More information on addressing performance regressions: http://g.co/ChromePerformanceRegressions Debug information about this bisect: https://chromeperf.appspot.com/buildbucket_job_status/8966935590799816112 For feedback, file a bug with component Speed>Bisection
,
Oct 2 2017
I believe that such a regression will be expected because we are intentionally adding more data to memory dumps so it makes sense that the CPU overhead would also rise.
,
Oct 2 2017
Yes it is expected. I thought tracing.tracing_with_background_memory_infra was not part of the sheriff queue. It should only send alert emails to me (there is a memory-infra sheriff email id). shend@ is it not true anymore?
,
Oct 2 2017
Hi ssid, apologies for the confusion, I'm not actually sheriffing. I landed a patch recently and I ran a few bisects to see if it caused any regressions. I must've ran this bisect by mistake as it's not relevant to my patch. I'm closing this now. Sorry for the confusion!
,
Oct 3 2017
|
|||||||||||||||||||||
►
Sign in to add a comment |
|||||||||||||||||||||
Comment 1 by 42576172...@developer.gserviceaccount.com
, Oct 1 2017