Sean Paul | 98e73c8 | 2015-06-24 14:38:49 -0700 | [diff] [blame] | 1 | /* |
| 2 | * Copyright (C) 2015 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 | |
Roman Stratiienko | 59bb481 | 2022-04-09 16:13:09 +0300 | [diff] [blame] | 17 | #undef NDEBUG /* Required for assert to work */ |
| 18 | |
Sean Paul | 98e73c8 | 2015-06-24 14:38:49 -0700 | [diff] [blame] | 19 | #define ATRACE_TAG ATRACE_TAG_GRAPHICS |
Sean Paul | 468a754 | 2024-07-16 19:50:58 +0000 | [diff] [blame] | 20 | #define LOG_TAG "drmhwc" |
Sean Paul | 98e73c8 | 2015-06-24 14:38:49 -0700 | [diff] [blame] | 21 | |
Roman Stratiienko | 4e99405 | 2022-02-09 17:40:35 +0200 | [diff] [blame] | 22 | #include "DrmAtomicStateManager.h" |
Zach Reizner | bff33ac | 2015-11-16 11:08:46 -0800 | [diff] [blame] | 23 | |
Roman Stratiienko | aa3cd54 | 2020-08-29 11:26:16 +0300 | [diff] [blame] | 24 | #include <drm/drm_mode.h> |
Roman Stratiienko | aa3cd54 | 2020-08-29 11:26:16 +0300 | [diff] [blame] | 25 | #include <sync/sync.h> |
Roman Stratiienko | aa3cd54 | 2020-08-29 11:26:16 +0300 | [diff] [blame] | 26 | #include <utils/Trace.h> |
| 27 | |
Roman Stratiienko | 59bb481 | 2022-04-09 16:13:09 +0300 | [diff] [blame] | 28 | #include <cassert> |
Zach Reizner | bff33ac | 2015-11-16 11:08:46 -0800 | [diff] [blame] | 29 | |
Roman Stratiienko | 13cc366 | 2020-08-29 21:35:39 +0300 | [diff] [blame] | 30 | #include "drm/DrmCrtc.h" |
| 31 | #include "drm/DrmDevice.h" |
| 32 | #include "drm/DrmPlane.h" |
Roman Stratiienko | 3e8ce57 | 2021-09-29 12:46:28 +0300 | [diff] [blame] | 33 | #include "drm/DrmUnique.h" |
Roman Stratiienko | d21071f | 2021-03-09 21:56:50 +0200 | [diff] [blame] | 34 | #include "utils/log.h" |
Sean Paul | 98e73c8 | 2015-06-24 14:38:49 -0700 | [diff] [blame] | 35 | |
Sean Paul | 98e73c8 | 2015-06-24 14:38:49 -0700 | [diff] [blame] | 36 | namespace android { |
| 37 | |
Roman Stratiienko | f818d4c | 2022-12-28 20:12:19 +0200 | [diff] [blame] | 38 | auto DrmAtomicStateManager::CreateInstance(DrmDisplayPipeline *pipe) |
| 39 | -> std::shared_ptr<DrmAtomicStateManager> { |
| 40 | auto dasm = std::shared_ptr<DrmAtomicStateManager>( |
| 41 | new DrmAtomicStateManager()); |
| 42 | |
| 43 | dasm->pipe_ = pipe; |
| 44 | std::thread(&DrmAtomicStateManager::ThreadFn, dasm.get(), dasm).detach(); |
| 45 | |
| 46 | return dasm; |
| 47 | } |
| 48 | |
Roman Stratiienko | 5f2f3ce | 2021-12-22 11:46:03 +0200 | [diff] [blame] | 49 | // NOLINTNEXTLINE (readability-function-cognitive-complexity): Fixme |
Roman Stratiienko | 4e99405 | 2022-02-09 17:40:35 +0200 | [diff] [blame] | 50 | auto DrmAtomicStateManager::CommitFrame(AtomicCommitArgs &args) -> int { |
Roman Stratiienko | a7913de | 2022-10-20 13:18:57 +0300 | [diff] [blame] | 51 | // NOLINTNEXTLINE(misc-const-correctness) |
Haixia Shi | 3979f7d | 2015-10-29 14:33:37 -0700 | [diff] [blame] | 52 | ATRACE_CALL(); |
| 53 | |
Roman Stratiienko | f815d38 | 2021-12-30 19:23:14 +0200 | [diff] [blame] | 54 | if (args.active && *args.active == active_frame_state_.crtc_active_state) { |
Roman Stratiienko | dccc6fb | 2021-10-23 17:35:44 +0300 | [diff] [blame] | 55 | /* Don't set the same state twice */ |
| 56 | args.active.reset(); |
| 57 | } |
Haixia Shi | dda2fab | 2015-10-22 18:12:49 -0700 | [diff] [blame] | 58 | |
Roman Stratiienko | dccc6fb | 2021-10-23 17:35:44 +0300 | [diff] [blame] | 59 | if (!args.HasInputs()) { |
| 60 | /* nothing to do */ |
| 61 | return 0; |
| 62 | } |
| 63 | |
Roman Stratiienko | f815d38 | 2021-12-30 19:23:14 +0200 | [diff] [blame] | 64 | if (!active_frame_state_.crtc_active_state) { |
Roman Stratiienko | dccc6fb | 2021-10-23 17:35:44 +0300 | [diff] [blame] | 65 | /* Force activate display */ |
| 66 | args.active = true; |
| 67 | } |
| 68 | |
Roman Stratiienko | 32685ba | 2021-12-15 13:46:05 +0200 | [diff] [blame] | 69 | auto new_frame_state = NewFrameState(); |
| 70 | |
Roman Stratiienko | 19c162f | 2022-02-01 09:35:08 +0200 | [diff] [blame] | 71 | auto *drm = pipe_->device; |
| 72 | auto *connector = pipe_->connector->Get(); |
| 73 | auto *crtc = pipe_->crtc->Get(); |
Sean Paul | 5735541 | 2015-09-19 09:14:34 -0400 | [diff] [blame] | 74 | |
Roman Stratiienko | 7fd8f88 | 2021-09-29 12:46:54 +0300 | [diff] [blame] | 75 | auto pset = MakeDrmModeAtomicReqUnique(); |
Sean Paul | 98e73c8 | 2015-06-24 14:38:49 -0700 | [diff] [blame] | 76 | if (!pset) { |
| 77 | ALOGE("Failed to allocate property set"); |
| 78 | return -ENOMEM; |
| 79 | } |
| 80 | |
Roman Stratiienko | 59bb481 | 2022-04-09 16:13:09 +0300 | [diff] [blame] | 81 | int out_fence = -1; |
Roman Stratiienko | f2c060f | 2023-09-18 22:46:08 +0300 | [diff] [blame] | 82 | if (!args.writeback_fb) { |
| 83 | if (!crtc->GetOutFencePtrProperty(). // |
| 84 | AtomicSet(*pset, uint64_t(&out_fence))) { |
| 85 | return -EINVAL; |
| 86 | } |
| 87 | } else { |
| 88 | if (!connector->GetWritebackOutFenceProperty(). // |
| 89 | AtomicSet(*pset, uint64_t(&out_fence))) { |
| 90 | return -EINVAL; |
| 91 | } |
| 92 | |
| 93 | if (!connector->GetWritebackFbIdProperty(). // |
| 94 | AtomicSet(*pset, args.writeback_fb->GetFbId())) { |
| 95 | return -EINVAL; |
| 96 | } |
| 97 | |
| 98 | if (args.writeback_release_fence) { |
| 99 | sync_wait(*args.writeback_release_fence, -1); |
| 100 | args.writeback_release_fence.reset(); |
| 101 | } |
Robert Foss | a1ade4e | 2017-09-27 19:28:15 +0200 | [diff] [blame] | 102 | } |
| 103 | |
Roman Stratiienko | 59bb481 | 2022-04-09 16:13:09 +0300 | [diff] [blame] | 104 | bool nonblock = true; |
| 105 | |
Roman Stratiienko | dccc6fb | 2021-10-23 17:35:44 +0300 | [diff] [blame] | 106 | if (args.active) { |
Roman Stratiienko | 59bb481 | 2022-04-09 16:13:09 +0300 | [diff] [blame] | 107 | nonblock = false; |
Roman Stratiienko | 32685ba | 2021-12-15 13:46:05 +0200 | [diff] [blame] | 108 | new_frame_state.crtc_active_state = *args.active; |
Roman Stratiienko | 4e99405 | 2022-02-09 17:40:35 +0200 | [diff] [blame] | 109 | if (!crtc->GetActiveProperty().AtomicSet(*pset, *args.active ? 1 : 0) || |
Roman Stratiienko | 650299a | 2022-01-30 23:46:10 +0200 | [diff] [blame] | 110 | !connector->GetCrtcIdProperty().AtomicSet(*pset, crtc->GetId())) { |
Roman Stratiienko | dccc6fb | 2021-10-23 17:35:44 +0300 | [diff] [blame] | 111 | return -EINVAL; |
| 112 | } |
Sean Paul | 5735541 | 2015-09-19 09:14:34 -0400 | [diff] [blame] | 113 | } |
| 114 | |
Roman Stratiienko | dccc6fb | 2021-10-23 17:35:44 +0300 | [diff] [blame] | 115 | if (args.display_mode) { |
Roman Stratiienko | 19c162f | 2022-02-01 09:35:08 +0200 | [diff] [blame] | 116 | new_frame_state.mode_blob = args.display_mode.value().CreateModeBlob(*drm); |
Zach Reizner | 92f8e63 | 2015-10-12 17:47:13 -0700 | [diff] [blame] | 117 | |
Roman Stratiienko | 32685ba | 2021-12-15 13:46:05 +0200 | [diff] [blame] | 118 | if (!new_frame_state.mode_blob) { |
Roman Stratiienko | dccc6fb | 2021-10-23 17:35:44 +0300 | [diff] [blame] | 119 | ALOGE("Failed to create mode_blob"); |
| 120 | return -EINVAL; |
| 121 | } |
Sean Paul | ca699be | 2016-05-11 16:29:45 -0400 | [diff] [blame] | 122 | |
Roman Stratiienko | 10be875 | 2022-01-30 20:28:46 +0200 | [diff] [blame] | 123 | if (!crtc->GetModeProperty().AtomicSet(*pset, *new_frame_state.mode_blob)) { |
Roman Stratiienko | dccc6fb | 2021-10-23 17:35:44 +0300 | [diff] [blame] | 124 | return -EINVAL; |
| 125 | } |
| 126 | } |
Roman Stratiienko | 0dbe639 | 2021-09-29 12:47:16 +0300 | [diff] [blame] | 127 | |
Roman Stratiienko | 0da91bf | 2023-01-17 18:06:04 +0200 | [diff] [blame] | 128 | if (args.color_matrix && crtc->GetCtmProperty()) { |
| 129 | auto blob = drm->RegisterUserPropertyBlob(args.color_matrix.get(), |
| 130 | sizeof(drm_color_ctm)); |
| 131 | new_frame_state.ctm_blob = std::move(blob); |
| 132 | |
| 133 | if (!new_frame_state.ctm_blob) { |
| 134 | ALOGE("Failed to create CTM blob"); |
| 135 | return -EINVAL; |
| 136 | } |
| 137 | |
| 138 | if (!crtc->GetCtmProperty().AtomicSet(*pset, *new_frame_state.ctm_blob)) |
| 139 | return -EINVAL; |
| 140 | } |
| 141 | |
Sasha McIntosh | 5294f09 | 2024-09-18 18:14:54 -0400 | [diff] [blame^] | 142 | if (args.colorspace && connector->GetColorspaceProperty()) { |
| 143 | if (!connector->GetColorspaceProperty() |
| 144 | .AtomicSet(*pset, connector->GetColorspacePropertyValue(*args.colorspace))) |
| 145 | return -EINVAL; |
| 146 | } |
| 147 | |
Sasha McIntosh | 173247b | 2024-09-18 18:06:52 -0400 | [diff] [blame] | 148 | if (args.content_type && connector->GetContentTypeProperty()) { |
| 149 | if (!connector->GetContentTypeProperty().AtomicSet(*pset, *args.content_type)) |
| 150 | return -EINVAL; |
| 151 | } |
| 152 | |
Roman Stratiienko | 67cebf5 | 2021-12-17 13:31:27 +0200 | [diff] [blame] | 153 | auto unused_planes = new_frame_state.used_planes; |
| 154 | |
Roman Stratiienko | dccc6fb | 2021-10-23 17:35:44 +0300 | [diff] [blame] | 155 | if (args.composition) { |
Roman Stratiienko | 32685ba | 2021-12-15 13:46:05 +0200 | [diff] [blame] | 156 | new_frame_state.used_planes.clear(); |
| 157 | |
Roman Stratiienko | 9362cef | 2022-02-02 09:53:50 +0200 | [diff] [blame] | 158 | for (auto &joining : args.composition->plan) { |
| 159 | DrmPlane *plane = joining.plane->Get(); |
Roman Stratiienko | 4b2cc48 | 2022-02-21 14:53:58 +0200 | [diff] [blame] | 160 | LayerData &layer = joining.layer; |
Roman Stratiienko | dccc6fb | 2021-10-23 17:35:44 +0300 | [diff] [blame] | 161 | |
Roman Stratiienko | 4b2cc48 | 2022-02-21 14:53:58 +0200 | [diff] [blame] | 162 | new_frame_state.used_framebuffers.emplace_back(layer.fb); |
Roman Stratiienko | 9362cef | 2022-02-02 09:53:50 +0200 | [diff] [blame] | 163 | new_frame_state.used_planes.emplace_back(joining.plane); |
Roman Stratiienko | 32685ba | 2021-12-15 13:46:05 +0200 | [diff] [blame] | 164 | |
Roman Stratiienko | 67cebf5 | 2021-12-17 13:31:27 +0200 | [diff] [blame] | 165 | /* Remove from 'unused' list, since plane is re-used */ |
| 166 | auto &v = unused_planes; |
Roman Stratiienko | 9362cef | 2022-02-02 09:53:50 +0200 | [diff] [blame] | 167 | v.erase(std::remove(v.begin(), v.end(), joining.plane), v.end()); |
Roman Stratiienko | 67cebf5 | 2021-12-17 13:31:27 +0200 | [diff] [blame] | 168 | |
Roman Stratiienko | 9362cef | 2022-02-02 09:53:50 +0200 | [diff] [blame] | 169 | if (plane->AtomicSetState(*pset, layer, joining.z_pos, crtc->GetId()) != |
Roman Stratiienko | 10be875 | 2022-01-30 20:28:46 +0200 | [diff] [blame] | 170 | 0) { |
Roman Stratiienko | 67cebf5 | 2021-12-17 13:31:27 +0200 | [diff] [blame] | 171 | return -EINVAL; |
Sean Paul | 39b3784 | 2016-05-11 13:50:28 -0400 | [diff] [blame] | 172 | } |
Roman Stratiienko | dccc6fb | 2021-10-23 17:35:44 +0300 | [diff] [blame] | 173 | } |
| 174 | } |
| 175 | |
Roman Stratiienko | ef5348b | 2022-02-09 17:19:56 +0200 | [diff] [blame] | 176 | if (args.composition) { |
Roman Stratiienko | 9362cef | 2022-02-02 09:53:50 +0200 | [diff] [blame] | 177 | for (auto &plane : unused_planes) { |
| 178 | if (plane->Get()->AtomicDisablePlane(*pset) != 0) { |
Roman Stratiienko | 0dbe639 | 2021-09-29 12:47:16 +0300 | [diff] [blame] | 179 | return -EINVAL; |
Matvii Zorin | 8338c34 | 2020-09-08 16:12:51 +0300 | [diff] [blame] | 180 | } |
| 181 | } |
Sean Paul | 98e73c8 | 2015-06-24 14:38:49 -0700 | [diff] [blame] | 182 | } |
| 183 | |
Roman Stratiienko | dccc6fb | 2021-10-23 17:35:44 +0300 | [diff] [blame] | 184 | uint32_t flags = DRM_MODE_ATOMIC_ALLOW_MODESET; |
Roman Stratiienko | 59bb481 | 2022-04-09 16:13:09 +0300 | [diff] [blame] | 185 | |
| 186 | if (args.test_only) { |
Roman Stratiienko | 7689278 | 2023-01-16 17:15:53 +0200 | [diff] [blame] | 187 | return drmModeAtomicCommit(*drm->GetFd(), pset.get(), |
Roman Stratiienko | 59bb481 | 2022-04-09 16:13:09 +0300 | [diff] [blame] | 188 | flags | DRM_MODE_ATOMIC_TEST_ONLY, drm); |
| 189 | } |
| 190 | |
| 191 | if (last_present_fence_) { |
Roman Stratiienko | a7913de | 2022-10-20 13:18:57 +0300 | [diff] [blame] | 192 | // NOLINTNEXTLINE(misc-const-correctness) |
Roman Stratiienko | 59bb481 | 2022-04-09 16:13:09 +0300 | [diff] [blame] | 193 | ATRACE_NAME("WaitPriorFramePresented"); |
| 194 | |
| 195 | constexpr int kTimeoutMs = 500; |
Roman Stratiienko | 7689278 | 2023-01-16 17:15:53 +0200 | [diff] [blame] | 196 | const int err = sync_wait(*last_present_fence_, kTimeoutMs); |
Roman Stratiienko | 59bb481 | 2022-04-09 16:13:09 +0300 | [diff] [blame] | 197 | if (err != 0) { |
Roman Stratiienko | 7689278 | 2023-01-16 17:15:53 +0200 | [diff] [blame] | 198 | ALOGE("sync_wait(fd=%i) returned: %i (errno: %i)", *last_present_fence_, |
| 199 | err, errno); |
Roman Stratiienko | 59bb481 | 2022-04-09 16:13:09 +0300 | [diff] [blame] | 200 | } |
| 201 | |
| 202 | CleanupPriorFrameResources(); |
| 203 | } |
| 204 | |
| 205 | if (nonblock) { |
| 206 | flags |= DRM_MODE_ATOMIC_NONBLOCK; |
| 207 | } |
Sean Paul | c07b211 | 2015-11-17 16:38:10 -0500 | [diff] [blame] | 208 | |
Roman Stratiienko | 7689278 | 2023-01-16 17:15:53 +0200 | [diff] [blame] | 209 | auto err = drmModeAtomicCommit(*drm->GetFd(), pset.get(), flags, drm); |
Roman Stratiienko | 59bb481 | 2022-04-09 16:13:09 +0300 | [diff] [blame] | 210 | |
Roman Stratiienko | 4e99405 | 2022-02-09 17:40:35 +0200 | [diff] [blame] | 211 | if (err != 0) { |
Roman Stratiienko | 59bb481 | 2022-04-09 16:13:09 +0300 | [diff] [blame] | 212 | ALOGE("Failed to commit pset ret=%d\n", err); |
Roman Stratiienko | dccc6fb | 2021-10-23 17:35:44 +0300 | [diff] [blame] | 213 | return err; |
Sean Paul | 98e73c8 | 2015-06-24 14:38:49 -0700 | [diff] [blame] | 214 | } |
Sean Paul | 98e73c8 | 2015-06-24 14:38:49 -0700 | [diff] [blame] | 215 | |
Roman Stratiienko | 7689278 | 2023-01-16 17:15:53 +0200 | [diff] [blame] | 216 | args.out_fence = MakeSharedFd(out_fence); |
| 217 | |
Roman Stratiienko | 59bb481 | 2022-04-09 16:13:09 +0300 | [diff] [blame] | 218 | if (nonblock) { |
Roman Stratiienko | f818d4c | 2022-12-28 20:12:19 +0200 | [diff] [blame] | 219 | { |
| 220 | const std::unique_lock lock(mutex_); |
Roman Stratiienko | 7689278 | 2023-01-16 17:15:53 +0200 | [diff] [blame] | 221 | last_present_fence_ = args.out_fence; |
Roman Stratiienko | f818d4c | 2022-12-28 20:12:19 +0200 | [diff] [blame] | 222 | staged_frame_state_ = std::move(new_frame_state); |
| 223 | frames_staged_++; |
| 224 | } |
| 225 | cv_.notify_all(); |
Roman Stratiienko | 59bb481 | 2022-04-09 16:13:09 +0300 | [diff] [blame] | 226 | } else { |
Roman Stratiienko | f815d38 | 2021-12-30 19:23:14 +0200 | [diff] [blame] | 227 | active_frame_state_ = std::move(new_frame_state); |
Robert Foss | a1ade4e | 2017-09-27 19:28:15 +0200 | [diff] [blame] | 228 | } |
| 229 | |
Roman Stratiienko | dccc6fb | 2021-10-23 17:35:44 +0300 | [diff] [blame] | 230 | return 0; |
Sean Paul | 98e73c8 | 2015-06-24 14:38:49 -0700 | [diff] [blame] | 231 | } |
| 232 | |
Roman Stratiienko | f818d4c | 2022-12-28 20:12:19 +0200 | [diff] [blame] | 233 | void DrmAtomicStateManager::ThreadFn( |
| 234 | const std::shared_ptr<DrmAtomicStateManager> &dasm) { |
Roman Stratiienko | 59bb481 | 2022-04-09 16:13:09 +0300 | [diff] [blame] | 235 | int tracking_at_the_moment = -1; |
Roman Stratiienko | f818d4c | 2022-12-28 20:12:19 +0200 | [diff] [blame] | 236 | auto &main_mutex = pipe_->device->GetResMan().GetMainLock(); |
Roman Stratiienko | 59bb481 | 2022-04-09 16:13:09 +0300 | [diff] [blame] | 237 | |
| 238 | for (;;) { |
Roman Stratiienko | 7689278 | 2023-01-16 17:15:53 +0200 | [diff] [blame] | 239 | SharedFd present_fence; |
Roman Stratiienko | 59bb481 | 2022-04-09 16:13:09 +0300 | [diff] [blame] | 240 | |
| 241 | { |
Roman Stratiienko | f818d4c | 2022-12-28 20:12:19 +0200 | [diff] [blame] | 242 | std::unique_lock lk(mutex_); |
| 243 | cv_.wait(lk); |
Roman Stratiienko | 59bb481 | 2022-04-09 16:13:09 +0300 | [diff] [blame] | 244 | |
Roman Stratiienko | f818d4c | 2022-12-28 20:12:19 +0200 | [diff] [blame] | 245 | if (exit_thread_ || dasm.use_count() == 1) |
Roman Stratiienko | 59bb481 | 2022-04-09 16:13:09 +0300 | [diff] [blame] | 246 | break; |
Roman Stratiienko | 59bb481 | 2022-04-09 16:13:09 +0300 | [diff] [blame] | 247 | |
Roman Stratiienko | f818d4c | 2022-12-28 20:12:19 +0200 | [diff] [blame] | 248 | if (frames_staged_ <= tracking_at_the_moment) |
Roman Stratiienko | 59bb481 | 2022-04-09 16:13:09 +0300 | [diff] [blame] | 249 | continue; |
Roman Stratiienko | f818d4c | 2022-12-28 20:12:19 +0200 | [diff] [blame] | 250 | |
| 251 | tracking_at_the_moment = frames_staged_; |
| 252 | |
Roman Stratiienko | 7689278 | 2023-01-16 17:15:53 +0200 | [diff] [blame] | 253 | present_fence = last_present_fence_; |
Roman Stratiienko | f818d4c | 2022-12-28 20:12:19 +0200 | [diff] [blame] | 254 | if (!present_fence) |
| 255 | continue; |
Roman Stratiienko | 59bb481 | 2022-04-09 16:13:09 +0300 | [diff] [blame] | 256 | } |
| 257 | |
| 258 | { |
Roman Stratiienko | a7913de | 2022-10-20 13:18:57 +0300 | [diff] [blame] | 259 | // NOLINTNEXTLINE(misc-const-correctness) |
Roman Stratiienko | 59bb481 | 2022-04-09 16:13:09 +0300 | [diff] [blame] | 260 | ATRACE_NAME("AsyncWaitForBuffersSwap"); |
| 261 | constexpr int kTimeoutMs = 500; |
Roman Stratiienko | 7689278 | 2023-01-16 17:15:53 +0200 | [diff] [blame] | 262 | auto err = sync_wait(*present_fence, kTimeoutMs); |
Roman Stratiienko | 59bb481 | 2022-04-09 16:13:09 +0300 | [diff] [blame] | 263 | if (err != 0) { |
Roman Stratiienko | 7689278 | 2023-01-16 17:15:53 +0200 | [diff] [blame] | 264 | ALOGE("sync_wait(fd=%i) returned: %i (errno: %i)", *present_fence, err, |
| 265 | errno); |
Roman Stratiienko | 59bb481 | 2022-04-09 16:13:09 +0300 | [diff] [blame] | 266 | } |
| 267 | } |
| 268 | |
| 269 | { |
Roman Stratiienko | f818d4c | 2022-12-28 20:12:19 +0200 | [diff] [blame] | 270 | const std::unique_lock mlk(main_mutex); |
| 271 | const std::unique_lock lk(mutex_); |
| 272 | if (exit_thread_) |
Roman Stratiienko | 59bb481 | 2022-04-09 16:13:09 +0300 | [diff] [blame] | 273 | break; |
Roman Stratiienko | 59bb481 | 2022-04-09 16:13:09 +0300 | [diff] [blame] | 274 | |
| 275 | /* If resources is already cleaned-up by main thread, skip */ |
Roman Stratiienko | f818d4c | 2022-12-28 20:12:19 +0200 | [diff] [blame] | 276 | if (tracking_at_the_moment > frames_tracked_) |
| 277 | CleanupPriorFrameResources(); |
Roman Stratiienko | 59bb481 | 2022-04-09 16:13:09 +0300 | [diff] [blame] | 278 | } |
| 279 | } |
Roman Stratiienko | f818d4c | 2022-12-28 20:12:19 +0200 | [diff] [blame] | 280 | |
| 281 | ALOGI("DrmAtomicStateManager thread exit"); |
Roman Stratiienko | 59bb481 | 2022-04-09 16:13:09 +0300 | [diff] [blame] | 282 | } |
| 283 | |
| 284 | void DrmAtomicStateManager::CleanupPriorFrameResources() { |
| 285 | assert(frames_staged_ - frames_tracked_ == 1); |
| 286 | assert(last_present_fence_); |
| 287 | |
Roman Stratiienko | a7913de | 2022-10-20 13:18:57 +0300 | [diff] [blame] | 288 | // NOLINTNEXTLINE(misc-const-correctness) |
Roman Stratiienko | 59bb481 | 2022-04-09 16:13:09 +0300 | [diff] [blame] | 289 | ATRACE_NAME("CleanupPriorFrameResources"); |
| 290 | frames_tracked_++; |
| 291 | active_frame_state_ = std::move(staged_frame_state_); |
| 292 | last_present_fence_ = {}; |
| 293 | } |
| 294 | |
Roman Stratiienko | 4e99405 | 2022-02-09 17:40:35 +0200 | [diff] [blame] | 295 | auto DrmAtomicStateManager::ExecuteAtomicCommit(AtomicCommitArgs &args) -> int { |
Roman Stratiienko | a7913de | 2022-10-20 13:18:57 +0300 | [diff] [blame] | 296 | auto err = CommitFrame(args); |
Sean Paul | 137a6a8 | 2016-06-22 22:48:22 -0400 | [diff] [blame] | 297 | |
Roman Stratiienko | dccc6fb | 2021-10-23 17:35:44 +0300 | [diff] [blame] | 298 | if (!args.test_only) { |
Roman Stratiienko | 4e99405 | 2022-02-09 17:40:35 +0200 | [diff] [blame] | 299 | if (err != 0) { |
Roman Stratiienko | 19c162f | 2022-02-01 09:35:08 +0200 | [diff] [blame] | 300 | ALOGE("Composite failed for pipeline %s", |
| 301 | pipe_->connector->Get()->GetName().c_str()); |
Roman Stratiienko | dccc6fb | 2021-10-23 17:35:44 +0300 | [diff] [blame] | 302 | // Disable the hw used by the last active composition. This allows us to |
| 303 | // signal the release fences from that composition to avoid hanging. |
Roman Stratiienko | ef5348b | 2022-02-09 17:19:56 +0200 | [diff] [blame] | 304 | AtomicCommitArgs cl_args{}; |
| 305 | cl_args.composition = std::make_shared<DrmKmsPlan>(); |
Roman Stratiienko | 4e99405 | 2022-02-09 17:40:35 +0200 | [diff] [blame] | 306 | if (CommitFrame(cl_args) != 0) { |
Roman Stratiienko | 19c162f | 2022-02-01 09:35:08 +0200 | [diff] [blame] | 307 | ALOGE("Failed to clean-up active composition for pipeline %s", |
| 308 | pipe_->connector->Get()->GetName().c_str()); |
Roman Stratiienko | dccc6fb | 2021-10-23 17:35:44 +0300 | [diff] [blame] | 309 | } |
| 310 | return err; |
| 311 | } |
Haixia Shi | dda2fab | 2015-10-22 18:12:49 -0700 | [diff] [blame] | 312 | } |
Haixia Shi | dda2fab | 2015-10-22 18:12:49 -0700 | [diff] [blame] | 313 | |
Roman Stratiienko | dccc6fb | 2021-10-23 17:35:44 +0300 | [diff] [blame] | 314 | return err; |
| 315 | } // namespace android |
Roman Stratiienko | 36a7f28 | 2021-10-05 18:17:53 +0300 | [diff] [blame] | 316 | |
Roman Stratiienko | 4e99405 | 2022-02-09 17:40:35 +0200 | [diff] [blame] | 317 | auto DrmAtomicStateManager::ActivateDisplayUsingDPMS() -> int { |
Roman Stratiienko | 7689278 | 2023-01-16 17:15:53 +0200 | [diff] [blame] | 318 | return drmModeConnectorSetProperty(*pipe_->device->GetFd(), |
Roman Stratiienko | 19c162f | 2022-02-01 09:35:08 +0200 | [diff] [blame] | 319 | pipe_->connector->Get()->GetId(), |
| 320 | pipe_->connector->Get() |
| 321 | ->GetDpmsProperty() |
Roman Stratiienko | abd8e53 | 2022-12-06 23:31:33 +0200 | [diff] [blame] | 322 | .GetId(), |
Roman Stratiienko | 19c162f | 2022-02-01 09:35:08 +0200 | [diff] [blame] | 323 | DRM_MODE_DPMS_ON); |
Roman Stratiienko | d37b308 | 2022-01-13 16:37:27 +0200 | [diff] [blame] | 324 | } |
| 325 | |
Sean Paul | f72cccd | 2018-08-27 13:59:08 -0400 | [diff] [blame] | 326 | } // namespace android |