diff options
| author | Jorim Jaggi <jjaggi@google.com> | 2021-01-19 00:08:02 +0100 |
|---|---|---|
| committer | Jorim Jaggi <jjaggi@google.com> | 2021-04-13 15:18:27 +0000 |
| commit | 10f328c580fe1e897b51a7e4b38ee4c341d970f1 (patch) | |
| tree | b5719df2aa8460ef2925440e72ecec07020557b1 /libs/hwui/FrameInfo.cpp | |
| parent | a373e1ed5a59997b2cc8ac86615963d8597e4a2b (diff) | |
Change hwui jank detection to use deadline & gpu completion (1/2)
- Use GPU finish time as well as actual deadline to determine jank
rate.
- Use dynamic interval to adjust for 60/90hz switching
- Move frame metrics reporting into JankTracker to adjust the
deadline communicated to the app when in stuffing scenario.
- Adjust double-stuffing detection to be a bit more readable.
Test: GraphicsStatsValidationTest.java
Test: adb shell dumpsys gfxinfo
Test: FrameMetricsListenerTest
Test: Log output of FrameMetricsObserver
Bug: 169858044
Change-Id: I3a6b8ed163e2cf9cf2b67667110340ebe35f98a1
Diffstat (limited to 'libs/hwui/FrameInfo.cpp')
| -rw-r--r-- | libs/hwui/FrameInfo.cpp | 18 |
1 files changed, 10 insertions, 8 deletions
diff --git a/libs/hwui/FrameInfo.cpp b/libs/hwui/FrameInfo.cpp index 51fbf363f51c..fecf26906c04 100644 --- a/libs/hwui/FrameInfo.cpp +++ b/libs/hwui/FrameInfo.cpp @@ -21,16 +21,18 @@ namespace android { namespace uirenderer { const std::array FrameInfoNames{ - "Flags", "FrameTimelineVsyncId", "IntendedVsync", - "Vsync", "InputEventId", "HandleInputStart", - "AnimationStart", "PerformTraversalsStart", "DrawStart", - "FrameDeadline", "FrameStartTime", "SyncQueued", - "SyncStart", "IssueDrawCommandsStart", "SwapBuffers", - "FrameCompleted", "DequeueBufferDuration", "QueueBufferDuration", - "GpuCompleted", "SwapBuffersCompleted", "DisplayPresentTime", + "Flags", "FrameTimelineVsyncId", "IntendedVsync", + "Vsync", "InputEventId", "HandleInputStart", + "AnimationStart", "PerformTraversalsStart", "DrawStart", + "FrameDeadline", "FrameInterval", "FrameStartTime", + "SyncQueued", "SyncStart", "IssueDrawCommandsStart", + "SwapBuffers", "FrameCompleted", "DequeueBufferDuration", + "QueueBufferDuration", "GpuCompleted", "SwapBuffersCompleted", + "DisplayPresentTime", + }; -static_assert(static_cast<int>(FrameInfoIndex::NumIndexes) == 21, +static_assert(static_cast<int>(FrameInfoIndex::NumIndexes) == 22, "Must update value in FrameMetrics.java#FRAME_STATS_COUNT (and here)"); void FrameInfo::importUiThreadInfo(int64_t* info) { |