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 | |
Roman Stratiienko | 67cebf5 | 2021-12-17 13:31:27 +0200 | [diff] [blame] | 142 | auto unused_planes = new_frame_state.used_planes; |
| 143 | |
Roman Stratiienko | dccc6fb | 2021-10-23 17:35:44 +0300 | [diff] [blame] | 144 | if (args.composition) { |
Roman Stratiienko | 32685ba | 2021-12-15 13:46:05 +0200 | [diff] [blame] | 145 | new_frame_state.used_planes.clear(); |
| 146 | |
Roman Stratiienko | 9362cef | 2022-02-02 09:53:50 +0200 | [diff] [blame] | 147 | for (auto &joining : args.composition->plan) { |
| 148 | DrmPlane *plane = joining.plane->Get(); |
Roman Stratiienko | 4b2cc48 | 2022-02-21 14:53:58 +0200 | [diff] [blame] | 149 | LayerData &layer = joining.layer; |
Roman Stratiienko | dccc6fb | 2021-10-23 17:35:44 +0300 | [diff] [blame] | 150 | |
Roman Stratiienko | 4b2cc48 | 2022-02-21 14:53:58 +0200 | [diff] [blame] | 151 | new_frame_state.used_framebuffers.emplace_back(layer.fb); |
Roman Stratiienko | 9362cef | 2022-02-02 09:53:50 +0200 | [diff] [blame] | 152 | new_frame_state.used_planes.emplace_back(joining.plane); |
Roman Stratiienko | 32685ba | 2021-12-15 13:46:05 +0200 | [diff] [blame] | 153 | |
Roman Stratiienko | 67cebf5 | 2021-12-17 13:31:27 +0200 | [diff] [blame] | 154 | /* Remove from 'unused' list, since plane is re-used */ |
| 155 | auto &v = unused_planes; |
Roman Stratiienko | 9362cef | 2022-02-02 09:53:50 +0200 | [diff] [blame] | 156 | v.erase(std::remove(v.begin(), v.end(), joining.plane), v.end()); |
Roman Stratiienko | 67cebf5 | 2021-12-17 13:31:27 +0200 | [diff] [blame] | 157 | |
Roman Stratiienko | 9362cef | 2022-02-02 09:53:50 +0200 | [diff] [blame] | 158 | if (plane->AtomicSetState(*pset, layer, joining.z_pos, crtc->GetId()) != |
Roman Stratiienko | 10be875 | 2022-01-30 20:28:46 +0200 | [diff] [blame] | 159 | 0) { |
Roman Stratiienko | 67cebf5 | 2021-12-17 13:31:27 +0200 | [diff] [blame] | 160 | return -EINVAL; |
Sean Paul | 39b3784 | 2016-05-11 13:50:28 -0400 | [diff] [blame] | 161 | } |
Roman Stratiienko | dccc6fb | 2021-10-23 17:35:44 +0300 | [diff] [blame] | 162 | } |
| 163 | } |
| 164 | |
Roman Stratiienko | ef5348b | 2022-02-09 17:19:56 +0200 | [diff] [blame] | 165 | if (args.composition) { |
Roman Stratiienko | 9362cef | 2022-02-02 09:53:50 +0200 | [diff] [blame] | 166 | for (auto &plane : unused_planes) { |
| 167 | if (plane->Get()->AtomicDisablePlane(*pset) != 0) { |
Roman Stratiienko | 0dbe639 | 2021-09-29 12:47:16 +0300 | [diff] [blame] | 168 | return -EINVAL; |
Matvii Zorin | 8338c34 | 2020-09-08 16:12:51 +0300 | [diff] [blame] | 169 | } |
| 170 | } |
Sean Paul | 98e73c8 | 2015-06-24 14:38:49 -0700 | [diff] [blame] | 171 | } |
| 172 | |
Roman Stratiienko | dccc6fb | 2021-10-23 17:35:44 +0300 | [diff] [blame] | 173 | uint32_t flags = DRM_MODE_ATOMIC_ALLOW_MODESET; |
Roman Stratiienko | 59bb481 | 2022-04-09 16:13:09 +0300 | [diff] [blame] | 174 | |
| 175 | if (args.test_only) { |
Roman Stratiienko | 7689278 | 2023-01-16 17:15:53 +0200 | [diff] [blame] | 176 | return drmModeAtomicCommit(*drm->GetFd(), pset.get(), |
Roman Stratiienko | 59bb481 | 2022-04-09 16:13:09 +0300 | [diff] [blame] | 177 | flags | DRM_MODE_ATOMIC_TEST_ONLY, drm); |
| 178 | } |
| 179 | |
| 180 | if (last_present_fence_) { |
Roman Stratiienko | a7913de | 2022-10-20 13:18:57 +0300 | [diff] [blame] | 181 | // NOLINTNEXTLINE(misc-const-correctness) |
Roman Stratiienko | 59bb481 | 2022-04-09 16:13:09 +0300 | [diff] [blame] | 182 | ATRACE_NAME("WaitPriorFramePresented"); |
| 183 | |
| 184 | constexpr int kTimeoutMs = 500; |
Roman Stratiienko | 7689278 | 2023-01-16 17:15:53 +0200 | [diff] [blame] | 185 | const int err = sync_wait(*last_present_fence_, kTimeoutMs); |
Roman Stratiienko | 59bb481 | 2022-04-09 16:13:09 +0300 | [diff] [blame] | 186 | if (err != 0) { |
Roman Stratiienko | 7689278 | 2023-01-16 17:15:53 +0200 | [diff] [blame] | 187 | ALOGE("sync_wait(fd=%i) returned: %i (errno: %i)", *last_present_fence_, |
| 188 | err, errno); |
Roman Stratiienko | 59bb481 | 2022-04-09 16:13:09 +0300 | [diff] [blame] | 189 | } |
| 190 | |
| 191 | CleanupPriorFrameResources(); |
| 192 | } |
| 193 | |
| 194 | if (nonblock) { |
| 195 | flags |= DRM_MODE_ATOMIC_NONBLOCK; |
| 196 | } |
Sean Paul | c07b211 | 2015-11-17 16:38:10 -0500 | [diff] [blame] | 197 | |
Roman Stratiienko | 7689278 | 2023-01-16 17:15:53 +0200 | [diff] [blame] | 198 | auto err = drmModeAtomicCommit(*drm->GetFd(), pset.get(), flags, drm); |
Roman Stratiienko | 59bb481 | 2022-04-09 16:13:09 +0300 | [diff] [blame] | 199 | |
Roman Stratiienko | 4e99405 | 2022-02-09 17:40:35 +0200 | [diff] [blame] | 200 | if (err != 0) { |
Roman Stratiienko | 59bb481 | 2022-04-09 16:13:09 +0300 | [diff] [blame] | 201 | ALOGE("Failed to commit pset ret=%d\n", err); |
Roman Stratiienko | dccc6fb | 2021-10-23 17:35:44 +0300 | [diff] [blame] | 202 | return err; |
Sean Paul | 98e73c8 | 2015-06-24 14:38:49 -0700 | [diff] [blame] | 203 | } |
Sean Paul | 98e73c8 | 2015-06-24 14:38:49 -0700 | [diff] [blame] | 204 | |
Roman Stratiienko | 7689278 | 2023-01-16 17:15:53 +0200 | [diff] [blame] | 205 | args.out_fence = MakeSharedFd(out_fence); |
| 206 | |
Roman Stratiienko | 59bb481 | 2022-04-09 16:13:09 +0300 | [diff] [blame] | 207 | if (nonblock) { |
Roman Stratiienko | f818d4c | 2022-12-28 20:12:19 +0200 | [diff] [blame] | 208 | { |
| 209 | const std::unique_lock lock(mutex_); |
Roman Stratiienko | 7689278 | 2023-01-16 17:15:53 +0200 | [diff] [blame] | 210 | last_present_fence_ = args.out_fence; |
Roman Stratiienko | f818d4c | 2022-12-28 20:12:19 +0200 | [diff] [blame] | 211 | staged_frame_state_ = std::move(new_frame_state); |
| 212 | frames_staged_++; |
| 213 | } |
| 214 | cv_.notify_all(); |
Roman Stratiienko | 59bb481 | 2022-04-09 16:13:09 +0300 | [diff] [blame] | 215 | } else { |
Roman Stratiienko | f815d38 | 2021-12-30 19:23:14 +0200 | [diff] [blame] | 216 | active_frame_state_ = std::move(new_frame_state); |
Robert Foss | a1ade4e | 2017-09-27 19:28:15 +0200 | [diff] [blame] | 217 | } |
| 218 | |
Roman Stratiienko | dccc6fb | 2021-10-23 17:35:44 +0300 | [diff] [blame] | 219 | return 0; |
Sean Paul | 98e73c8 | 2015-06-24 14:38:49 -0700 | [diff] [blame] | 220 | } |
| 221 | |
Roman Stratiienko | f818d4c | 2022-12-28 20:12:19 +0200 | [diff] [blame] | 222 | void DrmAtomicStateManager::ThreadFn( |
| 223 | const std::shared_ptr<DrmAtomicStateManager> &dasm) { |
Roman Stratiienko | 59bb481 | 2022-04-09 16:13:09 +0300 | [diff] [blame] | 224 | int tracking_at_the_moment = -1; |
Roman Stratiienko | f818d4c | 2022-12-28 20:12:19 +0200 | [diff] [blame] | 225 | auto &main_mutex = pipe_->device->GetResMan().GetMainLock(); |
Roman Stratiienko | 59bb481 | 2022-04-09 16:13:09 +0300 | [diff] [blame] | 226 | |
| 227 | for (;;) { |
Roman Stratiienko | 7689278 | 2023-01-16 17:15:53 +0200 | [diff] [blame] | 228 | SharedFd present_fence; |
Roman Stratiienko | 59bb481 | 2022-04-09 16:13:09 +0300 | [diff] [blame] | 229 | |
| 230 | { |
Roman Stratiienko | f818d4c | 2022-12-28 20:12:19 +0200 | [diff] [blame] | 231 | std::unique_lock lk(mutex_); |
| 232 | cv_.wait(lk); |
Roman Stratiienko | 59bb481 | 2022-04-09 16:13:09 +0300 | [diff] [blame] | 233 | |
Roman Stratiienko | f818d4c | 2022-12-28 20:12:19 +0200 | [diff] [blame] | 234 | if (exit_thread_ || dasm.use_count() == 1) |
Roman Stratiienko | 59bb481 | 2022-04-09 16:13:09 +0300 | [diff] [blame] | 235 | break; |
Roman Stratiienko | 59bb481 | 2022-04-09 16:13:09 +0300 | [diff] [blame] | 236 | |
Roman Stratiienko | f818d4c | 2022-12-28 20:12:19 +0200 | [diff] [blame] | 237 | if (frames_staged_ <= tracking_at_the_moment) |
Roman Stratiienko | 59bb481 | 2022-04-09 16:13:09 +0300 | [diff] [blame] | 238 | continue; |
Roman Stratiienko | f818d4c | 2022-12-28 20:12:19 +0200 | [diff] [blame] | 239 | |
| 240 | tracking_at_the_moment = frames_staged_; |
| 241 | |
Roman Stratiienko | 7689278 | 2023-01-16 17:15:53 +0200 | [diff] [blame] | 242 | present_fence = last_present_fence_; |
Roman Stratiienko | f818d4c | 2022-12-28 20:12:19 +0200 | [diff] [blame] | 243 | if (!present_fence) |
| 244 | continue; |
Roman Stratiienko | 59bb481 | 2022-04-09 16:13:09 +0300 | [diff] [blame] | 245 | } |
| 246 | |
| 247 | { |
Roman Stratiienko | a7913de | 2022-10-20 13:18:57 +0300 | [diff] [blame] | 248 | // NOLINTNEXTLINE(misc-const-correctness) |
Roman Stratiienko | 59bb481 | 2022-04-09 16:13:09 +0300 | [diff] [blame] | 249 | ATRACE_NAME("AsyncWaitForBuffersSwap"); |
| 250 | constexpr int kTimeoutMs = 500; |
Roman Stratiienko | 7689278 | 2023-01-16 17:15:53 +0200 | [diff] [blame] | 251 | auto err = sync_wait(*present_fence, kTimeoutMs); |
Roman Stratiienko | 59bb481 | 2022-04-09 16:13:09 +0300 | [diff] [blame] | 252 | if (err != 0) { |
Roman Stratiienko | 7689278 | 2023-01-16 17:15:53 +0200 | [diff] [blame] | 253 | ALOGE("sync_wait(fd=%i) returned: %i (errno: %i)", *present_fence, err, |
| 254 | errno); |
Roman Stratiienko | 59bb481 | 2022-04-09 16:13:09 +0300 | [diff] [blame] | 255 | } |
| 256 | } |
| 257 | |
| 258 | { |
Roman Stratiienko | f818d4c | 2022-12-28 20:12:19 +0200 | [diff] [blame] | 259 | const std::unique_lock mlk(main_mutex); |
| 260 | const std::unique_lock lk(mutex_); |
| 261 | if (exit_thread_) |
Roman Stratiienko | 59bb481 | 2022-04-09 16:13:09 +0300 | [diff] [blame] | 262 | break; |
Roman Stratiienko | 59bb481 | 2022-04-09 16:13:09 +0300 | [diff] [blame] | 263 | |
| 264 | /* If resources is already cleaned-up by main thread, skip */ |
Roman Stratiienko | f818d4c | 2022-12-28 20:12:19 +0200 | [diff] [blame] | 265 | if (tracking_at_the_moment > frames_tracked_) |
| 266 | CleanupPriorFrameResources(); |
Roman Stratiienko | 59bb481 | 2022-04-09 16:13:09 +0300 | [diff] [blame] | 267 | } |
| 268 | } |
Roman Stratiienko | f818d4c | 2022-12-28 20:12:19 +0200 | [diff] [blame] | 269 | |
| 270 | ALOGI("DrmAtomicStateManager thread exit"); |
Roman Stratiienko | 59bb481 | 2022-04-09 16:13:09 +0300 | [diff] [blame] | 271 | } |
| 272 | |
| 273 | void DrmAtomicStateManager::CleanupPriorFrameResources() { |
| 274 | assert(frames_staged_ - frames_tracked_ == 1); |
| 275 | assert(last_present_fence_); |
| 276 | |
Roman Stratiienko | a7913de | 2022-10-20 13:18:57 +0300 | [diff] [blame] | 277 | // NOLINTNEXTLINE(misc-const-correctness) |
Roman Stratiienko | 59bb481 | 2022-04-09 16:13:09 +0300 | [diff] [blame] | 278 | ATRACE_NAME("CleanupPriorFrameResources"); |
| 279 | frames_tracked_++; |
| 280 | active_frame_state_ = std::move(staged_frame_state_); |
| 281 | last_present_fence_ = {}; |
| 282 | } |
| 283 | |
Roman Stratiienko | 4e99405 | 2022-02-09 17:40:35 +0200 | [diff] [blame] | 284 | auto DrmAtomicStateManager::ExecuteAtomicCommit(AtomicCommitArgs &args) -> int { |
Roman Stratiienko | a7913de | 2022-10-20 13:18:57 +0300 | [diff] [blame] | 285 | auto err = CommitFrame(args); |
Sean Paul | 137a6a8 | 2016-06-22 22:48:22 -0400 | [diff] [blame] | 286 | |
Roman Stratiienko | dccc6fb | 2021-10-23 17:35:44 +0300 | [diff] [blame] | 287 | if (!args.test_only) { |
Roman Stratiienko | 4e99405 | 2022-02-09 17:40:35 +0200 | [diff] [blame] | 288 | if (err != 0) { |
Roman Stratiienko | 19c162f | 2022-02-01 09:35:08 +0200 | [diff] [blame] | 289 | ALOGE("Composite failed for pipeline %s", |
| 290 | pipe_->connector->Get()->GetName().c_str()); |
Roman Stratiienko | dccc6fb | 2021-10-23 17:35:44 +0300 | [diff] [blame] | 291 | // Disable the hw used by the last active composition. This allows us to |
| 292 | // signal the release fences from that composition to avoid hanging. |
Roman Stratiienko | ef5348b | 2022-02-09 17:19:56 +0200 | [diff] [blame] | 293 | AtomicCommitArgs cl_args{}; |
| 294 | cl_args.composition = std::make_shared<DrmKmsPlan>(); |
Roman Stratiienko | 4e99405 | 2022-02-09 17:40:35 +0200 | [diff] [blame] | 295 | if (CommitFrame(cl_args) != 0) { |
Roman Stratiienko | 19c162f | 2022-02-01 09:35:08 +0200 | [diff] [blame] | 296 | ALOGE("Failed to clean-up active composition for pipeline %s", |
| 297 | pipe_->connector->Get()->GetName().c_str()); |
Roman Stratiienko | dccc6fb | 2021-10-23 17:35:44 +0300 | [diff] [blame] | 298 | } |
| 299 | return err; |
| 300 | } |
Haixia Shi | dda2fab | 2015-10-22 18:12:49 -0700 | [diff] [blame] | 301 | } |
Haixia Shi | dda2fab | 2015-10-22 18:12:49 -0700 | [diff] [blame] | 302 | |
Roman Stratiienko | dccc6fb | 2021-10-23 17:35:44 +0300 | [diff] [blame] | 303 | return err; |
| 304 | } // namespace android |
Roman Stratiienko | 36a7f28 | 2021-10-05 18:17:53 +0300 | [diff] [blame] | 305 | |
Roman Stratiienko | 4e99405 | 2022-02-09 17:40:35 +0200 | [diff] [blame] | 306 | auto DrmAtomicStateManager::ActivateDisplayUsingDPMS() -> int { |
Roman Stratiienko | 7689278 | 2023-01-16 17:15:53 +0200 | [diff] [blame] | 307 | return drmModeConnectorSetProperty(*pipe_->device->GetFd(), |
Roman Stratiienko | 19c162f | 2022-02-01 09:35:08 +0200 | [diff] [blame] | 308 | pipe_->connector->Get()->GetId(), |
| 309 | pipe_->connector->Get() |
| 310 | ->GetDpmsProperty() |
Roman Stratiienko | abd8e53 | 2022-12-06 23:31:33 +0200 | [diff] [blame] | 311 | .GetId(), |
Roman Stratiienko | 19c162f | 2022-02-01 09:35:08 +0200 | [diff] [blame] | 312 | DRM_MODE_DPMS_ON); |
Roman Stratiienko | d37b308 | 2022-01-13 16:37:27 +0200 | [diff] [blame] | 313 | } |
| 314 | |
Sean Paul | f72cccd | 2018-08-27 13:59:08 -0400 | [diff] [blame] | 315 | } // namespace android |