From 10f328c580fe1e897b51a7e4b38ee4c341d970f1 Mon Sep 17 00:00:00 2001 From: Jorim Jaggi Date: Tue, 19 Jan 2021 00:08:02 +0100 Subject: 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 --- libs/hwui/FrameInfo.cpp | 18 ++++++++++-------- 1 file changed, 10 insertions(+), 8 deletions(-) (limited to 'libs/hwui/FrameInfo.cpp') 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(FrameInfoIndex::NumIndexes) == 21, +static_assert(static_cast(FrameInfoIndex::NumIndexes) == 22, "Must update value in FrameMetrics.java#FRAME_STATS_COUNT (and here)"); void FrameInfo::importUiThreadInfo(int64_t* info) { -- cgit v1.2.3