Issue metadata
Sign in to add a comment
|
100% improvement in rendering.mobile/thread_raster_cpu_time_per_frame at 603899:603942 |
||||||||||||||||||||||||
Issue descriptionSee the link to graphs below.
,
Nov 4
📍 Pinpoint job started. https://pinpoint-dot-chromeperf.appspot.com/job/12de8e69e40000
,
Nov 4
📍 Found a significant difference after 1 commit. https://pinpoint-dot-chromeperf.appspot.com/job/12de8e69e40000 Reland "Enable Perfetto by default for all telemetry tests" by oysteine@chromium.org https://chromium.googlesource.com/chromium/src/+/3bb24369bef996b43b2d2711f0033ba472f779ea thread_raster_cpu_time_per_frame: 0.6145 → 0 (-0.6145) Understanding performance regressions: http://g.co/ChromePerformanceRegressions Benchmark documentation link: https://bit.ly/rendering-benchmarks
,
Nov 5
Note oysteine@: this metric has gone up from zero, later (version range: 604426 - 604489). However, it's still signifficantly different (30% lower) than the metric value before the Perfetto CL. Could you please take a look?
,
Nov 7
#3: This is measuring trace event time on worker threads that don't have a current messageloop and hence non-Perfetto tracing needs to grab the global TraceLog lock to add each trace event, as far as I can tell that's what's causing this metric shift (no such lock with Perfetto). |
|||||||||||||||||||||||||
►
Sign in to add a comment |
|||||||||||||||||||||||||
Comment 1 by 42576172...@developer.gserviceaccount.com
, Nov 4