diff options
| author | Zhiyin Luo (罗植尹) <zhiyin.luo@mediatek.com> | 2020-01-11 17:14:05 +0800 |
|---|---|---|
| committer | Hung-ying Tyan <tyanh@google.com> | 2020-07-27 05:46:14 +0000 |
| commit | fc869035b6883bd9cfa6a83c5191405aebb426a6 (patch) | |
| tree | 259982b3029f21ba8316c6eb0322a3e56964b067 /libs/hwui/JankTracker.h | |
| parent | 95846eed9d84ec7fe51fd3272d56d8cc9c63e902 (diff) | |
fix uninitialization issue in JankTracker
We should initialize mSwapDeadline in JankTracker, or it can be a very
large randomized value and then makes jank tracker not collect concrete
jank type of frames appropriately as expected.
Bug: 161656795
Test: run cts -m CtsIncidentHostTestCases -t
com.android.server.cts.GraphicsStatsValidationTest#testJankyDrawFrame
Change-Id: I057a50a74502918619204f9164f6a954f8e9c5de
Merged-In: I057a50a74502918619204f9164f6a954f8e9c5de
(cherry picked from commit 7a0d224756195ce81875db0b481a77b87f860dd3)
Diffstat (limited to 'libs/hwui/JankTracker.h')
| -rw-r--r-- | libs/hwui/JankTracker.h | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/libs/hwui/JankTracker.h b/libs/hwui/JankTracker.h index 4460266276f9..b3fbbfe98669 100644 --- a/libs/hwui/JankTracker.h +++ b/libs/hwui/JankTracker.h @@ -75,7 +75,7 @@ private: std::array<int64_t, NUM_BUCKETS> mThresholds; int64_t mFrameInterval; - nsecs_t mSwapDeadline; + nsecs_t mSwapDeadline = -1; // The amount of time we will erase from the total duration to account // for SF vsync offsets with HWC2 blocking dequeueBuffers. // (Vsync + mDequeueBlockTolerance) is the point at which we expect |