Ana Krulec | 4593b69 | 2019-01-11 22:07:25 -0800 | [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 | |
Ady Abraham | b0dbdaa | 2020-01-06 16:19:42 -0800 | [diff] [blame] | 17 | // TODO(b/129481165): remove the #pragma below and fix conversion issues |
| 18 | #pragma clang diagnostic push |
| 19 | #pragma clang diagnostic ignored "-Wconversion" |
| 20 | |
Ana Krulec | 4593b69 | 2019-01-11 22:07:25 -0800 | [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 | |
| 28 | #include "Scheduler/RefreshRateStats.h" |
Alec Mouri | fb571ea | 2019-01-24 18:42:10 -0800 | [diff] [blame] | 29 | #include "mock/MockTimeStats.h" |
Ana Krulec | 4593b69 | 2019-01-11 22:07:25 -0800 | [diff] [blame] | 30 | |
| 31 | using namespace std::chrono_literals; |
Alec Mouri | fb571ea | 2019-01-24 18:42:10 -0800 | [diff] [blame] | 32 | using testing::_; |
Alec Mouri | a2957ea | 2019-03-16 21:05:23 -0700 | [diff] [blame] | 33 | using testing::AtLeast; |
Ana Krulec | 4593b69 | 2019-01-11 22:07:25 -0800 | [diff] [blame] | 34 | |
| 35 | namespace android { |
| 36 | namespace scheduler { |
| 37 | |
| 38 | class RefreshRateStatsTest : public testing::Test { |
| 39 | protected: |
Ady Abraham | 2139f73 | 2019-11-13 18:56:40 -0800 | [diff] [blame] | 40 | static inline const auto CONFIG_ID_0 = HwcConfigIndexType(0); |
| 41 | static inline const auto CONFIG_ID_1 = HwcConfigIndexType(1); |
| 42 | static inline const auto CONFIG_GROUP_0 = HwcConfigGroupType(0); |
Ana Krulec | 4593b69 | 2019-01-11 22:07:25 -0800 | [diff] [blame] | 43 | static constexpr int64_t VSYNC_90 = 11111111; |
| 44 | static constexpr int64_t VSYNC_60 = 16666667; |
| 45 | |
| 46 | RefreshRateStatsTest(); |
| 47 | ~RefreshRateStatsTest(); |
| 48 | |
Steven Thomas | 2bbaabe | 2019-08-28 16:08:35 -0700 | [diff] [blame] | 49 | void init(const std::vector<RefreshRateConfigs::InputConfig>& configs) { |
| 50 | mRefreshRateConfigs = std::make_unique<RefreshRateConfigs>( |
Ady Abraham | 2139f73 | 2019-11-13 18:56:40 -0800 | [diff] [blame] | 51 | /*refreshRateSwitching=*/true, configs, /*currentConfig=*/CONFIG_ID_0); |
Steven Thomas | 2bbaabe | 2019-08-28 16:08:35 -0700 | [diff] [blame] | 52 | mRefreshRateStats = |
| 53 | std::make_unique<RefreshRateStats>(*mRefreshRateConfigs, mTimeStats, |
Ady Abraham | 2139f73 | 2019-11-13 18:56:40 -0800 | [diff] [blame] | 54 | /*currentConfigId=*/CONFIG_ID_0, |
Steven Thomas | 2bbaabe | 2019-08-28 16:08:35 -0700 | [diff] [blame] | 55 | /*currentPowerMode=*/HWC_POWER_MODE_OFF); |
| 56 | } |
| 57 | |
Dominik Laskowski | 6453651 | 2019-03-28 09:53:04 -0700 | [diff] [blame] | 58 | mock::TimeStats mTimeStats; |
Steven Thomas | 2bbaabe | 2019-08-28 16:08:35 -0700 | [diff] [blame] | 59 | std::unique_ptr<RefreshRateConfigs> mRefreshRateConfigs; |
| 60 | std::unique_ptr<RefreshRateStats> mRefreshRateStats; |
Ana Krulec | 4593b69 | 2019-01-11 22:07:25 -0800 | [diff] [blame] | 61 | }; |
| 62 | |
| 63 | RefreshRateStatsTest::RefreshRateStatsTest() { |
| 64 | const ::testing::TestInfo* const test_info = |
| 65 | ::testing::UnitTest::GetInstance()->current_test_info(); |
| 66 | ALOGD("**** Setting up for %s.%s\n", test_info->test_case_name(), test_info->name()); |
| 67 | } |
| 68 | |
| 69 | RefreshRateStatsTest::~RefreshRateStatsTest() { |
| 70 | const ::testing::TestInfo* const test_info = |
| 71 | ::testing::UnitTest::GetInstance()->current_test_info(); |
| 72 | ALOGD("**** Tearing down after %s.%s\n", test_info->test_case_name(), test_info->name()); |
| 73 | } |
| 74 | |
Ana Krulec | 4593b69 | 2019-01-11 22:07:25 -0800 | [diff] [blame] | 75 | namespace { |
| 76 | /* ------------------------------------------------------------------------ |
| 77 | * Test cases |
| 78 | */ |
Ana Krulec | 4593b69 | 2019-01-11 22:07:25 -0800 | [diff] [blame] | 79 | TEST_F(RefreshRateStatsTest, oneConfigTest) { |
Ady Abraham | 2139f73 | 2019-11-13 18:56:40 -0800 | [diff] [blame] | 80 | init({{{CONFIG_ID_0, CONFIG_GROUP_0, VSYNC_90}}}); |
Ana Krulec | 4593b69 | 2019-01-11 22:07:25 -0800 | [diff] [blame] | 81 | |
Dominik Laskowski | 6453651 | 2019-03-28 09:53:04 -0700 | [diff] [blame] | 82 | EXPECT_CALL(mTimeStats, recordRefreshRate(0, _)).Times(AtLeast(1)); |
| 83 | EXPECT_CALL(mTimeStats, recordRefreshRate(90, _)).Times(AtLeast(1)); |
Alec Mouri | fb571ea | 2019-01-24 18:42:10 -0800 | [diff] [blame] | 84 | |
Steven Thomas | 2bbaabe | 2019-08-28 16:08:35 -0700 | [diff] [blame] | 85 | std::unordered_map<std::string, int64_t> times = mRefreshRateStats->getTotalTimes(); |
| 86 | ASSERT_EQ(1, times.size()); |
Alec Mouri | a2957ea | 2019-03-16 21:05:23 -0700 | [diff] [blame] | 87 | EXPECT_NE(0u, times.count("ScreenOff")); |
Ana Krulec | 4593b69 | 2019-01-11 22:07:25 -0800 | [diff] [blame] | 88 | // Setting up tests on mobile harness can be flaky with time passing, so testing for |
| 89 | // exact time changes can result in flaxy numbers. To avoid that remember old |
| 90 | // numbers to make sure the correct values are increasing in the next test. |
| 91 | int screenOff = times["ScreenOff"]; |
Ana Krulec | 4593b69 | 2019-01-11 22:07:25 -0800 | [diff] [blame] | 92 | |
| 93 | // Screen is off by default. |
| 94 | std::this_thread::sleep_for(std::chrono::milliseconds(2)); |
Steven Thomas | 2bbaabe | 2019-08-28 16:08:35 -0700 | [diff] [blame] | 95 | times = mRefreshRateStats->getTotalTimes(); |
Alec Mouri | a2957ea | 2019-03-16 21:05:23 -0700 | [diff] [blame] | 96 | EXPECT_LT(screenOff, times["ScreenOff"]); |
Steven Thomas | 2bbaabe | 2019-08-28 16:08:35 -0700 | [diff] [blame] | 97 | EXPECT_EQ(0u, times.count("90fps")); |
Ana Krulec | 4593b69 | 2019-01-11 22:07:25 -0800 | [diff] [blame] | 98 | |
Ady Abraham | 2139f73 | 2019-11-13 18:56:40 -0800 | [diff] [blame] | 99 | mRefreshRateStats->setConfigMode(CONFIG_ID_0); |
Steven Thomas | 2bbaabe | 2019-08-28 16:08:35 -0700 | [diff] [blame] | 100 | mRefreshRateStats->setPowerMode(HWC_POWER_MODE_NORMAL); |
| 101 | screenOff = mRefreshRateStats->getTotalTimes()["ScreenOff"]; |
Ana Krulec | 4593b69 | 2019-01-11 22:07:25 -0800 | [diff] [blame] | 102 | std::this_thread::sleep_for(std::chrono::milliseconds(2)); |
Steven Thomas | 2bbaabe | 2019-08-28 16:08:35 -0700 | [diff] [blame] | 103 | times = mRefreshRateStats->getTotalTimes(); |
Alec Mouri | a2957ea | 2019-03-16 21:05:23 -0700 | [diff] [blame] | 104 | EXPECT_EQ(screenOff, times["ScreenOff"]); |
Steven Thomas | 2bbaabe | 2019-08-28 16:08:35 -0700 | [diff] [blame] | 105 | ASSERT_EQ(1u, times.count("90fps")); |
| 106 | EXPECT_LT(0, times["90fps"]); |
Ana Krulec | 4593b69 | 2019-01-11 22:07:25 -0800 | [diff] [blame] | 107 | |
Steven Thomas | 2bbaabe | 2019-08-28 16:08:35 -0700 | [diff] [blame] | 108 | mRefreshRateStats->setPowerMode(HWC_POWER_MODE_DOZE); |
| 109 | int ninety = mRefreshRateStats->getTotalTimes()["90fps"]; |
Ana Krulec | 4593b69 | 2019-01-11 22:07:25 -0800 | [diff] [blame] | 110 | std::this_thread::sleep_for(std::chrono::milliseconds(2)); |
Steven Thomas | 2bbaabe | 2019-08-28 16:08:35 -0700 | [diff] [blame] | 111 | times = mRefreshRateStats->getTotalTimes(); |
Alec Mouri | a2957ea | 2019-03-16 21:05:23 -0700 | [diff] [blame] | 112 | EXPECT_LT(screenOff, times["ScreenOff"]); |
| 113 | EXPECT_EQ(ninety, times["90fps"]); |
Ana Krulec | 4593b69 | 2019-01-11 22:07:25 -0800 | [diff] [blame] | 114 | |
Ady Abraham | 2139f73 | 2019-11-13 18:56:40 -0800 | [diff] [blame] | 115 | mRefreshRateStats->setConfigMode(CONFIG_ID_0); |
Steven Thomas | 2bbaabe | 2019-08-28 16:08:35 -0700 | [diff] [blame] | 116 | screenOff = mRefreshRateStats->getTotalTimes()["ScreenOff"]; |
Ana Krulec | 4593b69 | 2019-01-11 22:07:25 -0800 | [diff] [blame] | 117 | std::this_thread::sleep_for(std::chrono::milliseconds(2)); |
Steven Thomas | 2bbaabe | 2019-08-28 16:08:35 -0700 | [diff] [blame] | 118 | times = mRefreshRateStats->getTotalTimes(); |
Ana Krulec | 4593b69 | 2019-01-11 22:07:25 -0800 | [diff] [blame] | 119 | // Because the power mode is not HWC_POWER_MODE_NORMAL, switching the config |
| 120 | // does not update refresh rates that come from the config. |
Alec Mouri | a2957ea | 2019-03-16 21:05:23 -0700 | [diff] [blame] | 121 | EXPECT_LT(screenOff, times["ScreenOff"]); |
| 122 | EXPECT_EQ(ninety, times["90fps"]); |
Ana Krulec | 4593b69 | 2019-01-11 22:07:25 -0800 | [diff] [blame] | 123 | } |
| 124 | |
| 125 | TEST_F(RefreshRateStatsTest, twoConfigsTest) { |
Ady Abraham | 2139f73 | 2019-11-13 18:56:40 -0800 | [diff] [blame] | 126 | init({{{CONFIG_ID_0, CONFIG_GROUP_0, VSYNC_90}, {CONFIG_ID_1, CONFIG_GROUP_0, VSYNC_60}}}); |
Ana Krulec | 4593b69 | 2019-01-11 22:07:25 -0800 | [diff] [blame] | 127 | |
Dominik Laskowski | 6453651 | 2019-03-28 09:53:04 -0700 | [diff] [blame] | 128 | EXPECT_CALL(mTimeStats, recordRefreshRate(0, _)).Times(AtLeast(1)); |
| 129 | EXPECT_CALL(mTimeStats, recordRefreshRate(60, _)).Times(AtLeast(1)); |
| 130 | EXPECT_CALL(mTimeStats, recordRefreshRate(90, _)).Times(AtLeast(1)); |
Alec Mouri | fb571ea | 2019-01-24 18:42:10 -0800 | [diff] [blame] | 131 | |
Steven Thomas | 2bbaabe | 2019-08-28 16:08:35 -0700 | [diff] [blame] | 132 | std::unordered_map<std::string, int64_t> times = mRefreshRateStats->getTotalTimes(); |
| 133 | ASSERT_EQ(1, times.size()); |
Alec Mouri | a2957ea | 2019-03-16 21:05:23 -0700 | [diff] [blame] | 134 | EXPECT_NE(0u, times.count("ScreenOff")); |
Ana Krulec | 4593b69 | 2019-01-11 22:07:25 -0800 | [diff] [blame] | 135 | // Setting up tests on mobile harness can be flaky with time passing, so testing for |
| 136 | // exact time changes can result in flaxy numbers. To avoid that remember old |
| 137 | // numbers to make sure the correct values are increasing in the next test. |
| 138 | int screenOff = times["ScreenOff"]; |
Ana Krulec | 4593b69 | 2019-01-11 22:07:25 -0800 | [diff] [blame] | 139 | |
| 140 | // Screen is off by default. |
| 141 | std::this_thread::sleep_for(std::chrono::milliseconds(2)); |
Steven Thomas | 2bbaabe | 2019-08-28 16:08:35 -0700 | [diff] [blame] | 142 | times = mRefreshRateStats->getTotalTimes(); |
Alec Mouri | a2957ea | 2019-03-16 21:05:23 -0700 | [diff] [blame] | 143 | EXPECT_LT(screenOff, times["ScreenOff"]); |
Ana Krulec | 4593b69 | 2019-01-11 22:07:25 -0800 | [diff] [blame] | 144 | |
Ady Abraham | 2139f73 | 2019-11-13 18:56:40 -0800 | [diff] [blame] | 145 | mRefreshRateStats->setConfigMode(CONFIG_ID_0); |
Steven Thomas | 2bbaabe | 2019-08-28 16:08:35 -0700 | [diff] [blame] | 146 | mRefreshRateStats->setPowerMode(HWC_POWER_MODE_NORMAL); |
| 147 | screenOff = mRefreshRateStats->getTotalTimes()["ScreenOff"]; |
Ana Krulec | 4593b69 | 2019-01-11 22:07:25 -0800 | [diff] [blame] | 148 | std::this_thread::sleep_for(std::chrono::milliseconds(2)); |
Steven Thomas | 2bbaabe | 2019-08-28 16:08:35 -0700 | [diff] [blame] | 149 | times = mRefreshRateStats->getTotalTimes(); |
Alec Mouri | a2957ea | 2019-03-16 21:05:23 -0700 | [diff] [blame] | 150 | EXPECT_EQ(screenOff, times["ScreenOff"]); |
Steven Thomas | 2bbaabe | 2019-08-28 16:08:35 -0700 | [diff] [blame] | 151 | ASSERT_EQ(1u, times.count("90fps")); |
| 152 | EXPECT_LT(0, times["90fps"]); |
Ana Krulec | 4593b69 | 2019-01-11 22:07:25 -0800 | [diff] [blame] | 153 | |
| 154 | // When power mode is normal, time for configs updates. |
Ady Abraham | 2139f73 | 2019-11-13 18:56:40 -0800 | [diff] [blame] | 155 | mRefreshRateStats->setConfigMode(CONFIG_ID_1); |
Steven Thomas | 2bbaabe | 2019-08-28 16:08:35 -0700 | [diff] [blame] | 156 | int ninety = mRefreshRateStats->getTotalTimes()["90fps"]; |
Ana Krulec | 4593b69 | 2019-01-11 22:07:25 -0800 | [diff] [blame] | 157 | std::this_thread::sleep_for(std::chrono::milliseconds(2)); |
Steven Thomas | 2bbaabe | 2019-08-28 16:08:35 -0700 | [diff] [blame] | 158 | times = mRefreshRateStats->getTotalTimes(); |
Alec Mouri | a2957ea | 2019-03-16 21:05:23 -0700 | [diff] [blame] | 159 | EXPECT_EQ(screenOff, times["ScreenOff"]); |
| 160 | EXPECT_EQ(ninety, times["90fps"]); |
Steven Thomas | 2bbaabe | 2019-08-28 16:08:35 -0700 | [diff] [blame] | 161 | ASSERT_EQ(1u, times.count("60fps")); |
| 162 | EXPECT_LT(0, times["60fps"]); |
Ana Krulec | 4593b69 | 2019-01-11 22:07:25 -0800 | [diff] [blame] | 163 | |
Ady Abraham | 2139f73 | 2019-11-13 18:56:40 -0800 | [diff] [blame] | 164 | mRefreshRateStats->setConfigMode(CONFIG_ID_0); |
Steven Thomas | 2bbaabe | 2019-08-28 16:08:35 -0700 | [diff] [blame] | 165 | int sixty = mRefreshRateStats->getTotalTimes()["60fps"]; |
Ana Krulec | 4593b69 | 2019-01-11 22:07:25 -0800 | [diff] [blame] | 166 | std::this_thread::sleep_for(std::chrono::milliseconds(2)); |
Steven Thomas | 2bbaabe | 2019-08-28 16:08:35 -0700 | [diff] [blame] | 167 | times = mRefreshRateStats->getTotalTimes(); |
Alec Mouri | a2957ea | 2019-03-16 21:05:23 -0700 | [diff] [blame] | 168 | EXPECT_EQ(screenOff, times["ScreenOff"]); |
| 169 | EXPECT_LT(ninety, times["90fps"]); |
| 170 | EXPECT_EQ(sixty, times["60fps"]); |
Ana Krulec | 4593b69 | 2019-01-11 22:07:25 -0800 | [diff] [blame] | 171 | |
Ady Abraham | 2139f73 | 2019-11-13 18:56:40 -0800 | [diff] [blame] | 172 | mRefreshRateStats->setConfigMode(CONFIG_ID_1); |
Steven Thomas | 2bbaabe | 2019-08-28 16:08:35 -0700 | [diff] [blame] | 173 | ninety = mRefreshRateStats->getTotalTimes()["90fps"]; |
Ana Krulec | 4593b69 | 2019-01-11 22:07:25 -0800 | [diff] [blame] | 174 | std::this_thread::sleep_for(std::chrono::milliseconds(2)); |
Steven Thomas | 2bbaabe | 2019-08-28 16:08:35 -0700 | [diff] [blame] | 175 | times = mRefreshRateStats->getTotalTimes(); |
Alec Mouri | a2957ea | 2019-03-16 21:05:23 -0700 | [diff] [blame] | 176 | EXPECT_EQ(screenOff, times["ScreenOff"]); |
| 177 | EXPECT_EQ(ninety, times["90fps"]); |
| 178 | EXPECT_LT(sixty, times["60fps"]); |
Ana Krulec | 4593b69 | 2019-01-11 22:07:25 -0800 | [diff] [blame] | 179 | |
| 180 | // Because the power mode is not HWC_POWER_MODE_NORMAL, switching the config |
| 181 | // does not update refresh rates that come from the config. |
Steven Thomas | 2bbaabe | 2019-08-28 16:08:35 -0700 | [diff] [blame] | 182 | mRefreshRateStats->setPowerMode(HWC_POWER_MODE_DOZE); |
Ady Abraham | 2139f73 | 2019-11-13 18:56:40 -0800 | [diff] [blame] | 183 | mRefreshRateStats->setConfigMode(CONFIG_ID_0); |
Steven Thomas | 2bbaabe | 2019-08-28 16:08:35 -0700 | [diff] [blame] | 184 | sixty = mRefreshRateStats->getTotalTimes()["60fps"]; |
Ana Krulec | 4593b69 | 2019-01-11 22:07:25 -0800 | [diff] [blame] | 185 | std::this_thread::sleep_for(std::chrono::milliseconds(2)); |
Steven Thomas | 2bbaabe | 2019-08-28 16:08:35 -0700 | [diff] [blame] | 186 | times = mRefreshRateStats->getTotalTimes(); |
Alec Mouri | a2957ea | 2019-03-16 21:05:23 -0700 | [diff] [blame] | 187 | EXPECT_LT(screenOff, times["ScreenOff"]); |
| 188 | EXPECT_EQ(ninety, times["90fps"]); |
| 189 | EXPECT_EQ(sixty, times["60fps"]); |
Ana Krulec | 4593b69 | 2019-01-11 22:07:25 -0800 | [diff] [blame] | 190 | |
Ady Abraham | 2139f73 | 2019-11-13 18:56:40 -0800 | [diff] [blame] | 191 | mRefreshRateStats->setConfigMode(CONFIG_ID_1); |
Steven Thomas | 2bbaabe | 2019-08-28 16:08:35 -0700 | [diff] [blame] | 192 | screenOff = mRefreshRateStats->getTotalTimes()["ScreenOff"]; |
Ana Krulec | 4593b69 | 2019-01-11 22:07:25 -0800 | [diff] [blame] | 193 | std::this_thread::sleep_for(std::chrono::milliseconds(2)); |
Steven Thomas | 2bbaabe | 2019-08-28 16:08:35 -0700 | [diff] [blame] | 194 | times = mRefreshRateStats->getTotalTimes(); |
Alec Mouri | a2957ea | 2019-03-16 21:05:23 -0700 | [diff] [blame] | 195 | EXPECT_LT(screenOff, times["ScreenOff"]); |
| 196 | EXPECT_EQ(ninety, times["90fps"]); |
| 197 | EXPECT_EQ(sixty, times["60fps"]); |
Ana Krulec | 4593b69 | 2019-01-11 22:07:25 -0800 | [diff] [blame] | 198 | } |
| 199 | } // namespace |
| 200 | } // namespace scheduler |
| 201 | } // namespace android |
Ady Abraham | b0dbdaa | 2020-01-06 16:19:42 -0800 | [diff] [blame] | 202 | |
| 203 | // TODO(b/129481165): remove the #pragma below and fix conversion issues |
| 204 | #pragma clang diagnostic pop // ignored "-Wconversion" |