| Lloyd Pique | 4603f3c | 2020-02-11 12:06:56 -0800 | [diff] [blame] | 1 | /* | 
|  | 2 | * Copyright 2020 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 |  | 
|  | 17 | // TODO(b/129481165): remove the #pragma below and fix conversion issues | 
|  | 18 | #pragma clang diagnostic push | 
|  | 19 | #pragma clang diagnostic ignored "-Wconversion" | 
|  | 20 |  | 
|  | 21 | #undef LOG_TAG | 
|  | 22 | #define LOG_TAG "LibSurfaceFlingerUnittests" | 
|  | 23 |  | 
|  | 24 | #include <vector> | 
|  | 25 |  | 
| Dominik Laskowski | 8b01cc0 | 2020-07-14 19:02:41 -0700 | [diff] [blame] | 26 | // StrictMock<T> derives from T and is not marked final, so the destructor of T is expected to be | 
|  | 27 | // virtual in case StrictMock<T> is used as a polymorphic base class. That is not the case here. | 
|  | 28 | #pragma clang diagnostic push | 
|  | 29 | #pragma clang diagnostic ignored "-Wnon-virtual-dtor" | 
| Lloyd Pique | 4603f3c | 2020-02-11 12:06:56 -0800 | [diff] [blame] | 30 | #include <gmock/gmock.h> | 
| Dominik Laskowski | 8b01cc0 | 2020-07-14 19:02:41 -0700 | [diff] [blame] | 31 | #pragma clang diagnostic pop | 
|  | 32 |  | 
| Lloyd Pique | 4603f3c | 2020-02-11 12:06:56 -0800 | [diff] [blame] | 33 | #include <gui/LayerMetadata.h> | 
|  | 34 | #include <log/log.h> | 
|  | 35 |  | 
| Marin Shalamanov | 23c4420 | 2020-12-22 19:09:20 +0100 | [diff] [blame] | 36 | #include "DisplayHardware/DisplayMode.h" | 
| Lloyd Pique | 4603f3c | 2020-02-11 12:06:56 -0800 | [diff] [blame] | 37 | #include "DisplayHardware/HWComposer.h" | 
| Marin Shalamanov | 12c9e5a | 2021-01-07 00:25:35 +0100 | [diff] [blame] | 38 | #include "DisplayHardware/Hal.h" | 
| Alec Mouri | ff79387 | 2022-01-13 17:45:06 -0800 | [diff] [blame] | 39 | #include "DisplayIdentificationTestHelpers.h" | 
| Lloyd Pique | 4603f3c | 2020-02-11 12:06:56 -0800 | [diff] [blame] | 40 | #include "mock/DisplayHardware/MockComposer.h" | 
| Lloyd Pique | a516c00 | 2021-05-07 14:36:58 -0700 | [diff] [blame] | 41 | #include "mock/DisplayHardware/MockHWC2.h" | 
| Lloyd Pique | 4603f3c | 2020-02-11 12:06:56 -0800 | [diff] [blame] | 42 |  | 
|  | 43 | // TODO(b/129481165): remove the #pragma below and fix conversion issues | 
|  | 44 | #pragma clang diagnostic pop // ignored "-Wconversion" | 
|  | 45 |  | 
|  | 46 | namespace android { | 
|  | 47 | namespace { | 
|  | 48 |  | 
| Marin Shalamanov | 6e84017 | 2020-12-14 22:13:28 +0100 | [diff] [blame] | 49 | namespace V2_1 = hardware::graphics::composer::V2_1; | 
|  | 50 | namespace V2_4 = hardware::graphics::composer::V2_4; | 
| Ady Abraham | de549d4 | 2022-01-26 19:19:17 -0800 | [diff] [blame] | 51 | namespace aidl = aidl::android::hardware::graphics::composer3; | 
| Marin Shalamanov | 6e84017 | 2020-12-14 22:13:28 +0100 | [diff] [blame] | 52 |  | 
|  | 53 | using Hwc2::Config; | 
| Peiyong Lin | e9d809e | 2020-04-14 13:10:48 -0700 | [diff] [blame] | 54 |  | 
| Lloyd Pique | 4603f3c | 2020-02-11 12:06:56 -0800 | [diff] [blame] | 55 | using ::testing::_; | 
|  | 56 | using ::testing::DoAll; | 
|  | 57 | using ::testing::ElementsAreArray; | 
|  | 58 | using ::testing::Return; | 
|  | 59 | using ::testing::SetArgPointee; | 
|  | 60 | using ::testing::StrictMock; | 
|  | 61 |  | 
| Dominik Laskowski | e2c5b0a | 2022-08-10 14:53:53 -0700 | [diff] [blame] | 62 | struct HWComposerTest : testing::Test { | 
|  | 63 | using HalError = hardware::graphics::composer::V2_1::Error; | 
| Marin Shalamanov | 8b19659 | 2021-08-09 16:24:42 +0200 | [diff] [blame] | 64 |  | 
| Dominik Laskowski | e2c5b0a | 2022-08-10 14:53:53 -0700 | [diff] [blame] | 65 | Hwc2::mock::Composer* const mHal = new StrictMock<Hwc2::mock::Composer>(); | 
|  | 66 | impl::HWComposer mHwc{std::unique_ptr<Hwc2::Composer>(mHal)}; | 
| Marin Shalamanov | 8b19659 | 2021-08-09 16:24:42 +0200 | [diff] [blame] | 67 |  | 
| Dominik Laskowski | e2c5b0a | 2022-08-10 14:53:53 -0700 | [diff] [blame] | 68 | void expectHotplugConnect(hal::HWDisplayId hwcDisplayId) { | 
|  | 69 | constexpr uint8_t kPort = 255; | 
|  | 70 | EXPECT_CALL(*mHal, getDisplayIdentificationData(hwcDisplayId, _, _)) | 
|  | 71 | .WillOnce(DoAll(SetArgPointee<1>(kPort), | 
|  | 72 | SetArgPointee<2>(getExternalEdid()), Return(HalError::NONE))); | 
| Marin Shalamanov | 8b19659 | 2021-08-09 16:24:42 +0200 | [diff] [blame] | 73 |  | 
| Dominik Laskowski | e2c5b0a | 2022-08-10 14:53:53 -0700 | [diff] [blame] | 74 | EXPECT_CALL(*mHal, setClientTargetSlotCount(_)); | 
|  | 75 | EXPECT_CALL(*mHal, setVsyncEnabled(hwcDisplayId, Hwc2::IComposerClient::Vsync::DISABLE)); | 
|  | 76 | } | 
|  | 77 | }; | 
| Marin Shalamanov | 8b19659 | 2021-08-09 16:24:42 +0200 | [diff] [blame] | 78 |  | 
| Dominik Laskowski | e2c5b0a | 2022-08-10 14:53:53 -0700 | [diff] [blame] | 79 | TEST_F(HWComposerTest, isHeadless) { | 
|  | 80 | ASSERT_TRUE(mHwc.isHeadless()); | 
|  | 81 |  | 
|  | 82 | constexpr hal::HWDisplayId kHwcDisplayId = 1; | 
|  | 83 | expectHotplugConnect(kHwcDisplayId); | 
|  | 84 |  | 
|  | 85 | const auto info = mHwc.onHotplug(kHwcDisplayId, hal::Connection::CONNECTED); | 
| Marin Shalamanov | 8b19659 | 2021-08-09 16:24:42 +0200 | [diff] [blame] | 86 | ASSERT_TRUE(info); | 
| Marin Shalamanov | 8b19659 | 2021-08-09 16:24:42 +0200 | [diff] [blame] | 87 |  | 
| Dominik Laskowski | e2c5b0a | 2022-08-10 14:53:53 -0700 | [diff] [blame] | 88 | ASSERT_FALSE(mHwc.isHeadless()); | 
|  | 89 |  | 
|  | 90 | mHwc.disconnectDisplay(info->id); | 
|  | 91 | ASSERT_TRUE(mHwc.isHeadless()); | 
|  | 92 | } | 
|  | 93 |  | 
|  | 94 | TEST_F(HWComposerTest, getActiveMode) { | 
|  | 95 | // Unknown display. | 
|  | 96 | EXPECT_EQ(mHwc.getActiveMode(PhysicalDisplayId::fromPort(0)), std::nullopt); | 
|  | 97 |  | 
|  | 98 | constexpr hal::HWDisplayId kHwcDisplayId = 2; | 
|  | 99 | expectHotplugConnect(kHwcDisplayId); | 
|  | 100 |  | 
|  | 101 | const auto info = mHwc.onHotplug(kHwcDisplayId, hal::Connection::CONNECTED); | 
|  | 102 | ASSERT_TRUE(info); | 
|  | 103 |  | 
|  | 104 | { | 
|  | 105 | // Display is known to SF but not HWC, e.g. the hotplug disconnect is pending. | 
|  | 106 | EXPECT_CALL(*mHal, getActiveConfig(kHwcDisplayId, _)) | 
|  | 107 | .WillOnce(Return(HalError::BAD_DISPLAY)); | 
|  | 108 |  | 
|  | 109 | EXPECT_EQ(mHwc.getActiveMode(info->id), std::nullopt); | 
|  | 110 | } | 
|  | 111 | { | 
|  | 112 | constexpr hal::HWConfigId kConfigId = 42; | 
|  | 113 | EXPECT_CALL(*mHal, getActiveConfig(kHwcDisplayId, _)) | 
|  | 114 | .WillOnce(DoAll(SetArgPointee<1>(kConfigId), Return(HalError::NONE))); | 
|  | 115 |  | 
|  | 116 | EXPECT_EQ(mHwc.getActiveMode(info->id), kConfigId); | 
|  | 117 | } | 
| Marin Shalamanov | 8b19659 | 2021-08-09 16:24:42 +0200 | [diff] [blame] | 118 | } | 
|  | 119 |  | 
| Dominik Laskowski | 8b01cc0 | 2020-07-14 19:02:41 -0700 | [diff] [blame] | 120 | struct MockHWC2ComposerCallback final : StrictMock<HWC2::ComposerCallback> { | 
| Dominik Laskowski | 0deb06e | 2021-04-16 23:18:31 -0700 | [diff] [blame] | 121 | MOCK_METHOD2(onComposerHalHotplug, void(hal::HWDisplayId, hal::Connection)); | 
|  | 122 | MOCK_METHOD1(onComposerHalRefresh, void(hal::HWDisplayId)); | 
|  | 123 | MOCK_METHOD3(onComposerHalVsync, | 
|  | 124 | void(hal::HWDisplayId, int64_t timestamp, std::optional<hal::VsyncPeriodNanos>)); | 
|  | 125 | MOCK_METHOD2(onComposerHalVsyncPeriodTimingChanged, | 
|  | 126 | void(hal::HWDisplayId, const hal::VsyncPeriodChangeTimeline&)); | 
|  | 127 | MOCK_METHOD1(onComposerHalSeamlessPossible, void(hal::HWDisplayId)); | 
| Yichi Chen | 1a417af | 2022-01-21 15:29:52 +0800 | [diff] [blame] | 128 | MOCK_METHOD1(onComposerHalVsyncIdle, void(hal::HWDisplayId)); | 
| Lloyd Pique | 4603f3c | 2020-02-11 12:06:56 -0800 | [diff] [blame] | 129 | }; | 
|  | 130 |  | 
| Dominik Laskowski | e2c5b0a | 2022-08-10 14:53:53 -0700 | [diff] [blame] | 131 | struct HWComposerSetCallbackTest : HWComposerTest { | 
| Dominik Laskowski | 8b01cc0 | 2020-07-14 19:02:41 -0700 | [diff] [blame] | 132 | MockHWC2ComposerCallback mCallback; | 
| Lloyd Pique | 4603f3c | 2020-02-11 12:06:56 -0800 | [diff] [blame] | 133 | }; | 
|  | 134 |  | 
| Dominik Laskowski | 0deb06e | 2021-04-16 23:18:31 -0700 | [diff] [blame] | 135 | TEST_F(HWComposerSetCallbackTest, loadsLayerMetadataSupport) { | 
| Lloyd Pique | 4603f3c | 2020-02-11 12:06:56 -0800 | [diff] [blame] | 136 | const std::string kMetadata1Name = "com.example.metadata.1"; | 
|  | 137 | constexpr bool kMetadata1Mandatory = false; | 
|  | 138 | const std::string kMetadata2Name = "com.example.metadata.2"; | 
|  | 139 | constexpr bool kMetadata2Mandatory = true; | 
|  | 140 |  | 
| Ady Abraham | de549d4 | 2022-01-26 19:19:17 -0800 | [diff] [blame] | 141 | EXPECT_CALL(*mHal, getCapabilities()).WillOnce(Return(std::vector<aidl::Capability>{})); | 
| Lloyd Pique | 4603f3c | 2020-02-11 12:06:56 -0800 | [diff] [blame] | 142 | EXPECT_CALL(*mHal, getLayerGenericMetadataKeys(_)) | 
| Peiyong Lin | e9d809e | 2020-04-14 13:10:48 -0700 | [diff] [blame] | 143 | .WillOnce(DoAll(SetArgPointee<0>(std::vector<hal::LayerGenericMetadataKey>{ | 
|  | 144 | {kMetadata1Name, kMetadata1Mandatory}, | 
|  | 145 | {kMetadata2Name, kMetadata2Mandatory}, | 
|  | 146 | }), | 
| Lloyd Pique | 4603f3c | 2020-02-11 12:06:56 -0800 | [diff] [blame] | 147 | Return(hardware::graphics::composer::V2_4::Error::NONE))); | 
|  | 148 | EXPECT_CALL(*mHal, registerCallback(_)); | 
| Lloyd Pique | 4603f3c | 2020-02-11 12:06:56 -0800 | [diff] [blame] | 149 |  | 
| Dominik Laskowski | e2c5b0a | 2022-08-10 14:53:53 -0700 | [diff] [blame] | 150 | mHwc.setCallback(mCallback); | 
| Lloyd Pique | 4603f3c | 2020-02-11 12:06:56 -0800 | [diff] [blame] | 151 |  | 
| Dominik Laskowski | e2c5b0a | 2022-08-10 14:53:53 -0700 | [diff] [blame] | 152 | const auto& supported = mHwc.getSupportedLayerGenericMetadata(); | 
| Lloyd Pique | 4603f3c | 2020-02-11 12:06:56 -0800 | [diff] [blame] | 153 | EXPECT_EQ(2u, supported.size()); | 
|  | 154 | EXPECT_EQ(1u, supported.count(kMetadata1Name)); | 
|  | 155 | EXPECT_EQ(kMetadata1Mandatory, supported.find(kMetadata1Name)->second); | 
|  | 156 | EXPECT_EQ(1u, supported.count(kMetadata2Name)); | 
|  | 157 | EXPECT_EQ(kMetadata2Mandatory, supported.find(kMetadata2Name)->second); | 
|  | 158 | } | 
|  | 159 |  | 
| Dominik Laskowski | 0deb06e | 2021-04-16 23:18:31 -0700 | [diff] [blame] | 160 | TEST_F(HWComposerSetCallbackTest, handlesUnsupportedCallToGetLayerGenericMetadataKeys) { | 
| Ady Abraham | de549d4 | 2022-01-26 19:19:17 -0800 | [diff] [blame] | 161 | EXPECT_CALL(*mHal, getCapabilities()).WillOnce(Return(std::vector<aidl::Capability>{})); | 
| Lloyd Pique | 4603f3c | 2020-02-11 12:06:56 -0800 | [diff] [blame] | 162 | EXPECT_CALL(*mHal, getLayerGenericMetadataKeys(_)) | 
|  | 163 | .WillOnce(Return(hardware::graphics::composer::V2_4::Error::UNSUPPORTED)); | 
|  | 164 | EXPECT_CALL(*mHal, registerCallback(_)); | 
| Lloyd Pique | 4603f3c | 2020-02-11 12:06:56 -0800 | [diff] [blame] | 165 |  | 
| Dominik Laskowski | e2c5b0a | 2022-08-10 14:53:53 -0700 | [diff] [blame] | 166 | mHwc.setCallback(mCallback); | 
| Lloyd Pique | 4603f3c | 2020-02-11 12:06:56 -0800 | [diff] [blame] | 167 |  | 
| Dominik Laskowski | e2c5b0a | 2022-08-10 14:53:53 -0700 | [diff] [blame] | 168 | const auto& supported = mHwc.getSupportedLayerGenericMetadata(); | 
|  | 169 | EXPECT_TRUE(supported.empty()); | 
| Lloyd Pique | 4603f3c | 2020-02-11 12:06:56 -0800 | [diff] [blame] | 170 | } | 
|  | 171 |  | 
|  | 172 | struct HWComposerLayerTest : public testing::Test { | 
| Peiyong Lin | e9d809e | 2020-04-14 13:10:48 -0700 | [diff] [blame] | 173 | static constexpr hal::HWDisplayId kDisplayId = static_cast<hal::HWDisplayId>(1001); | 
|  | 174 | static constexpr hal::HWLayerId kLayerId = static_cast<hal::HWLayerId>(1002); | 
| Lloyd Pique | 4603f3c | 2020-02-11 12:06:56 -0800 | [diff] [blame] | 175 |  | 
| Ady Abraham | de549d4 | 2022-01-26 19:19:17 -0800 | [diff] [blame] | 176 | HWComposerLayerTest(const std::unordered_set<aidl::Capability>& capabilities) | 
| Lloyd Pique | a516c00 | 2021-05-07 14:36:58 -0700 | [diff] [blame] | 177 | : mCapabilies(capabilities) { | 
|  | 178 | EXPECT_CALL(mDisplay, getId()).WillRepeatedly(Return(kDisplayId)); | 
|  | 179 | } | 
| Lloyd Pique | 4603f3c | 2020-02-11 12:06:56 -0800 | [diff] [blame] | 180 |  | 
| Lloyd Pique | a516c00 | 2021-05-07 14:36:58 -0700 | [diff] [blame] | 181 | ~HWComposerLayerTest() override { | 
|  | 182 | EXPECT_CALL(mDisplay, onLayerDestroyed(kLayerId)); | 
|  | 183 | EXPECT_CALL(*mHal, destroyLayer(kDisplayId, kLayerId)); | 
|  | 184 | } | 
| Lloyd Pique | 4603f3c | 2020-02-11 12:06:56 -0800 | [diff] [blame] | 185 |  | 
|  | 186 | std::unique_ptr<Hwc2::mock::Composer> mHal{new StrictMock<Hwc2::mock::Composer>()}; | 
| Ady Abraham | de549d4 | 2022-01-26 19:19:17 -0800 | [diff] [blame] | 187 | const std::unordered_set<aidl::Capability> mCapabilies; | 
| Lloyd Pique | a516c00 | 2021-05-07 14:36:58 -0700 | [diff] [blame] | 188 | StrictMock<HWC2::mock::Display> mDisplay; | 
|  | 189 | HWC2::impl::Layer mLayer{*mHal, mCapabilies, mDisplay, kLayerId}; | 
| Lloyd Pique | 4603f3c | 2020-02-11 12:06:56 -0800 | [diff] [blame] | 190 | }; | 
|  | 191 |  | 
|  | 192 | struct HWComposerLayerGenericMetadataTest : public HWComposerLayerTest { | 
|  | 193 | static const std::string kLayerGenericMetadata1Name; | 
|  | 194 | static constexpr bool kLayerGenericMetadata1Mandatory = false; | 
|  | 195 | static const std::vector<uint8_t> kLayerGenericMetadata1Value; | 
|  | 196 | static const std::string kLayerGenericMetadata2Name; | 
|  | 197 | static constexpr bool kLayerGenericMetadata2Mandatory = true; | 
|  | 198 | static const std::vector<uint8_t> kLayerGenericMetadata2Value; | 
|  | 199 |  | 
|  | 200 | HWComposerLayerGenericMetadataTest() : HWComposerLayerTest({}) {} | 
|  | 201 | }; | 
|  | 202 |  | 
|  | 203 | const std::string HWComposerLayerGenericMetadataTest::kLayerGenericMetadata1Name = | 
|  | 204 | "com.example.metadata.1"; | 
|  | 205 |  | 
|  | 206 | const std::vector<uint8_t> HWComposerLayerGenericMetadataTest::kLayerGenericMetadata1Value = {1u, | 
|  | 207 | 2u, | 
|  | 208 | 3u}; | 
|  | 209 |  | 
|  | 210 | const std::string HWComposerLayerGenericMetadataTest::kLayerGenericMetadata2Name = | 
|  | 211 | "com.example.metadata.2"; | 
|  | 212 |  | 
|  | 213 | const std::vector<uint8_t> HWComposerLayerGenericMetadataTest::kLayerGenericMetadata2Value = {45u, | 
|  | 214 | 67u}; | 
|  | 215 |  | 
|  | 216 | TEST_F(HWComposerLayerGenericMetadataTest, forwardsSupportedMetadata) { | 
|  | 217 | EXPECT_CALL(*mHal, | 
|  | 218 | setLayerGenericMetadata(kDisplayId, kLayerId, kLayerGenericMetadata1Name, | 
|  | 219 | kLayerGenericMetadata1Mandatory, | 
|  | 220 | kLayerGenericMetadata1Value)) | 
|  | 221 | .WillOnce(Return(hardware::graphics::composer::V2_4::Error::NONE)); | 
|  | 222 | auto result = mLayer.setLayerGenericMetadata(kLayerGenericMetadata1Name, | 
|  | 223 | kLayerGenericMetadata1Mandatory, | 
|  | 224 | kLayerGenericMetadata1Value); | 
| Peiyong Lin | e9d809e | 2020-04-14 13:10:48 -0700 | [diff] [blame] | 225 | EXPECT_EQ(hal::Error::NONE, result); | 
| Lloyd Pique | 4603f3c | 2020-02-11 12:06:56 -0800 | [diff] [blame] | 226 |  | 
|  | 227 | EXPECT_CALL(*mHal, | 
|  | 228 | setLayerGenericMetadata(kDisplayId, kLayerId, kLayerGenericMetadata2Name, | 
|  | 229 | kLayerGenericMetadata2Mandatory, | 
|  | 230 | kLayerGenericMetadata2Value)) | 
|  | 231 | .WillOnce(Return(hardware::graphics::composer::V2_4::Error::UNSUPPORTED)); | 
|  | 232 | result = mLayer.setLayerGenericMetadata(kLayerGenericMetadata2Name, | 
|  | 233 | kLayerGenericMetadata2Mandatory, | 
|  | 234 | kLayerGenericMetadata2Value); | 
| Peiyong Lin | e9d809e | 2020-04-14 13:10:48 -0700 | [diff] [blame] | 235 | EXPECT_EQ(hal::Error::UNSUPPORTED, result); | 
| Lloyd Pique | 4603f3c | 2020-02-11 12:06:56 -0800 | [diff] [blame] | 236 | } | 
|  | 237 |  | 
|  | 238 | } // namespace | 
| Dominik Laskowski | 3dce4f4 | 2021-03-08 20:48:28 -0800 | [diff] [blame] | 239 | } // namespace android |