Issue metadata
Sign in to add a comment
|
Scheduling.Browser.DrawDuration decrease in data |
||||||||||||||||||||
Issue descriptionOver the past month we've seen a 2x regression in Scheduling.Browser.DrawDuration (How long it takes the compositor to draw a frame) from the field. This is a regression which is unique to Chrome OS. UMA: https://uma.googleplex.com/timeline_v2?sid=c122636ff62e23f69826d3a579c29692
,
Jun 8 2016
I'm not aware of regressions in the compositor. danakj@, could you please take a look?
,
Jun 9 2016
Maybe it's not useful data. // Deprecated because they combine Browser and Renderer stats and have low // precision. // TODO(brianderson): Remove. https://cs.chromium.org/chromium/src/cc/scheduler/compositor_timing_history.cc?rcl=0&l=82 Handing this to Brian to triage as owner of the metric.
,
Jun 9 2016
+rschoen and rkaplow as an FYI that this metric may not be useful anymore?
,
Jun 9 2016
Hm, interesting. I think Sami had pointed me to this metric, any thoughts on the applicability of it?
,
Jun 9 2016
Even more striking than the regression, the volume of data has gone down https://uma.googleplex.com/timeline_v2?sid=c5a5c3e508a7ac68454cebf274730e5e 10X. That's probably the underlying cause of the "regression"
,
Jun 9 2016
Updating the name of the issue appropriately based on Comment 6. Will wait to here Sami's thoughts on the metric.
,
Jun 9 2016
Oh - right, just remembered this is the metric that we downsamples. Didn't we have a bug about this already, where alexei pointed out the new bucket scheme will cause a shift in the metric?
,
Jun 9 2016
A few comments: Scheduling.Browser.DrawDuration is useful - it's Scheduling.DrawDuration that isn't. I'm working on removing it now. It looks like the regression started before the reduced sampling rate landed, so I think that's a red herring. Is there a list of Scheduling metrics CrOS is tracking? Although DrawDuration is an important sub metric, you probably want to track the 99th percentile of DrawInterval instead (for both Renderer and Browser) which is a more accurate picture of the compositor's frame rate. Quick link to the 99% results from Stable for Linux, CrOS, Mac, Win, Android: https://uma.googleplex.com/timeline_v2?sid=8221d5c226a0783dadcdfdf50047e795 There's a regression across the board in Browser.DrawDuration, but it doesn't appear to show up in Browser.DrawInterval. The one bad thing about Browser.DrawInterval is that it won't include Browser animations that try to produce frames at less than 60fps since it looks like we keep going idle and that metric doesn't record an interval for the first frame coming out of an idle state. Not sure if that explains why there is a regression in DrawDuration but not DrawInterval.
,
Jun 9 2016
Perhaps this is material design related, then. I think that's ChromeOS only. It may have animations at < 60hz.
,
Jun 10 2016
MD is currently enabled on all desktop platforms. The enable points have been staggered in time. Without knowing what versions you're testing, I can't tell you if MD was only on for CrOS in those timeframes.
,
Jun 10 2016
@briananderson - There's no formal list of metrics we are tracking at the moment. Ryan was kind enough to bring this to our attention when he was looking at some Chrome metrics. We are looking at tracking a consistent set of metrics, but that's a WIP atm. My takeaway from what you're saying however, is that we shouldn't be too concerned w/ the DrawDuration regression as it hasn't manifested itself in DrawInterval? Or is there a concern that a further regression in this submetric or a related one could push things over a threshold such that we would then regress DrawInterval?
,
Jun 10 2016
@msheets: Correct. The general idea would be that DrawDuration should help narrow down the root cause if there's a regression in DrawInterval. It's weird though that there's no regression in DrawInterval here. @pkasting: Chrome Dev Browser.DrawDuration, by platform: https://uma.googleplex.com/timeline_v2?sid=8c68af64f01da80b8168c6e0a9d1bd16 On dev, it looks like Windows, CrOS, and Linux all regressed around the same time in mid February. Chrome Stable Browser.DrawDuration over time, by platform: https://uma.googleplex.com/timeline_v2?sid=91eefd5f1a9e60d37d893146b664897a On stable, It looks like the regression was staggered with Linux in mid April, then Windows followed quickly by CrOS in early May. Does that coincide with material design? @all: Chrome Dev Browser+Renderer DrawDuration+Interval https://uma.googleplex.com/timeline_v2?sid=b14c83cf5b3363732f774b0a64a58d71 In mid February, there wasn't the same kind of regression in Renderer.DrawDuration as Browser.DrawDuration, so a Browser content change like material design (rather than a change in the compositor itself) is probably what we are looking for. Interestingly, there's a sharp regression on Windows at the end of April in both Renderer DrawDuration and DrawInterval without a change in the Browser. I've opened a issue 619089 for that particular regression as it seems unrelated to this one.
,
Jun 10 2016
That data doesn't sound like this is MD. We didn't enable MD for Windows until just over a week ago, for example, while MD has been on for CrOS for a couple of months.
,
Jul 1 2016
Are there any updates here? Looks like the regression is continuing. Adding shrike, erikchen, ccameron. It seems too much like the DrawInterval regression on Mac to ignore...
,
Jul 8 2016
On the Mac the DrawInterval regression seems to be due to clearing of IOSurfaces before reuse. IOSurfaces are Mac-only so this would not cause a regression on other platforms (I believe that is what you are wondering?).
,
Sep 20 2017
,
Aug 3
sadrul@, could you please help triage? |
|||||||||||||||||||||
►
Sign in to add a comment |
|||||||||||||||||||||
Comment 1 by abodenha@chromium.org
, Jun 6 2016