Alec Mouri | 0a1cc96 | 2019-03-14 12:33:02 -0700 | [diff] [blame] | 1 | /* |
| 2 | * Copyright 2019 The Android Open Source Project |
| 3 | * |
| 4 | * Licensed under the Apache License, Version 2.0 (the "License"); |
| 5 | * you may not use this file except in compliance with the License. |
| 6 | * You may obtain a copy of the License at |
| 7 | * |
| 8 | * http://www.apache.org/licenses/LICENSE-2.0 |
| 9 | * |
| 10 | * Unless required by applicable law or agreed to in writing, software |
| 11 | * distributed under the License is distributed on an "AS IS" BASIS, |
| 12 | * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. |
| 13 | * See the License for the specific language governing permissions and |
| 14 | * limitations under the License. |
| 15 | */ |
| 16 | |
Marin Shalamanov | bed7fd3 | 2020-12-21 20:02:20 +0100 | [diff] [blame] | 17 | // TODO(b/129481165): remove the #pragma below and fix conversion issues |
| 18 | #pragma clang diagnostic push |
| 19 | #pragma clang diagnostic ignored "-Wextra" |
| 20 | |
Alec Mouri | 0a1cc96 | 2019-03-14 12:33:02 -0700 | [diff] [blame] | 21 | #undef LOG_TAG |
| 22 | #define LOG_TAG "SchedulerUnittests" |
| 23 | |
| 24 | #include <gmock/gmock.h> |
| 25 | #include <log/log.h> |
| 26 | #include <thread> |
| 27 | |
Marin Shalamanov | 3ea1d60 | 2020-12-16 19:59:39 +0100 | [diff] [blame] | 28 | #include <ui/Size.h> |
| 29 | |
Alec Mouri | 0a1cc96 | 2019-03-14 12:33:02 -0700 | [diff] [blame] | 30 | #include "DisplayHardware/HWC2.h" |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 31 | #include "FpsOps.h" |
Alec Mouri | 0a1cc96 | 2019-03-14 12:33:02 -0700 | [diff] [blame] | 32 | #include "Scheduler/RefreshRateConfigs.h" |
Alec Mouri | 0a1cc96 | 2019-03-14 12:33:02 -0700 | [diff] [blame] | 33 | |
| 34 | using namespace std::chrono_literals; |
Alec Mouri | 0a1cc96 | 2019-03-14 12:33:02 -0700 | [diff] [blame] | 35 | |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 36 | namespace android::scheduler { |
Alec Mouri | 0a1cc96 | 2019-03-14 12:33:02 -0700 | [diff] [blame] | 37 | |
Peiyong Lin | e9d809e | 2020-04-14 13:10:48 -0700 | [diff] [blame] | 38 | namespace hal = android::hardware::graphics::composer::hal; |
| 39 | |
Alec Mouri | 0a1cc96 | 2019-03-14 12:33:02 -0700 | [diff] [blame] | 40 | using RefreshRate = RefreshRateConfigs::RefreshRate; |
Ady Abraham | 8a82ba6 | 2020-01-17 12:43:17 -0800 | [diff] [blame] | 41 | using LayerVoteType = RefreshRateConfigs::LayerVoteType; |
| 42 | using LayerRequirement = RefreshRateConfigs::LayerRequirement; |
Alec Mouri | 0a1cc96 | 2019-03-14 12:33:02 -0700 | [diff] [blame] | 43 | |
| 44 | class RefreshRateConfigsTest : public testing::Test { |
| 45 | protected: |
Marin Shalamanov | 4c7831e | 2021-06-08 20:44:06 +0200 | [diff] [blame] | 46 | using GetBestRefreshRateInvocation = RefreshRateConfigs::GetBestRefreshRateInvocation; |
| 47 | |
Alec Mouri | 0a1cc96 | 2019-03-14 12:33:02 -0700 | [diff] [blame] | 48 | RefreshRateConfigsTest(); |
| 49 | ~RefreshRateConfigsTest(); |
Ady Abraham | abc2760 | 2020-04-08 17:20:29 -0700 | [diff] [blame] | 50 | |
Marin Shalamanov | 4c7831e | 2021-06-08 20:44:06 +0200 | [diff] [blame] | 51 | RefreshRate createRefreshRate(DisplayModePtr displayMode) { |
Ady Abraham | 6b7ad65 | 2021-06-23 17:34:57 -0700 | [diff] [blame] | 52 | return {displayMode, RefreshRate::ConstructorTag(0)}; |
Marin Shalamanov | 4c7831e | 2021-06-08 20:44:06 +0200 | [diff] [blame] | 53 | } |
| 54 | |
Marin Shalamanov | e8a663d | 2020-11-24 17:48:00 +0100 | [diff] [blame] | 55 | Fps findClosestKnownFrameRate(const RefreshRateConfigs& refreshRateConfigs, Fps frameRate) { |
Ady Abraham | b1b9d41 | 2020-06-01 19:53:52 -0700 | [diff] [blame] | 56 | return refreshRateConfigs.findClosestKnownFrameRate(frameRate); |
| 57 | } |
| 58 | |
Marin Shalamanov | e8a663d | 2020-11-24 17:48:00 +0100 | [diff] [blame] | 59 | std::vector<Fps> getKnownFrameRate(const RefreshRateConfigs& refreshRateConfigs) { |
Ady Abraham | b1b9d41 | 2020-06-01 19:53:52 -0700 | [diff] [blame] | 60 | return refreshRateConfigs.mKnownFrameRates; |
| 61 | } |
| 62 | |
Marin Shalamanov | eadf2e7 | 2020-12-10 15:35:28 +0100 | [diff] [blame] | 63 | RefreshRate getMinRefreshRateByPolicy(const RefreshRateConfigs& refreshRateConfigs) { |
| 64 | std::lock_guard lock(refreshRateConfigs.mLock); |
| 65 | return refreshRateConfigs.getMinRefreshRateByPolicyLocked(); |
| 66 | } |
| 67 | |
| 68 | RefreshRate getMinSupportedRefreshRate(const RefreshRateConfigs& refreshRateConfigs) { |
| 69 | std::lock_guard lock(refreshRateConfigs.mLock); |
| 70 | return *refreshRateConfigs.mMinSupportedRefreshRate; |
| 71 | } |
| 72 | |
| 73 | RefreshRate getMaxSupportedRefreshRate(const RefreshRateConfigs& refreshRateConfigs) { |
| 74 | std::lock_guard lock(refreshRateConfigs.mLock); |
| 75 | return *refreshRateConfigs.mMaxSupportedRefreshRate; |
| 76 | } |
| 77 | |
Marin Shalamanov | 4c7831e | 2021-06-08 20:44:06 +0200 | [diff] [blame] | 78 | void setLastBestRefreshRateInvocation(RefreshRateConfigs& refreshRateConfigs, |
| 79 | const GetBestRefreshRateInvocation& invocation) { |
| 80 | std::lock_guard lock(refreshRateConfigs.mLock); |
| 81 | refreshRateConfigs.lastBestRefreshRateInvocation.emplace( |
| 82 | GetBestRefreshRateInvocation(invocation)); |
| 83 | } |
| 84 | |
| 85 | std::optional<GetBestRefreshRateInvocation> getLastBestRefreshRateInvocation( |
| 86 | const RefreshRateConfigs& refreshRateConfigs) { |
| 87 | std::lock_guard lock(refreshRateConfigs.mLock); |
| 88 | return refreshRateConfigs.lastBestRefreshRateInvocation; |
| 89 | } |
| 90 | |
Ady Abraham | abc2760 | 2020-04-08 17:20:29 -0700 | [diff] [blame] | 91 | // Test config IDs |
Marin Shalamanov | 23c4420 | 2020-12-22 19:09:20 +0100 | [diff] [blame] | 92 | static inline const DisplayModeId HWC_CONFIG_ID_60 = DisplayModeId(0); |
| 93 | static inline const DisplayModeId HWC_CONFIG_ID_90 = DisplayModeId(1); |
| 94 | static inline const DisplayModeId HWC_CONFIG_ID_72 = DisplayModeId(2); |
| 95 | static inline const DisplayModeId HWC_CONFIG_ID_120 = DisplayModeId(3); |
| 96 | static inline const DisplayModeId HWC_CONFIG_ID_30 = DisplayModeId(4); |
| 97 | static inline const DisplayModeId HWC_CONFIG_ID_25 = DisplayModeId(5); |
| 98 | static inline const DisplayModeId HWC_CONFIG_ID_50 = DisplayModeId(6); |
Marin Shalamanov | 15a0fc6 | 2021-08-16 18:20:21 +0200 | [diff] [blame] | 99 | static inline const DisplayModeId HWC_CONFIG_ID_24 = DisplayModeId(7); |
| 100 | static inline const DisplayModeId HWC_CONFIG_ID_24_FRAC = DisplayModeId(8); |
| 101 | static inline const DisplayModeId HWC_CONFIG_ID_30_FRAC = DisplayModeId(9); |
| 102 | static inline const DisplayModeId HWC_CONFIG_ID_60_FRAC = DisplayModeId(10); |
Ady Abraham | abc2760 | 2020-04-08 17:20:29 -0700 | [diff] [blame] | 103 | |
| 104 | // Test configs |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 105 | DisplayModePtr mConfig60 = createDisplayMode(HWC_CONFIG_ID_60, 0, (60_Hz).getPeriodNsecs()); |
Marin Shalamanov | 15a0fc6 | 2021-08-16 18:20:21 +0200 | [diff] [blame] | 106 | DisplayModePtr mConfig60Frac = |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 107 | createDisplayMode(HWC_CONFIG_ID_60_FRAC, 0, (59.94_Hz).getPeriodNsecs()); |
| 108 | DisplayModePtr mConfig90 = createDisplayMode(HWC_CONFIG_ID_90, 0, (90_Hz).getPeriodNsecs()); |
Marin Shalamanov | 3ea1d60 | 2020-12-16 19:59:39 +0100 | [diff] [blame] | 109 | DisplayModePtr mConfig90DifferentGroup = |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 110 | createDisplayMode(HWC_CONFIG_ID_90, 1, (90_Hz).getPeriodNsecs()); |
Marin Shalamanov | 3ea1d60 | 2020-12-16 19:59:39 +0100 | [diff] [blame] | 111 | DisplayModePtr mConfig90DifferentResolution = |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 112 | createDisplayMode(HWC_CONFIG_ID_90, 0, (90_Hz).getPeriodNsecs(), ui::Size(111, 222)); |
| 113 | DisplayModePtr mConfig72 = createDisplayMode(HWC_CONFIG_ID_72, 0, (72_Hz).getPeriodNsecs()); |
Marin Shalamanov | 3ea1d60 | 2020-12-16 19:59:39 +0100 | [diff] [blame] | 114 | DisplayModePtr mConfig72DifferentGroup = |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 115 | createDisplayMode(HWC_CONFIG_ID_72, 1, (72_Hz).getPeriodNsecs()); |
| 116 | DisplayModePtr mConfig120 = createDisplayMode(HWC_CONFIG_ID_120, 0, (120_Hz).getPeriodNsecs()); |
Marin Shalamanov | 3ea1d60 | 2020-12-16 19:59:39 +0100 | [diff] [blame] | 117 | DisplayModePtr mConfig120DifferentGroup = |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 118 | createDisplayMode(HWC_CONFIG_ID_120, 1, (120_Hz).getPeriodNsecs()); |
| 119 | DisplayModePtr mConfig30 = createDisplayMode(HWC_CONFIG_ID_30, 0, (30_Hz).getPeriodNsecs()); |
Marin Shalamanov | 3ea1d60 | 2020-12-16 19:59:39 +0100 | [diff] [blame] | 120 | DisplayModePtr mConfig30DifferentGroup = |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 121 | createDisplayMode(HWC_CONFIG_ID_30, 1, (30_Hz).getPeriodNsecs()); |
Marin Shalamanov | 15a0fc6 | 2021-08-16 18:20:21 +0200 | [diff] [blame] | 122 | DisplayModePtr mConfig30Frac = |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 123 | createDisplayMode(HWC_CONFIG_ID_30_FRAC, 0, (29.97_Hz).getPeriodNsecs()); |
| 124 | DisplayModePtr mConfig25 = createDisplayMode(HWC_CONFIG_ID_25, 0, (25_Hz).getPeriodNsecs()); |
Marin Shalamanov | 3ea1d60 | 2020-12-16 19:59:39 +0100 | [diff] [blame] | 125 | DisplayModePtr mConfig25DifferentGroup = |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 126 | createDisplayMode(HWC_CONFIG_ID_25, 1, (25_Hz).getPeriodNsecs()); |
| 127 | DisplayModePtr mConfig50 = createDisplayMode(HWC_CONFIG_ID_50, 0, (50_Hz).getPeriodNsecs()); |
| 128 | DisplayModePtr mConfig24 = createDisplayMode(HWC_CONFIG_ID_24, 0, (24_Hz).getPeriodNsecs()); |
Marin Shalamanov | 15a0fc6 | 2021-08-16 18:20:21 +0200 | [diff] [blame] | 129 | DisplayModePtr mConfig24Frac = |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 130 | createDisplayMode(HWC_CONFIG_ID_24_FRAC, 0, (23.976_Hz).getPeriodNsecs()); |
Ady Abraham | abc2760 | 2020-04-08 17:20:29 -0700 | [diff] [blame] | 131 | |
| 132 | // Test device configurations |
Marin Shalamanov | 4608442 | 2020-10-13 12:33:42 +0200 | [diff] [blame] | 133 | // The positions of the configs in the arrays below MUST match their IDs. For example, |
| 134 | // the first config should always be 60Hz, the second 90Hz etc. |
Marin Shalamanov | 3ea1d60 | 2020-12-16 19:59:39 +0100 | [diff] [blame] | 135 | DisplayModes m60OnlyConfigDevice = {mConfig60}; |
| 136 | DisplayModes m60_90Device = {mConfig60, mConfig90}; |
| 137 | DisplayModes m60_90DeviceWithDifferentGroups = {mConfig60, mConfig90DifferentGroup}; |
| 138 | DisplayModes m60_90DeviceWithDifferentResolutions = {mConfig60, mConfig90DifferentResolution}; |
| 139 | DisplayModes m60_72_90Device = {mConfig60, mConfig90, mConfig72}; |
| 140 | DisplayModes m60_90_72_120Device = {mConfig60, mConfig90, mConfig72, mConfig120}; |
| 141 | DisplayModes m30_60_72_90_120Device = {mConfig60, mConfig90, mConfig72, mConfig120, mConfig30}; |
| 142 | DisplayModes m30_60Device = {mConfig60, mConfig90DifferentGroup, mConfig72DifferentGroup, |
| 143 | mConfig120DifferentGroup, mConfig30}; |
| 144 | DisplayModes m30_60_72_90Device = {mConfig60, mConfig90, mConfig72, mConfig120DifferentGroup, |
| 145 | mConfig30}; |
| 146 | DisplayModes m30_60_90Device = {mConfig60, mConfig90, mConfig72DifferentGroup, |
| 147 | mConfig120DifferentGroup, mConfig30}; |
| 148 | DisplayModes m25_30_50_60Device = {mConfig60, |
| 149 | mConfig90, |
| 150 | mConfig72DifferentGroup, |
| 151 | mConfig120DifferentGroup, |
| 152 | mConfig30DifferentGroup, |
| 153 | mConfig25DifferentGroup, |
| 154 | mConfig50}; |
Ady Abraham | 5e4e983 | 2021-06-14 13:40:56 -0700 | [diff] [blame] | 155 | DisplayModes m60_120Device = {mConfig60, mConfig120}; |
Ady Abraham | abc2760 | 2020-04-08 17:20:29 -0700 | [diff] [blame] | 156 | |
Marin Shalamanov | 15a0fc6 | 2021-08-16 18:20:21 +0200 | [diff] [blame] | 157 | // This is a typical TV configuration. |
| 158 | DisplayModes m24_25_30_50_60WithFracDevice = {mConfig24, mConfig24Frac, mConfig25, |
| 159 | mConfig30, mConfig30Frac, mConfig50, |
| 160 | mConfig60, mConfig60Frac}; |
| 161 | |
Ady Abraham | abc2760 | 2020-04-08 17:20:29 -0700 | [diff] [blame] | 162 | // Expected RefreshRate objects |
Ady Abraham | 6b7ad65 | 2021-06-23 17:34:57 -0700 | [diff] [blame] | 163 | RefreshRate mExpected60Config = {mConfig60, RefreshRate::ConstructorTag(0)}; |
| 164 | RefreshRate mExpectedAlmost60Config = {createDisplayMode(HWC_CONFIG_ID_60, 0, 16666665), |
| 165 | RefreshRate::ConstructorTag(0)}; |
| 166 | RefreshRate mExpected90Config = {mConfig90, RefreshRate::ConstructorTag(0)}; |
| 167 | RefreshRate mExpected90DifferentGroupConfig = {mConfig90DifferentGroup, |
| 168 | RefreshRate::ConstructorTag(0)}; |
| 169 | RefreshRate mExpected90DifferentResolutionConfig = {mConfig90DifferentResolution, |
Ady Abraham | abc2760 | 2020-04-08 17:20:29 -0700 | [diff] [blame] | 170 | RefreshRate::ConstructorTag(0)}; |
Ady Abraham | 6b7ad65 | 2021-06-23 17:34:57 -0700 | [diff] [blame] | 171 | RefreshRate mExpected72Config = {mConfig72, RefreshRate::ConstructorTag(0)}; |
| 172 | RefreshRate mExpected30Config = {mConfig30, RefreshRate::ConstructorTag(0)}; |
| 173 | RefreshRate mExpected120Config = {mConfig120, RefreshRate::ConstructorTag(0)}; |
rnlee | 3bd61066 | 2021-06-23 16:27:57 -0700 | [diff] [blame] | 174 | |
Marin Shalamanov | a7fe304 | 2021-01-29 21:02:08 +0100 | [diff] [blame] | 175 | DisplayModePtr createDisplayMode(DisplayModeId modeId, int32_t group, int64_t vsyncPeriod, |
| 176 | ui::Size resolution = ui::Size()); |
Alec Mouri | 0a1cc96 | 2019-03-14 12:33:02 -0700 | [diff] [blame] | 177 | }; |
| 178 | |
Marin Shalamanov | 3ea1d60 | 2020-12-16 19:59:39 +0100 | [diff] [blame] | 179 | using Builder = DisplayMode::Builder; |
Ady Abraham | abc2760 | 2020-04-08 17:20:29 -0700 | [diff] [blame] | 180 | |
Alec Mouri | 0a1cc96 | 2019-03-14 12:33:02 -0700 | [diff] [blame] | 181 | RefreshRateConfigsTest::RefreshRateConfigsTest() { |
| 182 | const ::testing::TestInfo* const test_info = |
| 183 | ::testing::UnitTest::GetInstance()->current_test_info(); |
| 184 | ALOGD("**** Setting up for %s.%s\n", test_info->test_case_name(), test_info->name()); |
| 185 | } |
| 186 | |
| 187 | RefreshRateConfigsTest::~RefreshRateConfigsTest() { |
| 188 | const ::testing::TestInfo* const test_info = |
| 189 | ::testing::UnitTest::GetInstance()->current_test_info(); |
| 190 | ALOGD("**** Tearing down after %s.%s\n", test_info->test_case_name(), test_info->name()); |
| 191 | } |
| 192 | |
Marin Shalamanov | a7fe304 | 2021-01-29 21:02:08 +0100 | [diff] [blame] | 193 | DisplayModePtr RefreshRateConfigsTest::createDisplayMode(DisplayModeId modeId, int32_t group, |
| 194 | int64_t vsyncPeriod, ui::Size resolution) { |
| 195 | return DisplayMode::Builder(hal::HWConfigId(modeId.value())) |
| 196 | .setId(modeId) |
Ady Abraham | 5e7ee86 | 2021-06-23 17:43:41 -0700 | [diff] [blame] | 197 | .setPhysicalDisplayId(PhysicalDisplayId::fromPort(0)) |
Ady Abraham | abc2760 | 2020-04-08 17:20:29 -0700 | [diff] [blame] | 198 | .setVsyncPeriod(int32_t(vsyncPeriod)) |
Marin Shalamanov | a7fe304 | 2021-01-29 21:02:08 +0100 | [diff] [blame] | 199 | .setGroup(group) |
Marin Shalamanov | 3ea1d60 | 2020-12-16 19:59:39 +0100 | [diff] [blame] | 200 | .setHeight(resolution.height) |
| 201 | .setWidth(resolution.width) |
Ady Abraham | abc2760 | 2020-04-08 17:20:29 -0700 | [diff] [blame] | 202 | .build(); |
| 203 | } |
| 204 | |
Alec Mouri | 0a1cc96 | 2019-03-14 12:33:02 -0700 | [diff] [blame] | 205 | namespace { |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 206 | |
Ady Abraham | 2139f73 | 2019-11-13 18:56:40 -0800 | [diff] [blame] | 207 | TEST_F(RefreshRateConfigsTest, oneDeviceConfig_SwitchingSupported) { |
Steven Thomas | 2bbaabe | 2019-08-28 16:08:35 -0700 | [diff] [blame] | 208 | auto refreshRateConfigs = |
Ady Abraham | abc2760 | 2020-04-08 17:20:29 -0700 | [diff] [blame] | 209 | std::make_unique<RefreshRateConfigs>(m60OnlyConfigDevice, |
| 210 | /*currentConfigId=*/HWC_CONFIG_ID_60); |
Alec Mouri | 0a1cc96 | 2019-03-14 12:33:02 -0700 | [diff] [blame] | 211 | } |
| 212 | |
Ana Krulec | ed3a8cc | 2019-11-14 00:55:07 +0100 | [diff] [blame] | 213 | TEST_F(RefreshRateConfigsTest, invalidPolicy) { |
Ana Krulec | ed3a8cc | 2019-11-14 00:55:07 +0100 | [diff] [blame] | 214 | auto refreshRateConfigs = |
Ady Abraham | abc2760 | 2020-04-08 17:20:29 -0700 | [diff] [blame] | 215 | std::make_unique<RefreshRateConfigs>(m60OnlyConfigDevice, |
| 216 | /*currentConfigId=*/HWC_CONFIG_ID_60); |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 217 | ASSERT_LT(refreshRateConfigs->setDisplayManagerPolicy({DisplayModeId(10), {60_Hz, 60_Hz}}), 0); |
| 218 | ASSERT_LT(refreshRateConfigs->setDisplayManagerPolicy({HWC_CONFIG_ID_60, {20_Hz, 40_Hz}}), 0); |
Ana Krulec | ed3a8cc | 2019-11-14 00:55:07 +0100 | [diff] [blame] | 219 | } |
| 220 | |
Steven Thomas | 2bbaabe | 2019-08-28 16:08:35 -0700 | [diff] [blame] | 221 | TEST_F(RefreshRateConfigsTest, twoDeviceConfigs_storesFullRefreshRateMap) { |
Steven Thomas | 2bbaabe | 2019-08-28 16:08:35 -0700 | [diff] [blame] | 222 | auto refreshRateConfigs = |
Ady Abraham | abc2760 | 2020-04-08 17:20:29 -0700 | [diff] [blame] | 223 | std::make_unique<RefreshRateConfigs>(m60_90Device, |
| 224 | /*currentConfigId=*/HWC_CONFIG_ID_60); |
Alec Mouri | 0a1cc96 | 2019-03-14 12:33:02 -0700 | [diff] [blame] | 225 | |
Marin Shalamanov | eadf2e7 | 2020-12-10 15:35:28 +0100 | [diff] [blame] | 226 | const auto& minRate = getMinSupportedRefreshRate(*refreshRateConfigs); |
| 227 | const auto& performanceRate = getMaxSupportedRefreshRate(*refreshRateConfigs); |
Alec Mouri | 0a1cc96 | 2019-03-14 12:33:02 -0700 | [diff] [blame] | 228 | |
Ady Abraham | abc2760 | 2020-04-08 17:20:29 -0700 | [diff] [blame] | 229 | ASSERT_EQ(mExpected60Config, minRate); |
| 230 | ASSERT_EQ(mExpected90Config, performanceRate); |
Ady Abraham | 2139f73 | 2019-11-13 18:56:40 -0800 | [diff] [blame] | 231 | |
Marin Shalamanov | eadf2e7 | 2020-12-10 15:35:28 +0100 | [diff] [blame] | 232 | const auto& minRateByPolicy = getMinRefreshRateByPolicy(*refreshRateConfigs); |
Ady Abraham | 2e1dd89 | 2020-03-05 13:48:36 -0800 | [diff] [blame] | 233 | const auto& performanceRateByPolicy = refreshRateConfigs->getMaxRefreshRateByPolicy(); |
Ady Abraham | 2139f73 | 2019-11-13 18:56:40 -0800 | [diff] [blame] | 234 | ASSERT_EQ(minRateByPolicy, minRate); |
| 235 | ASSERT_EQ(performanceRateByPolicy, performanceRate); |
Alec Mouri | 0a1cc96 | 2019-03-14 12:33:02 -0700 | [diff] [blame] | 236 | } |
Ady Abraham | 2139f73 | 2019-11-13 18:56:40 -0800 | [diff] [blame] | 237 | |
| 238 | TEST_F(RefreshRateConfigsTest, twoDeviceConfigs_storesFullRefreshRateMap_differentGroups) { |
Ady Abraham | 2139f73 | 2019-11-13 18:56:40 -0800 | [diff] [blame] | 239 | auto refreshRateConfigs = |
Ady Abraham | abc2760 | 2020-04-08 17:20:29 -0700 | [diff] [blame] | 240 | std::make_unique<RefreshRateConfigs>(m60_90DeviceWithDifferentGroups, |
| 241 | /*currentConfigId=*/HWC_CONFIG_ID_60); |
Ady Abraham | 2139f73 | 2019-11-13 18:56:40 -0800 | [diff] [blame] | 242 | |
Marin Shalamanov | eadf2e7 | 2020-12-10 15:35:28 +0100 | [diff] [blame] | 243 | const auto& minRate = getMinRefreshRateByPolicy(*refreshRateConfigs); |
| 244 | const auto& performanceRate = getMaxSupportedRefreshRate(*refreshRateConfigs); |
| 245 | const auto& minRate60 = getMinRefreshRateByPolicy(*refreshRateConfigs); |
Ady Abraham | 2e1dd89 | 2020-03-05 13:48:36 -0800 | [diff] [blame] | 246 | const auto& performanceRate60 = refreshRateConfigs->getMaxRefreshRateByPolicy(); |
Ady Abraham | 2139f73 | 2019-11-13 18:56:40 -0800 | [diff] [blame] | 247 | |
Ady Abraham | abc2760 | 2020-04-08 17:20:29 -0700 | [diff] [blame] | 248 | ASSERT_EQ(mExpected60Config, minRate); |
| 249 | ASSERT_EQ(mExpected60Config, minRate60); |
| 250 | ASSERT_EQ(mExpected60Config, performanceRate60); |
Ady Abraham | 2139f73 | 2019-11-13 18:56:40 -0800 | [diff] [blame] | 251 | |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 252 | ASSERT_GE(refreshRateConfigs->setDisplayManagerPolicy({HWC_CONFIG_ID_90, {60_Hz, 90_Hz}}), 0); |
Marin Shalamanov | a7fe304 | 2021-01-29 21:02:08 +0100 | [diff] [blame] | 253 | refreshRateConfigs->setCurrentModeId(HWC_CONFIG_ID_90); |
Ady Abraham | 2139f73 | 2019-11-13 18:56:40 -0800 | [diff] [blame] | 254 | |
Marin Shalamanov | eadf2e7 | 2020-12-10 15:35:28 +0100 | [diff] [blame] | 255 | const auto& minRate90 = getMinRefreshRateByPolicy(*refreshRateConfigs); |
Ady Abraham | 2e1dd89 | 2020-03-05 13:48:36 -0800 | [diff] [blame] | 256 | const auto& performanceRate90 = refreshRateConfigs->getMaxRefreshRateByPolicy(); |
Ady Abraham | 2139f73 | 2019-11-13 18:56:40 -0800 | [diff] [blame] | 257 | |
Ady Abraham | abc2760 | 2020-04-08 17:20:29 -0700 | [diff] [blame] | 258 | ASSERT_EQ(mExpected90DifferentGroupConfig, performanceRate); |
| 259 | ASSERT_EQ(mExpected90DifferentGroupConfig, minRate90); |
| 260 | ASSERT_EQ(mExpected90DifferentGroupConfig, performanceRate90); |
| 261 | } |
| 262 | |
| 263 | TEST_F(RefreshRateConfigsTest, twoDeviceConfigs_storesFullRefreshRateMap_differentResolutions) { |
| 264 | auto refreshRateConfigs = |
| 265 | std::make_unique<RefreshRateConfigs>(m60_90DeviceWithDifferentResolutions, |
| 266 | /*currentConfigId=*/HWC_CONFIG_ID_60); |
| 267 | |
Marin Shalamanov | eadf2e7 | 2020-12-10 15:35:28 +0100 | [diff] [blame] | 268 | const auto& minRate = getMinRefreshRateByPolicy(*refreshRateConfigs); |
| 269 | const auto& performanceRate = getMaxSupportedRefreshRate(*refreshRateConfigs); |
| 270 | const auto& minRate60 = getMinRefreshRateByPolicy(*refreshRateConfigs); |
Ady Abraham | abc2760 | 2020-04-08 17:20:29 -0700 | [diff] [blame] | 271 | const auto& performanceRate60 = refreshRateConfigs->getMaxRefreshRateByPolicy(); |
| 272 | |
| 273 | ASSERT_EQ(mExpected60Config, minRate); |
| 274 | ASSERT_EQ(mExpected60Config, minRate60); |
| 275 | ASSERT_EQ(mExpected60Config, performanceRate60); |
| 276 | |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 277 | ASSERT_GE(refreshRateConfigs->setDisplayManagerPolicy({HWC_CONFIG_ID_90, {60_Hz, 90_Hz}}), 0); |
Marin Shalamanov | a7fe304 | 2021-01-29 21:02:08 +0100 | [diff] [blame] | 278 | refreshRateConfigs->setCurrentModeId(HWC_CONFIG_ID_90); |
Ady Abraham | abc2760 | 2020-04-08 17:20:29 -0700 | [diff] [blame] | 279 | |
Marin Shalamanov | eadf2e7 | 2020-12-10 15:35:28 +0100 | [diff] [blame] | 280 | const auto& minRate90 = getMinRefreshRateByPolicy(*refreshRateConfigs); |
Ady Abraham | abc2760 | 2020-04-08 17:20:29 -0700 | [diff] [blame] | 281 | const auto& performanceRate90 = refreshRateConfigs->getMaxRefreshRateByPolicy(); |
| 282 | |
| 283 | ASSERT_EQ(mExpected90DifferentResolutionConfig, performanceRate); |
| 284 | ASSERT_EQ(mExpected90DifferentResolutionConfig, minRate90); |
| 285 | ASSERT_EQ(mExpected90DifferentResolutionConfig, performanceRate90); |
Ady Abraham | 2139f73 | 2019-11-13 18:56:40 -0800 | [diff] [blame] | 286 | } |
| 287 | |
| 288 | TEST_F(RefreshRateConfigsTest, twoDeviceConfigs_policyChange) { |
Ady Abraham | 2139f73 | 2019-11-13 18:56:40 -0800 | [diff] [blame] | 289 | auto refreshRateConfigs = |
Ady Abraham | abc2760 | 2020-04-08 17:20:29 -0700 | [diff] [blame] | 290 | std::make_unique<RefreshRateConfigs>(m60_90Device, |
| 291 | /*currentConfigId=*/HWC_CONFIG_ID_60); |
Ana Krulec | 3f6a206 | 2020-01-23 15:48:01 -0800 | [diff] [blame] | 292 | |
Marin Shalamanov | eadf2e7 | 2020-12-10 15:35:28 +0100 | [diff] [blame] | 293 | auto minRate = getMinRefreshRateByPolicy(*refreshRateConfigs); |
| 294 | auto performanceRate = refreshRateConfigs->getMaxRefreshRateByPolicy(); |
Ady Abraham | 2139f73 | 2019-11-13 18:56:40 -0800 | [diff] [blame] | 295 | |
Ady Abraham | abc2760 | 2020-04-08 17:20:29 -0700 | [diff] [blame] | 296 | ASSERT_EQ(mExpected60Config, minRate); |
| 297 | ASSERT_EQ(mExpected90Config, performanceRate); |
Ady Abraham | 2139f73 | 2019-11-13 18:56:40 -0800 | [diff] [blame] | 298 | |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 299 | ASSERT_GE(refreshRateConfigs->setDisplayManagerPolicy({HWC_CONFIG_ID_60, {60_Hz, 60_Hz}}), 0); |
Ady Abraham | 2139f73 | 2019-11-13 18:56:40 -0800 | [diff] [blame] | 300 | |
Marin Shalamanov | eadf2e7 | 2020-12-10 15:35:28 +0100 | [diff] [blame] | 301 | auto minRate60 = getMinRefreshRateByPolicy(*refreshRateConfigs); |
| 302 | auto performanceRate60 = refreshRateConfigs->getMaxRefreshRateByPolicy(); |
Ady Abraham | abc2760 | 2020-04-08 17:20:29 -0700 | [diff] [blame] | 303 | ASSERT_EQ(mExpected60Config, minRate60); |
| 304 | ASSERT_EQ(mExpected60Config, performanceRate60); |
Ady Abraham | 2139f73 | 2019-11-13 18:56:40 -0800 | [diff] [blame] | 305 | } |
| 306 | |
| 307 | TEST_F(RefreshRateConfigsTest, twoDeviceConfigs_getCurrentRefreshRate) { |
Ady Abraham | 2139f73 | 2019-11-13 18:56:40 -0800 | [diff] [blame] | 308 | auto refreshRateConfigs = |
Ady Abraham | abc2760 | 2020-04-08 17:20:29 -0700 | [diff] [blame] | 309 | std::make_unique<RefreshRateConfigs>(m60_90Device, |
| 310 | /*currentConfigId=*/HWC_CONFIG_ID_60); |
Ady Abraham | 2139f73 | 2019-11-13 18:56:40 -0800 | [diff] [blame] | 311 | { |
Marin Shalamanov | eadf2e7 | 2020-12-10 15:35:28 +0100 | [diff] [blame] | 312 | auto current = refreshRateConfigs->getCurrentRefreshRate(); |
Marin Shalamanov | a7fe304 | 2021-01-29 21:02:08 +0100 | [diff] [blame] | 313 | EXPECT_EQ(current.getModeId(), HWC_CONFIG_ID_60); |
Ady Abraham | 2139f73 | 2019-11-13 18:56:40 -0800 | [diff] [blame] | 314 | } |
| 315 | |
Marin Shalamanov | a7fe304 | 2021-01-29 21:02:08 +0100 | [diff] [blame] | 316 | refreshRateConfigs->setCurrentModeId(HWC_CONFIG_ID_90); |
Ady Abraham | 2139f73 | 2019-11-13 18:56:40 -0800 | [diff] [blame] | 317 | { |
Marin Shalamanov | eadf2e7 | 2020-12-10 15:35:28 +0100 | [diff] [blame] | 318 | auto current = refreshRateConfigs->getCurrentRefreshRate(); |
Marin Shalamanov | a7fe304 | 2021-01-29 21:02:08 +0100 | [diff] [blame] | 319 | EXPECT_EQ(current.getModeId(), HWC_CONFIG_ID_90); |
Ady Abraham | 2139f73 | 2019-11-13 18:56:40 -0800 | [diff] [blame] | 320 | } |
| 321 | |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 322 | ASSERT_GE(refreshRateConfigs->setDisplayManagerPolicy({HWC_CONFIG_ID_90, {90_Hz, 90_Hz}}), 0); |
Ady Abraham | 2139f73 | 2019-11-13 18:56:40 -0800 | [diff] [blame] | 323 | { |
Marin Shalamanov | eadf2e7 | 2020-12-10 15:35:28 +0100 | [diff] [blame] | 324 | auto current = refreshRateConfigs->getCurrentRefreshRate(); |
Marin Shalamanov | a7fe304 | 2021-01-29 21:02:08 +0100 | [diff] [blame] | 325 | EXPECT_EQ(current.getModeId(), HWC_CONFIG_ID_90); |
Ady Abraham | 2139f73 | 2019-11-13 18:56:40 -0800 | [diff] [blame] | 326 | } |
| 327 | } |
| 328 | |
Steven Thomas | bb37432 | 2020-04-28 22:47:16 -0700 | [diff] [blame] | 329 | TEST_F(RefreshRateConfigsTest, getBestRefreshRate_noLayers) { |
Ady Abraham | abc2760 | 2020-04-08 17:20:29 -0700 | [diff] [blame] | 330 | auto refreshRateConfigs = |
| 331 | std::make_unique<RefreshRateConfigs>(m60_72_90Device, /*currentConfigId=*/ |
| 332 | HWC_CONFIG_ID_72); |
Ana Krulec | 3d367c8 | 2020-02-25 15:02:01 -0800 | [diff] [blame] | 333 | |
Steven Thomas | debafed | 2020-05-18 17:30:35 -0700 | [diff] [blame] | 334 | // If there are no layers we select the default frame rate, which is the max of the primary |
| 335 | // range. |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 336 | EXPECT_EQ(mExpected90Config, refreshRateConfigs->getBestRefreshRate({}, {})); |
Ana Krulec | 3d367c8 | 2020-02-25 15:02:01 -0800 | [diff] [blame] | 337 | |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 338 | ASSERT_EQ(refreshRateConfigs->setDisplayManagerPolicy({HWC_CONFIG_ID_60, {60_Hz, 60_Hz}}), |
Marin Shalamanov | 8cd8a99 | 2021-09-14 23:22:49 +0200 | [diff] [blame] | 339 | NO_ERROR); |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 340 | EXPECT_EQ(mExpected60Config, refreshRateConfigs->getBestRefreshRate({}, {})); |
Marin Shalamanov | 8cd8a99 | 2021-09-14 23:22:49 +0200 | [diff] [blame] | 341 | |
| 342 | // We select max even when this will cause a non-seamless switch. |
| 343 | refreshRateConfigs = std::make_unique<RefreshRateConfigs>(m60_90DeviceWithDifferentGroups, |
| 344 | /*currentConfigId=*/HWC_CONFIG_ID_60); |
| 345 | ASSERT_EQ(refreshRateConfigs->setDisplayManagerPolicy( |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 346 | {HWC_CONFIG_ID_90, /*allowGroupSwitching*/ true, {0_Hz, 90_Hz}}), |
Marin Shalamanov | 8cd8a99 | 2021-09-14 23:22:49 +0200 | [diff] [blame] | 347 | NO_ERROR); |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 348 | EXPECT_EQ(mExpected90DifferentGroupConfig, refreshRateConfigs->getBestRefreshRate({}, {})); |
Ana Krulec | 3d367c8 | 2020-02-25 15:02:01 -0800 | [diff] [blame] | 349 | } |
| 350 | |
Steven Thomas | bb37432 | 2020-04-28 22:47:16 -0700 | [diff] [blame] | 351 | TEST_F(RefreshRateConfigsTest, getBestRefreshRate_60_90) { |
Ady Abraham | 8a82ba6 | 2020-01-17 12:43:17 -0800 | [diff] [blame] | 352 | auto refreshRateConfigs = |
Ady Abraham | abc2760 | 2020-04-08 17:20:29 -0700 | [diff] [blame] | 353 | std::make_unique<RefreshRateConfigs>(m60_90Device, |
| 354 | /*currentConfigId=*/HWC_CONFIG_ID_60); |
Ady Abraham | 8a82ba6 | 2020-01-17 12:43:17 -0800 | [diff] [blame] | 355 | |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 356 | std::vector<LayerRequirement> layers = {{.weight = 1.f}}; |
Ady Abraham | 8a82ba6 | 2020-01-17 12:43:17 -0800 | [diff] [blame] | 357 | auto& lr = layers[0]; |
| 358 | |
| 359 | lr.vote = LayerVoteType::Min; |
Ady Abraham | 4ccdcb4 | 2020-02-11 17:34:34 -0800 | [diff] [blame] | 360 | lr.name = "Min"; |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 361 | EXPECT_EQ(mExpected60Config, refreshRateConfigs->getBestRefreshRate(layers, {})); |
Ady Abraham | 8a82ba6 | 2020-01-17 12:43:17 -0800 | [diff] [blame] | 362 | |
| 363 | lr.vote = LayerVoteType::Max; |
Ady Abraham | 4ccdcb4 | 2020-02-11 17:34:34 -0800 | [diff] [blame] | 364 | lr.name = "Max"; |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 365 | EXPECT_EQ(mExpected90Config, refreshRateConfigs->getBestRefreshRate(layers, {})); |
Ady Abraham | 8a82ba6 | 2020-01-17 12:43:17 -0800 | [diff] [blame] | 366 | |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 367 | lr.desiredRefreshRate = 90_Hz; |
Ady Abraham | 8a82ba6 | 2020-01-17 12:43:17 -0800 | [diff] [blame] | 368 | lr.vote = LayerVoteType::Heuristic; |
Ady Abraham | 4ccdcb4 | 2020-02-11 17:34:34 -0800 | [diff] [blame] | 369 | lr.name = "90Hz Heuristic"; |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 370 | EXPECT_EQ(mExpected90Config, refreshRateConfigs->getBestRefreshRate(layers, {})); |
Ady Abraham | 8a82ba6 | 2020-01-17 12:43:17 -0800 | [diff] [blame] | 371 | |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 372 | lr.desiredRefreshRate = 60_Hz; |
Ady Abraham | 4ccdcb4 | 2020-02-11 17:34:34 -0800 | [diff] [blame] | 373 | lr.name = "60Hz Heuristic"; |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 374 | EXPECT_EQ(mExpected60Config, refreshRateConfigs->getBestRefreshRate(layers, {})); |
Ady Abraham | 8a82ba6 | 2020-01-17 12:43:17 -0800 | [diff] [blame] | 375 | |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 376 | lr.desiredRefreshRate = 45_Hz; |
Ady Abraham | 4ccdcb4 | 2020-02-11 17:34:34 -0800 | [diff] [blame] | 377 | lr.name = "45Hz Heuristic"; |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 378 | EXPECT_EQ(mExpected90Config, refreshRateConfigs->getBestRefreshRate(layers, {})); |
Ady Abraham | 8a82ba6 | 2020-01-17 12:43:17 -0800 | [diff] [blame] | 379 | |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 380 | lr.desiredRefreshRate = 30_Hz; |
Ady Abraham | 4ccdcb4 | 2020-02-11 17:34:34 -0800 | [diff] [blame] | 381 | lr.name = "30Hz Heuristic"; |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 382 | EXPECT_EQ(mExpected60Config, refreshRateConfigs->getBestRefreshRate(layers, {})); |
Ady Abraham | 8a82ba6 | 2020-01-17 12:43:17 -0800 | [diff] [blame] | 383 | |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 384 | lr.desiredRefreshRate = 24_Hz; |
Ady Abraham | 4ccdcb4 | 2020-02-11 17:34:34 -0800 | [diff] [blame] | 385 | lr.name = "24Hz Heuristic"; |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 386 | EXPECT_EQ(mExpected60Config, refreshRateConfigs->getBestRefreshRate(layers, {})); |
Ady Abraham | 8a82ba6 | 2020-01-17 12:43:17 -0800 | [diff] [blame] | 387 | |
Ady Abraham | 4ccdcb4 | 2020-02-11 17:34:34 -0800 | [diff] [blame] | 388 | lr.name = ""; |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 389 | ASSERT_GE(refreshRateConfigs->setDisplayManagerPolicy({HWC_CONFIG_ID_60, {60_Hz, 60_Hz}}), 0); |
Ady Abraham | 8a82ba6 | 2020-01-17 12:43:17 -0800 | [diff] [blame] | 390 | |
| 391 | lr.vote = LayerVoteType::Min; |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 392 | EXPECT_EQ(mExpected60Config, refreshRateConfigs->getBestRefreshRate(layers, {})); |
Ady Abraham | 8a82ba6 | 2020-01-17 12:43:17 -0800 | [diff] [blame] | 393 | |
| 394 | lr.vote = LayerVoteType::Max; |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 395 | EXPECT_EQ(mExpected60Config, refreshRateConfigs->getBestRefreshRate(layers, {})); |
Ady Abraham | 8a82ba6 | 2020-01-17 12:43:17 -0800 | [diff] [blame] | 396 | |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 397 | lr.desiredRefreshRate = 90_Hz; |
Ady Abraham | 8a82ba6 | 2020-01-17 12:43:17 -0800 | [diff] [blame] | 398 | lr.vote = LayerVoteType::Heuristic; |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 399 | EXPECT_EQ(mExpected60Config, refreshRateConfigs->getBestRefreshRate(layers, {})); |
Ady Abraham | 8a82ba6 | 2020-01-17 12:43:17 -0800 | [diff] [blame] | 400 | |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 401 | lr.desiredRefreshRate = 60_Hz; |
| 402 | EXPECT_EQ(mExpected60Config, refreshRateConfigs->getBestRefreshRate(layers, {})); |
Ady Abraham | 8a82ba6 | 2020-01-17 12:43:17 -0800 | [diff] [blame] | 403 | |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 404 | lr.desiredRefreshRate = 45_Hz; |
| 405 | EXPECT_EQ(mExpected60Config, refreshRateConfigs->getBestRefreshRate(layers, {})); |
Ady Abraham | 8a82ba6 | 2020-01-17 12:43:17 -0800 | [diff] [blame] | 406 | |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 407 | lr.desiredRefreshRate = 30_Hz; |
| 408 | EXPECT_EQ(mExpected60Config, refreshRateConfigs->getBestRefreshRate(layers, {})); |
Ady Abraham | 8a82ba6 | 2020-01-17 12:43:17 -0800 | [diff] [blame] | 409 | |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 410 | lr.desiredRefreshRate = 24_Hz; |
| 411 | EXPECT_EQ(mExpected60Config, refreshRateConfigs->getBestRefreshRate(layers, {})); |
Ady Abraham | 8a82ba6 | 2020-01-17 12:43:17 -0800 | [diff] [blame] | 412 | |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 413 | ASSERT_GE(refreshRateConfigs->setDisplayManagerPolicy({HWC_CONFIG_ID_90, {90_Hz, 90_Hz}}), 0); |
Ady Abraham | 8a82ba6 | 2020-01-17 12:43:17 -0800 | [diff] [blame] | 414 | |
| 415 | lr.vote = LayerVoteType::Min; |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 416 | EXPECT_EQ(mExpected90Config, refreshRateConfigs->getBestRefreshRate(layers, {})); |
Ady Abraham | 8a82ba6 | 2020-01-17 12:43:17 -0800 | [diff] [blame] | 417 | |
| 418 | lr.vote = LayerVoteType::Max; |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 419 | EXPECT_EQ(mExpected90Config, refreshRateConfigs->getBestRefreshRate(layers, {})); |
Ady Abraham | 8a82ba6 | 2020-01-17 12:43:17 -0800 | [diff] [blame] | 420 | |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 421 | lr.desiredRefreshRate = 90_Hz; |
Ady Abraham | 8a82ba6 | 2020-01-17 12:43:17 -0800 | [diff] [blame] | 422 | lr.vote = LayerVoteType::Heuristic; |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 423 | EXPECT_EQ(mExpected90Config, refreshRateConfigs->getBestRefreshRate(layers, {})); |
Ady Abraham | 8a82ba6 | 2020-01-17 12:43:17 -0800 | [diff] [blame] | 424 | |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 425 | lr.desiredRefreshRate = 60_Hz; |
| 426 | EXPECT_EQ(mExpected90Config, refreshRateConfigs->getBestRefreshRate(layers, {})); |
Ady Abraham | 8a82ba6 | 2020-01-17 12:43:17 -0800 | [diff] [blame] | 427 | |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 428 | lr.desiredRefreshRate = 45_Hz; |
| 429 | EXPECT_EQ(mExpected90Config, refreshRateConfigs->getBestRefreshRate(layers, {})); |
Ady Abraham | 8a82ba6 | 2020-01-17 12:43:17 -0800 | [diff] [blame] | 430 | |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 431 | lr.desiredRefreshRate = 30_Hz; |
| 432 | EXPECT_EQ(mExpected90Config, refreshRateConfigs->getBestRefreshRate(layers, {})); |
Ady Abraham | 8a82ba6 | 2020-01-17 12:43:17 -0800 | [diff] [blame] | 433 | |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 434 | lr.desiredRefreshRate = 24_Hz; |
| 435 | EXPECT_EQ(mExpected90Config, refreshRateConfigs->getBestRefreshRate(layers, {})); |
Ady Abraham | 8a82ba6 | 2020-01-17 12:43:17 -0800 | [diff] [blame] | 436 | |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 437 | ASSERT_GE(refreshRateConfigs->setDisplayManagerPolicy({HWC_CONFIG_ID_60, {0_Hz, 120_Hz}}), 0); |
Ady Abraham | 8a82ba6 | 2020-01-17 12:43:17 -0800 | [diff] [blame] | 438 | lr.vote = LayerVoteType::Min; |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 439 | EXPECT_EQ(mExpected60Config, refreshRateConfigs->getBestRefreshRate(layers, {})); |
Ady Abraham | 8a82ba6 | 2020-01-17 12:43:17 -0800 | [diff] [blame] | 440 | |
| 441 | lr.vote = LayerVoteType::Max; |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 442 | EXPECT_EQ(mExpected90Config, refreshRateConfigs->getBestRefreshRate(layers, {})); |
Ady Abraham | 8a82ba6 | 2020-01-17 12:43:17 -0800 | [diff] [blame] | 443 | |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 444 | lr.desiredRefreshRate = 90_Hz; |
Ady Abraham | 8a82ba6 | 2020-01-17 12:43:17 -0800 | [diff] [blame] | 445 | lr.vote = LayerVoteType::Heuristic; |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 446 | EXPECT_EQ(mExpected90Config, refreshRateConfigs->getBestRefreshRate(layers, {})); |
Ady Abraham | 8a82ba6 | 2020-01-17 12:43:17 -0800 | [diff] [blame] | 447 | |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 448 | lr.desiredRefreshRate = 60_Hz; |
| 449 | EXPECT_EQ(mExpected60Config, refreshRateConfigs->getBestRefreshRate(layers, {})); |
Ady Abraham | 8a82ba6 | 2020-01-17 12:43:17 -0800 | [diff] [blame] | 450 | |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 451 | lr.desiredRefreshRate = 45_Hz; |
| 452 | EXPECT_EQ(mExpected90Config, refreshRateConfigs->getBestRefreshRate(layers, {})); |
Ady Abraham | 8a82ba6 | 2020-01-17 12:43:17 -0800 | [diff] [blame] | 453 | |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 454 | lr.desiredRefreshRate = 30_Hz; |
| 455 | EXPECT_EQ(mExpected60Config, refreshRateConfigs->getBestRefreshRate(layers, {})); |
Ady Abraham | 8a82ba6 | 2020-01-17 12:43:17 -0800 | [diff] [blame] | 456 | |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 457 | lr.desiredRefreshRate = 24_Hz; |
| 458 | EXPECT_EQ(mExpected60Config, refreshRateConfigs->getBestRefreshRate(layers, {})); |
Ady Abraham | 8a82ba6 | 2020-01-17 12:43:17 -0800 | [diff] [blame] | 459 | } |
| 460 | |
rnlee | 3bd61066 | 2021-06-23 16:27:57 -0700 | [diff] [blame] | 461 | TEST_F(RefreshRateConfigsTest, getBestRefreshRate_multipleThreshold_60_90) { |
| 462 | RefreshRateConfigs::Config config = {.frameRateMultipleThreshold = 90}; |
| 463 | auto refreshRateConfigs = |
| 464 | std::make_unique<RefreshRateConfigs>(m60_90Device, |
| 465 | /*currentConfigId=*/HWC_CONFIG_ID_60, config); |
| 466 | |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 467 | std::vector<LayerRequirement> layers = {{.weight = 1.f}}; |
rnlee | 3bd61066 | 2021-06-23 16:27:57 -0700 | [diff] [blame] | 468 | auto& lr = layers[0]; |
| 469 | |
| 470 | lr.vote = LayerVoteType::Min; |
| 471 | lr.name = "Min"; |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 472 | EXPECT_EQ(mExpected60Config, refreshRateConfigs->getBestRefreshRate(layers, {})); |
rnlee | 3bd61066 | 2021-06-23 16:27:57 -0700 | [diff] [blame] | 473 | |
| 474 | lr.vote = LayerVoteType::Max; |
| 475 | lr.name = "Max"; |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 476 | EXPECT_EQ(mExpected90Config, refreshRateConfigs->getBestRefreshRate(layers, {})); |
rnlee | 3bd61066 | 2021-06-23 16:27:57 -0700 | [diff] [blame] | 477 | |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 478 | lr.desiredRefreshRate = 90_Hz; |
rnlee | 3bd61066 | 2021-06-23 16:27:57 -0700 | [diff] [blame] | 479 | lr.vote = LayerVoteType::Heuristic; |
| 480 | lr.name = "90Hz Heuristic"; |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 481 | EXPECT_EQ(mExpected90Config, refreshRateConfigs->getBestRefreshRate(layers, {})); |
rnlee | 3bd61066 | 2021-06-23 16:27:57 -0700 | [diff] [blame] | 482 | |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 483 | lr.desiredRefreshRate = 60_Hz; |
rnlee | 3bd61066 | 2021-06-23 16:27:57 -0700 | [diff] [blame] | 484 | lr.name = "60Hz Heuristic"; |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 485 | EXPECT_EQ(mExpected60Config, refreshRateConfigs->getBestRefreshRate(layers, {})); |
rnlee | 3bd61066 | 2021-06-23 16:27:57 -0700 | [diff] [blame] | 486 | |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 487 | lr.desiredRefreshRate = 45_Hz; |
rnlee | 3bd61066 | 2021-06-23 16:27:57 -0700 | [diff] [blame] | 488 | lr.name = "45Hz Heuristic"; |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 489 | EXPECT_EQ(mExpected90Config, refreshRateConfigs->getBestRefreshRate(layers, {})); |
rnlee | 3bd61066 | 2021-06-23 16:27:57 -0700 | [diff] [blame] | 490 | |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 491 | lr.desiredRefreshRate = 30_Hz; |
rnlee | 3bd61066 | 2021-06-23 16:27:57 -0700 | [diff] [blame] | 492 | lr.name = "30Hz Heuristic"; |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 493 | EXPECT_EQ(mExpected60Config, refreshRateConfigs->getBestRefreshRate(layers, {})); |
rnlee | 3bd61066 | 2021-06-23 16:27:57 -0700 | [diff] [blame] | 494 | |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 495 | lr.desiredRefreshRate = 24_Hz; |
rnlee | 3bd61066 | 2021-06-23 16:27:57 -0700 | [diff] [blame] | 496 | lr.name = "24Hz Heuristic"; |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 497 | EXPECT_EQ(mExpected60Config, refreshRateConfigs->getBestRefreshRate(layers, {})); |
rnlee | 3bd61066 | 2021-06-23 16:27:57 -0700 | [diff] [blame] | 498 | } |
| 499 | |
Steven Thomas | bb37432 | 2020-04-28 22:47:16 -0700 | [diff] [blame] | 500 | TEST_F(RefreshRateConfigsTest, getBestRefreshRate_60_72_90) { |
Ady Abraham | 8a82ba6 | 2020-01-17 12:43:17 -0800 | [diff] [blame] | 501 | auto refreshRateConfigs = |
Ady Abraham | abc2760 | 2020-04-08 17:20:29 -0700 | [diff] [blame] | 502 | std::make_unique<RefreshRateConfigs>(m60_72_90Device, |
| 503 | /*currentConfigId=*/HWC_CONFIG_ID_60); |
Ady Abraham | 8a82ba6 | 2020-01-17 12:43:17 -0800 | [diff] [blame] | 504 | |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 505 | std::vector<LayerRequirement> layers = {{.weight = 1.f}}; |
Ady Abraham | 8a82ba6 | 2020-01-17 12:43:17 -0800 | [diff] [blame] | 506 | auto& lr = layers[0]; |
| 507 | |
| 508 | lr.vote = LayerVoteType::Min; |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 509 | EXPECT_EQ(mExpected60Config, refreshRateConfigs->getBestRefreshRate(layers, {})); |
Ady Abraham | 8a82ba6 | 2020-01-17 12:43:17 -0800 | [diff] [blame] | 510 | |
| 511 | lr.vote = LayerVoteType::Max; |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 512 | EXPECT_EQ(mExpected90Config, refreshRateConfigs->getBestRefreshRate(layers, {})); |
Ady Abraham | 8a82ba6 | 2020-01-17 12:43:17 -0800 | [diff] [blame] | 513 | |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 514 | lr.desiredRefreshRate = 90_Hz; |
Ady Abraham | 8a82ba6 | 2020-01-17 12:43:17 -0800 | [diff] [blame] | 515 | lr.vote = LayerVoteType::Heuristic; |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 516 | EXPECT_EQ(mExpected90Config, refreshRateConfigs->getBestRefreshRate(layers, {})); |
Ady Abraham | 8a82ba6 | 2020-01-17 12:43:17 -0800 | [diff] [blame] | 517 | |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 518 | lr.desiredRefreshRate = 60_Hz; |
| 519 | EXPECT_EQ(mExpected60Config, refreshRateConfigs->getBestRefreshRate(layers, {})); |
Ady Abraham | 8a82ba6 | 2020-01-17 12:43:17 -0800 | [diff] [blame] | 520 | |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 521 | lr.desiredRefreshRate = 45_Hz; |
| 522 | EXPECT_EQ(mExpected90Config, refreshRateConfigs->getBestRefreshRate(layers, {})); |
Ady Abraham | 8a82ba6 | 2020-01-17 12:43:17 -0800 | [diff] [blame] | 523 | |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 524 | lr.desiredRefreshRate = 30_Hz; |
| 525 | EXPECT_EQ(mExpected60Config, refreshRateConfigs->getBestRefreshRate(layers, {})); |
Ady Abraham | 8a82ba6 | 2020-01-17 12:43:17 -0800 | [diff] [blame] | 526 | |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 527 | lr.desiredRefreshRate = 24_Hz; |
| 528 | EXPECT_EQ(mExpected72Config, refreshRateConfigs->getBestRefreshRate(layers, {})); |
Ady Abraham | 8a82ba6 | 2020-01-17 12:43:17 -0800 | [diff] [blame] | 529 | } |
| 530 | |
Steven Thomas | bb37432 | 2020-04-28 22:47:16 -0700 | [diff] [blame] | 531 | TEST_F(RefreshRateConfigsTest, getBestRefreshRate_30_60_72_90_120) { |
Ady Abraham | 8a82ba6 | 2020-01-17 12:43:17 -0800 | [diff] [blame] | 532 | auto refreshRateConfigs = |
Ady Abraham | abc2760 | 2020-04-08 17:20:29 -0700 | [diff] [blame] | 533 | std::make_unique<RefreshRateConfigs>(m30_60_72_90_120Device, |
| 534 | /*currentConfigId=*/HWC_CONFIG_ID_60); |
Ady Abraham | 8a82ba6 | 2020-01-17 12:43:17 -0800 | [diff] [blame] | 535 | |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 536 | std::vector<LayerRequirement> layers = {{.weight = 1.f}, {.weight = 1.f}}; |
Ady Abraham | 8a82ba6 | 2020-01-17 12:43:17 -0800 | [diff] [blame] | 537 | auto& lr1 = layers[0]; |
| 538 | auto& lr2 = layers[1]; |
| 539 | |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 540 | lr1.desiredRefreshRate = 24_Hz; |
Ady Abraham | 8a82ba6 | 2020-01-17 12:43:17 -0800 | [diff] [blame] | 541 | lr1.vote = LayerVoteType::Heuristic; |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 542 | lr2.desiredRefreshRate = 60_Hz; |
Ady Abraham | 8a82ba6 | 2020-01-17 12:43:17 -0800 | [diff] [blame] | 543 | lr2.vote = LayerVoteType::Heuristic; |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 544 | EXPECT_EQ(mExpected120Config, refreshRateConfigs->getBestRefreshRate(layers, {})); |
Ady Abraham | 8a82ba6 | 2020-01-17 12:43:17 -0800 | [diff] [blame] | 545 | |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 546 | lr1.desiredRefreshRate = 24_Hz; |
Ady Abraham | 8a82ba6 | 2020-01-17 12:43:17 -0800 | [diff] [blame] | 547 | lr1.vote = LayerVoteType::Heuristic; |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 548 | lr2.desiredRefreshRate = 48_Hz; |
Ady Abraham | 8a82ba6 | 2020-01-17 12:43:17 -0800 | [diff] [blame] | 549 | lr2.vote = LayerVoteType::Heuristic; |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 550 | EXPECT_EQ(mExpected72Config, refreshRateConfigs->getBestRefreshRate(layers, {})); |
Ady Abraham | 8a82ba6 | 2020-01-17 12:43:17 -0800 | [diff] [blame] | 551 | |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 552 | lr1.desiredRefreshRate = 24_Hz; |
Ady Abraham | 8a82ba6 | 2020-01-17 12:43:17 -0800 | [diff] [blame] | 553 | lr1.vote = LayerVoteType::Heuristic; |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 554 | lr2.desiredRefreshRate = 48_Hz; |
Ady Abraham | 8a82ba6 | 2020-01-17 12:43:17 -0800 | [diff] [blame] | 555 | lr2.vote = LayerVoteType::Heuristic; |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 556 | EXPECT_EQ(mExpected72Config, refreshRateConfigs->getBestRefreshRate(layers, {})); |
Ady Abraham | 8a82ba6 | 2020-01-17 12:43:17 -0800 | [diff] [blame] | 557 | } |
| 558 | |
Steven Thomas | bb37432 | 2020-04-28 22:47:16 -0700 | [diff] [blame] | 559 | TEST_F(RefreshRateConfigsTest, getBestRefreshRate_30_60_90_120_DifferentTypes) { |
Ady Abraham | 71c437d | 2020-01-31 15:56:57 -0800 | [diff] [blame] | 560 | auto refreshRateConfigs = |
Ady Abraham | abc2760 | 2020-04-08 17:20:29 -0700 | [diff] [blame] | 561 | std::make_unique<RefreshRateConfigs>(m30_60_72_90_120Device, |
| 562 | /*currentConfigId=*/HWC_CONFIG_ID_60); |
Ady Abraham | 71c437d | 2020-01-31 15:56:57 -0800 | [diff] [blame] | 563 | |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 564 | std::vector<LayerRequirement> layers = {{.weight = 1.f}, {.weight = 1.f}}; |
Ady Abraham | 71c437d | 2020-01-31 15:56:57 -0800 | [diff] [blame] | 565 | auto& lr1 = layers[0]; |
| 566 | auto& lr2 = layers[1]; |
| 567 | |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 568 | lr1.desiredRefreshRate = 24_Hz; |
Ady Abraham | 71c437d | 2020-01-31 15:56:57 -0800 | [diff] [blame] | 569 | lr1.vote = LayerVoteType::ExplicitDefault; |
Ady Abraham | 4ccdcb4 | 2020-02-11 17:34:34 -0800 | [diff] [blame] | 570 | lr1.name = "24Hz ExplicitDefault"; |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 571 | lr2.desiredRefreshRate = 60_Hz; |
Ady Abraham | 71c437d | 2020-01-31 15:56:57 -0800 | [diff] [blame] | 572 | lr2.vote = LayerVoteType::Heuristic; |
Ady Abraham | 4ccdcb4 | 2020-02-11 17:34:34 -0800 | [diff] [blame] | 573 | lr2.name = "60Hz Heuristic"; |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 574 | EXPECT_EQ(mExpected120Config, refreshRateConfigs->getBestRefreshRate(layers, {})); |
Ady Abraham | 71c437d | 2020-01-31 15:56:57 -0800 | [diff] [blame] | 575 | |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 576 | lr1.desiredRefreshRate = 24_Hz; |
Ady Abraham | 71c437d | 2020-01-31 15:56:57 -0800 | [diff] [blame] | 577 | lr1.vote = LayerVoteType::ExplicitExactOrMultiple; |
Ady Abraham | 4ccdcb4 | 2020-02-11 17:34:34 -0800 | [diff] [blame] | 578 | lr1.name = "24Hz ExplicitExactOrMultiple"; |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 579 | lr2.desiredRefreshRate = 60_Hz; |
Ady Abraham | 71c437d | 2020-01-31 15:56:57 -0800 | [diff] [blame] | 580 | lr2.vote = LayerVoteType::Heuristic; |
Ady Abraham | 4ccdcb4 | 2020-02-11 17:34:34 -0800 | [diff] [blame] | 581 | lr2.name = "60Hz Heuristic"; |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 582 | EXPECT_EQ(mExpected120Config, refreshRateConfigs->getBestRefreshRate(layers, {})); |
Ady Abraham | 71c437d | 2020-01-31 15:56:57 -0800 | [diff] [blame] | 583 | |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 584 | lr1.desiredRefreshRate = 24_Hz; |
Ady Abraham | 71c437d | 2020-01-31 15:56:57 -0800 | [diff] [blame] | 585 | lr1.vote = LayerVoteType::ExplicitExactOrMultiple; |
Ady Abraham | 4ccdcb4 | 2020-02-11 17:34:34 -0800 | [diff] [blame] | 586 | lr1.name = "24Hz ExplicitExactOrMultiple"; |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 587 | lr2.desiredRefreshRate = 60_Hz; |
Ady Abraham | 71c437d | 2020-01-31 15:56:57 -0800 | [diff] [blame] | 588 | lr2.vote = LayerVoteType::ExplicitDefault; |
Ady Abraham | 4ccdcb4 | 2020-02-11 17:34:34 -0800 | [diff] [blame] | 589 | lr2.name = "60Hz ExplicitDefault"; |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 590 | EXPECT_EQ(mExpected120Config, refreshRateConfigs->getBestRefreshRate(layers, {})); |
Ady Abraham | 71c437d | 2020-01-31 15:56:57 -0800 | [diff] [blame] | 591 | |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 592 | lr1.desiredRefreshRate = 24_Hz; |
Ady Abraham | 71c437d | 2020-01-31 15:56:57 -0800 | [diff] [blame] | 593 | lr1.vote = LayerVoteType::ExplicitExactOrMultiple; |
Ady Abraham | 4ccdcb4 | 2020-02-11 17:34:34 -0800 | [diff] [blame] | 594 | lr1.name = "24Hz ExplicitExactOrMultiple"; |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 595 | lr2.desiredRefreshRate = 90_Hz; |
Ady Abraham | 71c437d | 2020-01-31 15:56:57 -0800 | [diff] [blame] | 596 | lr2.vote = LayerVoteType::Heuristic; |
Ady Abraham | 4ccdcb4 | 2020-02-11 17:34:34 -0800 | [diff] [blame] | 597 | lr2.name = "90Hz Heuristic"; |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 598 | EXPECT_EQ(mExpected90Config, refreshRateConfigs->getBestRefreshRate(layers, {})); |
Ady Abraham | 4ccdcb4 | 2020-02-11 17:34:34 -0800 | [diff] [blame] | 599 | |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 600 | lr1.desiredRefreshRate = 24_Hz; |
Ady Abraham | 4ccdcb4 | 2020-02-11 17:34:34 -0800 | [diff] [blame] | 601 | lr1.vote = LayerVoteType::ExplicitExactOrMultiple; |
| 602 | lr1.name = "24Hz ExplicitExactOrMultiple"; |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 603 | lr2.desiredRefreshRate = 90_Hz; |
Ady Abraham | 4ccdcb4 | 2020-02-11 17:34:34 -0800 | [diff] [blame] | 604 | lr2.vote = LayerVoteType::ExplicitDefault; |
| 605 | lr2.name = "90Hz Heuristic"; |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 606 | EXPECT_EQ(mExpected72Config, refreshRateConfigs->getBestRefreshRate(layers, {})); |
Ady Abraham | 71c437d | 2020-01-31 15:56:57 -0800 | [diff] [blame] | 607 | |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 608 | lr1.desiredRefreshRate = 24_Hz; |
Ady Abraham | 71c437d | 2020-01-31 15:56:57 -0800 | [diff] [blame] | 609 | lr1.vote = LayerVoteType::ExplicitDefault; |
Ady Abraham | 4ccdcb4 | 2020-02-11 17:34:34 -0800 | [diff] [blame] | 610 | lr1.name = "24Hz ExplicitDefault"; |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 611 | lr2.desiredRefreshRate = 90_Hz; |
Ady Abraham | 71c437d | 2020-01-31 15:56:57 -0800 | [diff] [blame] | 612 | lr2.vote = LayerVoteType::Heuristic; |
Ady Abraham | 4ccdcb4 | 2020-02-11 17:34:34 -0800 | [diff] [blame] | 613 | lr2.name = "90Hz Heuristic"; |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 614 | EXPECT_EQ(mExpected90Config, refreshRateConfigs->getBestRefreshRate(layers, {})); |
Ady Abraham | 71c437d | 2020-01-31 15:56:57 -0800 | [diff] [blame] | 615 | |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 616 | lr1.desiredRefreshRate = 24_Hz; |
Ady Abraham | 71c437d | 2020-01-31 15:56:57 -0800 | [diff] [blame] | 617 | lr1.vote = LayerVoteType::Heuristic; |
Ady Abraham | 4ccdcb4 | 2020-02-11 17:34:34 -0800 | [diff] [blame] | 618 | lr1.name = "24Hz Heuristic"; |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 619 | lr2.desiredRefreshRate = 90_Hz; |
Ady Abraham | 71c437d | 2020-01-31 15:56:57 -0800 | [diff] [blame] | 620 | lr2.vote = LayerVoteType::ExplicitDefault; |
Ady Abraham | 4ccdcb4 | 2020-02-11 17:34:34 -0800 | [diff] [blame] | 621 | lr2.name = "90Hz ExplicitDefault"; |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 622 | EXPECT_EQ(mExpected72Config, refreshRateConfigs->getBestRefreshRate(layers, {})); |
Ady Abraham | 71c437d | 2020-01-31 15:56:57 -0800 | [diff] [blame] | 623 | |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 624 | lr1.desiredRefreshRate = 24_Hz; |
Ady Abraham | 71c437d | 2020-01-31 15:56:57 -0800 | [diff] [blame] | 625 | lr1.vote = LayerVoteType::ExplicitExactOrMultiple; |
Ady Abraham | 4ccdcb4 | 2020-02-11 17:34:34 -0800 | [diff] [blame] | 626 | lr1.name = "24Hz ExplicitExactOrMultiple"; |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 627 | lr2.desiredRefreshRate = 90_Hz; |
Ady Abraham | 71c437d | 2020-01-31 15:56:57 -0800 | [diff] [blame] | 628 | lr2.vote = LayerVoteType::ExplicitDefault; |
Ady Abraham | 4ccdcb4 | 2020-02-11 17:34:34 -0800 | [diff] [blame] | 629 | lr2.name = "90Hz ExplicitDefault"; |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 630 | EXPECT_EQ(mExpected72Config, refreshRateConfigs->getBestRefreshRate(layers, {})); |
Ady Abraham | 71c437d | 2020-01-31 15:56:57 -0800 | [diff] [blame] | 631 | |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 632 | lr1.desiredRefreshRate = 24_Hz; |
Ady Abraham | 71c437d | 2020-01-31 15:56:57 -0800 | [diff] [blame] | 633 | lr1.vote = LayerVoteType::ExplicitDefault; |
Ady Abraham | 4ccdcb4 | 2020-02-11 17:34:34 -0800 | [diff] [blame] | 634 | lr1.name = "24Hz ExplicitDefault"; |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 635 | lr2.desiredRefreshRate = 90_Hz; |
Ady Abraham | 71c437d | 2020-01-31 15:56:57 -0800 | [diff] [blame] | 636 | lr2.vote = LayerVoteType::ExplicitExactOrMultiple; |
Ady Abraham | 4ccdcb4 | 2020-02-11 17:34:34 -0800 | [diff] [blame] | 637 | lr2.name = "90Hz ExplicitExactOrMultiple"; |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 638 | EXPECT_EQ(mExpected90Config, refreshRateConfigs->getBestRefreshRate(layers, {})); |
Ady Abraham | 71c437d | 2020-01-31 15:56:57 -0800 | [diff] [blame] | 639 | } |
| 640 | |
rnlee | 3bd61066 | 2021-06-23 16:27:57 -0700 | [diff] [blame] | 641 | TEST_F(RefreshRateConfigsTest, getBestRefreshRate_30_60_90_120_DifferentTypes_multipleThreshold) { |
| 642 | RefreshRateConfigs::Config config = {.frameRateMultipleThreshold = 120}; |
| 643 | auto refreshRateConfigs = |
| 644 | std::make_unique<RefreshRateConfigs>(m30_60_72_90_120Device, |
| 645 | /*currentConfigId=*/HWC_CONFIG_ID_60, config); |
| 646 | |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 647 | std::vector<LayerRequirement> layers = {{.weight = 1.f}, {.weight = 1.f}}; |
rnlee | 3bd61066 | 2021-06-23 16:27:57 -0700 | [diff] [blame] | 648 | auto& lr1 = layers[0]; |
| 649 | auto& lr2 = layers[1]; |
| 650 | |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 651 | lr1.desiredRefreshRate = 24_Hz; |
rnlee | 3bd61066 | 2021-06-23 16:27:57 -0700 | [diff] [blame] | 652 | lr1.vote = LayerVoteType::ExplicitDefault; |
| 653 | lr1.name = "24Hz ExplicitDefault"; |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 654 | lr2.desiredRefreshRate = 60_Hz; |
rnlee | 3bd61066 | 2021-06-23 16:27:57 -0700 | [diff] [blame] | 655 | lr2.vote = LayerVoteType::Heuristic; |
| 656 | lr2.name = "60Hz Heuristic"; |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 657 | EXPECT_EQ(mExpected120Config, refreshRateConfigs->getBestRefreshRate(layers, {})); |
rnlee | 3bd61066 | 2021-06-23 16:27:57 -0700 | [diff] [blame] | 658 | |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 659 | lr1.desiredRefreshRate = 24_Hz; |
rnlee | 3bd61066 | 2021-06-23 16:27:57 -0700 | [diff] [blame] | 660 | lr1.vote = LayerVoteType::ExplicitExactOrMultiple; |
| 661 | lr1.name = "24Hz ExplicitExactOrMultiple"; |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 662 | lr2.desiredRefreshRate = 60_Hz; |
rnlee | 3bd61066 | 2021-06-23 16:27:57 -0700 | [diff] [blame] | 663 | lr2.vote = LayerVoteType::Heuristic; |
| 664 | lr2.name = "60Hz Heuristic"; |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 665 | EXPECT_EQ(mExpected60Config, refreshRateConfigs->getBestRefreshRate(layers, {})); |
rnlee | 3bd61066 | 2021-06-23 16:27:57 -0700 | [diff] [blame] | 666 | |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 667 | lr1.desiredRefreshRate = 24_Hz; |
rnlee | 3bd61066 | 2021-06-23 16:27:57 -0700 | [diff] [blame] | 668 | lr1.vote = LayerVoteType::ExplicitExactOrMultiple; |
| 669 | lr1.name = "24Hz ExplicitExactOrMultiple"; |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 670 | lr2.desiredRefreshRate = 60_Hz; |
rnlee | 3bd61066 | 2021-06-23 16:27:57 -0700 | [diff] [blame] | 671 | lr2.vote = LayerVoteType::ExplicitDefault; |
| 672 | lr2.name = "60Hz ExplicitDefault"; |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 673 | EXPECT_EQ(mExpected72Config, refreshRateConfigs->getBestRefreshRate(layers, {})); |
rnlee | 3bd61066 | 2021-06-23 16:27:57 -0700 | [diff] [blame] | 674 | |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 675 | lr1.desiredRefreshRate = 24_Hz; |
rnlee | 3bd61066 | 2021-06-23 16:27:57 -0700 | [diff] [blame] | 676 | lr1.vote = LayerVoteType::ExplicitExactOrMultiple; |
| 677 | lr1.name = "24Hz ExplicitExactOrMultiple"; |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 678 | lr2.desiredRefreshRate = 90_Hz; |
rnlee | 3bd61066 | 2021-06-23 16:27:57 -0700 | [diff] [blame] | 679 | lr2.vote = LayerVoteType::Heuristic; |
| 680 | lr2.name = "90Hz Heuristic"; |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 681 | EXPECT_EQ(mExpected90Config, refreshRateConfigs->getBestRefreshRate(layers, {})); |
rnlee | 3bd61066 | 2021-06-23 16:27:57 -0700 | [diff] [blame] | 682 | |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 683 | lr1.desiredRefreshRate = 24_Hz; |
rnlee | 3bd61066 | 2021-06-23 16:27:57 -0700 | [diff] [blame] | 684 | lr1.vote = LayerVoteType::ExplicitExactOrMultiple; |
| 685 | lr1.name = "24Hz ExplicitExactOrMultiple"; |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 686 | lr2.desiredRefreshRate = 90_Hz; |
rnlee | 3bd61066 | 2021-06-23 16:27:57 -0700 | [diff] [blame] | 687 | lr2.vote = LayerVoteType::ExplicitDefault; |
| 688 | lr2.name = "90Hz Heuristic"; |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 689 | EXPECT_EQ(mExpected72Config, refreshRateConfigs->getBestRefreshRate(layers, {})); |
rnlee | 3bd61066 | 2021-06-23 16:27:57 -0700 | [diff] [blame] | 690 | |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 691 | lr1.desiredRefreshRate = 24_Hz; |
rnlee | 3bd61066 | 2021-06-23 16:27:57 -0700 | [diff] [blame] | 692 | lr1.vote = LayerVoteType::ExplicitDefault; |
| 693 | lr1.name = "24Hz ExplicitDefault"; |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 694 | lr2.desiredRefreshRate = 90_Hz; |
rnlee | 3bd61066 | 2021-06-23 16:27:57 -0700 | [diff] [blame] | 695 | lr2.vote = LayerVoteType::Heuristic; |
| 696 | lr2.name = "90Hz Heuristic"; |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 697 | EXPECT_EQ(mExpected90Config, refreshRateConfigs->getBestRefreshRate(layers, {})); |
rnlee | 3bd61066 | 2021-06-23 16:27:57 -0700 | [diff] [blame] | 698 | |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 699 | lr1.desiredRefreshRate = 24_Hz; |
rnlee | 3bd61066 | 2021-06-23 16:27:57 -0700 | [diff] [blame] | 700 | lr1.vote = LayerVoteType::Heuristic; |
| 701 | lr1.name = "24Hz Heuristic"; |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 702 | lr2.desiredRefreshRate = 90_Hz; |
rnlee | 3bd61066 | 2021-06-23 16:27:57 -0700 | [diff] [blame] | 703 | lr2.vote = LayerVoteType::ExplicitDefault; |
| 704 | lr2.name = "90Hz ExplicitDefault"; |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 705 | EXPECT_EQ(mExpected72Config, refreshRateConfigs->getBestRefreshRate(layers, {})); |
rnlee | 3bd61066 | 2021-06-23 16:27:57 -0700 | [diff] [blame] | 706 | |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 707 | lr1.desiredRefreshRate = 24_Hz; |
rnlee | 3bd61066 | 2021-06-23 16:27:57 -0700 | [diff] [blame] | 708 | lr1.vote = LayerVoteType::ExplicitExactOrMultiple; |
| 709 | lr1.name = "24Hz ExplicitExactOrMultiple"; |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 710 | lr2.desiredRefreshRate = 90_Hz; |
rnlee | 3bd61066 | 2021-06-23 16:27:57 -0700 | [diff] [blame] | 711 | lr2.vote = LayerVoteType::ExplicitDefault; |
| 712 | lr2.name = "90Hz ExplicitDefault"; |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 713 | EXPECT_EQ(mExpected72Config, refreshRateConfigs->getBestRefreshRate(layers, {})); |
rnlee | 3bd61066 | 2021-06-23 16:27:57 -0700 | [diff] [blame] | 714 | |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 715 | lr1.desiredRefreshRate = 24_Hz; |
rnlee | 3bd61066 | 2021-06-23 16:27:57 -0700 | [diff] [blame] | 716 | lr1.vote = LayerVoteType::ExplicitDefault; |
| 717 | lr1.name = "24Hz ExplicitDefault"; |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 718 | lr2.desiredRefreshRate = 90_Hz; |
rnlee | 3bd61066 | 2021-06-23 16:27:57 -0700 | [diff] [blame] | 719 | lr2.vote = LayerVoteType::ExplicitExactOrMultiple; |
| 720 | lr2.name = "90Hz ExplicitExactOrMultiple"; |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 721 | EXPECT_EQ(mExpected90Config, refreshRateConfigs->getBestRefreshRate(layers, {})); |
rnlee | 3bd61066 | 2021-06-23 16:27:57 -0700 | [diff] [blame] | 722 | } |
| 723 | |
Steven Thomas | bb37432 | 2020-04-28 22:47:16 -0700 | [diff] [blame] | 724 | TEST_F(RefreshRateConfigsTest, getBestRefreshRate_30_60) { |
Ady Abraham | 8a82ba6 | 2020-01-17 12:43:17 -0800 | [diff] [blame] | 725 | auto refreshRateConfigs = |
Ady Abraham | abc2760 | 2020-04-08 17:20:29 -0700 | [diff] [blame] | 726 | std::make_unique<RefreshRateConfigs>(m30_60Device, |
| 727 | /*currentConfigId=*/HWC_CONFIG_ID_60); |
Ady Abraham | 8a82ba6 | 2020-01-17 12:43:17 -0800 | [diff] [blame] | 728 | |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 729 | std::vector<LayerRequirement> layers = {{.weight = 1.f}}; |
Ady Abraham | 8a82ba6 | 2020-01-17 12:43:17 -0800 | [diff] [blame] | 730 | auto& lr = layers[0]; |
| 731 | |
| 732 | lr.vote = LayerVoteType::Min; |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 733 | EXPECT_EQ(mExpected30Config, refreshRateConfigs->getBestRefreshRate(layers, {})); |
Ady Abraham | 8a82ba6 | 2020-01-17 12:43:17 -0800 | [diff] [blame] | 734 | |
| 735 | lr.vote = LayerVoteType::Max; |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 736 | EXPECT_EQ(mExpected60Config, refreshRateConfigs->getBestRefreshRate(layers, {})); |
Ady Abraham | 8a82ba6 | 2020-01-17 12:43:17 -0800 | [diff] [blame] | 737 | |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 738 | lr.desiredRefreshRate = 90_Hz; |
Ady Abraham | 8a82ba6 | 2020-01-17 12:43:17 -0800 | [diff] [blame] | 739 | lr.vote = LayerVoteType::Heuristic; |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 740 | EXPECT_EQ(mExpected60Config, refreshRateConfigs->getBestRefreshRate(layers, {})); |
Ady Abraham | 8a82ba6 | 2020-01-17 12:43:17 -0800 | [diff] [blame] | 741 | |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 742 | lr.desiredRefreshRate = 60_Hz; |
| 743 | EXPECT_EQ(mExpected60Config, refreshRateConfigs->getBestRefreshRate(layers, {})); |
Ady Abraham | 8a82ba6 | 2020-01-17 12:43:17 -0800 | [diff] [blame] | 744 | |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 745 | lr.desiredRefreshRate = 45_Hz; |
| 746 | EXPECT_EQ(mExpected60Config, refreshRateConfigs->getBestRefreshRate(layers, {})); |
Ady Abraham | 8a82ba6 | 2020-01-17 12:43:17 -0800 | [diff] [blame] | 747 | |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 748 | lr.desiredRefreshRate = 30_Hz; |
| 749 | EXPECT_EQ(mExpected30Config, refreshRateConfigs->getBestRefreshRate(layers, {})); |
Ady Abraham | 8a82ba6 | 2020-01-17 12:43:17 -0800 | [diff] [blame] | 750 | |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 751 | lr.desiredRefreshRate = 24_Hz; |
| 752 | EXPECT_EQ(mExpected60Config, refreshRateConfigs->getBestRefreshRate(layers, {})); |
Ady Abraham | 8a82ba6 | 2020-01-17 12:43:17 -0800 | [diff] [blame] | 753 | } |
| 754 | |
Steven Thomas | bb37432 | 2020-04-28 22:47:16 -0700 | [diff] [blame] | 755 | TEST_F(RefreshRateConfigsTest, getBestRefreshRate_30_60_72_90) { |
Ady Abraham | 8a82ba6 | 2020-01-17 12:43:17 -0800 | [diff] [blame] | 756 | auto refreshRateConfigs = |
Ady Abraham | abc2760 | 2020-04-08 17:20:29 -0700 | [diff] [blame] | 757 | std::make_unique<RefreshRateConfigs>(m30_60_72_90Device, |
| 758 | /*currentConfigId=*/HWC_CONFIG_ID_60); |
Ady Abraham | 8a82ba6 | 2020-01-17 12:43:17 -0800 | [diff] [blame] | 759 | |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 760 | std::vector<LayerRequirement> layers = {{.weight = 1.f}}; |
Ady Abraham | 8a82ba6 | 2020-01-17 12:43:17 -0800 | [diff] [blame] | 761 | auto& lr = layers[0]; |
| 762 | |
| 763 | lr.vote = LayerVoteType::Min; |
Ady Abraham | 4ccdcb4 | 2020-02-11 17:34:34 -0800 | [diff] [blame] | 764 | lr.name = "Min"; |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 765 | EXPECT_EQ(mExpected30Config, refreshRateConfigs->getBestRefreshRate(layers, {})); |
Ady Abraham | 8a82ba6 | 2020-01-17 12:43:17 -0800 | [diff] [blame] | 766 | |
| 767 | lr.vote = LayerVoteType::Max; |
Ady Abraham | 4ccdcb4 | 2020-02-11 17:34:34 -0800 | [diff] [blame] | 768 | lr.name = "Max"; |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 769 | EXPECT_EQ(mExpected90Config, refreshRateConfigs->getBestRefreshRate(layers, {})); |
Ady Abraham | 8a82ba6 | 2020-01-17 12:43:17 -0800 | [diff] [blame] | 770 | |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 771 | lr.desiredRefreshRate = 90_Hz; |
Ady Abraham | 8a82ba6 | 2020-01-17 12:43:17 -0800 | [diff] [blame] | 772 | lr.vote = LayerVoteType::Heuristic; |
Ady Abraham | 4ccdcb4 | 2020-02-11 17:34:34 -0800 | [diff] [blame] | 773 | lr.name = "90Hz Heuristic"; |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 774 | EXPECT_EQ(mExpected90Config, refreshRateConfigs->getBestRefreshRate(layers, {})); |
Ady Abraham | 8a82ba6 | 2020-01-17 12:43:17 -0800 | [diff] [blame] | 775 | |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 776 | lr.desiredRefreshRate = 60_Hz; |
Ady Abraham | 4ccdcb4 | 2020-02-11 17:34:34 -0800 | [diff] [blame] | 777 | lr.name = "60Hz Heuristic"; |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 778 | EXPECT_EQ(mExpected60Config, refreshRateConfigs->getBestRefreshRate(layers, {})); |
| 779 | EXPECT_EQ(mExpected90Config, refreshRateConfigs->getBestRefreshRate(layers, {.touch = true})); |
Ady Abraham | 8a82ba6 | 2020-01-17 12:43:17 -0800 | [diff] [blame] | 780 | |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 781 | lr.desiredRefreshRate = 45_Hz; |
Ady Abraham | 4ccdcb4 | 2020-02-11 17:34:34 -0800 | [diff] [blame] | 782 | lr.name = "45Hz Heuristic"; |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 783 | EXPECT_EQ(mExpected90Config, refreshRateConfigs->getBestRefreshRate(layers, {})); |
| 784 | EXPECT_EQ(mExpected90Config, refreshRateConfigs->getBestRefreshRate(layers, {.touch = true})); |
Ady Abraham | 8a82ba6 | 2020-01-17 12:43:17 -0800 | [diff] [blame] | 785 | |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 786 | lr.desiredRefreshRate = 30_Hz; |
Ady Abraham | 4ccdcb4 | 2020-02-11 17:34:34 -0800 | [diff] [blame] | 787 | lr.name = "30Hz Heuristic"; |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 788 | EXPECT_EQ(mExpected30Config, refreshRateConfigs->getBestRefreshRate(layers, {})); |
| 789 | EXPECT_EQ(mExpected90Config, refreshRateConfigs->getBestRefreshRate(layers, {.touch = true})); |
Ady Abraham | 8a82ba6 | 2020-01-17 12:43:17 -0800 | [diff] [blame] | 790 | |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 791 | lr.desiredRefreshRate = 24_Hz; |
Ady Abraham | 4ccdcb4 | 2020-02-11 17:34:34 -0800 | [diff] [blame] | 792 | lr.name = "24Hz Heuristic"; |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 793 | EXPECT_EQ(mExpected72Config, refreshRateConfigs->getBestRefreshRate(layers, {})); |
| 794 | EXPECT_EQ(mExpected90Config, refreshRateConfigs->getBestRefreshRate(layers, {.touch = true})); |
Ady Abraham | 4ccdcb4 | 2020-02-11 17:34:34 -0800 | [diff] [blame] | 795 | |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 796 | lr.desiredRefreshRate = 24_Hz; |
Ady Abraham | 4ccdcb4 | 2020-02-11 17:34:34 -0800 | [diff] [blame] | 797 | lr.vote = LayerVoteType::ExplicitExactOrMultiple; |
| 798 | lr.name = "24Hz ExplicitExactOrMultiple"; |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 799 | EXPECT_EQ(mExpected72Config, refreshRateConfigs->getBestRefreshRate(layers, {})); |
| 800 | EXPECT_EQ(mExpected90Config, refreshRateConfigs->getBestRefreshRate(layers, {.touch = true})); |
Ady Abraham | 8a82ba6 | 2020-01-17 12:43:17 -0800 | [diff] [blame] | 801 | } |
| 802 | |
Steven Thomas | bb37432 | 2020-04-28 22:47:16 -0700 | [diff] [blame] | 803 | TEST_F(RefreshRateConfigsTest, getBestRefreshRate_PriorityTest) { |
Ady Abraham | 8a82ba6 | 2020-01-17 12:43:17 -0800 | [diff] [blame] | 804 | auto refreshRateConfigs = |
Ady Abraham | abc2760 | 2020-04-08 17:20:29 -0700 | [diff] [blame] | 805 | std::make_unique<RefreshRateConfigs>(m30_60_90Device, |
| 806 | /*currentConfigId=*/HWC_CONFIG_ID_60); |
Ady Abraham | 8a82ba6 | 2020-01-17 12:43:17 -0800 | [diff] [blame] | 807 | |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 808 | std::vector<LayerRequirement> layers = {{.weight = 1.f}, {.weight = 1.f}}; |
Ady Abraham | 8a82ba6 | 2020-01-17 12:43:17 -0800 | [diff] [blame] | 809 | auto& lr1 = layers[0]; |
| 810 | auto& lr2 = layers[1]; |
| 811 | |
| 812 | lr1.vote = LayerVoteType::Min; |
| 813 | lr2.vote = LayerVoteType::Max; |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 814 | EXPECT_EQ(mExpected90Config, refreshRateConfigs->getBestRefreshRate(layers, {})); |
Ady Abraham | 8a82ba6 | 2020-01-17 12:43:17 -0800 | [diff] [blame] | 815 | |
| 816 | lr1.vote = LayerVoteType::Min; |
| 817 | lr2.vote = LayerVoteType::Heuristic; |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 818 | lr2.desiredRefreshRate = 24_Hz; |
| 819 | EXPECT_EQ(mExpected60Config, refreshRateConfigs->getBestRefreshRate(layers, {})); |
Ady Abraham | 8a82ba6 | 2020-01-17 12:43:17 -0800 | [diff] [blame] | 820 | |
| 821 | lr1.vote = LayerVoteType::Min; |
Ady Abraham | 71c437d | 2020-01-31 15:56:57 -0800 | [diff] [blame] | 822 | lr2.vote = LayerVoteType::ExplicitExactOrMultiple; |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 823 | lr2.desiredRefreshRate = 24_Hz; |
| 824 | EXPECT_EQ(mExpected60Config, refreshRateConfigs->getBestRefreshRate(layers, {})); |
Ady Abraham | 8a82ba6 | 2020-01-17 12:43:17 -0800 | [diff] [blame] | 825 | |
| 826 | lr1.vote = LayerVoteType::Max; |
| 827 | lr2.vote = LayerVoteType::Heuristic; |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 828 | lr2.desiredRefreshRate = 60_Hz; |
| 829 | EXPECT_EQ(mExpected90Config, refreshRateConfigs->getBestRefreshRate(layers, {})); |
Ady Abraham | 8a82ba6 | 2020-01-17 12:43:17 -0800 | [diff] [blame] | 830 | |
| 831 | lr1.vote = LayerVoteType::Max; |
Ady Abraham | 71c437d | 2020-01-31 15:56:57 -0800 | [diff] [blame] | 832 | lr2.vote = LayerVoteType::ExplicitExactOrMultiple; |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 833 | lr2.desiredRefreshRate = 60_Hz; |
| 834 | EXPECT_EQ(mExpected90Config, refreshRateConfigs->getBestRefreshRate(layers, {})); |
Ady Abraham | 8a82ba6 | 2020-01-17 12:43:17 -0800 | [diff] [blame] | 835 | |
| 836 | lr1.vote = LayerVoteType::Heuristic; |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 837 | lr1.desiredRefreshRate = 15_Hz; |
Ady Abraham | 8a82ba6 | 2020-01-17 12:43:17 -0800 | [diff] [blame] | 838 | lr2.vote = LayerVoteType::Heuristic; |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 839 | lr2.desiredRefreshRate = 45_Hz; |
| 840 | EXPECT_EQ(mExpected90Config, refreshRateConfigs->getBestRefreshRate(layers, {})); |
Ady Abraham | 8a82ba6 | 2020-01-17 12:43:17 -0800 | [diff] [blame] | 841 | |
| 842 | lr1.vote = LayerVoteType::Heuristic; |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 843 | lr1.desiredRefreshRate = 30_Hz; |
Ady Abraham | 71c437d | 2020-01-31 15:56:57 -0800 | [diff] [blame] | 844 | lr2.vote = LayerVoteType::ExplicitExactOrMultiple; |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 845 | lr2.desiredRefreshRate = 45_Hz; |
| 846 | EXPECT_EQ(mExpected90Config, refreshRateConfigs->getBestRefreshRate(layers, {})); |
Ady Abraham | 8a82ba6 | 2020-01-17 12:43:17 -0800 | [diff] [blame] | 847 | } |
| 848 | |
Steven Thomas | bb37432 | 2020-04-28 22:47:16 -0700 | [diff] [blame] | 849 | TEST_F(RefreshRateConfigsTest, getBestRefreshRate_24FpsVideo) { |
Ady Abraham | 8a82ba6 | 2020-01-17 12:43:17 -0800 | [diff] [blame] | 850 | auto refreshRateConfigs = |
Ady Abraham | abc2760 | 2020-04-08 17:20:29 -0700 | [diff] [blame] | 851 | std::make_unique<RefreshRateConfigs>(m60_90Device, |
| 852 | /*currentConfigId=*/HWC_CONFIG_ID_60); |
Ady Abraham | 8a82ba6 | 2020-01-17 12:43:17 -0800 | [diff] [blame] | 853 | |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 854 | std::vector<LayerRequirement> layers = {{.weight = 1.f}}; |
Ady Abraham | 8a82ba6 | 2020-01-17 12:43:17 -0800 | [diff] [blame] | 855 | auto& lr = layers[0]; |
| 856 | |
Ady Abraham | 71c437d | 2020-01-31 15:56:57 -0800 | [diff] [blame] | 857 | lr.vote = LayerVoteType::ExplicitExactOrMultiple; |
Ady Abraham | 8a82ba6 | 2020-01-17 12:43:17 -0800 | [diff] [blame] | 858 | for (float fps = 23.0f; fps < 25.0f; fps += 0.1f) { |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 859 | lr.desiredRefreshRate = Fps::fromValue(fps); |
| 860 | const auto refreshRate = refreshRateConfigs->getBestRefreshRate(layers, {}); |
| 861 | EXPECT_EQ(mExpected60Config, refreshRate) |
| 862 | << lr.desiredRefreshRate << " chooses " << refreshRate.getName(); |
Ady Abraham | 8a82ba6 | 2020-01-17 12:43:17 -0800 | [diff] [blame] | 863 | } |
| 864 | } |
| 865 | |
rnlee | 3bd61066 | 2021-06-23 16:27:57 -0700 | [diff] [blame] | 866 | TEST_F(RefreshRateConfigsTest, getBestRefreshRate_24FpsVideo_multipleThreshold_60_120) { |
| 867 | RefreshRateConfigs::Config config = {.frameRateMultipleThreshold = 120}; |
| 868 | auto refreshRateConfigs = |
| 869 | std::make_unique<RefreshRateConfigs>(m60_120Device, |
| 870 | /*currentConfigId=*/HWC_CONFIG_ID_60, config); |
| 871 | |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 872 | std::vector<LayerRequirement> layers = {{.weight = 1.f}}; |
rnlee | 3bd61066 | 2021-06-23 16:27:57 -0700 | [diff] [blame] | 873 | auto& lr = layers[0]; |
| 874 | |
| 875 | lr.vote = LayerVoteType::ExplicitExactOrMultiple; |
| 876 | for (float fps = 23.0f; fps < 25.0f; fps += 0.1f) { |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 877 | lr.desiredRefreshRate = Fps::fromValue(fps); |
| 878 | const auto refreshRate = refreshRateConfigs->getBestRefreshRate(layers, {}); |
| 879 | EXPECT_EQ(mExpected60Config, refreshRate) |
| 880 | << lr.desiredRefreshRate << " chooses " << refreshRate.getName(); |
rnlee | 3bd61066 | 2021-06-23 16:27:57 -0700 | [diff] [blame] | 881 | } |
| 882 | } |
| 883 | |
Steven Thomas | bb37432 | 2020-04-28 22:47:16 -0700 | [diff] [blame] | 884 | TEST_F(RefreshRateConfigsTest, twoDeviceConfigs_getBestRefreshRate_Explicit) { |
Ady Abraham | 8a82ba6 | 2020-01-17 12:43:17 -0800 | [diff] [blame] | 885 | auto refreshRateConfigs = |
Ady Abraham | abc2760 | 2020-04-08 17:20:29 -0700 | [diff] [blame] | 886 | std::make_unique<RefreshRateConfigs>(m60_90Device, |
| 887 | /*currentConfigId=*/HWC_CONFIG_ID_60); |
Ady Abraham | 8a82ba6 | 2020-01-17 12:43:17 -0800 | [diff] [blame] | 888 | |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 889 | std::vector<LayerRequirement> layers = {{.weight = 1.f}, {.weight = 1.f}}; |
Ady Abraham | 8a82ba6 | 2020-01-17 12:43:17 -0800 | [diff] [blame] | 890 | auto& lr1 = layers[0]; |
| 891 | auto& lr2 = layers[1]; |
| 892 | |
| 893 | lr1.vote = LayerVoteType::Heuristic; |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 894 | lr1.desiredRefreshRate = 60_Hz; |
Ady Abraham | 71c437d | 2020-01-31 15:56:57 -0800 | [diff] [blame] | 895 | lr2.vote = LayerVoteType::ExplicitExactOrMultiple; |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 896 | lr2.desiredRefreshRate = 90_Hz; |
| 897 | EXPECT_EQ(mExpected90Config, refreshRateConfigs->getBestRefreshRate(layers, {})); |
Ady Abraham | 4ccdcb4 | 2020-02-11 17:34:34 -0800 | [diff] [blame] | 898 | |
| 899 | lr1.vote = LayerVoteType::ExplicitDefault; |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 900 | lr1.desiredRefreshRate = 90_Hz; |
Ady Abraham | 4ccdcb4 | 2020-02-11 17:34:34 -0800 | [diff] [blame] | 901 | lr2.vote = LayerVoteType::ExplicitExactOrMultiple; |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 902 | lr2.desiredRefreshRate = 60_Hz; |
| 903 | EXPECT_EQ(mExpected60Config, refreshRateConfigs->getBestRefreshRate(layers, {})); |
Ady Abraham | 8a82ba6 | 2020-01-17 12:43:17 -0800 | [diff] [blame] | 904 | |
| 905 | lr1.vote = LayerVoteType::Heuristic; |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 906 | lr1.desiredRefreshRate = 90_Hz; |
Ady Abraham | 71c437d | 2020-01-31 15:56:57 -0800 | [diff] [blame] | 907 | lr2.vote = LayerVoteType::ExplicitExactOrMultiple; |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 908 | lr2.desiredRefreshRate = 60_Hz; |
| 909 | EXPECT_EQ(mExpected90Config, refreshRateConfigs->getBestRefreshRate(layers, {})); |
Ady Abraham | 2139f73 | 2019-11-13 18:56:40 -0800 | [diff] [blame] | 910 | } |
| 911 | |
Ana Krulec | 72f0d6e | 2020-01-06 15:24:47 -0800 | [diff] [blame] | 912 | TEST_F(RefreshRateConfigsTest, testInPolicy) { |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 913 | ASSERT_TRUE(mExpectedAlmost60Config.inPolicy(60.000004_Hz, 60.000004_Hz)); |
| 914 | ASSERT_TRUE(mExpectedAlmost60Config.inPolicy(59_Hz, 60.1_Hz)); |
| 915 | ASSERT_FALSE(mExpectedAlmost60Config.inPolicy(75_Hz, 90_Hz)); |
| 916 | ASSERT_FALSE(mExpectedAlmost60Config.inPolicy(60.0011_Hz, 90_Hz)); |
| 917 | ASSERT_FALSE(mExpectedAlmost60Config.inPolicy(50_Hz, 59.998_Hz)); |
Ana Krulec | 72f0d6e | 2020-01-06 15:24:47 -0800 | [diff] [blame] | 918 | } |
| 919 | |
Steven Thomas | bb37432 | 2020-04-28 22:47:16 -0700 | [diff] [blame] | 920 | TEST_F(RefreshRateConfigsTest, getBestRefreshRate_75HzContent) { |
Ady Abraham | f6b7707 | 2020-01-30 14:22:54 -0800 | [diff] [blame] | 921 | auto refreshRateConfigs = |
Ady Abraham | abc2760 | 2020-04-08 17:20:29 -0700 | [diff] [blame] | 922 | std::make_unique<RefreshRateConfigs>(m60_90Device, |
| 923 | /*currentConfigId=*/HWC_CONFIG_ID_60); |
Ady Abraham | f6b7707 | 2020-01-30 14:22:54 -0800 | [diff] [blame] | 924 | |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 925 | std::vector<LayerRequirement> layers = {{.weight = 1.f}}; |
Ady Abraham | f6b7707 | 2020-01-30 14:22:54 -0800 | [diff] [blame] | 926 | auto& lr = layers[0]; |
| 927 | |
Ady Abraham | 71c437d | 2020-01-31 15:56:57 -0800 | [diff] [blame] | 928 | lr.vote = LayerVoteType::ExplicitExactOrMultiple; |
Ady Abraham | f6b7707 | 2020-01-30 14:22:54 -0800 | [diff] [blame] | 929 | for (float fps = 75.0f; fps < 100.0f; fps += 0.1f) { |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 930 | lr.desiredRefreshRate = Fps::fromValue(fps); |
| 931 | const auto refreshRate = refreshRateConfigs->getBestRefreshRate(layers, {}); |
| 932 | EXPECT_EQ(mExpected90Config, refreshRate) |
| 933 | << lr.desiredRefreshRate << " chooses " << refreshRate.getName(); |
Ady Abraham | f6b7707 | 2020-01-30 14:22:54 -0800 | [diff] [blame] | 934 | } |
| 935 | } |
| 936 | |
Steven Thomas | bb37432 | 2020-04-28 22:47:16 -0700 | [diff] [blame] | 937 | TEST_F(RefreshRateConfigsTest, getBestRefreshRate_Multiples) { |
Ady Abraham | 3470210 | 2020-02-10 14:12:05 -0800 | [diff] [blame] | 938 | auto refreshRateConfigs = |
Ady Abraham | abc2760 | 2020-04-08 17:20:29 -0700 | [diff] [blame] | 939 | std::make_unique<RefreshRateConfigs>(m60_90Device, |
| 940 | /*currentConfigId=*/HWC_CONFIG_ID_60); |
Ady Abraham | 3470210 | 2020-02-10 14:12:05 -0800 | [diff] [blame] | 941 | |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 942 | std::vector<LayerRequirement> layers = {{.weight = 1.f}, {.weight = 1.f}}; |
Ady Abraham | 3470210 | 2020-02-10 14:12:05 -0800 | [diff] [blame] | 943 | auto& lr1 = layers[0]; |
| 944 | auto& lr2 = layers[1]; |
| 945 | |
| 946 | lr1.vote = LayerVoteType::ExplicitExactOrMultiple; |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 947 | lr1.desiredRefreshRate = 60_Hz; |
Ady Abraham | 4ccdcb4 | 2020-02-11 17:34:34 -0800 | [diff] [blame] | 948 | lr1.name = "60Hz ExplicitExactOrMultiple"; |
Ady Abraham | 3470210 | 2020-02-10 14:12:05 -0800 | [diff] [blame] | 949 | lr2.vote = LayerVoteType::Heuristic; |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 950 | lr2.desiredRefreshRate = 90_Hz; |
Ady Abraham | 4ccdcb4 | 2020-02-11 17:34:34 -0800 | [diff] [blame] | 951 | lr2.name = "90Hz Heuristic"; |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 952 | EXPECT_EQ(mExpected90Config, refreshRateConfigs->getBestRefreshRate(layers, {})); |
Ady Abraham | 3470210 | 2020-02-10 14:12:05 -0800 | [diff] [blame] | 953 | |
| 954 | lr1.vote = LayerVoteType::ExplicitExactOrMultiple; |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 955 | lr1.desiredRefreshRate = 60_Hz; |
Ady Abraham | 4ccdcb4 | 2020-02-11 17:34:34 -0800 | [diff] [blame] | 956 | lr1.name = "60Hz ExplicitExactOrMultiple"; |
| 957 | lr2.vote = LayerVoteType::ExplicitDefault; |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 958 | lr2.desiredRefreshRate = 90_Hz; |
Ady Abraham | 4ccdcb4 | 2020-02-11 17:34:34 -0800 | [diff] [blame] | 959 | lr2.name = "90Hz ExplicitDefault"; |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 960 | EXPECT_EQ(mExpected60Config, refreshRateConfigs->getBestRefreshRate(layers, {})); |
Ady Abraham | 4ccdcb4 | 2020-02-11 17:34:34 -0800 | [diff] [blame] | 961 | |
| 962 | lr1.vote = LayerVoteType::ExplicitExactOrMultiple; |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 963 | lr1.desiredRefreshRate = 60_Hz; |
Ady Abraham | 4ccdcb4 | 2020-02-11 17:34:34 -0800 | [diff] [blame] | 964 | lr1.name = "60Hz ExplicitExactOrMultiple"; |
Ady Abraham | 3470210 | 2020-02-10 14:12:05 -0800 | [diff] [blame] | 965 | lr2.vote = LayerVoteType::Max; |
Ady Abraham | 4ccdcb4 | 2020-02-11 17:34:34 -0800 | [diff] [blame] | 966 | lr2.name = "Max"; |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 967 | EXPECT_EQ(mExpected90Config, refreshRateConfigs->getBestRefreshRate(layers, {})); |
Ady Abraham | 3470210 | 2020-02-10 14:12:05 -0800 | [diff] [blame] | 968 | |
| 969 | lr1.vote = LayerVoteType::ExplicitExactOrMultiple; |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 970 | lr1.desiredRefreshRate = 30_Hz; |
Ady Abraham | 4ccdcb4 | 2020-02-11 17:34:34 -0800 | [diff] [blame] | 971 | lr1.name = "30Hz ExplicitExactOrMultiple"; |
Ady Abraham | 3470210 | 2020-02-10 14:12:05 -0800 | [diff] [blame] | 972 | lr2.vote = LayerVoteType::Heuristic; |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 973 | lr2.desiredRefreshRate = 90_Hz; |
Ady Abraham | 4ccdcb4 | 2020-02-11 17:34:34 -0800 | [diff] [blame] | 974 | lr2.name = "90Hz Heuristic"; |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 975 | EXPECT_EQ(mExpected90Config, refreshRateConfigs->getBestRefreshRate(layers, {})); |
Ady Abraham | 3470210 | 2020-02-10 14:12:05 -0800 | [diff] [blame] | 976 | |
| 977 | lr1.vote = LayerVoteType::ExplicitExactOrMultiple; |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 978 | lr1.desiredRefreshRate = 30_Hz; |
Ady Abraham | 4ccdcb4 | 2020-02-11 17:34:34 -0800 | [diff] [blame] | 979 | lr1.name = "30Hz ExplicitExactOrMultiple"; |
Ady Abraham | 3470210 | 2020-02-10 14:12:05 -0800 | [diff] [blame] | 980 | lr2.vote = LayerVoteType::Max; |
Ady Abraham | 4ccdcb4 | 2020-02-11 17:34:34 -0800 | [diff] [blame] | 981 | lr2.name = "Max"; |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 982 | EXPECT_EQ(mExpected90Config, refreshRateConfigs->getBestRefreshRate(layers, {})); |
Ady Abraham | 4ccdcb4 | 2020-02-11 17:34:34 -0800 | [diff] [blame] | 983 | } |
| 984 | |
| 985 | TEST_F(RefreshRateConfigsTest, scrollWhileWatching60fps_60_90) { |
Ady Abraham | 4ccdcb4 | 2020-02-11 17:34:34 -0800 | [diff] [blame] | 986 | auto refreshRateConfigs = |
Ady Abraham | abc2760 | 2020-04-08 17:20:29 -0700 | [diff] [blame] | 987 | std::make_unique<RefreshRateConfigs>(m60_90Device, |
| 988 | /*currentConfigId=*/HWC_CONFIG_ID_60); |
Ady Abraham | 4ccdcb4 | 2020-02-11 17:34:34 -0800 | [diff] [blame] | 989 | |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 990 | std::vector<LayerRequirement> layers = {{.weight = 1.f}, {.weight = 1.f}}; |
Ady Abraham | 4ccdcb4 | 2020-02-11 17:34:34 -0800 | [diff] [blame] | 991 | auto& lr1 = layers[0]; |
| 992 | auto& lr2 = layers[1]; |
| 993 | |
| 994 | lr1.vote = LayerVoteType::ExplicitExactOrMultiple; |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 995 | lr1.desiredRefreshRate = 60_Hz; |
Ady Abraham | 4ccdcb4 | 2020-02-11 17:34:34 -0800 | [diff] [blame] | 996 | lr1.name = "60Hz ExplicitExactOrMultiple"; |
| 997 | lr2.vote = LayerVoteType::NoVote; |
| 998 | lr2.name = "NoVote"; |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 999 | EXPECT_EQ(mExpected60Config, refreshRateConfigs->getBestRefreshRate(layers, {})); |
Ady Abraham | 4ccdcb4 | 2020-02-11 17:34:34 -0800 | [diff] [blame] | 1000 | |
| 1001 | lr1.vote = LayerVoteType::ExplicitExactOrMultiple; |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 1002 | lr1.desiredRefreshRate = 60_Hz; |
Ady Abraham | 4ccdcb4 | 2020-02-11 17:34:34 -0800 | [diff] [blame] | 1003 | lr1.name = "60Hz ExplicitExactOrMultiple"; |
| 1004 | lr2.vote = LayerVoteType::NoVote; |
| 1005 | lr2.name = "NoVote"; |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 1006 | EXPECT_EQ(mExpected90Config, refreshRateConfigs->getBestRefreshRate(layers, {.touch = true})); |
Ady Abraham | 4ccdcb4 | 2020-02-11 17:34:34 -0800 | [diff] [blame] | 1007 | |
| 1008 | lr1.vote = LayerVoteType::ExplicitExactOrMultiple; |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 1009 | lr1.desiredRefreshRate = 60_Hz; |
Ady Abraham | 4ccdcb4 | 2020-02-11 17:34:34 -0800 | [diff] [blame] | 1010 | lr1.name = "60Hz ExplicitExactOrMultiple"; |
| 1011 | lr2.vote = LayerVoteType::Max; |
| 1012 | lr2.name = "Max"; |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 1013 | EXPECT_EQ(mExpected90Config, refreshRateConfigs->getBestRefreshRate(layers, {.touch = true})); |
Ady Abraham | 4ccdcb4 | 2020-02-11 17:34:34 -0800 | [diff] [blame] | 1014 | |
| 1015 | lr1.vote = LayerVoteType::ExplicitExactOrMultiple; |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 1016 | lr1.desiredRefreshRate = 60_Hz; |
Ady Abraham | 4ccdcb4 | 2020-02-11 17:34:34 -0800 | [diff] [blame] | 1017 | lr1.name = "60Hz ExplicitExactOrMultiple"; |
| 1018 | lr2.vote = LayerVoteType::Max; |
| 1019 | lr2.name = "Max"; |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 1020 | EXPECT_EQ(mExpected90Config, refreshRateConfigs->getBestRefreshRate(layers, {})); |
Ady Abraham | 4ccdcb4 | 2020-02-11 17:34:34 -0800 | [diff] [blame] | 1021 | |
| 1022 | // The other layer starts to provide buffers |
| 1023 | lr1.vote = LayerVoteType::ExplicitExactOrMultiple; |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 1024 | lr1.desiredRefreshRate = 60_Hz; |
Ady Abraham | 4ccdcb4 | 2020-02-11 17:34:34 -0800 | [diff] [blame] | 1025 | lr1.name = "60Hz ExplicitExactOrMultiple"; |
| 1026 | lr2.vote = LayerVoteType::Heuristic; |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 1027 | lr2.desiredRefreshRate = 90_Hz; |
Ady Abraham | 4ccdcb4 | 2020-02-11 17:34:34 -0800 | [diff] [blame] | 1028 | lr2.name = "90Hz Heuristic"; |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 1029 | EXPECT_EQ(mExpected90Config, refreshRateConfigs->getBestRefreshRate(layers, {})); |
Ady Abraham | 6fb599b | 2020-03-05 13:48:22 -0800 | [diff] [blame] | 1030 | } |
| 1031 | |
| 1032 | TEST_F(RefreshRateConfigsTest, touchConsidered) { |
Ady Abraham | dfd6216 | 2020-06-10 16:11:56 -0700 | [diff] [blame] | 1033 | RefreshRateConfigs::GlobalSignals consideredSignals; |
Ady Abraham | 6fb599b | 2020-03-05 13:48:22 -0800 | [diff] [blame] | 1034 | auto refreshRateConfigs = |
Ady Abraham | abc2760 | 2020-04-08 17:20:29 -0700 | [diff] [blame] | 1035 | std::make_unique<RefreshRateConfigs>(m60_90Device, |
| 1036 | /*currentConfigId=*/HWC_CONFIG_ID_60); |
Ady Abraham | 6fb599b | 2020-03-05 13:48:22 -0800 | [diff] [blame] | 1037 | |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 1038 | refreshRateConfigs->getBestRefreshRate({}, {}, &consideredSignals); |
Ady Abraham | dfd6216 | 2020-06-10 16:11:56 -0700 | [diff] [blame] | 1039 | EXPECT_EQ(false, consideredSignals.touch); |
Ady Abraham | 6fb599b | 2020-03-05 13:48:22 -0800 | [diff] [blame] | 1040 | |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 1041 | refreshRateConfigs->getBestRefreshRate({}, {.touch = true}, &consideredSignals); |
Ady Abraham | dfd6216 | 2020-06-10 16:11:56 -0700 | [diff] [blame] | 1042 | EXPECT_EQ(true, consideredSignals.touch); |
Ady Abraham | 6fb599b | 2020-03-05 13:48:22 -0800 | [diff] [blame] | 1043 | |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 1044 | std::vector<LayerRequirement> layers = {{.weight = 1.f}, {.weight = 1.f}}; |
Ady Abraham | 6fb599b | 2020-03-05 13:48:22 -0800 | [diff] [blame] | 1045 | auto& lr1 = layers[0]; |
| 1046 | auto& lr2 = layers[1]; |
| 1047 | |
| 1048 | lr1.vote = LayerVoteType::ExplicitExactOrMultiple; |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 1049 | lr1.desiredRefreshRate = 60_Hz; |
Ady Abraham | 6fb599b | 2020-03-05 13:48:22 -0800 | [diff] [blame] | 1050 | lr1.name = "60Hz ExplicitExactOrMultiple"; |
| 1051 | lr2.vote = LayerVoteType::Heuristic; |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 1052 | lr2.desiredRefreshRate = 60_Hz; |
Steven Thomas | f734df4 | 2020-04-13 21:09:28 -0700 | [diff] [blame] | 1053 | lr2.name = "60Hz Heuristic"; |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 1054 | refreshRateConfigs->getBestRefreshRate(layers, {.touch = true}, &consideredSignals); |
Ady Abraham | dfd6216 | 2020-06-10 16:11:56 -0700 | [diff] [blame] | 1055 | EXPECT_EQ(true, consideredSignals.touch); |
Ady Abraham | 6fb599b | 2020-03-05 13:48:22 -0800 | [diff] [blame] | 1056 | |
| 1057 | lr1.vote = LayerVoteType::ExplicitDefault; |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 1058 | lr1.desiredRefreshRate = 60_Hz; |
Ady Abraham | 6fb599b | 2020-03-05 13:48:22 -0800 | [diff] [blame] | 1059 | lr1.name = "60Hz ExplicitExactOrMultiple"; |
| 1060 | lr2.vote = LayerVoteType::Heuristic; |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 1061 | lr2.desiredRefreshRate = 60_Hz; |
Steven Thomas | f734df4 | 2020-04-13 21:09:28 -0700 | [diff] [blame] | 1062 | lr2.name = "60Hz Heuristic"; |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 1063 | refreshRateConfigs->getBestRefreshRate(layers, {.touch = true}, &consideredSignals); |
Ady Abraham | dfd6216 | 2020-06-10 16:11:56 -0700 | [diff] [blame] | 1064 | EXPECT_EQ(false, consideredSignals.touch); |
Ady Abraham | 6fb599b | 2020-03-05 13:48:22 -0800 | [diff] [blame] | 1065 | |
| 1066 | lr1.vote = LayerVoteType::ExplicitExactOrMultiple; |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 1067 | lr1.desiredRefreshRate = 60_Hz; |
Ady Abraham | 6fb599b | 2020-03-05 13:48:22 -0800 | [diff] [blame] | 1068 | lr1.name = "60Hz ExplicitExactOrMultiple"; |
| 1069 | lr2.vote = LayerVoteType::Heuristic; |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 1070 | lr2.desiredRefreshRate = 60_Hz; |
Steven Thomas | f734df4 | 2020-04-13 21:09:28 -0700 | [diff] [blame] | 1071 | lr2.name = "60Hz Heuristic"; |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 1072 | refreshRateConfigs->getBestRefreshRate(layers, {.touch = true}, &consideredSignals); |
Ady Abraham | dfd6216 | 2020-06-10 16:11:56 -0700 | [diff] [blame] | 1073 | EXPECT_EQ(true, consideredSignals.touch); |
Ady Abraham | 6fb599b | 2020-03-05 13:48:22 -0800 | [diff] [blame] | 1074 | |
| 1075 | lr1.vote = LayerVoteType::ExplicitDefault; |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 1076 | lr1.desiredRefreshRate = 60_Hz; |
Steven Thomas | f734df4 | 2020-04-13 21:09:28 -0700 | [diff] [blame] | 1077 | lr1.name = "60Hz ExplicitExactOrMultiple"; |
Ady Abraham | 6fb599b | 2020-03-05 13:48:22 -0800 | [diff] [blame] | 1078 | lr2.vote = LayerVoteType::Heuristic; |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 1079 | lr2.desiredRefreshRate = 60_Hz; |
Steven Thomas | f734df4 | 2020-04-13 21:09:28 -0700 | [diff] [blame] | 1080 | lr2.name = "60Hz Heuristic"; |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 1081 | refreshRateConfigs->getBestRefreshRate(layers, {.touch = true}, &consideredSignals); |
Ady Abraham | dfd6216 | 2020-06-10 16:11:56 -0700 | [diff] [blame] | 1082 | EXPECT_EQ(false, consideredSignals.touch); |
Ady Abraham | 3470210 | 2020-02-10 14:12:05 -0800 | [diff] [blame] | 1083 | } |
| 1084 | |
Steven Thomas | bb37432 | 2020-04-28 22:47:16 -0700 | [diff] [blame] | 1085 | TEST_F(RefreshRateConfigsTest, getBestRefreshRate_ExplicitDefault) { |
Ady Abraham | abc2760 | 2020-04-08 17:20:29 -0700 | [diff] [blame] | 1086 | auto refreshRateConfigs = |
| 1087 | std::make_unique<RefreshRateConfigs>(m60_90_72_120Device, /*currentConfigId=*/ |
| 1088 | HWC_CONFIG_ID_60); |
Ady Abraham | 5b8afb5a | 2020-03-06 14:57:26 -0800 | [diff] [blame] | 1089 | |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 1090 | std::vector<LayerRequirement> layers = {{.weight = 1.f}}; |
Ady Abraham | 5b8afb5a | 2020-03-06 14:57:26 -0800 | [diff] [blame] | 1091 | auto& lr = layers[0]; |
| 1092 | |
| 1093 | // Prepare a table with the vote and the expected refresh rate |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 1094 | const std::initializer_list<std::pair<Fps, Fps>> testCases = { |
| 1095 | {130_Hz, 120_Hz}, {120_Hz, 120_Hz}, {119_Hz, 120_Hz}, {110_Hz, 120_Hz}, |
Ady Abraham | 5b8afb5a | 2020-03-06 14:57:26 -0800 | [diff] [blame] | 1096 | |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 1097 | {100_Hz, 90_Hz}, {90_Hz, 90_Hz}, {89_Hz, 90_Hz}, |
Ady Abraham | 5b8afb5a | 2020-03-06 14:57:26 -0800 | [diff] [blame] | 1098 | |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 1099 | {80_Hz, 72_Hz}, {73_Hz, 72_Hz}, {72_Hz, 72_Hz}, {71_Hz, 72_Hz}, {70_Hz, 72_Hz}, |
Ady Abraham | 5b8afb5a | 2020-03-06 14:57:26 -0800 | [diff] [blame] | 1100 | |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 1101 | {65_Hz, 60_Hz}, {60_Hz, 60_Hz}, {59_Hz, 60_Hz}, {58_Hz, 60_Hz}, |
Ady Abraham | 5b8afb5a | 2020-03-06 14:57:26 -0800 | [diff] [blame] | 1102 | |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 1103 | {55_Hz, 90_Hz}, {50_Hz, 90_Hz}, {45_Hz, 90_Hz}, |
Ady Abraham | 5b8afb5a | 2020-03-06 14:57:26 -0800 | [diff] [blame] | 1104 | |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 1105 | {42_Hz, 120_Hz}, {40_Hz, 120_Hz}, {39_Hz, 120_Hz}, |
Ady Abraham | 5b8afb5a | 2020-03-06 14:57:26 -0800 | [diff] [blame] | 1106 | |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 1107 | {37_Hz, 72_Hz}, {36_Hz, 72_Hz}, {35_Hz, 72_Hz}, |
Ady Abraham | 5b8afb5a | 2020-03-06 14:57:26 -0800 | [diff] [blame] | 1108 | |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 1109 | {30_Hz, 60_Hz}, |
Ady Abraham | 5b8afb5a | 2020-03-06 14:57:26 -0800 | [diff] [blame] | 1110 | }; |
| 1111 | |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 1112 | for (auto [desired, expected] : testCases) { |
Ady Abraham | 5b8afb5a | 2020-03-06 14:57:26 -0800 | [diff] [blame] | 1113 | lr.vote = LayerVoteType::ExplicitDefault; |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 1114 | lr.desiredRefreshRate = desired; |
Ady Abraham | 5b8afb5a | 2020-03-06 14:57:26 -0800 | [diff] [blame] | 1115 | |
| 1116 | std::stringstream ss; |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 1117 | ss << "ExplicitDefault " << desired; |
Ady Abraham | 5b8afb5a | 2020-03-06 14:57:26 -0800 | [diff] [blame] | 1118 | lr.name = ss.str(); |
| 1119 | |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 1120 | const auto refreshRate = refreshRateConfigs->getBestRefreshRate(layers, {}); |
| 1121 | EXPECT_EQ(refreshRate.getFps(), expected); |
Marin Shalamanov | 15a0fc6 | 2021-08-16 18:20:21 +0200 | [diff] [blame] | 1122 | } |
| 1123 | } |
| 1124 | |
| 1125 | TEST_F(RefreshRateConfigsTest, |
| 1126 | getBestRefreshRate_ExplicitExactOrMultiple_WithFractionalRefreshRates) { |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 1127 | std::vector<LayerRequirement> layers = {{.weight = 1.f}}; |
Marin Shalamanov | 15a0fc6 | 2021-08-16 18:20:21 +0200 | [diff] [blame] | 1128 | auto& lr = layers[0]; |
| 1129 | |
| 1130 | // Test that 23.976 will choose 24 if 23.976 is not supported |
| 1131 | { |
| 1132 | android::DisplayModes modes = {mConfig24, mConfig25, mConfig30, |
| 1133 | mConfig30Frac, mConfig60, mConfig60Frac}; |
| 1134 | auto refreshRateConfigs = |
| 1135 | std::make_unique<RefreshRateConfigs>(modes, /*currentConfigId=*/HWC_CONFIG_ID_60); |
| 1136 | |
| 1137 | lr.vote = LayerVoteType::ExplicitExactOrMultiple; |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 1138 | lr.desiredRefreshRate = 23.976_Hz; |
| 1139 | lr.name = "ExplicitExactOrMultiple 23.976 Hz"; |
| 1140 | EXPECT_EQ(HWC_CONFIG_ID_24, refreshRateConfigs->getBestRefreshRate(layers, {}).getModeId()); |
Marin Shalamanov | 15a0fc6 | 2021-08-16 18:20:21 +0200 | [diff] [blame] | 1141 | } |
| 1142 | |
| 1143 | // Test that 24 will choose 23.976 if 24 is not supported |
| 1144 | { |
| 1145 | android::DisplayModes modes = {mConfig24Frac, mConfig25, mConfig30, |
| 1146 | mConfig30Frac, mConfig60, mConfig60Frac}; |
| 1147 | auto refreshRateConfigs = |
| 1148 | std::make_unique<RefreshRateConfigs>(modes, /*currentConfigId=*/HWC_CONFIG_ID_60); |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 1149 | lr.desiredRefreshRate = 24_Hz; |
| 1150 | lr.name = "ExplicitExactOrMultiple 24 Hz"; |
Marin Shalamanov | 15a0fc6 | 2021-08-16 18:20:21 +0200 | [diff] [blame] | 1151 | EXPECT_EQ(HWC_CONFIG_ID_24_FRAC, |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 1152 | refreshRateConfigs->getBestRefreshRate(layers, {}).getModeId()); |
Marin Shalamanov | 15a0fc6 | 2021-08-16 18:20:21 +0200 | [diff] [blame] | 1153 | } |
| 1154 | |
| 1155 | // Test that 29.97 will prefer 59.94 over 60 and 30 |
| 1156 | { |
| 1157 | android::DisplayModes modes = {mConfig24, mConfig24Frac, mConfig25, |
| 1158 | mConfig30, mConfig60, mConfig60Frac}; |
| 1159 | auto refreshRateConfigs = |
| 1160 | std::make_unique<RefreshRateConfigs>(modes, /*currentConfigId=*/HWC_CONFIG_ID_60); |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 1161 | lr.desiredRefreshRate = 29.97_Hz; |
| 1162 | lr.name = "ExplicitExactOrMultiple 29.97 Hz"; |
Marin Shalamanov | 15a0fc6 | 2021-08-16 18:20:21 +0200 | [diff] [blame] | 1163 | EXPECT_EQ(HWC_CONFIG_ID_60_FRAC, |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 1164 | refreshRateConfigs->getBestRefreshRate(layers, {}).getModeId()); |
Marin Shalamanov | 15a0fc6 | 2021-08-16 18:20:21 +0200 | [diff] [blame] | 1165 | } |
| 1166 | } |
| 1167 | |
| 1168 | TEST_F(RefreshRateConfigsTest, getBestRefreshRate_ExplicitExact_WithFractionalRefreshRates) { |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 1169 | std::vector<LayerRequirement> layers = {{.weight = 1.f}}; |
Marin Shalamanov | 15a0fc6 | 2021-08-16 18:20:21 +0200 | [diff] [blame] | 1170 | auto& lr = layers[0]; |
| 1171 | |
| 1172 | // Test that voting for supported refresh rate will select this refresh rate |
| 1173 | { |
| 1174 | auto refreshRateConfigs = |
| 1175 | std::make_unique<RefreshRateConfigs>(m24_25_30_50_60WithFracDevice, |
| 1176 | /*currentConfigId=*/HWC_CONFIG_ID_60); |
| 1177 | |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 1178 | for (auto desired : {23.976_Hz, 24_Hz, 25_Hz, 29.97_Hz, 30_Hz, 50_Hz, 59.94_Hz, 60_Hz}) { |
Marin Shalamanov | 15a0fc6 | 2021-08-16 18:20:21 +0200 | [diff] [blame] | 1179 | lr.vote = LayerVoteType::ExplicitExact; |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 1180 | lr.desiredRefreshRate = desired; |
Marin Shalamanov | 15a0fc6 | 2021-08-16 18:20:21 +0200 | [diff] [blame] | 1181 | std::stringstream ss; |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 1182 | ss << "ExplicitExact " << desired; |
Marin Shalamanov | 15a0fc6 | 2021-08-16 18:20:21 +0200 | [diff] [blame] | 1183 | lr.name = ss.str(); |
| 1184 | |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 1185 | auto selectedRefreshRate = refreshRateConfigs->getBestRefreshRate(layers, {}); |
| 1186 | EXPECT_EQ(selectedRefreshRate.getFps(), lr.desiredRefreshRate); |
Marin Shalamanov | 15a0fc6 | 2021-08-16 18:20:21 +0200 | [diff] [blame] | 1187 | } |
| 1188 | } |
| 1189 | |
| 1190 | // Test that 23.976 will choose 24 if 23.976 is not supported |
| 1191 | { |
| 1192 | android::DisplayModes modes = {mConfig24, mConfig25, mConfig30, |
| 1193 | mConfig30Frac, mConfig60, mConfig60Frac}; |
| 1194 | auto refreshRateConfigs = |
| 1195 | std::make_unique<RefreshRateConfigs>(modes, /*currentConfigId=*/HWC_CONFIG_ID_60); |
| 1196 | lr.vote = LayerVoteType::ExplicitExact; |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 1197 | lr.desiredRefreshRate = 23.976_Hz; |
| 1198 | lr.name = "ExplicitExact 23.976 Hz"; |
| 1199 | EXPECT_EQ(HWC_CONFIG_ID_24, refreshRateConfigs->getBestRefreshRate(layers, {}).getModeId()); |
Marin Shalamanov | 15a0fc6 | 2021-08-16 18:20:21 +0200 | [diff] [blame] | 1200 | } |
| 1201 | |
| 1202 | // Test that 24 will choose 23.976 if 24 is not supported |
| 1203 | { |
| 1204 | android::DisplayModes modes = {mConfig24Frac, mConfig25, mConfig30, |
| 1205 | mConfig30Frac, mConfig60, mConfig60Frac}; |
| 1206 | auto refreshRateConfigs = |
| 1207 | std::make_unique<RefreshRateConfigs>(modes, /*currentConfigId=*/HWC_CONFIG_ID_60); |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 1208 | lr.desiredRefreshRate = 24_Hz; |
| 1209 | lr.name = "ExplicitExact 24 Hz"; |
Marin Shalamanov | 15a0fc6 | 2021-08-16 18:20:21 +0200 | [diff] [blame] | 1210 | EXPECT_EQ(HWC_CONFIG_ID_24_FRAC, |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 1211 | refreshRateConfigs->getBestRefreshRate(layers, {}).getModeId()); |
Ady Abraham | 5b8afb5a | 2020-03-06 14:57:26 -0800 | [diff] [blame] | 1212 | } |
Alec Mouri | 0a1cc96 | 2019-03-14 12:33:02 -0700 | [diff] [blame] | 1213 | } |
| 1214 | |
Alec Mouri | 11232a2 | 2020-05-14 18:06:25 -0700 | [diff] [blame] | 1215 | TEST_F(RefreshRateConfigsTest, |
| 1216 | getBestRefreshRate_withDisplayManagerRequestingSingleRate_ignoresTouchFlag) { |
| 1217 | auto refreshRateConfigs = |
| 1218 | std::make_unique<RefreshRateConfigs>(m60_90Device, |
| 1219 | /*currentConfigId=*/HWC_CONFIG_ID_90); |
| 1220 | |
| 1221 | ASSERT_GE(refreshRateConfigs->setDisplayManagerPolicy( |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 1222 | {HWC_CONFIG_ID_90, {90_Hz, 90_Hz}, {60_Hz, 90_Hz}}), |
Alec Mouri | 11232a2 | 2020-05-14 18:06:25 -0700 | [diff] [blame] | 1223 | 0); |
| 1224 | |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 1225 | std::vector<LayerRequirement> layers = {{.weight = 1.f}}; |
Alec Mouri | 11232a2 | 2020-05-14 18:06:25 -0700 | [diff] [blame] | 1226 | auto& lr = layers[0]; |
| 1227 | |
Ady Abraham | dfd6216 | 2020-06-10 16:11:56 -0700 | [diff] [blame] | 1228 | RefreshRateConfigs::GlobalSignals consideredSignals; |
Alec Mouri | 11232a2 | 2020-05-14 18:06:25 -0700 | [diff] [blame] | 1229 | lr.vote = LayerVoteType::ExplicitDefault; |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 1230 | lr.desiredRefreshRate = 60_Hz; |
Alec Mouri | 11232a2 | 2020-05-14 18:06:25 -0700 | [diff] [blame] | 1231 | lr.name = "60Hz ExplicitDefault"; |
Ady Abraham | aae5ed5 | 2020-06-26 09:32:43 -0700 | [diff] [blame] | 1232 | lr.focused = true; |
Alec Mouri | 11232a2 | 2020-05-14 18:06:25 -0700 | [diff] [blame] | 1233 | EXPECT_EQ(mExpected60Config, |
Ady Abraham | dfd6216 | 2020-06-10 16:11:56 -0700 | [diff] [blame] | 1234 | refreshRateConfigs->getBestRefreshRate(layers, {.touch = true, .idle = true}, |
| 1235 | &consideredSignals)); |
| 1236 | EXPECT_EQ(false, consideredSignals.touch); |
Alec Mouri | 11232a2 | 2020-05-14 18:06:25 -0700 | [diff] [blame] | 1237 | } |
| 1238 | |
| 1239 | TEST_F(RefreshRateConfigsTest, |
| 1240 | getBestRefreshRate_withDisplayManagerRequestingSingleRate_ignoresIdleFlag) { |
| 1241 | auto refreshRateConfigs = |
| 1242 | std::make_unique<RefreshRateConfigs>(m60_90Device, |
| 1243 | /*currentConfigId=*/HWC_CONFIG_ID_60); |
| 1244 | |
| 1245 | ASSERT_GE(refreshRateConfigs->setDisplayManagerPolicy( |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 1246 | {HWC_CONFIG_ID_60, {60_Hz, 60_Hz}, {60_Hz, 90_Hz}}), |
Alec Mouri | 11232a2 | 2020-05-14 18:06:25 -0700 | [diff] [blame] | 1247 | 0); |
| 1248 | |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 1249 | std::vector<LayerRequirement> layers = {{.weight = 1.f}}; |
Alec Mouri | 11232a2 | 2020-05-14 18:06:25 -0700 | [diff] [blame] | 1250 | auto& lr = layers[0]; |
| 1251 | |
Alec Mouri | 11232a2 | 2020-05-14 18:06:25 -0700 | [diff] [blame] | 1252 | lr.vote = LayerVoteType::ExplicitDefault; |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 1253 | lr.desiredRefreshRate = 90_Hz; |
Alec Mouri | 11232a2 | 2020-05-14 18:06:25 -0700 | [diff] [blame] | 1254 | lr.name = "90Hz ExplicitDefault"; |
Ady Abraham | aae5ed5 | 2020-06-26 09:32:43 -0700 | [diff] [blame] | 1255 | lr.focused = true; |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 1256 | EXPECT_EQ(mExpected90Config, refreshRateConfigs->getBestRefreshRate(layers, {.idle = true})); |
Alec Mouri | 11232a2 | 2020-05-14 18:06:25 -0700 | [diff] [blame] | 1257 | } |
| 1258 | |
| 1259 | TEST_F(RefreshRateConfigsTest, |
Ady Abraham | aae5ed5 | 2020-06-26 09:32:43 -0700 | [diff] [blame] | 1260 | getBestRefreshRate_withDisplayManagerRequestingSingleRate_onlySwitchesRatesForExplicitFocusedLayers) { |
Alec Mouri | 11232a2 | 2020-05-14 18:06:25 -0700 | [diff] [blame] | 1261 | auto refreshRateConfigs = |
| 1262 | std::make_unique<RefreshRateConfigs>(m60_90Device, |
| 1263 | /*currentConfigId=*/HWC_CONFIG_ID_90); |
| 1264 | |
| 1265 | ASSERT_GE(refreshRateConfigs->setDisplayManagerPolicy( |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 1266 | {HWC_CONFIG_ID_90, {90_Hz, 90_Hz}, {60_Hz, 90_Hz}}), |
Alec Mouri | 11232a2 | 2020-05-14 18:06:25 -0700 | [diff] [blame] | 1267 | 0); |
| 1268 | |
Ady Abraham | dfd6216 | 2020-06-10 16:11:56 -0700 | [diff] [blame] | 1269 | RefreshRateConfigs::GlobalSignals consideredSignals; |
Alec Mouri | 11232a2 | 2020-05-14 18:06:25 -0700 | [diff] [blame] | 1270 | EXPECT_EQ(mExpected90Config, |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 1271 | refreshRateConfigs->getBestRefreshRate({}, {}, &consideredSignals)); |
Ady Abraham | dfd6216 | 2020-06-10 16:11:56 -0700 | [diff] [blame] | 1272 | EXPECT_EQ(false, consideredSignals.touch); |
Alec Mouri | 11232a2 | 2020-05-14 18:06:25 -0700 | [diff] [blame] | 1273 | |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 1274 | std::vector<LayerRequirement> layers = {{.weight = 1.f}}; |
Alec Mouri | 11232a2 | 2020-05-14 18:06:25 -0700 | [diff] [blame] | 1275 | auto& lr = layers[0]; |
| 1276 | |
| 1277 | lr.vote = LayerVoteType::ExplicitExactOrMultiple; |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 1278 | lr.desiredRefreshRate = 60_Hz; |
Alec Mouri | 11232a2 | 2020-05-14 18:06:25 -0700 | [diff] [blame] | 1279 | lr.name = "60Hz ExplicitExactOrMultiple"; |
Ady Abraham | aae5ed5 | 2020-06-26 09:32:43 -0700 | [diff] [blame] | 1280 | lr.focused = false; |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 1281 | EXPECT_EQ(mExpected90Config, refreshRateConfigs->getBestRefreshRate(layers, {})); |
Ady Abraham | aae5ed5 | 2020-06-26 09:32:43 -0700 | [diff] [blame] | 1282 | |
| 1283 | lr.focused = true; |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 1284 | EXPECT_EQ(mExpected90Config, refreshRateConfigs->getBestRefreshRate(layers, {})); |
Alec Mouri | 11232a2 | 2020-05-14 18:06:25 -0700 | [diff] [blame] | 1285 | |
| 1286 | lr.vote = LayerVoteType::ExplicitDefault; |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 1287 | lr.desiredRefreshRate = 60_Hz; |
Alec Mouri | 11232a2 | 2020-05-14 18:06:25 -0700 | [diff] [blame] | 1288 | lr.name = "60Hz ExplicitDefault"; |
Ady Abraham | aae5ed5 | 2020-06-26 09:32:43 -0700 | [diff] [blame] | 1289 | lr.focused = false; |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 1290 | EXPECT_EQ(mExpected90Config, refreshRateConfigs->getBestRefreshRate(layers, {})); |
Ady Abraham | aae5ed5 | 2020-06-26 09:32:43 -0700 | [diff] [blame] | 1291 | |
| 1292 | lr.focused = true; |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 1293 | EXPECT_EQ(mExpected60Config, refreshRateConfigs->getBestRefreshRate(layers, {})); |
Alec Mouri | 11232a2 | 2020-05-14 18:06:25 -0700 | [diff] [blame] | 1294 | |
| 1295 | lr.vote = LayerVoteType::Heuristic; |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 1296 | lr.desiredRefreshRate = 60_Hz; |
Alec Mouri | 11232a2 | 2020-05-14 18:06:25 -0700 | [diff] [blame] | 1297 | lr.name = "60Hz Heuristic"; |
Ady Abraham | aae5ed5 | 2020-06-26 09:32:43 -0700 | [diff] [blame] | 1298 | lr.focused = false; |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 1299 | EXPECT_EQ(mExpected90Config, refreshRateConfigs->getBestRefreshRate(layers, {})); |
Ady Abraham | aae5ed5 | 2020-06-26 09:32:43 -0700 | [diff] [blame] | 1300 | |
| 1301 | lr.focused = true; |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 1302 | EXPECT_EQ(mExpected90Config, refreshRateConfigs->getBestRefreshRate(layers, {})); |
Alec Mouri | 11232a2 | 2020-05-14 18:06:25 -0700 | [diff] [blame] | 1303 | |
| 1304 | lr.vote = LayerVoteType::Max; |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 1305 | lr.desiredRefreshRate = 60_Hz; |
Alec Mouri | 11232a2 | 2020-05-14 18:06:25 -0700 | [diff] [blame] | 1306 | lr.name = "60Hz Max"; |
Ady Abraham | aae5ed5 | 2020-06-26 09:32:43 -0700 | [diff] [blame] | 1307 | lr.focused = false; |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 1308 | EXPECT_EQ(mExpected90Config, refreshRateConfigs->getBestRefreshRate(layers, {})); |
Ady Abraham | aae5ed5 | 2020-06-26 09:32:43 -0700 | [diff] [blame] | 1309 | |
| 1310 | lr.focused = true; |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 1311 | EXPECT_EQ(mExpected90Config, refreshRateConfigs->getBestRefreshRate(layers, {})); |
Alec Mouri | 11232a2 | 2020-05-14 18:06:25 -0700 | [diff] [blame] | 1312 | |
| 1313 | lr.vote = LayerVoteType::Min; |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 1314 | lr.desiredRefreshRate = 60_Hz; |
Alec Mouri | 11232a2 | 2020-05-14 18:06:25 -0700 | [diff] [blame] | 1315 | lr.name = "60Hz Min"; |
Ady Abraham | aae5ed5 | 2020-06-26 09:32:43 -0700 | [diff] [blame] | 1316 | lr.focused = false; |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 1317 | EXPECT_EQ(mExpected90Config, refreshRateConfigs->getBestRefreshRate(layers, {})); |
Ady Abraham | aae5ed5 | 2020-06-26 09:32:43 -0700 | [diff] [blame] | 1318 | |
| 1319 | lr.focused = true; |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 1320 | EXPECT_EQ(mExpected90Config, refreshRateConfigs->getBestRefreshRate(layers, {})); |
Alec Mouri | 11232a2 | 2020-05-14 18:06:25 -0700 | [diff] [blame] | 1321 | } |
| 1322 | |
Marin Shalamanov | bb65d05 | 2021-03-23 19:21:52 +0100 | [diff] [blame] | 1323 | TEST_F(RefreshRateConfigsTest, groupSwitchingNotAllowed) { |
Steven Thomas | d407190 | 2020-03-24 16:02:53 -0700 | [diff] [blame] | 1324 | auto refreshRateConfigs = |
Ady Abraham | abc2760 | 2020-04-08 17:20:29 -0700 | [diff] [blame] | 1325 | std::make_unique<RefreshRateConfigs>(m60_90DeviceWithDifferentGroups, |
| 1326 | /*currentConfigId=*/HWC_CONFIG_ID_60); |
Steven Thomas | d407190 | 2020-03-24 16:02:53 -0700 | [diff] [blame] | 1327 | |
Marin Shalamanov | bb65d05 | 2021-03-23 19:21:52 +0100 | [diff] [blame] | 1328 | // The default policy doesn't allow group switching. Verify that no |
| 1329 | // group switches are performed. |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 1330 | std::vector<LayerRequirement> layers = {{.weight = 1.f}}; |
Steven Thomas | d407190 | 2020-03-24 16:02:53 -0700 | [diff] [blame] | 1331 | auto& layer = layers[0]; |
| 1332 | layer.vote = LayerVoteType::ExplicitDefault; |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 1333 | layer.desiredRefreshRate = 90_Hz; |
Marin Shalamanov | 53fc11d | 2020-11-20 14:00:13 +0100 | [diff] [blame] | 1334 | layer.seamlessness = Seamlessness::SeamedAndSeamless; |
Steven Thomas | d407190 | 2020-03-24 16:02:53 -0700 | [diff] [blame] | 1335 | layer.name = "90Hz ExplicitDefault"; |
Marin Shalamanov | 4608442 | 2020-10-13 12:33:42 +0200 | [diff] [blame] | 1336 | layer.focused = true; |
Steven Thomas | d407190 | 2020-03-24 16:02:53 -0700 | [diff] [blame] | 1337 | |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 1338 | ASSERT_EQ(HWC_CONFIG_ID_60, refreshRateConfigs->getBestRefreshRate(layers, {}).getModeId()); |
Marin Shalamanov | bb65d05 | 2021-03-23 19:21:52 +0100 | [diff] [blame] | 1339 | } |
Steven Thomas | d407190 | 2020-03-24 16:02:53 -0700 | [diff] [blame] | 1340 | |
Marin Shalamanov | bb65d05 | 2021-03-23 19:21:52 +0100 | [diff] [blame] | 1341 | TEST_F(RefreshRateConfigsTest, groupSwitchingWithOneLayer) { |
| 1342 | auto refreshRateConfigs = |
| 1343 | std::make_unique<RefreshRateConfigs>(m60_90DeviceWithDifferentGroups, |
| 1344 | /*currentConfigId=*/HWC_CONFIG_ID_60); |
Steven Thomas | d407190 | 2020-03-24 16:02:53 -0700 | [diff] [blame] | 1345 | RefreshRateConfigs::Policy policy; |
Marin Shalamanov | a7fe304 | 2021-01-29 21:02:08 +0100 | [diff] [blame] | 1346 | policy.defaultMode = refreshRateConfigs->getCurrentPolicy().defaultMode; |
Steven Thomas | d407190 | 2020-03-24 16:02:53 -0700 | [diff] [blame] | 1347 | policy.allowGroupSwitching = true; |
| 1348 | ASSERT_GE(refreshRateConfigs->setDisplayManagerPolicy(policy), 0); |
Marin Shalamanov | bb65d05 | 2021-03-23 19:21:52 +0100 | [diff] [blame] | 1349 | |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 1350 | std::vector<LayerRequirement> layers = {{.weight = 1.f}}; |
Marin Shalamanov | bb65d05 | 2021-03-23 19:21:52 +0100 | [diff] [blame] | 1351 | auto& layer = layers[0]; |
| 1352 | layer.vote = LayerVoteType::ExplicitDefault; |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 1353 | layer.desiredRefreshRate = 90_Hz; |
Marin Shalamanov | bb65d05 | 2021-03-23 19:21:52 +0100 | [diff] [blame] | 1354 | layer.seamlessness = Seamlessness::SeamedAndSeamless; |
| 1355 | layer.name = "90Hz ExplicitDefault"; |
| 1356 | layer.focused = true; |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 1357 | ASSERT_EQ(HWC_CONFIG_ID_90, refreshRateConfigs->getBestRefreshRate(layers, {}).getModeId()); |
Marin Shalamanov | bb65d05 | 2021-03-23 19:21:52 +0100 | [diff] [blame] | 1358 | } |
| 1359 | |
| 1360 | TEST_F(RefreshRateConfigsTest, groupSwitchingWithOneLayerOnlySeamless) { |
| 1361 | auto refreshRateConfigs = |
| 1362 | std::make_unique<RefreshRateConfigs>(m60_90DeviceWithDifferentGroups, |
| 1363 | /*currentConfigId=*/HWC_CONFIG_ID_60); |
| 1364 | RefreshRateConfigs::Policy policy; |
| 1365 | policy.defaultMode = refreshRateConfigs->getCurrentPolicy().defaultMode; |
| 1366 | policy.allowGroupSwitching = true; |
| 1367 | ASSERT_GE(refreshRateConfigs->setDisplayManagerPolicy(policy), 0); |
Marin Shalamanov | 4608442 | 2020-10-13 12:33:42 +0200 | [diff] [blame] | 1368 | |
| 1369 | // Verify that we won't change the group if seamless switch is required. |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 1370 | std::vector<LayerRequirement> layers = {{.weight = 1.f}}; |
Marin Shalamanov | bb65d05 | 2021-03-23 19:21:52 +0100 | [diff] [blame] | 1371 | auto& layer = layers[0]; |
| 1372 | layer.vote = LayerVoteType::ExplicitDefault; |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 1373 | layer.desiredRefreshRate = 90_Hz; |
Marin Shalamanov | 53fc11d | 2020-11-20 14:00:13 +0100 | [diff] [blame] | 1374 | layer.seamlessness = Seamlessness::OnlySeamless; |
Marin Shalamanov | bb65d05 | 2021-03-23 19:21:52 +0100 | [diff] [blame] | 1375 | layer.name = "90Hz ExplicitDefault"; |
| 1376 | layer.focused = true; |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 1377 | ASSERT_EQ(HWC_CONFIG_ID_60, refreshRateConfigs->getBestRefreshRate(layers, {}).getModeId()); |
Marin Shalamanov | bb65d05 | 2021-03-23 19:21:52 +0100 | [diff] [blame] | 1378 | } |
| 1379 | |
| 1380 | TEST_F(RefreshRateConfigsTest, groupSwitchingWithOneLayerOnlySeamlessDefaultFps) { |
| 1381 | auto refreshRateConfigs = |
| 1382 | std::make_unique<RefreshRateConfigs>(m60_90DeviceWithDifferentGroups, |
| 1383 | /*currentConfigId=*/HWC_CONFIG_ID_60); |
| 1384 | RefreshRateConfigs::Policy policy; |
| 1385 | policy.defaultMode = refreshRateConfigs->getCurrentPolicy().defaultMode; |
| 1386 | policy.allowGroupSwitching = true; |
| 1387 | ASSERT_GE(refreshRateConfigs->setDisplayManagerPolicy(policy), 0); |
| 1388 | |
| 1389 | refreshRateConfigs->setCurrentModeId(HWC_CONFIG_ID_90); |
Marin Shalamanov | 4608442 | 2020-10-13 12:33:42 +0200 | [diff] [blame] | 1390 | |
Marin Shalamanov | 53fc11d | 2020-11-20 14:00:13 +0100 | [diff] [blame] | 1391 | // Verify that we won't do a seamless switch if we request the same mode as the default |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 1392 | std::vector<LayerRequirement> layers = {{.weight = 1.f}}; |
Marin Shalamanov | bb65d05 | 2021-03-23 19:21:52 +0100 | [diff] [blame] | 1393 | auto& layer = layers[0]; |
| 1394 | layer.vote = LayerVoteType::ExplicitDefault; |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 1395 | layer.desiredRefreshRate = 60_Hz; |
Marin Shalamanov | 53fc11d | 2020-11-20 14:00:13 +0100 | [diff] [blame] | 1396 | layer.seamlessness = Seamlessness::OnlySeamless; |
Marin Shalamanov | bb65d05 | 2021-03-23 19:21:52 +0100 | [diff] [blame] | 1397 | layer.name = "60Hz ExplicitDefault"; |
| 1398 | layer.focused = true; |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 1399 | ASSERT_EQ(HWC_CONFIG_ID_90, refreshRateConfigs->getBestRefreshRate(layers, {}).getModeId()); |
Marin Shalamanov | bb65d05 | 2021-03-23 19:21:52 +0100 | [diff] [blame] | 1400 | } |
| 1401 | |
| 1402 | TEST_F(RefreshRateConfigsTest, groupSwitchingWithOneLayerDefaultSeamlessness) { |
| 1403 | auto refreshRateConfigs = |
| 1404 | std::make_unique<RefreshRateConfigs>(m60_90DeviceWithDifferentGroups, |
| 1405 | /*currentConfigId=*/HWC_CONFIG_ID_60); |
| 1406 | RefreshRateConfigs::Policy policy; |
| 1407 | policy.defaultMode = refreshRateConfigs->getCurrentPolicy().defaultMode; |
| 1408 | policy.allowGroupSwitching = true; |
| 1409 | ASSERT_GE(refreshRateConfigs->setDisplayManagerPolicy(policy), 0); |
| 1410 | |
| 1411 | refreshRateConfigs->setCurrentModeId(HWC_CONFIG_ID_90); |
Marin Shalamanov | 53fc11d | 2020-11-20 14:00:13 +0100 | [diff] [blame] | 1412 | |
| 1413 | // Verify that if the current config is in another group and there are no layers with |
| 1414 | // seamlessness=SeamedAndSeamless we'll go back to the default group. |
Marin Shalamanov | bb65d05 | 2021-03-23 19:21:52 +0100 | [diff] [blame] | 1415 | |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 1416 | std::vector<LayerRequirement> layers = {{.weight = 1.f}}; |
Marin Shalamanov | bb65d05 | 2021-03-23 19:21:52 +0100 | [diff] [blame] | 1417 | auto& layer = layers[0]; |
| 1418 | layer.vote = LayerVoteType::ExplicitDefault; |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 1419 | layer.desiredRefreshRate = 60_Hz; |
Marin Shalamanov | 53fc11d | 2020-11-20 14:00:13 +0100 | [diff] [blame] | 1420 | layer.seamlessness = Seamlessness::Default; |
Marin Shalamanov | bb65d05 | 2021-03-23 19:21:52 +0100 | [diff] [blame] | 1421 | layer.name = "60Hz ExplicitDefault"; |
| 1422 | layer.focused = true; |
| 1423 | |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 1424 | ASSERT_EQ(HWC_CONFIG_ID_60, refreshRateConfigs->getBestRefreshRate(layers, {}).getModeId()); |
Marin Shalamanov | bb65d05 | 2021-03-23 19:21:52 +0100 | [diff] [blame] | 1425 | } |
| 1426 | |
| 1427 | TEST_F(RefreshRateConfigsTest, groupSwitchingWithTwoLayersOnlySeamlessAndSeamed) { |
| 1428 | auto refreshRateConfigs = |
| 1429 | std::make_unique<RefreshRateConfigs>(m60_90DeviceWithDifferentGroups, |
| 1430 | /*currentConfigId=*/HWC_CONFIG_ID_60); |
| 1431 | RefreshRateConfigs::Policy policy; |
| 1432 | policy.defaultMode = refreshRateConfigs->getCurrentPolicy().defaultMode; |
| 1433 | policy.allowGroupSwitching = true; |
| 1434 | ASSERT_GE(refreshRateConfigs->setDisplayManagerPolicy(policy), 0); |
| 1435 | |
| 1436 | refreshRateConfigs->setCurrentModeId(HWC_CONFIG_ID_90); |
Marin Shalamanov | 4608442 | 2020-10-13 12:33:42 +0200 | [diff] [blame] | 1437 | |
Marin Shalamanov | 53fc11d | 2020-11-20 14:00:13 +0100 | [diff] [blame] | 1438 | // If there's a layer with seamlessness=SeamedAndSeamless, another layer with |
Marin Shalamanov | a7fe304 | 2021-01-29 21:02:08 +0100 | [diff] [blame] | 1439 | // seamlessness=OnlySeamless can't change the mode group. |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 1440 | std::vector<LayerRequirement> layers = {{.weight = 1.f}}; |
Marin Shalamanov | bb65d05 | 2021-03-23 19:21:52 +0100 | [diff] [blame] | 1441 | layers[0].vote = LayerVoteType::ExplicitDefault; |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 1442 | layers[0].desiredRefreshRate = 60_Hz; |
Marin Shalamanov | bb65d05 | 2021-03-23 19:21:52 +0100 | [diff] [blame] | 1443 | layers[0].seamlessness = Seamlessness::OnlySeamless; |
| 1444 | layers[0].name = "60Hz ExplicitDefault"; |
| 1445 | layers[0].focused = true; |
Marin Shalamanov | 53fc11d | 2020-11-20 14:00:13 +0100 | [diff] [blame] | 1446 | |
| 1447 | layers.push_back(LayerRequirement{.weight = 0.5f}); |
Marin Shalamanov | bb65d05 | 2021-03-23 19:21:52 +0100 | [diff] [blame] | 1448 | layers[1].vote = LayerVoteType::ExplicitDefault; |
| 1449 | layers[1].seamlessness = Seamlessness::SeamedAndSeamless; |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 1450 | layers[1].desiredRefreshRate = 90_Hz; |
Marin Shalamanov | bb65d05 | 2021-03-23 19:21:52 +0100 | [diff] [blame] | 1451 | layers[1].name = "90Hz ExplicitDefault"; |
| 1452 | layers[1].focused = false; |
Marin Shalamanov | 53fc11d | 2020-11-20 14:00:13 +0100 | [diff] [blame] | 1453 | |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 1454 | ASSERT_EQ(HWC_CONFIG_ID_90, refreshRateConfigs->getBestRefreshRate(layers, {}).getModeId()); |
Marin Shalamanov | bb65d05 | 2021-03-23 19:21:52 +0100 | [diff] [blame] | 1455 | } |
Marin Shalamanov | 53fc11d | 2020-11-20 14:00:13 +0100 | [diff] [blame] | 1456 | |
Marin Shalamanov | ae0b535 | 2021-03-24 12:56:08 +0100 | [diff] [blame] | 1457 | TEST_F(RefreshRateConfigsTest, groupSwitchingWithTwoLayersDefaultFocusedAndSeamed) { |
Marin Shalamanov | bb65d05 | 2021-03-23 19:21:52 +0100 | [diff] [blame] | 1458 | auto refreshRateConfigs = |
| 1459 | std::make_unique<RefreshRateConfigs>(m60_90DeviceWithDifferentGroups, |
| 1460 | /*currentConfigId=*/HWC_CONFIG_ID_60); |
| 1461 | RefreshRateConfigs::Policy policy; |
| 1462 | policy.defaultMode = refreshRateConfigs->getCurrentPolicy().defaultMode; |
| 1463 | policy.allowGroupSwitching = true; |
| 1464 | ASSERT_GE(refreshRateConfigs->setDisplayManagerPolicy(policy), 0); |
| 1465 | |
| 1466 | refreshRateConfigs->setCurrentModeId(HWC_CONFIG_ID_90); |
Marin Shalamanov | 53fc11d | 2020-11-20 14:00:13 +0100 | [diff] [blame] | 1467 | |
Marin Shalamanov | ae0b535 | 2021-03-24 12:56:08 +0100 | [diff] [blame] | 1468 | // If there's a focused layer with seamlessness=SeamedAndSeamless, another layer with |
| 1469 | // seamlessness=Default can't change the mode group back to the group of the default |
| 1470 | // mode. |
| 1471 | // For example, this may happen when a video playback requests and gets a seamed switch, |
| 1472 | // but another layer (with default seamlessness) starts animating. The animating layer |
| 1473 | // should not cause a seamed switch. |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 1474 | std::vector<LayerRequirement> layers = {{.weight = 1.f}}; |
Marin Shalamanov | 53fc11d | 2020-11-20 14:00:13 +0100 | [diff] [blame] | 1475 | layers[0].seamlessness = Seamlessness::Default; |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 1476 | layers[0].desiredRefreshRate = 60_Hz; |
Marin Shalamanov | bb65d05 | 2021-03-23 19:21:52 +0100 | [diff] [blame] | 1477 | layers[0].focused = true; |
| 1478 | layers[0].vote = LayerVoteType::ExplicitDefault; |
| 1479 | layers[0].name = "60Hz ExplicitDefault"; |
| 1480 | |
Marin Shalamanov | ae0b535 | 2021-03-24 12:56:08 +0100 | [diff] [blame] | 1481 | layers.push_back(LayerRequirement{.weight = 0.1f}); |
| 1482 | layers[1].seamlessness = Seamlessness::SeamedAndSeamless; |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 1483 | layers[1].desiredRefreshRate = 90_Hz; |
Marin Shalamanov | ae0b535 | 2021-03-24 12:56:08 +0100 | [diff] [blame] | 1484 | layers[1].focused = true; |
| 1485 | layers[1].vote = LayerVoteType::ExplicitDefault; |
| 1486 | layers[1].name = "90Hz ExplicitDefault"; |
| 1487 | |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 1488 | ASSERT_EQ(HWC_CONFIG_ID_90, refreshRateConfigs->getBestRefreshRate(layers, {}).getModeId()); |
Marin Shalamanov | 4608442 | 2020-10-13 12:33:42 +0200 | [diff] [blame] | 1489 | } |
| 1490 | |
Marin Shalamanov | ae0b535 | 2021-03-24 12:56:08 +0100 | [diff] [blame] | 1491 | TEST_F(RefreshRateConfigsTest, groupSwitchingWithTwoLayersDefaultNotFocusedAndSeamed) { |
| 1492 | auto refreshRateConfigs = |
| 1493 | std::make_unique<RefreshRateConfigs>(m60_90DeviceWithDifferentGroups, |
| 1494 | /*currentConfigId=*/HWC_CONFIG_ID_60); |
| 1495 | RefreshRateConfigs::Policy policy; |
| 1496 | policy.defaultMode = refreshRateConfigs->getCurrentPolicy().defaultMode; |
| 1497 | policy.allowGroupSwitching = true; |
| 1498 | ASSERT_GE(refreshRateConfigs->setDisplayManagerPolicy(policy), 0); |
| 1499 | |
| 1500 | refreshRateConfigs->setCurrentModeId(HWC_CONFIG_ID_90); |
| 1501 | |
| 1502 | // Layer with seamlessness=Default can change the mode group if there's a not |
| 1503 | // focused layer with seamlessness=SeamedAndSeamless. This happens for example, |
| 1504 | // when in split screen mode the user switches between the two visible applications. |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 1505 | std::vector<LayerRequirement> layers = {{.weight = 1.f}}; |
Marin Shalamanov | ae0b535 | 2021-03-24 12:56:08 +0100 | [diff] [blame] | 1506 | layers[0].seamlessness = Seamlessness::Default; |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 1507 | layers[0].desiredRefreshRate = 60_Hz; |
Marin Shalamanov | ae0b535 | 2021-03-24 12:56:08 +0100 | [diff] [blame] | 1508 | layers[0].focused = true; |
| 1509 | layers[0].vote = LayerVoteType::ExplicitDefault; |
| 1510 | layers[0].name = "60Hz ExplicitDefault"; |
| 1511 | |
| 1512 | layers.push_back(LayerRequirement{.weight = 0.7f}); |
Marin Shalamanov | bb65d05 | 2021-03-23 19:21:52 +0100 | [diff] [blame] | 1513 | layers[1].seamlessness = Seamlessness::SeamedAndSeamless; |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 1514 | layers[1].desiredRefreshRate = 90_Hz; |
Marin Shalamanov | bb65d05 | 2021-03-23 19:21:52 +0100 | [diff] [blame] | 1515 | layers[1].focused = false; |
| 1516 | layers[1].vote = LayerVoteType::ExplicitDefault; |
| 1517 | layers[1].name = "90Hz ExplicitDefault"; |
| 1518 | |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 1519 | ASSERT_EQ(HWC_CONFIG_ID_60, refreshRateConfigs->getBestRefreshRate(layers, {}).getModeId()); |
Marin Shalamanov | 4608442 | 2020-10-13 12:33:42 +0200 | [diff] [blame] | 1520 | } |
| 1521 | |
| 1522 | TEST_F(RefreshRateConfigsTest, nonSeamlessVotePrefersSeamlessSwitches) { |
| 1523 | auto refreshRateConfigs = |
| 1524 | std::make_unique<RefreshRateConfigs>(m30_60Device, |
| 1525 | /*currentConfigId=*/HWC_CONFIG_ID_60); |
| 1526 | |
| 1527 | // Allow group switching. |
| 1528 | RefreshRateConfigs::Policy policy; |
Marin Shalamanov | a7fe304 | 2021-01-29 21:02:08 +0100 | [diff] [blame] | 1529 | policy.defaultMode = refreshRateConfigs->getCurrentPolicy().defaultMode; |
Marin Shalamanov | 4608442 | 2020-10-13 12:33:42 +0200 | [diff] [blame] | 1530 | policy.allowGroupSwitching = true; |
| 1531 | ASSERT_GE(refreshRateConfigs->setDisplayManagerPolicy(policy), 0); |
| 1532 | |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 1533 | std::vector<LayerRequirement> layers = {{.weight = 1.f}}; |
Marin Shalamanov | 4608442 | 2020-10-13 12:33:42 +0200 | [diff] [blame] | 1534 | auto& layer = layers[0]; |
| 1535 | layer.vote = LayerVoteType::ExplicitExactOrMultiple; |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 1536 | layer.desiredRefreshRate = 60_Hz; |
Marin Shalamanov | 53fc11d | 2020-11-20 14:00:13 +0100 | [diff] [blame] | 1537 | layer.seamlessness = Seamlessness::SeamedAndSeamless; |
Marin Shalamanov | 4608442 | 2020-10-13 12:33:42 +0200 | [diff] [blame] | 1538 | layer.name = "60Hz ExplicitExactOrMultiple"; |
| 1539 | layer.focused = true; |
| 1540 | |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 1541 | ASSERT_EQ(HWC_CONFIG_ID_60, refreshRateConfigs->getBestRefreshRate(layers, {}).getModeId()); |
Marin Shalamanov | 4608442 | 2020-10-13 12:33:42 +0200 | [diff] [blame] | 1542 | |
Marin Shalamanov | a7fe304 | 2021-01-29 21:02:08 +0100 | [diff] [blame] | 1543 | refreshRateConfigs->setCurrentModeId(HWC_CONFIG_ID_120); |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 1544 | ASSERT_EQ(HWC_CONFIG_ID_120, refreshRateConfigs->getBestRefreshRate(layers, {}).getModeId()); |
Marin Shalamanov | 4608442 | 2020-10-13 12:33:42 +0200 | [diff] [blame] | 1545 | } |
| 1546 | |
| 1547 | TEST_F(RefreshRateConfigsTest, nonSeamlessExactAndSeamlessMultipleLayers) { |
| 1548 | auto refreshRateConfigs = |
| 1549 | std::make_unique<RefreshRateConfigs>(m25_30_50_60Device, |
| 1550 | /*currentConfigId=*/HWC_CONFIG_ID_60); |
| 1551 | |
| 1552 | // Allow group switching. |
| 1553 | RefreshRateConfigs::Policy policy; |
Marin Shalamanov | a7fe304 | 2021-01-29 21:02:08 +0100 | [diff] [blame] | 1554 | policy.defaultMode = refreshRateConfigs->getCurrentPolicy().defaultMode; |
Marin Shalamanov | 4608442 | 2020-10-13 12:33:42 +0200 | [diff] [blame] | 1555 | policy.allowGroupSwitching = true; |
| 1556 | ASSERT_GE(refreshRateConfigs->setDisplayManagerPolicy(policy), 0); |
| 1557 | |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 1558 | std::vector<LayerRequirement> layers = {{.name = "60Hz ExplicitDefault", |
| 1559 | .vote = LayerVoteType::ExplicitDefault, |
| 1560 | .desiredRefreshRate = 60_Hz, |
| 1561 | .seamlessness = Seamlessness::SeamedAndSeamless, |
| 1562 | .weight = 0.5f, |
| 1563 | .focused = false}, |
| 1564 | {.name = "25Hz ExplicitExactOrMultiple", |
| 1565 | .vote = LayerVoteType::ExplicitExactOrMultiple, |
| 1566 | .desiredRefreshRate = 25_Hz, |
| 1567 | .seamlessness = Seamlessness::OnlySeamless, |
| 1568 | .weight = 1.f, |
| 1569 | .focused = true}}; |
Marin Shalamanov | 4608442 | 2020-10-13 12:33:42 +0200 | [diff] [blame] | 1570 | |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 1571 | ASSERT_EQ(HWC_CONFIG_ID_50, refreshRateConfigs->getBestRefreshRate(layers, {}).getModeId()); |
Marin Shalamanov | 4608442 | 2020-10-13 12:33:42 +0200 | [diff] [blame] | 1572 | |
Marin Shalamanov | e7aadcc | 2021-04-09 18:58:38 +0200 | [diff] [blame] | 1573 | auto& seamedLayer = layers[0]; |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 1574 | seamedLayer.desiredRefreshRate = 30_Hz; |
| 1575 | seamedLayer.name = "30Hz ExplicitDefault"; |
Marin Shalamanov | a7fe304 | 2021-01-29 21:02:08 +0100 | [diff] [blame] | 1576 | refreshRateConfigs->setCurrentModeId(HWC_CONFIG_ID_30); |
Marin Shalamanov | 4608442 | 2020-10-13 12:33:42 +0200 | [diff] [blame] | 1577 | |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 1578 | ASSERT_EQ(HWC_CONFIG_ID_25, refreshRateConfigs->getBestRefreshRate(layers, {}).getModeId()); |
Steven Thomas | d407190 | 2020-03-24 16:02:53 -0700 | [diff] [blame] | 1579 | } |
| 1580 | |
Marin Shalamanov | e7aadcc | 2021-04-09 18:58:38 +0200 | [diff] [blame] | 1581 | TEST_F(RefreshRateConfigsTest, minLayersDontTrigerSeamedSwitch) { |
| 1582 | auto refreshRateConfigs = |
| 1583 | std::make_unique<RefreshRateConfigs>(m60_90DeviceWithDifferentGroups, |
| 1584 | /*currentConfigId=*/HWC_CONFIG_ID_90); |
| 1585 | |
| 1586 | // Allow group switching. |
| 1587 | RefreshRateConfigs::Policy policy; |
| 1588 | policy.defaultMode = refreshRateConfigs->getCurrentPolicy().defaultMode; |
| 1589 | policy.allowGroupSwitching = true; |
| 1590 | ASSERT_GE(refreshRateConfigs->setDisplayManagerPolicy(policy), 0); |
| 1591 | |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 1592 | std::vector<LayerRequirement> layers = { |
| 1593 | {.name = "Min", .vote = LayerVoteType::Min, .weight = 1.f, .focused = true}}; |
Marin Shalamanov | e7aadcc | 2021-04-09 18:58:38 +0200 | [diff] [blame] | 1594 | |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 1595 | ASSERT_EQ(HWC_CONFIG_ID_90, refreshRateConfigs->getBestRefreshRate(layers, {}).getModeId()); |
Marin Shalamanov | e7aadcc | 2021-04-09 18:58:38 +0200 | [diff] [blame] | 1596 | } |
| 1597 | |
Steven Thomas | f734df4 | 2020-04-13 21:09:28 -0700 | [diff] [blame] | 1598 | TEST_F(RefreshRateConfigsTest, primaryVsAppRequestPolicy) { |
| 1599 | auto refreshRateConfigs = |
Alec Mouri | 11232a2 | 2020-05-14 18:06:25 -0700 | [diff] [blame] | 1600 | std::make_unique<RefreshRateConfigs>(m30_60_90Device, |
Steven Thomas | f734df4 | 2020-04-13 21:09:28 -0700 | [diff] [blame] | 1601 | /*currentConfigId=*/HWC_CONFIG_ID_60); |
| 1602 | |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 1603 | std::vector<LayerRequirement> layers = {{.weight = 1.f}}; |
Steven Thomas | f734df4 | 2020-04-13 21:09:28 -0700 | [diff] [blame] | 1604 | layers[0].name = "Test layer"; |
| 1605 | |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 1606 | struct Args { |
| 1607 | bool touch = false; |
| 1608 | bool focused = true; |
| 1609 | }; |
| 1610 | |
Steven Thomas | bb37432 | 2020-04-28 22:47:16 -0700 | [diff] [blame] | 1611 | // Return the config ID from calling getBestRefreshRate() for a single layer with the |
Steven Thomas | f734df4 | 2020-04-13 21:09:28 -0700 | [diff] [blame] | 1612 | // given voteType and fps. |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 1613 | auto getFrameRate = [&](LayerVoteType voteType, Fps fps, Args args = {}) -> DisplayModeId { |
Steven Thomas | f734df4 | 2020-04-13 21:09:28 -0700 | [diff] [blame] | 1614 | layers[0].vote = voteType; |
| 1615 | layers[0].desiredRefreshRate = fps; |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 1616 | layers[0].focused = args.focused; |
| 1617 | return refreshRateConfigs->getBestRefreshRate(layers, {.touch = args.touch}).getModeId(); |
Steven Thomas | f734df4 | 2020-04-13 21:09:28 -0700 | [diff] [blame] | 1618 | }; |
| 1619 | |
| 1620 | ASSERT_GE(refreshRateConfigs->setDisplayManagerPolicy( |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 1621 | {HWC_CONFIG_ID_60, {30_Hz, 60_Hz}, {30_Hz, 90_Hz}}), |
Steven Thomas | f734df4 | 2020-04-13 21:09:28 -0700 | [diff] [blame] | 1622 | 0); |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 1623 | EXPECT_EQ(HWC_CONFIG_ID_60, refreshRateConfigs->getBestRefreshRate({}, {}).getModeId()); |
| 1624 | EXPECT_EQ(HWC_CONFIG_ID_60, getFrameRate(LayerVoteType::NoVote, 90_Hz)); |
| 1625 | EXPECT_EQ(HWC_CONFIG_ID_30, getFrameRate(LayerVoteType::Min, 90_Hz)); |
| 1626 | EXPECT_EQ(HWC_CONFIG_ID_60, getFrameRate(LayerVoteType::Max, 90_Hz)); |
| 1627 | EXPECT_EQ(HWC_CONFIG_ID_60, getFrameRate(LayerVoteType::Heuristic, 90_Hz)); |
| 1628 | EXPECT_EQ(HWC_CONFIG_ID_90, getFrameRate(LayerVoteType::ExplicitDefault, 90_Hz)); |
| 1629 | EXPECT_EQ(HWC_CONFIG_ID_60, getFrameRate(LayerVoteType::ExplicitExactOrMultiple, 90_Hz)); |
Steven Thomas | f734df4 | 2020-04-13 21:09:28 -0700 | [diff] [blame] | 1630 | |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 1631 | // Unfocused layers are not allowed to override primary config. |
Ady Abraham | aae5ed5 | 2020-06-26 09:32:43 -0700 | [diff] [blame] | 1632 | EXPECT_EQ(HWC_CONFIG_ID_60, |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 1633 | getFrameRate(LayerVoteType::ExplicitDefault, 90_Hz, {.focused = false})); |
Ady Abraham | aae5ed5 | 2020-06-26 09:32:43 -0700 | [diff] [blame] | 1634 | EXPECT_EQ(HWC_CONFIG_ID_60, |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 1635 | getFrameRate(LayerVoteType::ExplicitExactOrMultiple, 90_Hz, {.focused = false})); |
Ady Abraham | aae5ed5 | 2020-06-26 09:32:43 -0700 | [diff] [blame] | 1636 | |
Steven Thomas | f734df4 | 2020-04-13 21:09:28 -0700 | [diff] [blame] | 1637 | // Touch boost should be restricted to the primary range. |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 1638 | EXPECT_EQ(HWC_CONFIG_ID_60, getFrameRate(LayerVoteType::Max, 90_Hz, {.touch = true})); |
Steven Thomas | f734df4 | 2020-04-13 21:09:28 -0700 | [diff] [blame] | 1639 | // When we're higher than the primary range max due to a layer frame rate setting, touch boost |
| 1640 | // shouldn't drag us back down to the primary range max. |
Marin Shalamanov | e8a663d | 2020-11-24 17:48:00 +0100 | [diff] [blame] | 1641 | EXPECT_EQ(HWC_CONFIG_ID_90, |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 1642 | getFrameRate(LayerVoteType::ExplicitDefault, 90_Hz, {.touch = true})); |
Ady Abraham | 20c029c | 2020-07-06 12:58:05 -0700 | [diff] [blame] | 1643 | EXPECT_EQ(HWC_CONFIG_ID_60, |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 1644 | getFrameRate(LayerVoteType::ExplicitExactOrMultiple, 90_Hz, {.touch = true})); |
Steven Thomas | f734df4 | 2020-04-13 21:09:28 -0700 | [diff] [blame] | 1645 | |
| 1646 | ASSERT_GE(refreshRateConfigs->setDisplayManagerPolicy( |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 1647 | {HWC_CONFIG_ID_60, {60_Hz, 60_Hz}, {60_Hz, 60_Hz}}), |
Steven Thomas | f734df4 | 2020-04-13 21:09:28 -0700 | [diff] [blame] | 1648 | 0); |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 1649 | EXPECT_EQ(HWC_CONFIG_ID_60, getFrameRate(LayerVoteType::NoVote, 90_Hz)); |
| 1650 | EXPECT_EQ(HWC_CONFIG_ID_60, getFrameRate(LayerVoteType::Min, 90_Hz)); |
| 1651 | EXPECT_EQ(HWC_CONFIG_ID_60, getFrameRate(LayerVoteType::Max, 90_Hz)); |
| 1652 | EXPECT_EQ(HWC_CONFIG_ID_60, getFrameRate(LayerVoteType::Heuristic, 90_Hz)); |
| 1653 | EXPECT_EQ(HWC_CONFIG_ID_60, getFrameRate(LayerVoteType::ExplicitDefault, 90_Hz)); |
| 1654 | EXPECT_EQ(HWC_CONFIG_ID_60, getFrameRate(LayerVoteType::ExplicitExactOrMultiple, 90_Hz)); |
Steven Thomas | f734df4 | 2020-04-13 21:09:28 -0700 | [diff] [blame] | 1655 | } |
| 1656 | |
Steven Thomas | bb37432 | 2020-04-28 22:47:16 -0700 | [diff] [blame] | 1657 | TEST_F(RefreshRateConfigsTest, idle) { |
| 1658 | auto refreshRateConfigs = |
| 1659 | std::make_unique<RefreshRateConfigs>(m60_90Device, |
| 1660 | /*currentConfigId=*/HWC_CONFIG_ID_60); |
| 1661 | |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 1662 | std::vector<LayerRequirement> layers = {{.weight = 1.f}}; |
Steven Thomas | bb37432 | 2020-04-28 22:47:16 -0700 | [diff] [blame] | 1663 | layers[0].name = "Test layer"; |
| 1664 | |
Marin Shalamanov | 23c4420 | 2020-12-22 19:09:20 +0100 | [diff] [blame] | 1665 | const auto getIdleFrameRate = [&](LayerVoteType voteType, bool touchActive) -> DisplayModeId { |
Steven Thomas | bb37432 | 2020-04-28 22:47:16 -0700 | [diff] [blame] | 1666 | layers[0].vote = voteType; |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 1667 | layers[0].desiredRefreshRate = 90_Hz; |
Ady Abraham | dfd6216 | 2020-06-10 16:11:56 -0700 | [diff] [blame] | 1668 | RefreshRateConfigs::GlobalSignals consideredSignals; |
| 1669 | const auto configId = |
| 1670 | refreshRateConfigs |
| 1671 | ->getBestRefreshRate(layers, {.touch = touchActive, .idle = true}, |
| 1672 | &consideredSignals) |
Marin Shalamanov | a7fe304 | 2021-01-29 21:02:08 +0100 | [diff] [blame] | 1673 | .getModeId(); |
Ady Abraham | dfd6216 | 2020-06-10 16:11:56 -0700 | [diff] [blame] | 1674 | // Refresh rate will be chosen by either touch state or idle state |
| 1675 | EXPECT_EQ(!touchActive, consideredSignals.idle); |
| 1676 | return configId; |
Steven Thomas | bb37432 | 2020-04-28 22:47:16 -0700 | [diff] [blame] | 1677 | }; |
| 1678 | |
| 1679 | ASSERT_GE(refreshRateConfigs->setDisplayManagerPolicy( |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 1680 | {HWC_CONFIG_ID_60, {60_Hz, 90_Hz}, {60_Hz, 90_Hz}}), |
Steven Thomas | bb37432 | 2020-04-28 22:47:16 -0700 | [diff] [blame] | 1681 | 0); |
| 1682 | |
| 1683 | // Idle should be lower priority than touch boost. |
Ady Abraham | dfd6216 | 2020-06-10 16:11:56 -0700 | [diff] [blame] | 1684 | EXPECT_EQ(HWC_CONFIG_ID_90, getIdleFrameRate(LayerVoteType::NoVote, /*touchActive=*/true)); |
| 1685 | EXPECT_EQ(HWC_CONFIG_ID_90, getIdleFrameRate(LayerVoteType::Min, /*touchActive=*/true)); |
| 1686 | EXPECT_EQ(HWC_CONFIG_ID_90, getIdleFrameRate(LayerVoteType::Max, /*touchActive=*/true)); |
| 1687 | EXPECT_EQ(HWC_CONFIG_ID_90, getIdleFrameRate(LayerVoteType::Heuristic, /*touchActive=*/true)); |
| 1688 | EXPECT_EQ(HWC_CONFIG_ID_90, |
| 1689 | getIdleFrameRate(LayerVoteType::ExplicitDefault, /*touchActive=*/true)); |
| 1690 | EXPECT_EQ(HWC_CONFIG_ID_90, |
| 1691 | getIdleFrameRate(LayerVoteType::ExplicitExactOrMultiple, /*touchActive=*/true)); |
Steven Thomas | bb37432 | 2020-04-28 22:47:16 -0700 | [diff] [blame] | 1692 | |
| 1693 | // With no layers, idle should still be lower priority than touch boost. |
Steven Thomas | bb37432 | 2020-04-28 22:47:16 -0700 | [diff] [blame] | 1694 | EXPECT_EQ(HWC_CONFIG_ID_90, |
Ady Abraham | dfd6216 | 2020-06-10 16:11:56 -0700 | [diff] [blame] | 1695 | refreshRateConfigs->getBestRefreshRate({}, {.touch = true, .idle = true}) |
Marin Shalamanov | a7fe304 | 2021-01-29 21:02:08 +0100 | [diff] [blame] | 1696 | .getModeId()); |
Steven Thomas | bb37432 | 2020-04-28 22:47:16 -0700 | [diff] [blame] | 1697 | |
| 1698 | // Idle should be higher precedence than other layer frame rate considerations. |
Marin Shalamanov | a7fe304 | 2021-01-29 21:02:08 +0100 | [diff] [blame] | 1699 | refreshRateConfigs->setCurrentModeId(HWC_CONFIG_ID_90); |
Ady Abraham | dfd6216 | 2020-06-10 16:11:56 -0700 | [diff] [blame] | 1700 | EXPECT_EQ(HWC_CONFIG_ID_60, getIdleFrameRate(LayerVoteType::NoVote, /*touchActive=*/false)); |
| 1701 | EXPECT_EQ(HWC_CONFIG_ID_60, getIdleFrameRate(LayerVoteType::Min, /*touchActive=*/false)); |
| 1702 | EXPECT_EQ(HWC_CONFIG_ID_60, getIdleFrameRate(LayerVoteType::Max, /*touchActive=*/false)); |
| 1703 | EXPECT_EQ(HWC_CONFIG_ID_60, getIdleFrameRate(LayerVoteType::Heuristic, /*touchActive=*/false)); |
| 1704 | EXPECT_EQ(HWC_CONFIG_ID_60, |
| 1705 | getIdleFrameRate(LayerVoteType::ExplicitDefault, /*touchActive=*/false)); |
| 1706 | EXPECT_EQ(HWC_CONFIG_ID_60, |
| 1707 | getIdleFrameRate(LayerVoteType::ExplicitExactOrMultiple, /*touchActive=*/false)); |
Steven Thomas | bb37432 | 2020-04-28 22:47:16 -0700 | [diff] [blame] | 1708 | |
| 1709 | // Idle should be applied rather than the current config when there are no layers. |
| 1710 | EXPECT_EQ(HWC_CONFIG_ID_60, |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 1711 | refreshRateConfigs->getBestRefreshRate({}, {.idle = true}).getModeId()); |
Steven Thomas | bb37432 | 2020-04-28 22:47:16 -0700 | [diff] [blame] | 1712 | } |
| 1713 | |
Ady Abraham | b1b9d41 | 2020-06-01 19:53:52 -0700 | [diff] [blame] | 1714 | TEST_F(RefreshRateConfigsTest, findClosestKnownFrameRate) { |
| 1715 | auto refreshRateConfigs = |
| 1716 | std::make_unique<RefreshRateConfigs>(m60_90Device, |
| 1717 | /*currentConfigId=*/HWC_CONFIG_ID_60); |
| 1718 | |
| 1719 | for (float fps = 1.0f; fps <= 120.0f; fps += 0.1f) { |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 1720 | const auto knownFrameRate = |
| 1721 | findClosestKnownFrameRate(*refreshRateConfigs, Fps::fromValue(fps)); |
Marin Shalamanov | e8a663d | 2020-11-24 17:48:00 +0100 | [diff] [blame] | 1722 | Fps expectedFrameRate; |
Ady Abraham | b1b9d41 | 2020-06-01 19:53:52 -0700 | [diff] [blame] | 1723 | if (fps < 26.91f) { |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 1724 | expectedFrameRate = 24_Hz; |
Ady Abraham | b1b9d41 | 2020-06-01 19:53:52 -0700 | [diff] [blame] | 1725 | } else if (fps < 37.51f) { |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 1726 | expectedFrameRate = 30_Hz; |
Ady Abraham | b1b9d41 | 2020-06-01 19:53:52 -0700 | [diff] [blame] | 1727 | } else if (fps < 52.51f) { |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 1728 | expectedFrameRate = 45_Hz; |
Ady Abraham | b1b9d41 | 2020-06-01 19:53:52 -0700 | [diff] [blame] | 1729 | } else if (fps < 66.01f) { |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 1730 | expectedFrameRate = 60_Hz; |
Ady Abraham | b1b9d41 | 2020-06-01 19:53:52 -0700 | [diff] [blame] | 1731 | } else if (fps < 81.01f) { |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 1732 | expectedFrameRate = 72_Hz; |
Ady Abraham | b1b9d41 | 2020-06-01 19:53:52 -0700 | [diff] [blame] | 1733 | } else { |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 1734 | expectedFrameRate = 90_Hz; |
Ady Abraham | b1b9d41 | 2020-06-01 19:53:52 -0700 | [diff] [blame] | 1735 | } |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 1736 | EXPECT_EQ(expectedFrameRate, knownFrameRate); |
Ady Abraham | b1b9d41 | 2020-06-01 19:53:52 -0700 | [diff] [blame] | 1737 | } |
| 1738 | } |
| 1739 | |
| 1740 | TEST_F(RefreshRateConfigsTest, getBestRefreshRate_KnownFrameRate) { |
Ady Abraham | b1b9d41 | 2020-06-01 19:53:52 -0700 | [diff] [blame] | 1741 | auto refreshRateConfigs = |
| 1742 | std::make_unique<RefreshRateConfigs>(m60_90Device, |
| 1743 | /*currentConfigId=*/HWC_CONFIG_ID_60); |
| 1744 | |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 1745 | struct Expectation { |
| 1746 | Fps fps; |
| 1747 | const RefreshRate& refreshRate; |
Ady Abraham | b1b9d41 | 2020-06-01 19:53:52 -0700 | [diff] [blame] | 1748 | }; |
| 1749 | |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 1750 | const std::initializer_list<Expectation> knownFrameRatesExpectations = { |
| 1751 | {24_Hz, mExpected60Config}, {30_Hz, mExpected60Config}, {45_Hz, mExpected90Config}, |
| 1752 | {60_Hz, mExpected60Config}, {72_Hz, mExpected90Config}, {90_Hz, mExpected90Config}, |
Ady Abraham | b1b9d41 | 2020-06-01 19:53:52 -0700 | [diff] [blame] | 1753 | }; |
Ady Abraham | b1b9d41 | 2020-06-01 19:53:52 -0700 | [diff] [blame] | 1754 | |
| 1755 | // Make sure the test tests all the known frame rate |
| 1756 | const auto knownFrameRateList = getKnownFrameRate(*refreshRateConfigs); |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 1757 | const bool equal = std::equal(knownFrameRateList.begin(), knownFrameRateList.end(), |
| 1758 | knownFrameRatesExpectations.begin(), |
| 1759 | [](Fps fps, const Expectation& expected) { |
| 1760 | return isApproxEqual(fps, expected.fps); |
| 1761 | }); |
Ady Abraham | b1b9d41 | 2020-06-01 19:53:52 -0700 | [diff] [blame] | 1762 | EXPECT_TRUE(equal); |
| 1763 | |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 1764 | std::vector<LayerRequirement> layers = {{.weight = 1.f}}; |
Ady Abraham | b1b9d41 | 2020-06-01 19:53:52 -0700 | [diff] [blame] | 1765 | auto& layer = layers[0]; |
| 1766 | layer.vote = LayerVoteType::Heuristic; |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 1767 | |
| 1768 | for (const auto& [fps, refreshRate] : knownFrameRatesExpectations) { |
| 1769 | layer.desiredRefreshRate = fps; |
| 1770 | EXPECT_EQ(refreshRate, refreshRateConfigs->getBestRefreshRate(layers, {})); |
Ady Abraham | b1b9d41 | 2020-06-01 19:53:52 -0700 | [diff] [blame] | 1771 | } |
| 1772 | } |
| 1773 | |
Ady Abraham | dd5bfa9 | 2021-01-07 17:56:08 -0800 | [diff] [blame] | 1774 | TEST_F(RefreshRateConfigsTest, getBestRefreshRate_ExplicitExact) { |
| 1775 | auto refreshRateConfigs = |
| 1776 | std::make_unique<RefreshRateConfigs>(m30_60_72_90_120Device, |
| 1777 | /*currentConfigId=*/HWC_CONFIG_ID_60); |
| 1778 | |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 1779 | std::vector<LayerRequirement> layers = {{.weight = 1.f}, {.weight = 0.5f}}; |
Ady Abraham | dd5bfa9 | 2021-01-07 17:56:08 -0800 | [diff] [blame] | 1780 | auto& explicitExactLayer = layers[0]; |
| 1781 | auto& explicitExactOrMultipleLayer = layers[1]; |
| 1782 | |
| 1783 | explicitExactOrMultipleLayer.vote = LayerVoteType::ExplicitExactOrMultiple; |
| 1784 | explicitExactOrMultipleLayer.name = "ExplicitExactOrMultiple"; |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 1785 | explicitExactOrMultipleLayer.desiredRefreshRate = 60_Hz; |
Ady Abraham | dd5bfa9 | 2021-01-07 17:56:08 -0800 | [diff] [blame] | 1786 | |
| 1787 | explicitExactLayer.vote = LayerVoteType::ExplicitExact; |
| 1788 | explicitExactLayer.name = "ExplicitExact"; |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 1789 | explicitExactLayer.desiredRefreshRate = 30_Hz; |
Ady Abraham | dd5bfa9 | 2021-01-07 17:56:08 -0800 | [diff] [blame] | 1790 | |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 1791 | EXPECT_EQ(mExpected30Config, refreshRateConfigs->getBestRefreshRate(layers, {})); |
| 1792 | EXPECT_EQ(mExpected30Config, refreshRateConfigs->getBestRefreshRate(layers, {.touch = true})); |
Ady Abraham | dd5bfa9 | 2021-01-07 17:56:08 -0800 | [diff] [blame] | 1793 | |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 1794 | explicitExactOrMultipleLayer.desiredRefreshRate = 120_Hz; |
| 1795 | explicitExactLayer.desiredRefreshRate = 60_Hz; |
| 1796 | EXPECT_EQ(mExpected60Config, refreshRateConfigs->getBestRefreshRate(layers, {})); |
Ady Abraham | dd5bfa9 | 2021-01-07 17:56:08 -0800 | [diff] [blame] | 1797 | |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 1798 | explicitExactLayer.desiredRefreshRate = 72_Hz; |
| 1799 | EXPECT_EQ(mExpected72Config, refreshRateConfigs->getBestRefreshRate(layers, {})); |
Ady Abraham | dd5bfa9 | 2021-01-07 17:56:08 -0800 | [diff] [blame] | 1800 | |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 1801 | explicitExactLayer.desiredRefreshRate = 90_Hz; |
| 1802 | EXPECT_EQ(mExpected90Config, refreshRateConfigs->getBestRefreshRate(layers, {})); |
Ady Abraham | dd5bfa9 | 2021-01-07 17:56:08 -0800 | [diff] [blame] | 1803 | |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 1804 | explicitExactLayer.desiredRefreshRate = 120_Hz; |
| 1805 | EXPECT_EQ(mExpected120Config, refreshRateConfigs->getBestRefreshRate(layers, {})); |
Ady Abraham | dd5bfa9 | 2021-01-07 17:56:08 -0800 | [diff] [blame] | 1806 | } |
| 1807 | |
| 1808 | TEST_F(RefreshRateConfigsTest, getBestRefreshRate_ExplicitExactEnableFrameRateOverride) { |
rnlee | 3bd61066 | 2021-06-23 16:27:57 -0700 | [diff] [blame] | 1809 | RefreshRateConfigs::Config config = {.enableFrameRateOverride = true}; |
Ady Abraham | dd5bfa9 | 2021-01-07 17:56:08 -0800 | [diff] [blame] | 1810 | auto refreshRateConfigs = |
| 1811 | std::make_unique<RefreshRateConfigs>(m30_60_72_90_120Device, |
rnlee | 3bd61066 | 2021-06-23 16:27:57 -0700 | [diff] [blame] | 1812 | /*currentConfigId=*/HWC_CONFIG_ID_60, config); |
Ady Abraham | dd5bfa9 | 2021-01-07 17:56:08 -0800 | [diff] [blame] | 1813 | |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 1814 | std::vector<LayerRequirement> layers = {{.weight = 1.f}, {.weight = 0.5f}}; |
Ady Abraham | dd5bfa9 | 2021-01-07 17:56:08 -0800 | [diff] [blame] | 1815 | auto& explicitExactLayer = layers[0]; |
| 1816 | auto& explicitExactOrMultipleLayer = layers[1]; |
| 1817 | |
| 1818 | explicitExactOrMultipleLayer.vote = LayerVoteType::ExplicitExactOrMultiple; |
| 1819 | explicitExactOrMultipleLayer.name = "ExplicitExactOrMultiple"; |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 1820 | explicitExactOrMultipleLayer.desiredRefreshRate = 60_Hz; |
Ady Abraham | dd5bfa9 | 2021-01-07 17:56:08 -0800 | [diff] [blame] | 1821 | |
| 1822 | explicitExactLayer.vote = LayerVoteType::ExplicitExact; |
| 1823 | explicitExactLayer.name = "ExplicitExact"; |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 1824 | explicitExactLayer.desiredRefreshRate = 30_Hz; |
Ady Abraham | dd5bfa9 | 2021-01-07 17:56:08 -0800 | [diff] [blame] | 1825 | |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 1826 | EXPECT_EQ(mExpected60Config, refreshRateConfigs->getBestRefreshRate(layers, {})); |
| 1827 | EXPECT_EQ(mExpected120Config, refreshRateConfigs->getBestRefreshRate(layers, {.touch = true})); |
Ady Abraham | dd5bfa9 | 2021-01-07 17:56:08 -0800 | [diff] [blame] | 1828 | |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 1829 | explicitExactOrMultipleLayer.desiredRefreshRate = 120_Hz; |
| 1830 | explicitExactLayer.desiredRefreshRate = 60_Hz; |
| 1831 | EXPECT_EQ(mExpected120Config, refreshRateConfigs->getBestRefreshRate(layers, {})); |
Ady Abraham | dd5bfa9 | 2021-01-07 17:56:08 -0800 | [diff] [blame] | 1832 | |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 1833 | explicitExactLayer.desiredRefreshRate = 72_Hz; |
| 1834 | EXPECT_EQ(mExpected72Config, refreshRateConfigs->getBestRefreshRate(layers, {})); |
Ady Abraham | dd5bfa9 | 2021-01-07 17:56:08 -0800 | [diff] [blame] | 1835 | |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 1836 | explicitExactLayer.desiredRefreshRate = 90_Hz; |
| 1837 | EXPECT_EQ(mExpected90Config, refreshRateConfigs->getBestRefreshRate(layers, {})); |
Ady Abraham | dd5bfa9 | 2021-01-07 17:56:08 -0800 | [diff] [blame] | 1838 | |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 1839 | explicitExactLayer.desiredRefreshRate = 120_Hz; |
| 1840 | EXPECT_EQ(mExpected120Config, refreshRateConfigs->getBestRefreshRate(layers, {})); |
Ady Abraham | dd5bfa9 | 2021-01-07 17:56:08 -0800 | [diff] [blame] | 1841 | } |
| 1842 | |
Marin Shalamanov | 4c7831e | 2021-06-08 20:44:06 +0200 | [diff] [blame] | 1843 | TEST_F(RefreshRateConfigsTest, getBestRefreshRate_ReadsCached) { |
| 1844 | using GlobalSignals = RefreshRateConfigs::GlobalSignals; |
| 1845 | |
| 1846 | auto refreshRateConfigs = |
| 1847 | std::make_unique<RefreshRateConfigs>(m30_60_72_90_120Device, |
| 1848 | /*currentConfigId=*/HWC_CONFIG_ID_60); |
| 1849 | |
| 1850 | setLastBestRefreshRateInvocation(*refreshRateConfigs, |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 1851 | GetBestRefreshRateInvocation{.globalSignals = {.touch = true, |
Marin Shalamanov | 4c7831e | 2021-06-08 20:44:06 +0200 | [diff] [blame] | 1852 | .idle = true}, |
| 1853 | .outSignalsConsidered = |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 1854 | {.touch = true}, |
Marin Shalamanov | 4c7831e | 2021-06-08 20:44:06 +0200 | [diff] [blame] | 1855 | .resultingBestRefreshRate = |
| 1856 | createRefreshRate( |
| 1857 | mConfig90)}); |
| 1858 | |
| 1859 | EXPECT_EQ(createRefreshRate(mConfig90), |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 1860 | refreshRateConfigs->getBestRefreshRate({}, {.touch = true, .idle = true})); |
Marin Shalamanov | 4c7831e | 2021-06-08 20:44:06 +0200 | [diff] [blame] | 1861 | |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 1862 | const GlobalSignals cachedSignalsConsidered{.touch = true}; |
Marin Shalamanov | 4c7831e | 2021-06-08 20:44:06 +0200 | [diff] [blame] | 1863 | setLastBestRefreshRateInvocation(*refreshRateConfigs, |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 1864 | GetBestRefreshRateInvocation{.globalSignals = {.touch = true, |
Marin Shalamanov | 4c7831e | 2021-06-08 20:44:06 +0200 | [diff] [blame] | 1865 | .idle = true}, |
| 1866 | .outSignalsConsidered = |
| 1867 | cachedSignalsConsidered, |
| 1868 | .resultingBestRefreshRate = |
| 1869 | createRefreshRate( |
| 1870 | mConfig30)}); |
| 1871 | |
| 1872 | GlobalSignals signalsConsidered; |
| 1873 | EXPECT_EQ(createRefreshRate(mConfig30), |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 1874 | refreshRateConfigs->getBestRefreshRate({}, {.touch = true, .idle = true}, |
Marin Shalamanov | 4c7831e | 2021-06-08 20:44:06 +0200 | [diff] [blame] | 1875 | &signalsConsidered)); |
| 1876 | |
| 1877 | EXPECT_EQ(cachedSignalsConsidered, signalsConsidered); |
| 1878 | } |
| 1879 | |
| 1880 | TEST_F(RefreshRateConfigsTest, getBestRefreshRate_WritesCache) { |
| 1881 | using GlobalSignals = RefreshRateConfigs::GlobalSignals; |
| 1882 | |
| 1883 | auto refreshRateConfigs = |
| 1884 | std::make_unique<RefreshRateConfigs>(m30_60_72_90_120Device, |
| 1885 | /*currentConfigId=*/HWC_CONFIG_ID_60); |
| 1886 | ASSERT_FALSE(getLastBestRefreshRateInvocation(*refreshRateConfigs).has_value()); |
| 1887 | |
| 1888 | GlobalSignals globalSignals{.touch = true, .idle = true}; |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 1889 | std::vector<LayerRequirement> layers = {{.weight = 1.f}, {.weight = 0.5f}}; |
Marin Shalamanov | 4c7831e | 2021-06-08 20:44:06 +0200 | [diff] [blame] | 1890 | const auto lastResult = |
| 1891 | refreshRateConfigs->getBestRefreshRate(layers, globalSignals, |
| 1892 | /* outSignalsConsidered */ nullptr); |
| 1893 | |
| 1894 | const auto lastInvocation = getLastBestRefreshRateInvocation(*refreshRateConfigs); |
| 1895 | |
| 1896 | ASSERT_TRUE(lastInvocation.has_value()); |
| 1897 | ASSERT_EQ(layers, lastInvocation->layerRequirements); |
| 1898 | ASSERT_EQ(globalSignals, lastInvocation->globalSignals); |
| 1899 | ASSERT_EQ(lastResult, lastInvocation->resultingBestRefreshRate); |
| 1900 | |
| 1901 | // outSignalsConsidered needs to be populated even tho earlier we gave nullptr |
| 1902 | // to getBestRefreshRate() |
| 1903 | GlobalSignals detaultSignals; |
| 1904 | ASSERT_FALSE(detaultSignals == lastInvocation->outSignalsConsidered); |
| 1905 | } |
| 1906 | |
Ady Abraham | 5e4e983 | 2021-06-14 13:40:56 -0700 | [diff] [blame] | 1907 | TEST_F(RefreshRateConfigsTest, getBestRefreshRate_ExplicitExactTouchBoost) { |
rnlee | 3bd61066 | 2021-06-23 16:27:57 -0700 | [diff] [blame] | 1908 | RefreshRateConfigs::Config config = {.enableFrameRateOverride = true}; |
Ady Abraham | 5e4e983 | 2021-06-14 13:40:56 -0700 | [diff] [blame] | 1909 | auto refreshRateConfigs = |
| 1910 | std::make_unique<RefreshRateConfigs>(m60_120Device, |
rnlee | 3bd61066 | 2021-06-23 16:27:57 -0700 | [diff] [blame] | 1911 | /*currentConfigId=*/HWC_CONFIG_ID_60, config); |
Ady Abraham | 5e4e983 | 2021-06-14 13:40:56 -0700 | [diff] [blame] | 1912 | |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 1913 | std::vector<LayerRequirement> layers = {{.weight = 1.f}, {.weight = 0.5f}}; |
Ady Abraham | 5e4e983 | 2021-06-14 13:40:56 -0700 | [diff] [blame] | 1914 | auto& explicitExactLayer = layers[0]; |
| 1915 | auto& explicitExactOrMultipleLayer = layers[1]; |
| 1916 | |
| 1917 | explicitExactOrMultipleLayer.vote = LayerVoteType::ExplicitExactOrMultiple; |
| 1918 | explicitExactOrMultipleLayer.name = "ExplicitExactOrMultiple"; |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 1919 | explicitExactOrMultipleLayer.desiredRefreshRate = 60_Hz; |
Ady Abraham | 5e4e983 | 2021-06-14 13:40:56 -0700 | [diff] [blame] | 1920 | |
| 1921 | explicitExactLayer.vote = LayerVoteType::ExplicitExact; |
| 1922 | explicitExactLayer.name = "ExplicitExact"; |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 1923 | explicitExactLayer.desiredRefreshRate = 30_Hz; |
Ady Abraham | 5e4e983 | 2021-06-14 13:40:56 -0700 | [diff] [blame] | 1924 | |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 1925 | EXPECT_EQ(mExpected60Config, refreshRateConfigs->getBestRefreshRate(layers, {})); |
| 1926 | EXPECT_EQ(mExpected120Config, refreshRateConfigs->getBestRefreshRate(layers, {.touch = true})); |
Ady Abraham | 5e4e983 | 2021-06-14 13:40:56 -0700 | [diff] [blame] | 1927 | |
| 1928 | explicitExactOrMultipleLayer.vote = LayerVoteType::NoVote; |
| 1929 | |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 1930 | EXPECT_EQ(mExpected60Config, refreshRateConfigs->getBestRefreshRate(layers, {})); |
| 1931 | EXPECT_EQ(mExpected60Config, refreshRateConfigs->getBestRefreshRate(layers, {.touch = true})); |
Ady Abraham | 5e4e983 | 2021-06-14 13:40:56 -0700 | [diff] [blame] | 1932 | } |
| 1933 | |
Marin Shalamanov | 15a0fc6 | 2021-08-16 18:20:21 +0200 | [diff] [blame] | 1934 | TEST_F(RefreshRateConfigsTest, getBestRefreshRate_FractionalRefreshRates_ExactAndDefault) { |
| 1935 | RefreshRateConfigs::Config config = {.enableFrameRateOverride = true}; |
| 1936 | auto refreshRateConfigs = |
| 1937 | std::make_unique<RefreshRateConfigs>(m24_25_30_50_60WithFracDevice, |
| 1938 | /*currentConfigId=*/HWC_CONFIG_ID_60, config); |
| 1939 | |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 1940 | std::vector<LayerRequirement> layers = {{.weight = 0.5f}, {.weight = 0.5f}}; |
Marin Shalamanov | 15a0fc6 | 2021-08-16 18:20:21 +0200 | [diff] [blame] | 1941 | auto& explicitDefaultLayer = layers[0]; |
| 1942 | auto& explicitExactOrMultipleLayer = layers[1]; |
| 1943 | |
| 1944 | explicitExactOrMultipleLayer.vote = LayerVoteType::ExplicitExactOrMultiple; |
| 1945 | explicitExactOrMultipleLayer.name = "ExplicitExactOrMultiple"; |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 1946 | explicitExactOrMultipleLayer.desiredRefreshRate = 60_Hz; |
Marin Shalamanov | 15a0fc6 | 2021-08-16 18:20:21 +0200 | [diff] [blame] | 1947 | |
| 1948 | explicitDefaultLayer.vote = LayerVoteType::ExplicitDefault; |
| 1949 | explicitDefaultLayer.name = "ExplicitDefault"; |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 1950 | explicitDefaultLayer.desiredRefreshRate = 59.94_Hz; |
Marin Shalamanov | 15a0fc6 | 2021-08-16 18:20:21 +0200 | [diff] [blame] | 1951 | |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 1952 | EXPECT_EQ(mExpected60Config, refreshRateConfigs->getBestRefreshRate(layers, {})); |
Marin Shalamanov | 15a0fc6 | 2021-08-16 18:20:21 +0200 | [diff] [blame] | 1953 | } |
| 1954 | |
Ady Abraham | 05243be | 2021-09-16 15:58:52 -0700 | [diff] [blame] | 1955 | // b/190578904 |
| 1956 | TEST_F(RefreshRateConfigsTest, getBestRefreshRate_deviceWithCloseRefreshRates) { |
| 1957 | constexpr int kMinRefreshRate = 10; |
| 1958 | constexpr int kMaxRefreshRate = 240; |
| 1959 | |
| 1960 | DisplayModes displayModes; |
| 1961 | for (int fps = kMinRefreshRate; fps < kMaxRefreshRate; fps++) { |
| 1962 | constexpr int32_t kGroup = 0; |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 1963 | const auto refreshRate = Fps::fromValue(static_cast<float>(fps)); |
Ady Abraham | 05243be | 2021-09-16 15:58:52 -0700 | [diff] [blame] | 1964 | displayModes.push_back( |
| 1965 | createDisplayMode(DisplayModeId(fps), kGroup, refreshRate.getPeriodNsecs())); |
| 1966 | } |
| 1967 | |
Ady Abraham | 05243be | 2021-09-16 15:58:52 -0700 | [diff] [blame] | 1968 | auto refreshRateConfigs = |
| 1969 | std::make_unique<RefreshRateConfigs>(displayModes, |
| 1970 | /*currentConfigId=*/displayModes[0]->getId()); |
| 1971 | |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 1972 | std::vector<LayerRequirement> layers = {{.weight = 1.f}}; |
Ady Abraham | 05243be | 2021-09-16 15:58:52 -0700 | [diff] [blame] | 1973 | const auto testRefreshRate = [&](Fps fps, LayerVoteType vote) { |
| 1974 | layers[0].desiredRefreshRate = fps; |
| 1975 | layers[0].vote = vote; |
| 1976 | EXPECT_EQ(fps.getIntValue(), |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 1977 | refreshRateConfigs->getBestRefreshRate(layers, {}).getFps().getIntValue()) |
Ady Abraham | 05243be | 2021-09-16 15:58:52 -0700 | [diff] [blame] | 1978 | << "Failed for " << RefreshRateConfigs::layerVoteTypeString(vote); |
| 1979 | }; |
| 1980 | |
| 1981 | for (int fps = kMinRefreshRate; fps < kMaxRefreshRate; fps++) { |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 1982 | const auto refreshRate = Fps::fromValue(static_cast<float>(fps)); |
Ady Abraham | 05243be | 2021-09-16 15:58:52 -0700 | [diff] [blame] | 1983 | testRefreshRate(refreshRate, LayerVoteType::Heuristic); |
| 1984 | testRefreshRate(refreshRate, LayerVoteType::ExplicitDefault); |
| 1985 | testRefreshRate(refreshRate, LayerVoteType::ExplicitExactOrMultiple); |
| 1986 | testRefreshRate(refreshRate, LayerVoteType::ExplicitExact); |
| 1987 | } |
| 1988 | } |
| 1989 | |
Ana Krulec | b9afd79 | 2020-06-11 13:16:15 -0700 | [diff] [blame] | 1990 | TEST_F(RefreshRateConfigsTest, testComparisonOperator) { |
| 1991 | EXPECT_TRUE(mExpected60Config < mExpected90Config); |
| 1992 | EXPECT_FALSE(mExpected60Config < mExpected60Config); |
| 1993 | EXPECT_FALSE(mExpected90Config < mExpected90Config); |
| 1994 | } |
| 1995 | |
| 1996 | TEST_F(RefreshRateConfigsTest, testKernelIdleTimerAction) { |
| 1997 | using KernelIdleTimerAction = scheduler::RefreshRateConfigs::KernelIdleTimerAction; |
| 1998 | |
| 1999 | auto refreshRateConfigs = |
| 2000 | std::make_unique<RefreshRateConfigs>(m60_90Device, |
| 2001 | /*currentConfigId=*/HWC_CONFIG_ID_90); |
| 2002 | // SetPolicy(60, 90), current 90Hz => TurnOn. |
| 2003 | EXPECT_EQ(KernelIdleTimerAction::TurnOn, refreshRateConfigs->getIdleTimerAction()); |
| 2004 | |
| 2005 | // SetPolicy(60, 90), current 60Hz => TurnOn. |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 2006 | ASSERT_GE(refreshRateConfigs->setDisplayManagerPolicy({HWC_CONFIG_ID_60, {60_Hz, 90_Hz}}), 0); |
Ana Krulec | b9afd79 | 2020-06-11 13:16:15 -0700 | [diff] [blame] | 2007 | EXPECT_EQ(KernelIdleTimerAction::TurnOn, refreshRateConfigs->getIdleTimerAction()); |
| 2008 | |
Ady Abraham | a91605e | 2021-06-18 11:41:47 -0700 | [diff] [blame] | 2009 | // SetPolicy(60, 60), current 60Hz => TurnOff |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 2010 | ASSERT_GE(refreshRateConfigs->setDisplayManagerPolicy({HWC_CONFIG_ID_60, {60_Hz, 60_Hz}}), 0); |
Ady Abraham | a91605e | 2021-06-18 11:41:47 -0700 | [diff] [blame] | 2011 | EXPECT_EQ(KernelIdleTimerAction::TurnOff, refreshRateConfigs->getIdleTimerAction()); |
Ana Krulec | b9afd79 | 2020-06-11 13:16:15 -0700 | [diff] [blame] | 2012 | |
| 2013 | // SetPolicy(90, 90), current 90Hz => TurnOff. |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 2014 | ASSERT_GE(refreshRateConfigs->setDisplayManagerPolicy({HWC_CONFIG_ID_90, {90_Hz, 90_Hz}}), 0); |
Ana Krulec | b9afd79 | 2020-06-11 13:16:15 -0700 | [diff] [blame] | 2015 | EXPECT_EQ(KernelIdleTimerAction::TurnOff, refreshRateConfigs->getIdleTimerAction()); |
| 2016 | } |
| 2017 | |
TreeHugger Robot | 758ab61 | 2021-06-22 19:17:29 +0000 | [diff] [blame] | 2018 | TEST_F(RefreshRateConfigsTest, testKernelIdleTimerActionFor120Hz) { |
| 2019 | using KernelIdleTimerAction = scheduler::RefreshRateConfigs::KernelIdleTimerAction; |
| 2020 | |
| 2021 | // Tests with 120Hz |
| 2022 | auto refreshRateConfigs = |
| 2023 | std::make_unique<RefreshRateConfigs>(m60_120Device, |
| 2024 | /*currentConfigId=*/HWC_CONFIG_ID_120); |
| 2025 | // SetPolicy(0, 60), current 60Hz => TurnOn. |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 2026 | ASSERT_GE(refreshRateConfigs->setDisplayManagerPolicy({HWC_CONFIG_ID_60, {0_Hz, 60_Hz}}), 0); |
TreeHugger Robot | 758ab61 | 2021-06-22 19:17:29 +0000 | [diff] [blame] | 2027 | EXPECT_EQ(KernelIdleTimerAction::TurnOn, refreshRateConfigs->getIdleTimerAction()); |
| 2028 | |
| 2029 | // SetPolicy(60, 60), current 60Hz => TurnOff. |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 2030 | ASSERT_GE(refreshRateConfigs->setDisplayManagerPolicy({HWC_CONFIG_ID_60, {60_Hz, 60_Hz}}), 0); |
TreeHugger Robot | 758ab61 | 2021-06-22 19:17:29 +0000 | [diff] [blame] | 2031 | EXPECT_EQ(KernelIdleTimerAction::TurnOff, refreshRateConfigs->getIdleTimerAction()); |
| 2032 | |
| 2033 | // SetPolicy(60, 120), current 60Hz => TurnOn. |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 2034 | ASSERT_GE(refreshRateConfigs->setDisplayManagerPolicy({HWC_CONFIG_ID_60, {60_Hz, 120_Hz}}), 0); |
TreeHugger Robot | 758ab61 | 2021-06-22 19:17:29 +0000 | [diff] [blame] | 2035 | EXPECT_EQ(KernelIdleTimerAction::TurnOn, refreshRateConfigs->getIdleTimerAction()); |
| 2036 | |
| 2037 | // SetPolicy(120, 120), current 120Hz => TurnOff. |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 2038 | ASSERT_GE(refreshRateConfigs->setDisplayManagerPolicy({HWC_CONFIG_ID_120, {120_Hz, 120_Hz}}), |
TreeHugger Robot | 758ab61 | 2021-06-22 19:17:29 +0000 | [diff] [blame] | 2039 | 0); |
| 2040 | EXPECT_EQ(KernelIdleTimerAction::TurnOff, refreshRateConfigs->getIdleTimerAction()); |
| 2041 | } |
| 2042 | |
Ady Abraham | 5cc2e26 | 2021-03-25 13:09:17 -0700 | [diff] [blame] | 2043 | TEST_F(RefreshRateConfigsTest, getFrameRateDivider) { |
Ady Abraham | 0bb6a47 | 2020-10-12 10:22:13 -0700 | [diff] [blame] | 2044 | auto refreshRateConfigs = |
| 2045 | std::make_unique<RefreshRateConfigs>(m30_60_72_90_120Device, |
| 2046 | /*currentConfigId=*/HWC_CONFIG_ID_30); |
Ady Abraham | 62a0be2 | 2020-12-08 16:54:10 -0800 | [diff] [blame] | 2047 | |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 2048 | const auto frameRate = 30_Hz; |
Ady Abraham | 5cc2e26 | 2021-03-25 13:09:17 -0700 | [diff] [blame] | 2049 | Fps displayRefreshRate = refreshRateConfigs->getCurrentRefreshRate().getFps(); |
| 2050 | EXPECT_EQ(1, RefreshRateConfigs::getFrameRateDivider(displayRefreshRate, frameRate)); |
Ady Abraham | 0bb6a47 | 2020-10-12 10:22:13 -0700 | [diff] [blame] | 2051 | |
Marin Shalamanov | a7fe304 | 2021-01-29 21:02:08 +0100 | [diff] [blame] | 2052 | refreshRateConfigs->setCurrentModeId(HWC_CONFIG_ID_60); |
Ady Abraham | 5cc2e26 | 2021-03-25 13:09:17 -0700 | [diff] [blame] | 2053 | displayRefreshRate = refreshRateConfigs->getCurrentRefreshRate().getFps(); |
| 2054 | EXPECT_EQ(2, RefreshRateConfigs::getFrameRateDivider(displayRefreshRate, frameRate)); |
Ady Abraham | 0bb6a47 | 2020-10-12 10:22:13 -0700 | [diff] [blame] | 2055 | |
Marin Shalamanov | a7fe304 | 2021-01-29 21:02:08 +0100 | [diff] [blame] | 2056 | refreshRateConfigs->setCurrentModeId(HWC_CONFIG_ID_72); |
Ady Abraham | 5cc2e26 | 2021-03-25 13:09:17 -0700 | [diff] [blame] | 2057 | displayRefreshRate = refreshRateConfigs->getCurrentRefreshRate().getFps(); |
| 2058 | EXPECT_EQ(0, RefreshRateConfigs::getFrameRateDivider(displayRefreshRate, frameRate)); |
Ady Abraham | 0bb6a47 | 2020-10-12 10:22:13 -0700 | [diff] [blame] | 2059 | |
Marin Shalamanov | a7fe304 | 2021-01-29 21:02:08 +0100 | [diff] [blame] | 2060 | refreshRateConfigs->setCurrentModeId(HWC_CONFIG_ID_90); |
Ady Abraham | 5cc2e26 | 2021-03-25 13:09:17 -0700 | [diff] [blame] | 2061 | displayRefreshRate = refreshRateConfigs->getCurrentRefreshRate().getFps(); |
| 2062 | EXPECT_EQ(3, RefreshRateConfigs::getFrameRateDivider(displayRefreshRate, frameRate)); |
Ady Abraham | 0bb6a47 | 2020-10-12 10:22:13 -0700 | [diff] [blame] | 2063 | |
Marin Shalamanov | a7fe304 | 2021-01-29 21:02:08 +0100 | [diff] [blame] | 2064 | refreshRateConfigs->setCurrentModeId(HWC_CONFIG_ID_120); |
Ady Abraham | 5cc2e26 | 2021-03-25 13:09:17 -0700 | [diff] [blame] | 2065 | displayRefreshRate = refreshRateConfigs->getCurrentRefreshRate().getFps(); |
| 2066 | EXPECT_EQ(4, RefreshRateConfigs::getFrameRateDivider(displayRefreshRate, frameRate)); |
Ady Abraham | 62f216c | 2020-10-13 19:07:23 -0700 | [diff] [blame] | 2067 | |
Marin Shalamanov | a7fe304 | 2021-01-29 21:02:08 +0100 | [diff] [blame] | 2068 | refreshRateConfigs->setCurrentModeId(HWC_CONFIG_ID_90); |
Ady Abraham | 5cc2e26 | 2021-03-25 13:09:17 -0700 | [diff] [blame] | 2069 | displayRefreshRate = refreshRateConfigs->getCurrentRefreshRate().getFps(); |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 2070 | EXPECT_EQ(4, RefreshRateConfigs::getFrameRateDivider(displayRefreshRate, 22.5_Hz)); |
Marin Shalamanov | 15a0fc6 | 2021-08-16 18:20:21 +0200 | [diff] [blame] | 2071 | |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 2072 | EXPECT_EQ(0, RefreshRateConfigs::getFrameRateDivider(24_Hz, 25_Hz)); |
| 2073 | EXPECT_EQ(0, RefreshRateConfigs::getFrameRateDivider(24_Hz, 23.976_Hz)); |
| 2074 | EXPECT_EQ(0, RefreshRateConfigs::getFrameRateDivider(30_Hz, 29.97_Hz)); |
| 2075 | EXPECT_EQ(0, RefreshRateConfigs::getFrameRateDivider(60_Hz, 59.94_Hz)); |
Marin Shalamanov | 15a0fc6 | 2021-08-16 18:20:21 +0200 | [diff] [blame] | 2076 | } |
| 2077 | |
| 2078 | TEST_F(RefreshRateConfigsTest, isFractionalPairOrMultiple) { |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 2079 | EXPECT_TRUE(RefreshRateConfigs::isFractionalPairOrMultiple(23.976_Hz, 24_Hz)); |
| 2080 | EXPECT_TRUE(RefreshRateConfigs::isFractionalPairOrMultiple(24_Hz, 23.976_Hz)); |
Marin Shalamanov | 15a0fc6 | 2021-08-16 18:20:21 +0200 | [diff] [blame] | 2081 | |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 2082 | EXPECT_TRUE(RefreshRateConfigs::isFractionalPairOrMultiple(29.97_Hz, 30_Hz)); |
| 2083 | EXPECT_TRUE(RefreshRateConfigs::isFractionalPairOrMultiple(30_Hz, 29.97_Hz)); |
Marin Shalamanov | 15a0fc6 | 2021-08-16 18:20:21 +0200 | [diff] [blame] | 2084 | |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 2085 | EXPECT_TRUE(RefreshRateConfigs::isFractionalPairOrMultiple(59.94_Hz, 60_Hz)); |
| 2086 | EXPECT_TRUE(RefreshRateConfigs::isFractionalPairOrMultiple(60_Hz, 59.94_Hz)); |
Marin Shalamanov | 15a0fc6 | 2021-08-16 18:20:21 +0200 | [diff] [blame] | 2087 | |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 2088 | EXPECT_TRUE(RefreshRateConfigs::isFractionalPairOrMultiple(29.97_Hz, 60_Hz)); |
| 2089 | EXPECT_TRUE(RefreshRateConfigs::isFractionalPairOrMultiple(60_Hz, 29.97_Hz)); |
Marin Shalamanov | 15a0fc6 | 2021-08-16 18:20:21 +0200 | [diff] [blame] | 2090 | |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 2091 | EXPECT_TRUE(RefreshRateConfigs::isFractionalPairOrMultiple(59.94_Hz, 30_Hz)); |
| 2092 | EXPECT_TRUE(RefreshRateConfigs::isFractionalPairOrMultiple(30_Hz, 59.94_Hz)); |
Marin Shalamanov | 15a0fc6 | 2021-08-16 18:20:21 +0200 | [diff] [blame] | 2093 | |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 2094 | const auto refreshRates = {23.976_Hz, 24_Hz, 25_Hz, 29.97_Hz, 30_Hz, 50_Hz, 59.94_Hz, 60_Hz}; |
Marin Shalamanov | 15a0fc6 | 2021-08-16 18:20:21 +0200 | [diff] [blame] | 2095 | for (auto refreshRate : refreshRates) { |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 2096 | EXPECT_FALSE(RefreshRateConfigs::isFractionalPairOrMultiple(refreshRate, refreshRate)); |
Marin Shalamanov | 15a0fc6 | 2021-08-16 18:20:21 +0200 | [diff] [blame] | 2097 | } |
| 2098 | |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 2099 | EXPECT_FALSE(RefreshRateConfigs::isFractionalPairOrMultiple(24_Hz, 25_Hz)); |
| 2100 | EXPECT_FALSE(RefreshRateConfigs::isFractionalPairOrMultiple(23.978_Hz, 25_Hz)); |
| 2101 | EXPECT_FALSE(RefreshRateConfigs::isFractionalPairOrMultiple(29.97_Hz, 59.94_Hz)); |
Ady Abraham | 62a0be2 | 2020-12-08 16:54:10 -0800 | [diff] [blame] | 2102 | } |
| 2103 | |
Ady Abraham | dd5bfa9 | 2021-01-07 17:56:08 -0800 | [diff] [blame] | 2104 | TEST_F(RefreshRateConfigsTest, getFrameRateOverrides_noLayers) { |
Ady Abraham | 62a0be2 | 2020-12-08 16:54:10 -0800 | [diff] [blame] | 2105 | auto refreshRateConfigs = |
| 2106 | std::make_unique<RefreshRateConfigs>(m30_60_72_90_120Device, /*currentConfigId=*/ |
| 2107 | HWC_CONFIG_ID_120); |
| 2108 | |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 2109 | ASSERT_TRUE(refreshRateConfigs->getFrameRateOverrides({}, 120_Hz, {}).empty()); |
Ady Abraham | 62a0be2 | 2020-12-08 16:54:10 -0800 | [diff] [blame] | 2110 | } |
| 2111 | |
| 2112 | TEST_F(RefreshRateConfigsTest, getFrameRateOverrides_60on120) { |
rnlee | 3bd61066 | 2021-06-23 16:27:57 -0700 | [diff] [blame] | 2113 | RefreshRateConfigs::Config config = {.enableFrameRateOverride = true}; |
Ady Abraham | 62a0be2 | 2020-12-08 16:54:10 -0800 | [diff] [blame] | 2114 | auto refreshRateConfigs = |
| 2115 | std::make_unique<RefreshRateConfigs>(m30_60_72_90_120Device, /*currentConfigId=*/ |
rnlee | 3bd61066 | 2021-06-23 16:27:57 -0700 | [diff] [blame] | 2116 | HWC_CONFIG_ID_120, config); |
Ady Abraham | 62a0be2 | 2020-12-08 16:54:10 -0800 | [diff] [blame] | 2117 | |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 2118 | std::vector<LayerRequirement> layers = {{.weight = 1.f}}; |
Ady Abraham | 62a0be2 | 2020-12-08 16:54:10 -0800 | [diff] [blame] | 2119 | layers[0].name = "Test layer"; |
| 2120 | layers[0].ownerUid = 1234; |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 2121 | layers[0].desiredRefreshRate = 60_Hz; |
Ady Abraham | 62a0be2 | 2020-12-08 16:54:10 -0800 | [diff] [blame] | 2122 | layers[0].vote = LayerVoteType::ExplicitDefault; |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 2123 | auto frameRateOverrides = refreshRateConfigs->getFrameRateOverrides(layers, 120_Hz, {}); |
Ady Abraham | 62a0be2 | 2020-12-08 16:54:10 -0800 | [diff] [blame] | 2124 | ASSERT_EQ(1, frameRateOverrides.size()); |
| 2125 | ASSERT_EQ(1, frameRateOverrides.count(1234)); |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 2126 | ASSERT_EQ(60_Hz, frameRateOverrides.at(1234)); |
Ady Abraham | 62a0be2 | 2020-12-08 16:54:10 -0800 | [diff] [blame] | 2127 | |
| 2128 | layers[0].vote = LayerVoteType::ExplicitExactOrMultiple; |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 2129 | frameRateOverrides = refreshRateConfigs->getFrameRateOverrides(layers, 120_Hz, {}); |
Ady Abraham | 62a0be2 | 2020-12-08 16:54:10 -0800 | [diff] [blame] | 2130 | ASSERT_EQ(1, frameRateOverrides.size()); |
| 2131 | ASSERT_EQ(1, frameRateOverrides.count(1234)); |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 2132 | ASSERT_EQ(60_Hz, frameRateOverrides.at(1234)); |
Ady Abraham | 62a0be2 | 2020-12-08 16:54:10 -0800 | [diff] [blame] | 2133 | |
| 2134 | layers[0].vote = LayerVoteType::NoVote; |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 2135 | frameRateOverrides = refreshRateConfigs->getFrameRateOverrides(layers, 120_Hz, {}); |
Ady Abraham | 62a0be2 | 2020-12-08 16:54:10 -0800 | [diff] [blame] | 2136 | ASSERT_TRUE(frameRateOverrides.empty()); |
| 2137 | |
| 2138 | layers[0].vote = LayerVoteType::Min; |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 2139 | frameRateOverrides = refreshRateConfigs->getFrameRateOverrides(layers, 120_Hz, {}); |
Ady Abraham | 62a0be2 | 2020-12-08 16:54:10 -0800 | [diff] [blame] | 2140 | ASSERT_TRUE(frameRateOverrides.empty()); |
| 2141 | |
| 2142 | layers[0].vote = LayerVoteType::Max; |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 2143 | frameRateOverrides = refreshRateConfigs->getFrameRateOverrides(layers, 120_Hz, {}); |
Ady Abraham | 62a0be2 | 2020-12-08 16:54:10 -0800 | [diff] [blame] | 2144 | ASSERT_TRUE(frameRateOverrides.empty()); |
| 2145 | |
| 2146 | layers[0].vote = LayerVoteType::Heuristic; |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 2147 | frameRateOverrides = refreshRateConfigs->getFrameRateOverrides(layers, 120_Hz, {}); |
Ady Abraham | 62a0be2 | 2020-12-08 16:54:10 -0800 | [diff] [blame] | 2148 | ASSERT_TRUE(frameRateOverrides.empty()); |
| 2149 | } |
| 2150 | |
Ady Abraham | dd5bfa9 | 2021-01-07 17:56:08 -0800 | [diff] [blame] | 2151 | TEST_F(RefreshRateConfigsTest, getFrameRateOverrides_twoUids) { |
rnlee | 3bd61066 | 2021-06-23 16:27:57 -0700 | [diff] [blame] | 2152 | RefreshRateConfigs::Config config = {.enableFrameRateOverride = true}; |
Ady Abraham | 62a0be2 | 2020-12-08 16:54:10 -0800 | [diff] [blame] | 2153 | auto refreshRateConfigs = |
| 2154 | std::make_unique<RefreshRateConfigs>(m30_60_72_90_120Device, /*currentConfigId=*/ |
rnlee | 3bd61066 | 2021-06-23 16:27:57 -0700 | [diff] [blame] | 2155 | HWC_CONFIG_ID_120, config); |
Ady Abraham | 62a0be2 | 2020-12-08 16:54:10 -0800 | [diff] [blame] | 2156 | |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 2157 | std::vector<LayerRequirement> layers = {{.ownerUid = 1234, .weight = 1.f}, |
| 2158 | {.ownerUid = 5678, .weight = 1.f}}; |
Ady Abraham | 62a0be2 | 2020-12-08 16:54:10 -0800 | [diff] [blame] | 2159 | |
| 2160 | layers[0].name = "Test layer 1234"; |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 2161 | layers[0].desiredRefreshRate = 60_Hz; |
Ady Abraham | 62a0be2 | 2020-12-08 16:54:10 -0800 | [diff] [blame] | 2162 | layers[0].vote = LayerVoteType::ExplicitDefault; |
| 2163 | |
| 2164 | layers[1].name = "Test layer 5678"; |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 2165 | layers[1].desiredRefreshRate = 30_Hz; |
Ady Abraham | 62a0be2 | 2020-12-08 16:54:10 -0800 | [diff] [blame] | 2166 | layers[1].vote = LayerVoteType::ExplicitDefault; |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 2167 | auto frameRateOverrides = refreshRateConfigs->getFrameRateOverrides(layers, 120_Hz, {}); |
Ady Abraham | 62a0be2 | 2020-12-08 16:54:10 -0800 | [diff] [blame] | 2168 | |
| 2169 | ASSERT_EQ(2, frameRateOverrides.size()); |
| 2170 | ASSERT_EQ(1, frameRateOverrides.count(1234)); |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 2171 | ASSERT_EQ(60_Hz, frameRateOverrides.at(1234)); |
Ady Abraham | 62a0be2 | 2020-12-08 16:54:10 -0800 | [diff] [blame] | 2172 | ASSERT_EQ(1, frameRateOverrides.count(5678)); |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 2173 | ASSERT_EQ(30_Hz, frameRateOverrides.at(5678)); |
Ady Abraham | 62a0be2 | 2020-12-08 16:54:10 -0800 | [diff] [blame] | 2174 | |
| 2175 | layers[1].vote = LayerVoteType::Heuristic; |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 2176 | frameRateOverrides = refreshRateConfigs->getFrameRateOverrides(layers, 120_Hz, {}); |
Ady Abraham | 62a0be2 | 2020-12-08 16:54:10 -0800 | [diff] [blame] | 2177 | ASSERT_EQ(1, frameRateOverrides.size()); |
| 2178 | ASSERT_EQ(1, frameRateOverrides.count(1234)); |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 2179 | ASSERT_EQ(60_Hz, frameRateOverrides.at(1234)); |
Ady Abraham | 62a0be2 | 2020-12-08 16:54:10 -0800 | [diff] [blame] | 2180 | |
| 2181 | layers[1].ownerUid = 1234; |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 2182 | frameRateOverrides = refreshRateConfigs->getFrameRateOverrides(layers, 120_Hz, {}); |
Ady Abraham | dd5bfa9 | 2021-01-07 17:56:08 -0800 | [diff] [blame] | 2183 | ASSERT_TRUE(frameRateOverrides.empty()); |
| 2184 | } |
| 2185 | |
| 2186 | TEST_F(RefreshRateConfigsTest, getFrameRateOverrides_touch) { |
rnlee | 3bd61066 | 2021-06-23 16:27:57 -0700 | [diff] [blame] | 2187 | RefreshRateConfigs::Config config = {.enableFrameRateOverride = true}; |
Ady Abraham | dd5bfa9 | 2021-01-07 17:56:08 -0800 | [diff] [blame] | 2188 | auto refreshRateConfigs = |
| 2189 | std::make_unique<RefreshRateConfigs>(m30_60_72_90_120Device, /*currentConfigId=*/ |
rnlee | 3bd61066 | 2021-06-23 16:27:57 -0700 | [diff] [blame] | 2190 | HWC_CONFIG_ID_120, config); |
Ady Abraham | dd5bfa9 | 2021-01-07 17:56:08 -0800 | [diff] [blame] | 2191 | |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 2192 | std::vector<LayerRequirement> layers = {{.ownerUid = 1234, .weight = 1.f}}; |
Ady Abraham | dd5bfa9 | 2021-01-07 17:56:08 -0800 | [diff] [blame] | 2193 | layers[0].name = "Test layer"; |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 2194 | layers[0].desiredRefreshRate = 60_Hz; |
Ady Abraham | dd5bfa9 | 2021-01-07 17:56:08 -0800 | [diff] [blame] | 2195 | layers[0].vote = LayerVoteType::ExplicitDefault; |
| 2196 | |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 2197 | auto frameRateOverrides = refreshRateConfigs->getFrameRateOverrides(layers, 120_Hz, {}); |
Ady Abraham | dd5bfa9 | 2021-01-07 17:56:08 -0800 | [diff] [blame] | 2198 | ASSERT_EQ(1, frameRateOverrides.size()); |
| 2199 | ASSERT_EQ(1, frameRateOverrides.count(1234)); |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 2200 | ASSERT_EQ(60_Hz, frameRateOverrides.at(1234)); |
Ady Abraham | dd5bfa9 | 2021-01-07 17:56:08 -0800 | [diff] [blame] | 2201 | |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 2202 | frameRateOverrides = refreshRateConfigs->getFrameRateOverrides(layers, 120_Hz, {.touch = true}); |
Ady Abraham | dd5bfa9 | 2021-01-07 17:56:08 -0800 | [diff] [blame] | 2203 | ASSERT_EQ(1, frameRateOverrides.size()); |
| 2204 | ASSERT_EQ(1, frameRateOverrides.count(1234)); |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 2205 | ASSERT_EQ(60_Hz, frameRateOverrides.at(1234)); |
Ady Abraham | dd5bfa9 | 2021-01-07 17:56:08 -0800 | [diff] [blame] | 2206 | |
| 2207 | layers[0].vote = LayerVoteType::ExplicitExact; |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 2208 | frameRateOverrides = refreshRateConfigs->getFrameRateOverrides(layers, 120_Hz, {}); |
Ady Abraham | dd5bfa9 | 2021-01-07 17:56:08 -0800 | [diff] [blame] | 2209 | ASSERT_EQ(1, frameRateOverrides.size()); |
| 2210 | ASSERT_EQ(1, frameRateOverrides.count(1234)); |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 2211 | ASSERT_EQ(60_Hz, frameRateOverrides.at(1234)); |
Ady Abraham | dd5bfa9 | 2021-01-07 17:56:08 -0800 | [diff] [blame] | 2212 | |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 2213 | frameRateOverrides = refreshRateConfigs->getFrameRateOverrides(layers, 120_Hz, {.touch = true}); |
Ady Abraham | dd5bfa9 | 2021-01-07 17:56:08 -0800 | [diff] [blame] | 2214 | ASSERT_EQ(1, frameRateOverrides.size()); |
| 2215 | ASSERT_EQ(1, frameRateOverrides.count(1234)); |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 2216 | ASSERT_EQ(60_Hz, frameRateOverrides.at(1234)); |
Ady Abraham | dd5bfa9 | 2021-01-07 17:56:08 -0800 | [diff] [blame] | 2217 | |
| 2218 | layers[0].vote = LayerVoteType::ExplicitExactOrMultiple; |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 2219 | frameRateOverrides = refreshRateConfigs->getFrameRateOverrides(layers, 120_Hz, {}); |
Ady Abraham | dd5bfa9 | 2021-01-07 17:56:08 -0800 | [diff] [blame] | 2220 | ASSERT_EQ(1, frameRateOverrides.size()); |
| 2221 | ASSERT_EQ(1, frameRateOverrides.count(1234)); |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 2222 | ASSERT_EQ(60_Hz, frameRateOverrides.at(1234)); |
Ady Abraham | dd5bfa9 | 2021-01-07 17:56:08 -0800 | [diff] [blame] | 2223 | |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 2224 | frameRateOverrides = refreshRateConfigs->getFrameRateOverrides(layers, 120_Hz, {.touch = true}); |
Ady Abraham | 62a0be2 | 2020-12-08 16:54:10 -0800 | [diff] [blame] | 2225 | ASSERT_TRUE(frameRateOverrides.empty()); |
Ady Abraham | 0bb6a47 | 2020-10-12 10:22:13 -0700 | [diff] [blame] | 2226 | } |
| 2227 | |
Alec Mouri | 0a1cc96 | 2019-03-14 12:33:02 -0700 | [diff] [blame] | 2228 | } // namespace |
Dominik Laskowski | 6eab42d | 2021-09-13 14:34:13 -0700 | [diff] [blame] | 2229 | } // namespace android::scheduler |
Marin Shalamanov | bed7fd3 | 2020-12-21 20:02:20 +0100 | [diff] [blame] | 2230 | |
| 2231 | // TODO(b/129481165): remove the #pragma below and fix conversion issues |
| 2232 | #pragma clang diagnostic pop // ignored "-Wextra" |