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 |
Roman Stratiienko | 4e99405 | 2022-02-09 17:40:35 +0200 | [diff] [blame] | 20 | #define LOG_TAG "hwc-drm-atomic-state-manager" |
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; |
| 82 | if (!crtc->GetOutFencePtrProperty().AtomicSet(*pset, uint64_t(&out_fence))) { |
Roman Stratiienko | 7fd8f88 | 2021-09-29 12:46:54 +0300 | [diff] [blame] | 83 | return -EINVAL; |
Robert Foss | a1ade4e | 2017-09-27 19:28:15 +0200 | [diff] [blame] | 84 | } |
| 85 | |
Roman Stratiienko | 59bb481 | 2022-04-09 16:13:09 +0300 | [diff] [blame] | 86 | bool nonblock = true; |
| 87 | |
Roman Stratiienko | dccc6fb | 2021-10-23 17:35:44 +0300 | [diff] [blame] | 88 | if (args.active) { |
Roman Stratiienko | 59bb481 | 2022-04-09 16:13:09 +0300 | [diff] [blame] | 89 | nonblock = false; |
Roman Stratiienko | 32685ba | 2021-12-15 13:46:05 +0200 | [diff] [blame] | 90 | new_frame_state.crtc_active_state = *args.active; |
Roman Stratiienko | 4e99405 | 2022-02-09 17:40:35 +0200 | [diff] [blame] | 91 | if (!crtc->GetActiveProperty().AtomicSet(*pset, *args.active ? 1 : 0) || |
Roman Stratiienko | 650299a | 2022-01-30 23:46:10 +0200 | [diff] [blame] | 92 | !connector->GetCrtcIdProperty().AtomicSet(*pset, crtc->GetId())) { |
Roman Stratiienko | dccc6fb | 2021-10-23 17:35:44 +0300 | [diff] [blame] | 93 | return -EINVAL; |
| 94 | } |
Sean Paul | 5735541 | 2015-09-19 09:14:34 -0400 | [diff] [blame] | 95 | } |
| 96 | |
Roman Stratiienko | dccc6fb | 2021-10-23 17:35:44 +0300 | [diff] [blame] | 97 | if (args.display_mode) { |
Roman Stratiienko | 19c162f | 2022-02-01 09:35:08 +0200 | [diff] [blame] | 98 | new_frame_state.mode_blob = args.display_mode.value().CreateModeBlob(*drm); |
Zach Reizner | 92f8e63 | 2015-10-12 17:47:13 -0700 | [diff] [blame] | 99 | |
Roman Stratiienko | 32685ba | 2021-12-15 13:46:05 +0200 | [diff] [blame] | 100 | if (!new_frame_state.mode_blob) { |
Roman Stratiienko | dccc6fb | 2021-10-23 17:35:44 +0300 | [diff] [blame] | 101 | ALOGE("Failed to create mode_blob"); |
| 102 | return -EINVAL; |
| 103 | } |
Sean Paul | ca699be | 2016-05-11 16:29:45 -0400 | [diff] [blame] | 104 | |
Roman Stratiienko | 10be875 | 2022-01-30 20:28:46 +0200 | [diff] [blame] | 105 | if (!crtc->GetModeProperty().AtomicSet(*pset, *new_frame_state.mode_blob)) { |
Roman Stratiienko | dccc6fb | 2021-10-23 17:35:44 +0300 | [diff] [blame] | 106 | return -EINVAL; |
| 107 | } |
| 108 | } |
Roman Stratiienko | 0dbe639 | 2021-09-29 12:47:16 +0300 | [diff] [blame] | 109 | |
Roman Stratiienko | 67cebf5 | 2021-12-17 13:31:27 +0200 | [diff] [blame] | 110 | auto unused_planes = new_frame_state.used_planes; |
| 111 | |
Roman Stratiienko | dccc6fb | 2021-10-23 17:35:44 +0300 | [diff] [blame] | 112 | if (args.composition) { |
Roman Stratiienko | 32685ba | 2021-12-15 13:46:05 +0200 | [diff] [blame] | 113 | new_frame_state.used_planes.clear(); |
| 114 | |
Roman Stratiienko | 9362cef | 2022-02-02 09:53:50 +0200 | [diff] [blame] | 115 | for (auto &joining : args.composition->plan) { |
| 116 | DrmPlane *plane = joining.plane->Get(); |
Roman Stratiienko | 4b2cc48 | 2022-02-21 14:53:58 +0200 | [diff] [blame] | 117 | LayerData &layer = joining.layer; |
Roman Stratiienko | dccc6fb | 2021-10-23 17:35:44 +0300 | [diff] [blame] | 118 | |
Roman Stratiienko | 4b2cc48 | 2022-02-21 14:53:58 +0200 | [diff] [blame] | 119 | new_frame_state.used_framebuffers.emplace_back(layer.fb); |
Roman Stratiienko | 9362cef | 2022-02-02 09:53:50 +0200 | [diff] [blame] | 120 | new_frame_state.used_planes.emplace_back(joining.plane); |
Roman Stratiienko | 32685ba | 2021-12-15 13:46:05 +0200 | [diff] [blame] | 121 | |
Roman Stratiienko | 67cebf5 | 2021-12-17 13:31:27 +0200 | [diff] [blame] | 122 | /* Remove from 'unused' list, since plane is re-used */ |
| 123 | auto &v = unused_planes; |
Roman Stratiienko | 9362cef | 2022-02-02 09:53:50 +0200 | [diff] [blame] | 124 | v.erase(std::remove(v.begin(), v.end(), joining.plane), v.end()); |
Roman Stratiienko | 67cebf5 | 2021-12-17 13:31:27 +0200 | [diff] [blame] | 125 | |
Roman Stratiienko | 9362cef | 2022-02-02 09:53:50 +0200 | [diff] [blame] | 126 | if (plane->AtomicSetState(*pset, layer, joining.z_pos, crtc->GetId()) != |
Roman Stratiienko | 10be875 | 2022-01-30 20:28:46 +0200 | [diff] [blame] | 127 | 0) { |
Roman Stratiienko | 67cebf5 | 2021-12-17 13:31:27 +0200 | [diff] [blame] | 128 | return -EINVAL; |
Sean Paul | 39b3784 | 2016-05-11 13:50:28 -0400 | [diff] [blame] | 129 | } |
Roman Stratiienko | dccc6fb | 2021-10-23 17:35:44 +0300 | [diff] [blame] | 130 | } |
| 131 | } |
| 132 | |
Roman Stratiienko | ef5348b | 2022-02-09 17:19:56 +0200 | [diff] [blame] | 133 | if (args.composition) { |
Roman Stratiienko | 9362cef | 2022-02-02 09:53:50 +0200 | [diff] [blame] | 134 | for (auto &plane : unused_planes) { |
| 135 | if (plane->Get()->AtomicDisablePlane(*pset) != 0) { |
Roman Stratiienko | 0dbe639 | 2021-09-29 12:47:16 +0300 | [diff] [blame] | 136 | return -EINVAL; |
Matvii Zorin | 8338c34 | 2020-09-08 16:12:51 +0300 | [diff] [blame] | 137 | } |
| 138 | } |
Sean Paul | 98e73c8 | 2015-06-24 14:38:49 -0700 | [diff] [blame] | 139 | } |
| 140 | |
Roman Stratiienko | dccc6fb | 2021-10-23 17:35:44 +0300 | [diff] [blame] | 141 | uint32_t flags = DRM_MODE_ATOMIC_ALLOW_MODESET; |
Roman Stratiienko | 59bb481 | 2022-04-09 16:13:09 +0300 | [diff] [blame] | 142 | |
| 143 | if (args.test_only) { |
Roman Stratiienko | 7689278 | 2023-01-16 17:15:53 +0200 | [diff] [blame] | 144 | return drmModeAtomicCommit(*drm->GetFd(), pset.get(), |
Roman Stratiienko | 59bb481 | 2022-04-09 16:13:09 +0300 | [diff] [blame] | 145 | flags | DRM_MODE_ATOMIC_TEST_ONLY, drm); |
| 146 | } |
| 147 | |
| 148 | if (last_present_fence_) { |
Roman Stratiienko | a7913de | 2022-10-20 13:18:57 +0300 | [diff] [blame] | 149 | // NOLINTNEXTLINE(misc-const-correctness) |
Roman Stratiienko | 59bb481 | 2022-04-09 16:13:09 +0300 | [diff] [blame] | 150 | ATRACE_NAME("WaitPriorFramePresented"); |
| 151 | |
| 152 | constexpr int kTimeoutMs = 500; |
Roman Stratiienko | 7689278 | 2023-01-16 17:15:53 +0200 | [diff] [blame] | 153 | const int err = sync_wait(*last_present_fence_, kTimeoutMs); |
Roman Stratiienko | 59bb481 | 2022-04-09 16:13:09 +0300 | [diff] [blame] | 154 | if (err != 0) { |
Roman Stratiienko | 7689278 | 2023-01-16 17:15:53 +0200 | [diff] [blame] | 155 | ALOGE("sync_wait(fd=%i) returned: %i (errno: %i)", *last_present_fence_, |
| 156 | err, errno); |
Roman Stratiienko | 59bb481 | 2022-04-09 16:13:09 +0300 | [diff] [blame] | 157 | } |
| 158 | |
| 159 | CleanupPriorFrameResources(); |
| 160 | } |
| 161 | |
| 162 | if (nonblock) { |
| 163 | flags |= DRM_MODE_ATOMIC_NONBLOCK; |
| 164 | } |
Sean Paul | c07b211 | 2015-11-17 16:38:10 -0500 | [diff] [blame] | 165 | |
Roman Stratiienko | 7689278 | 2023-01-16 17:15:53 +0200 | [diff] [blame] | 166 | auto err = drmModeAtomicCommit(*drm->GetFd(), pset.get(), flags, drm); |
Roman Stratiienko | 59bb481 | 2022-04-09 16:13:09 +0300 | [diff] [blame] | 167 | |
Roman Stratiienko | 4e99405 | 2022-02-09 17:40:35 +0200 | [diff] [blame] | 168 | if (err != 0) { |
Roman Stratiienko | 59bb481 | 2022-04-09 16:13:09 +0300 | [diff] [blame] | 169 | ALOGE("Failed to commit pset ret=%d\n", err); |
Roman Stratiienko | dccc6fb | 2021-10-23 17:35:44 +0300 | [diff] [blame] | 170 | return err; |
Sean Paul | 98e73c8 | 2015-06-24 14:38:49 -0700 | [diff] [blame] | 171 | } |
Sean Paul | 98e73c8 | 2015-06-24 14:38:49 -0700 | [diff] [blame] | 172 | |
Roman Stratiienko | 7689278 | 2023-01-16 17:15:53 +0200 | [diff] [blame] | 173 | args.out_fence = MakeSharedFd(out_fence); |
| 174 | |
Roman Stratiienko | 59bb481 | 2022-04-09 16:13:09 +0300 | [diff] [blame] | 175 | if (nonblock) { |
Roman Stratiienko | f818d4c | 2022-12-28 20:12:19 +0200 | [diff] [blame] | 176 | { |
| 177 | const std::unique_lock lock(mutex_); |
Roman Stratiienko | 7689278 | 2023-01-16 17:15:53 +0200 | [diff] [blame] | 178 | last_present_fence_ = args.out_fence; |
Roman Stratiienko | f818d4c | 2022-12-28 20:12:19 +0200 | [diff] [blame] | 179 | staged_frame_state_ = std::move(new_frame_state); |
| 180 | frames_staged_++; |
| 181 | } |
| 182 | cv_.notify_all(); |
Roman Stratiienko | 59bb481 | 2022-04-09 16:13:09 +0300 | [diff] [blame] | 183 | } else { |
Roman Stratiienko | f815d38 | 2021-12-30 19:23:14 +0200 | [diff] [blame] | 184 | active_frame_state_ = std::move(new_frame_state); |
Robert Foss | a1ade4e | 2017-09-27 19:28:15 +0200 | [diff] [blame] | 185 | } |
| 186 | |
Roman Stratiienko | dccc6fb | 2021-10-23 17:35:44 +0300 | [diff] [blame] | 187 | return 0; |
Sean Paul | 98e73c8 | 2015-06-24 14:38:49 -0700 | [diff] [blame] | 188 | } |
| 189 | |
Roman Stratiienko | f818d4c | 2022-12-28 20:12:19 +0200 | [diff] [blame] | 190 | void DrmAtomicStateManager::ThreadFn( |
| 191 | const std::shared_ptr<DrmAtomicStateManager> &dasm) { |
Roman Stratiienko | 59bb481 | 2022-04-09 16:13:09 +0300 | [diff] [blame] | 192 | int tracking_at_the_moment = -1; |
Roman Stratiienko | f818d4c | 2022-12-28 20:12:19 +0200 | [diff] [blame] | 193 | auto &main_mutex = pipe_->device->GetResMan().GetMainLock(); |
Roman Stratiienko | 59bb481 | 2022-04-09 16:13:09 +0300 | [diff] [blame] | 194 | |
| 195 | for (;;) { |
Roman Stratiienko | 7689278 | 2023-01-16 17:15:53 +0200 | [diff] [blame] | 196 | SharedFd present_fence; |
Roman Stratiienko | 59bb481 | 2022-04-09 16:13:09 +0300 | [diff] [blame] | 197 | |
| 198 | { |
Roman Stratiienko | f818d4c | 2022-12-28 20:12:19 +0200 | [diff] [blame] | 199 | std::unique_lock lk(mutex_); |
| 200 | cv_.wait(lk); |
Roman Stratiienko | 59bb481 | 2022-04-09 16:13:09 +0300 | [diff] [blame] | 201 | |
Roman Stratiienko | f818d4c | 2022-12-28 20:12:19 +0200 | [diff] [blame] | 202 | if (exit_thread_ || dasm.use_count() == 1) |
Roman Stratiienko | 59bb481 | 2022-04-09 16:13:09 +0300 | [diff] [blame] | 203 | break; |
Roman Stratiienko | 59bb481 | 2022-04-09 16:13:09 +0300 | [diff] [blame] | 204 | |
Roman Stratiienko | f818d4c | 2022-12-28 20:12:19 +0200 | [diff] [blame] | 205 | if (frames_staged_ <= tracking_at_the_moment) |
Roman Stratiienko | 59bb481 | 2022-04-09 16:13:09 +0300 | [diff] [blame] | 206 | continue; |
Roman Stratiienko | f818d4c | 2022-12-28 20:12:19 +0200 | [diff] [blame] | 207 | |
| 208 | tracking_at_the_moment = frames_staged_; |
| 209 | |
Roman Stratiienko | 7689278 | 2023-01-16 17:15:53 +0200 | [diff] [blame] | 210 | present_fence = last_present_fence_; |
Roman Stratiienko | f818d4c | 2022-12-28 20:12:19 +0200 | [diff] [blame] | 211 | if (!present_fence) |
| 212 | continue; |
Roman Stratiienko | 59bb481 | 2022-04-09 16:13:09 +0300 | [diff] [blame] | 213 | } |
| 214 | |
| 215 | { |
Roman Stratiienko | a7913de | 2022-10-20 13:18:57 +0300 | [diff] [blame] | 216 | // NOLINTNEXTLINE(misc-const-correctness) |
Roman Stratiienko | 59bb481 | 2022-04-09 16:13:09 +0300 | [diff] [blame] | 217 | ATRACE_NAME("AsyncWaitForBuffersSwap"); |
| 218 | constexpr int kTimeoutMs = 500; |
Roman Stratiienko | 7689278 | 2023-01-16 17:15:53 +0200 | [diff] [blame] | 219 | auto err = sync_wait(*present_fence, kTimeoutMs); |
Roman Stratiienko | 59bb481 | 2022-04-09 16:13:09 +0300 | [diff] [blame] | 220 | if (err != 0) { |
Roman Stratiienko | 7689278 | 2023-01-16 17:15:53 +0200 | [diff] [blame] | 221 | ALOGE("sync_wait(fd=%i) returned: %i (errno: %i)", *present_fence, err, |
| 222 | errno); |
Roman Stratiienko | 59bb481 | 2022-04-09 16:13:09 +0300 | [diff] [blame] | 223 | } |
| 224 | } |
| 225 | |
| 226 | { |
Roman Stratiienko | f818d4c | 2022-12-28 20:12:19 +0200 | [diff] [blame] | 227 | const std::unique_lock mlk(main_mutex); |
| 228 | const std::unique_lock lk(mutex_); |
| 229 | if (exit_thread_) |
Roman Stratiienko | 59bb481 | 2022-04-09 16:13:09 +0300 | [diff] [blame] | 230 | break; |
Roman Stratiienko | 59bb481 | 2022-04-09 16:13:09 +0300 | [diff] [blame] | 231 | |
| 232 | /* If resources is already cleaned-up by main thread, skip */ |
Roman Stratiienko | f818d4c | 2022-12-28 20:12:19 +0200 | [diff] [blame] | 233 | if (tracking_at_the_moment > frames_tracked_) |
| 234 | CleanupPriorFrameResources(); |
Roman Stratiienko | 59bb481 | 2022-04-09 16:13:09 +0300 | [diff] [blame] | 235 | } |
| 236 | } |
Roman Stratiienko | f818d4c | 2022-12-28 20:12:19 +0200 | [diff] [blame] | 237 | |
| 238 | ALOGI("DrmAtomicStateManager thread exit"); |
Roman Stratiienko | 59bb481 | 2022-04-09 16:13:09 +0300 | [diff] [blame] | 239 | } |
| 240 | |
| 241 | void DrmAtomicStateManager::CleanupPriorFrameResources() { |
| 242 | assert(frames_staged_ - frames_tracked_ == 1); |
| 243 | assert(last_present_fence_); |
| 244 | |
Roman Stratiienko | a7913de | 2022-10-20 13:18:57 +0300 | [diff] [blame] | 245 | // NOLINTNEXTLINE(misc-const-correctness) |
Roman Stratiienko | 59bb481 | 2022-04-09 16:13:09 +0300 | [diff] [blame] | 246 | ATRACE_NAME("CleanupPriorFrameResources"); |
| 247 | frames_tracked_++; |
| 248 | active_frame_state_ = std::move(staged_frame_state_); |
| 249 | last_present_fence_ = {}; |
| 250 | } |
| 251 | |
Roman Stratiienko | 4e99405 | 2022-02-09 17:40:35 +0200 | [diff] [blame] | 252 | auto DrmAtomicStateManager::ExecuteAtomicCommit(AtomicCommitArgs &args) -> int { |
Roman Stratiienko | a7913de | 2022-10-20 13:18:57 +0300 | [diff] [blame] | 253 | auto err = CommitFrame(args); |
Sean Paul | 137a6a8 | 2016-06-22 22:48:22 -0400 | [diff] [blame] | 254 | |
Roman Stratiienko | dccc6fb | 2021-10-23 17:35:44 +0300 | [diff] [blame] | 255 | if (!args.test_only) { |
Roman Stratiienko | 4e99405 | 2022-02-09 17:40:35 +0200 | [diff] [blame] | 256 | if (err != 0) { |
Roman Stratiienko | 19c162f | 2022-02-01 09:35:08 +0200 | [diff] [blame] | 257 | ALOGE("Composite failed for pipeline %s", |
| 258 | pipe_->connector->Get()->GetName().c_str()); |
Roman Stratiienko | dccc6fb | 2021-10-23 17:35:44 +0300 | [diff] [blame] | 259 | // Disable the hw used by the last active composition. This allows us to |
| 260 | // signal the release fences from that composition to avoid hanging. |
Roman Stratiienko | ef5348b | 2022-02-09 17:19:56 +0200 | [diff] [blame] | 261 | AtomicCommitArgs cl_args{}; |
| 262 | cl_args.composition = std::make_shared<DrmKmsPlan>(); |
Roman Stratiienko | 4e99405 | 2022-02-09 17:40:35 +0200 | [diff] [blame] | 263 | if (CommitFrame(cl_args) != 0) { |
Roman Stratiienko | 19c162f | 2022-02-01 09:35:08 +0200 | [diff] [blame] | 264 | ALOGE("Failed to clean-up active composition for pipeline %s", |
| 265 | pipe_->connector->Get()->GetName().c_str()); |
Roman Stratiienko | dccc6fb | 2021-10-23 17:35:44 +0300 | [diff] [blame] | 266 | } |
| 267 | return err; |
| 268 | } |
Haixia Shi | dda2fab | 2015-10-22 18:12:49 -0700 | [diff] [blame] | 269 | } |
Haixia Shi | dda2fab | 2015-10-22 18:12:49 -0700 | [diff] [blame] | 270 | |
Roman Stratiienko | dccc6fb | 2021-10-23 17:35:44 +0300 | [diff] [blame] | 271 | return err; |
| 272 | } // namespace android |
Roman Stratiienko | 36a7f28 | 2021-10-05 18:17:53 +0300 | [diff] [blame] | 273 | |
Roman Stratiienko | 4e99405 | 2022-02-09 17:40:35 +0200 | [diff] [blame] | 274 | auto DrmAtomicStateManager::ActivateDisplayUsingDPMS() -> int { |
Roman Stratiienko | 7689278 | 2023-01-16 17:15:53 +0200 | [diff] [blame] | 275 | return drmModeConnectorSetProperty(*pipe_->device->GetFd(), |
Roman Stratiienko | 19c162f | 2022-02-01 09:35:08 +0200 | [diff] [blame] | 276 | pipe_->connector->Get()->GetId(), |
| 277 | pipe_->connector->Get() |
| 278 | ->GetDpmsProperty() |
Roman Stratiienko | abd8e53 | 2022-12-06 23:31:33 +0200 | [diff] [blame] | 279 | .GetId(), |
Roman Stratiienko | 19c162f | 2022-02-01 09:35:08 +0200 | [diff] [blame] | 280 | DRM_MODE_DPMS_ON); |
Roman Stratiienko | d37b308 | 2022-01-13 16:37:27 +0200 | [diff] [blame] | 281 | } |
| 282 | |
Sean Paul | f72cccd | 2018-08-27 13:59:08 -0400 | [diff] [blame] | 283 | } // namespace android |