Daniel Vetter | cc4ceb4 | 2014-07-25 21:30:38 +0200 | [diff] [blame] | 1 | /* |
| 2 | * Copyright (C) 2014 Red Hat |
| 3 | * Copyright (C) 2014 Intel Corp. |
| 4 | * |
| 5 | * Permission is hereby granted, free of charge, to any person obtaining a |
| 6 | * copy of this software and associated documentation files (the "Software"), |
| 7 | * to deal in the Software without restriction, including without limitation |
| 8 | * the rights to use, copy, modify, merge, publish, distribute, sublicense, |
| 9 | * and/or sell copies of the Software, and to permit persons to whom the |
| 10 | * Software is furnished to do so, subject to the following conditions: |
| 11 | * |
| 12 | * The above copyright notice and this permission notice shall be included in |
| 13 | * all copies or substantial portions of the Software. |
| 14 | * |
| 15 | * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR |
| 16 | * IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY, |
| 17 | * FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL |
| 18 | * THE COPYRIGHT HOLDER(S) OR AUTHOR(S) BE LIABLE FOR ANY CLAIM, DAMAGES OR |
| 19 | * OTHER LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, |
| 20 | * ARISING FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR |
| 21 | * OTHER DEALINGS IN THE SOFTWARE. |
| 22 | * |
| 23 | * Authors: |
| 24 | * Rob Clark <robdclark@gmail.com> |
| 25 | * Daniel Vetter <daniel.vetter@ffwll.ch> |
| 26 | */ |
| 27 | |
| 28 | |
| 29 | #include <drm/drmP.h> |
| 30 | #include <drm/drm_atomic.h> |
Lionel Landwerlin | 5488dc1 | 2016-02-26 17:05:00 +0000 | [diff] [blame] | 31 | #include <drm/drm_mode.h> |
Daniel Vetter | cc4ceb4 | 2014-07-25 21:30:38 +0200 | [diff] [blame] | 32 | #include <drm/drm_plane_helper.h> |
| 33 | |
Thierry Reding | be35f94 | 2016-04-28 15:19:56 +0200 | [diff] [blame] | 34 | #include "drm_crtc_internal.h" |
| 35 | |
Daniel Vetter | 3b24f7d | 2016-06-08 14:19:00 +0200 | [diff] [blame] | 36 | static void crtc_commit_free(struct kref *kref) |
| 37 | { |
| 38 | struct drm_crtc_commit *commit = |
| 39 | container_of(kref, struct drm_crtc_commit, ref); |
| 40 | |
| 41 | kfree(commit); |
| 42 | } |
| 43 | |
| 44 | void drm_crtc_commit_put(struct drm_crtc_commit *commit) |
| 45 | { |
| 46 | kref_put(&commit->ref, crtc_commit_free); |
| 47 | } |
| 48 | EXPORT_SYMBOL(drm_crtc_commit_put); |
| 49 | |
Maarten Lankhorst | 036ef57 | 2015-05-18 10:06:40 +0200 | [diff] [blame] | 50 | /** |
| 51 | * drm_atomic_state_default_release - |
| 52 | * release memory initialized by drm_atomic_state_init |
| 53 | * @state: atomic state |
| 54 | * |
| 55 | * Free all the memory allocated by drm_atomic_state_init. |
| 56 | * This is useful for drivers that subclass the atomic state. |
| 57 | */ |
| 58 | void drm_atomic_state_default_release(struct drm_atomic_state *state) |
Daniel Vetter | cc4ceb4 | 2014-07-25 21:30:38 +0200 | [diff] [blame] | 59 | { |
| 60 | kfree(state->connectors); |
Daniel Vetter | cc4ceb4 | 2014-07-25 21:30:38 +0200 | [diff] [blame] | 61 | kfree(state->crtcs); |
Daniel Vetter | cc4ceb4 | 2014-07-25 21:30:38 +0200 | [diff] [blame] | 62 | kfree(state->planes); |
Daniel Vetter | cc4ceb4 | 2014-07-25 21:30:38 +0200 | [diff] [blame] | 63 | } |
Maarten Lankhorst | 036ef57 | 2015-05-18 10:06:40 +0200 | [diff] [blame] | 64 | EXPORT_SYMBOL(drm_atomic_state_default_release); |
Daniel Vetter | cc4ceb4 | 2014-07-25 21:30:38 +0200 | [diff] [blame] | 65 | |
| 66 | /** |
Maarten Lankhorst | 036ef57 | 2015-05-18 10:06:40 +0200 | [diff] [blame] | 67 | * drm_atomic_state_init - init new atomic state |
Daniel Vetter | cc4ceb4 | 2014-07-25 21:30:38 +0200 | [diff] [blame] | 68 | * @dev: DRM device |
Maarten Lankhorst | 036ef57 | 2015-05-18 10:06:40 +0200 | [diff] [blame] | 69 | * @state: atomic state |
Daniel Vetter | cc4ceb4 | 2014-07-25 21:30:38 +0200 | [diff] [blame] | 70 | * |
Maarten Lankhorst | 036ef57 | 2015-05-18 10:06:40 +0200 | [diff] [blame] | 71 | * Default implementation for filling in a new atomic state. |
| 72 | * This is useful for drivers that subclass the atomic state. |
Daniel Vetter | cc4ceb4 | 2014-07-25 21:30:38 +0200 | [diff] [blame] | 73 | */ |
Maarten Lankhorst | 036ef57 | 2015-05-18 10:06:40 +0200 | [diff] [blame] | 74 | int |
| 75 | drm_atomic_state_init(struct drm_device *dev, struct drm_atomic_state *state) |
Daniel Vetter | cc4ceb4 | 2014-07-25 21:30:38 +0200 | [diff] [blame] | 76 | { |
Chris Wilson | 0853695 | 2016-10-14 13:18:18 +0100 | [diff] [blame] | 77 | kref_init(&state->ref); |
| 78 | |
Rob Clark | d34f20d | 2014-12-18 16:01:56 -0500 | [diff] [blame] | 79 | /* TODO legacy paths should maybe do a better job about |
| 80 | * setting this appropriately? |
| 81 | */ |
| 82 | state->allow_modeset = true; |
| 83 | |
Daniel Vetter | cc4ceb4 | 2014-07-25 21:30:38 +0200 | [diff] [blame] | 84 | state->crtcs = kcalloc(dev->mode_config.num_crtc, |
| 85 | sizeof(*state->crtcs), GFP_KERNEL); |
| 86 | if (!state->crtcs) |
| 87 | goto fail; |
Daniel Vetter | cc4ceb4 | 2014-07-25 21:30:38 +0200 | [diff] [blame] | 88 | state->planes = kcalloc(dev->mode_config.num_total_plane, |
| 89 | sizeof(*state->planes), GFP_KERNEL); |
| 90 | if (!state->planes) |
| 91 | goto fail; |
Daniel Vetter | cc4ceb4 | 2014-07-25 21:30:38 +0200 | [diff] [blame] | 92 | |
| 93 | state->dev = dev; |
| 94 | |
Maarten Lankhorst | 036ef57 | 2015-05-18 10:06:40 +0200 | [diff] [blame] | 95 | DRM_DEBUG_ATOMIC("Allocated atomic state %p\n", state); |
Daniel Vetter | cc4ceb4 | 2014-07-25 21:30:38 +0200 | [diff] [blame] | 96 | |
Maarten Lankhorst | 036ef57 | 2015-05-18 10:06:40 +0200 | [diff] [blame] | 97 | return 0; |
Daniel Vetter | cc4ceb4 | 2014-07-25 21:30:38 +0200 | [diff] [blame] | 98 | fail: |
Maarten Lankhorst | 036ef57 | 2015-05-18 10:06:40 +0200 | [diff] [blame] | 99 | drm_atomic_state_default_release(state); |
| 100 | return -ENOMEM; |
| 101 | } |
| 102 | EXPORT_SYMBOL(drm_atomic_state_init); |
Daniel Vetter | cc4ceb4 | 2014-07-25 21:30:38 +0200 | [diff] [blame] | 103 | |
Maarten Lankhorst | 036ef57 | 2015-05-18 10:06:40 +0200 | [diff] [blame] | 104 | /** |
| 105 | * drm_atomic_state_alloc - allocate atomic state |
| 106 | * @dev: DRM device |
| 107 | * |
| 108 | * This allocates an empty atomic state to track updates. |
| 109 | */ |
| 110 | struct drm_atomic_state * |
| 111 | drm_atomic_state_alloc(struct drm_device *dev) |
| 112 | { |
| 113 | struct drm_mode_config *config = &dev->mode_config; |
| 114 | struct drm_atomic_state *state; |
| 115 | |
| 116 | if (!config->funcs->atomic_state_alloc) { |
| 117 | state = kzalloc(sizeof(*state), GFP_KERNEL); |
| 118 | if (!state) |
| 119 | return NULL; |
| 120 | if (drm_atomic_state_init(dev, state) < 0) { |
| 121 | kfree(state); |
| 122 | return NULL; |
| 123 | } |
| 124 | return state; |
| 125 | } |
| 126 | |
| 127 | return config->funcs->atomic_state_alloc(dev); |
Daniel Vetter | cc4ceb4 | 2014-07-25 21:30:38 +0200 | [diff] [blame] | 128 | } |
| 129 | EXPORT_SYMBOL(drm_atomic_state_alloc); |
| 130 | |
| 131 | /** |
Maarten Lankhorst | 036ef57 | 2015-05-18 10:06:40 +0200 | [diff] [blame] | 132 | * drm_atomic_state_default_clear - clear base atomic state |
Daniel Vetter | cc4ceb4 | 2014-07-25 21:30:38 +0200 | [diff] [blame] | 133 | * @state: atomic state |
| 134 | * |
Maarten Lankhorst | 036ef57 | 2015-05-18 10:06:40 +0200 | [diff] [blame] | 135 | * Default implementation for clearing atomic state. |
| 136 | * This is useful for drivers that subclass the atomic state. |
Daniel Vetter | cc4ceb4 | 2014-07-25 21:30:38 +0200 | [diff] [blame] | 137 | */ |
Maarten Lankhorst | 036ef57 | 2015-05-18 10:06:40 +0200 | [diff] [blame] | 138 | void drm_atomic_state_default_clear(struct drm_atomic_state *state) |
Daniel Vetter | cc4ceb4 | 2014-07-25 21:30:38 +0200 | [diff] [blame] | 139 | { |
| 140 | struct drm_device *dev = state->dev; |
Daniel Vetter | 6f75cea | 2014-11-19 18:38:07 +0100 | [diff] [blame] | 141 | struct drm_mode_config *config = &dev->mode_config; |
Daniel Vetter | cc4ceb4 | 2014-07-25 21:30:38 +0200 | [diff] [blame] | 142 | int i; |
| 143 | |
Daniel Vetter | 17a38d9 | 2015-02-22 12:24:16 +0100 | [diff] [blame] | 144 | DRM_DEBUG_ATOMIC("Clearing atomic state %p\n", state); |
Daniel Vetter | cc4ceb4 | 2014-07-25 21:30:38 +0200 | [diff] [blame] | 145 | |
Daniel Vetter | f52b69f1 | 2014-11-19 18:38:08 +0100 | [diff] [blame] | 146 | for (i = 0; i < state->num_connector; i++) { |
Daniel Vetter | 63e83c1 | 2016-06-02 00:06:32 +0200 | [diff] [blame] | 147 | struct drm_connector *connector = state->connectors[i].ptr; |
Daniel Vetter | cc4ceb4 | 2014-07-25 21:30:38 +0200 | [diff] [blame] | 148 | |
| 149 | if (!connector) |
| 150 | continue; |
| 151 | |
Dave Airlie | d2307de | 2016-04-27 11:27:39 +1000 | [diff] [blame] | 152 | connector->funcs->atomic_destroy_state(connector, |
Daniel Vetter | 63e83c1 | 2016-06-02 00:06:32 +0200 | [diff] [blame] | 153 | state->connectors[i].state); |
| 154 | state->connectors[i].ptr = NULL; |
| 155 | state->connectors[i].state = NULL; |
Dave Airlie | b164d31 | 2016-04-27 11:10:09 +1000 | [diff] [blame] | 156 | drm_connector_unreference(connector); |
Daniel Vetter | cc4ceb4 | 2014-07-25 21:30:38 +0200 | [diff] [blame] | 157 | } |
| 158 | |
Daniel Vetter | 6f75cea | 2014-11-19 18:38:07 +0100 | [diff] [blame] | 159 | for (i = 0; i < config->num_crtc; i++) { |
Daniel Vetter | 5d943aa6 | 2016-06-02 00:06:34 +0200 | [diff] [blame] | 160 | struct drm_crtc *crtc = state->crtcs[i].ptr; |
Daniel Vetter | cc4ceb4 | 2014-07-25 21:30:38 +0200 | [diff] [blame] | 161 | |
| 162 | if (!crtc) |
| 163 | continue; |
| 164 | |
| 165 | crtc->funcs->atomic_destroy_state(crtc, |
Daniel Vetter | 5d943aa6 | 2016-06-02 00:06:34 +0200 | [diff] [blame] | 166 | state->crtcs[i].state); |
Daniel Vetter | 3b24f7d | 2016-06-08 14:19:00 +0200 | [diff] [blame] | 167 | |
| 168 | if (state->crtcs[i].commit) { |
| 169 | kfree(state->crtcs[i].commit->event); |
| 170 | state->crtcs[i].commit->event = NULL; |
| 171 | drm_crtc_commit_put(state->crtcs[i].commit); |
| 172 | } |
| 173 | |
| 174 | state->crtcs[i].commit = NULL; |
Daniel Vetter | 5d943aa6 | 2016-06-02 00:06:34 +0200 | [diff] [blame] | 175 | state->crtcs[i].ptr = NULL; |
| 176 | state->crtcs[i].state = NULL; |
Daniel Vetter | cc4ceb4 | 2014-07-25 21:30:38 +0200 | [diff] [blame] | 177 | } |
| 178 | |
Daniel Vetter | 6f75cea | 2014-11-19 18:38:07 +0100 | [diff] [blame] | 179 | for (i = 0; i < config->num_total_plane; i++) { |
Daniel Vetter | b8b5342 | 2016-06-02 00:06:33 +0200 | [diff] [blame] | 180 | struct drm_plane *plane = state->planes[i].ptr; |
Daniel Vetter | cc4ceb4 | 2014-07-25 21:30:38 +0200 | [diff] [blame] | 181 | |
| 182 | if (!plane) |
| 183 | continue; |
| 184 | |
| 185 | plane->funcs->atomic_destroy_state(plane, |
Daniel Vetter | b8b5342 | 2016-06-02 00:06:33 +0200 | [diff] [blame] | 186 | state->planes[i].state); |
| 187 | state->planes[i].ptr = NULL; |
| 188 | state->planes[i].state = NULL; |
Daniel Vetter | cc4ceb4 | 2014-07-25 21:30:38 +0200 | [diff] [blame] | 189 | } |
| 190 | } |
Maarten Lankhorst | 036ef57 | 2015-05-18 10:06:40 +0200 | [diff] [blame] | 191 | EXPORT_SYMBOL(drm_atomic_state_default_clear); |
| 192 | |
| 193 | /** |
| 194 | * drm_atomic_state_clear - clear state object |
| 195 | * @state: atomic state |
| 196 | * |
| 197 | * When the w/w mutex algorithm detects a deadlock we need to back off and drop |
| 198 | * all locks. So someone else could sneak in and change the current modeset |
| 199 | * configuration. Which means that all the state assembled in @state is no |
| 200 | * longer an atomic update to the current state, but to some arbitrary earlier |
| 201 | * state. Which could break assumptions the driver's ->atomic_check likely |
| 202 | * relies on. |
| 203 | * |
| 204 | * Hence we must clear all cached state and completely start over, using this |
| 205 | * function. |
| 206 | */ |
| 207 | void drm_atomic_state_clear(struct drm_atomic_state *state) |
| 208 | { |
| 209 | struct drm_device *dev = state->dev; |
| 210 | struct drm_mode_config *config = &dev->mode_config; |
| 211 | |
| 212 | if (config->funcs->atomic_state_clear) |
| 213 | config->funcs->atomic_state_clear(state); |
| 214 | else |
| 215 | drm_atomic_state_default_clear(state); |
| 216 | } |
Daniel Vetter | cc4ceb4 | 2014-07-25 21:30:38 +0200 | [diff] [blame] | 217 | EXPORT_SYMBOL(drm_atomic_state_clear); |
| 218 | |
| 219 | /** |
Chris Wilson | 0853695 | 2016-10-14 13:18:18 +0100 | [diff] [blame] | 220 | * __drm_atomic_state_free - free all memory for an atomic state |
| 221 | * @ref: This atomic state to deallocate |
Daniel Vetter | cc4ceb4 | 2014-07-25 21:30:38 +0200 | [diff] [blame] | 222 | * |
| 223 | * This frees all memory associated with an atomic state, including all the |
| 224 | * per-object state for planes, crtcs and connectors. |
| 225 | */ |
Chris Wilson | 0853695 | 2016-10-14 13:18:18 +0100 | [diff] [blame] | 226 | void __drm_atomic_state_free(struct kref *ref) |
Daniel Vetter | cc4ceb4 | 2014-07-25 21:30:38 +0200 | [diff] [blame] | 227 | { |
Chris Wilson | 0853695 | 2016-10-14 13:18:18 +0100 | [diff] [blame] | 228 | struct drm_atomic_state *state = container_of(ref, typeof(*state), ref); |
| 229 | struct drm_mode_config *config = &state->dev->mode_config; |
Maarten Lankhorst | 036ef57 | 2015-05-18 10:06:40 +0200 | [diff] [blame] | 230 | |
Daniel Vetter | cc4ceb4 | 2014-07-25 21:30:38 +0200 | [diff] [blame] | 231 | drm_atomic_state_clear(state); |
| 232 | |
Daniel Vetter | 17a38d9 | 2015-02-22 12:24:16 +0100 | [diff] [blame] | 233 | DRM_DEBUG_ATOMIC("Freeing atomic state %p\n", state); |
Daniel Vetter | cc4ceb4 | 2014-07-25 21:30:38 +0200 | [diff] [blame] | 234 | |
Maarten Lankhorst | 036ef57 | 2015-05-18 10:06:40 +0200 | [diff] [blame] | 235 | if (config->funcs->atomic_state_free) { |
| 236 | config->funcs->atomic_state_free(state); |
| 237 | } else { |
| 238 | drm_atomic_state_default_release(state); |
| 239 | kfree(state); |
| 240 | } |
Daniel Vetter | cc4ceb4 | 2014-07-25 21:30:38 +0200 | [diff] [blame] | 241 | } |
Chris Wilson | 0853695 | 2016-10-14 13:18:18 +0100 | [diff] [blame] | 242 | EXPORT_SYMBOL(__drm_atomic_state_free); |
Daniel Vetter | cc4ceb4 | 2014-07-25 21:30:38 +0200 | [diff] [blame] | 243 | |
| 244 | /** |
| 245 | * drm_atomic_get_crtc_state - get crtc state |
| 246 | * @state: global atomic state object |
| 247 | * @crtc: crtc to get state object for |
| 248 | * |
| 249 | * This function returns the crtc state for the given crtc, allocating it if |
| 250 | * needed. It will also grab the relevant crtc lock to make sure that the state |
| 251 | * is consistent. |
| 252 | * |
| 253 | * Returns: |
| 254 | * |
| 255 | * Either the allocated state or the error code encoded into the pointer. When |
| 256 | * the error is EDEADLK then the w/w mutex code has detected a deadlock and the |
| 257 | * entire atomic sequence must be restarted. All other errors are fatal. |
| 258 | */ |
| 259 | struct drm_crtc_state * |
| 260 | drm_atomic_get_crtc_state(struct drm_atomic_state *state, |
| 261 | struct drm_crtc *crtc) |
| 262 | { |
Maarten Lankhorst | 1b26a5e | 2015-05-13 10:37:25 +0200 | [diff] [blame] | 263 | int ret, index = drm_crtc_index(crtc); |
Daniel Vetter | cc4ceb4 | 2014-07-25 21:30:38 +0200 | [diff] [blame] | 264 | struct drm_crtc_state *crtc_state; |
| 265 | |
Maarten Lankhorst | 7f4eaa89 | 2016-05-03 11:12:31 +0200 | [diff] [blame] | 266 | WARN_ON(!state->acquire_ctx); |
| 267 | |
Maarten Lankhorst | 1b26a5e | 2015-05-13 10:37:25 +0200 | [diff] [blame] | 268 | crtc_state = drm_atomic_get_existing_crtc_state(state, crtc); |
| 269 | if (crtc_state) |
| 270 | return crtc_state; |
Daniel Vetter | cc4ceb4 | 2014-07-25 21:30:38 +0200 | [diff] [blame] | 271 | |
| 272 | ret = drm_modeset_lock(&crtc->mutex, state->acquire_ctx); |
| 273 | if (ret) |
| 274 | return ERR_PTR(ret); |
| 275 | |
| 276 | crtc_state = crtc->funcs->atomic_duplicate_state(crtc); |
| 277 | if (!crtc_state) |
| 278 | return ERR_PTR(-ENOMEM); |
| 279 | |
Daniel Vetter | 5d943aa6 | 2016-06-02 00:06:34 +0200 | [diff] [blame] | 280 | state->crtcs[index].state = crtc_state; |
| 281 | state->crtcs[index].ptr = crtc; |
Daniel Vetter | cc4ceb4 | 2014-07-25 21:30:38 +0200 | [diff] [blame] | 282 | crtc_state->state = state; |
| 283 | |
Ville Syrjälä | fa3ab4c | 2015-12-08 18:41:53 +0200 | [diff] [blame] | 284 | DRM_DEBUG_ATOMIC("Added [CRTC:%d:%s] %p state to %p\n", |
| 285 | crtc->base.id, crtc->name, crtc_state, state); |
Daniel Vetter | cc4ceb4 | 2014-07-25 21:30:38 +0200 | [diff] [blame] | 286 | |
| 287 | return crtc_state; |
| 288 | } |
| 289 | EXPORT_SYMBOL(drm_atomic_get_crtc_state); |
| 290 | |
| 291 | /** |
Daniel Stone | 819364d | 2015-05-26 14:36:48 +0100 | [diff] [blame] | 292 | * drm_atomic_set_mode_for_crtc - set mode for CRTC |
| 293 | * @state: the CRTC whose incoming state to update |
| 294 | * @mode: kernel-internal mode to use for the CRTC, or NULL to disable |
| 295 | * |
| 296 | * Set a mode (originating from the kernel) on the desired CRTC state. Does |
| 297 | * not change any other state properties, including enable, active, or |
| 298 | * mode_changed. |
| 299 | * |
| 300 | * RETURNS: |
| 301 | * Zero on success, error code on failure. Cannot return -EDEADLK. |
| 302 | */ |
| 303 | int drm_atomic_set_mode_for_crtc(struct drm_crtc_state *state, |
| 304 | struct drm_display_mode *mode) |
| 305 | { |
Daniel Stone | 99cf4a2 | 2015-05-25 19:11:51 +0100 | [diff] [blame] | 306 | struct drm_mode_modeinfo umode; |
| 307 | |
Daniel Stone | 819364d | 2015-05-26 14:36:48 +0100 | [diff] [blame] | 308 | /* Early return for no change. */ |
| 309 | if (mode && memcmp(&state->mode, mode, sizeof(*mode)) == 0) |
| 310 | return 0; |
| 311 | |
Markus Elfring | 5f91190 | 2015-11-06 12:03:46 +0100 | [diff] [blame] | 312 | drm_property_unreference_blob(state->mode_blob); |
Daniel Stone | 99cf4a2 | 2015-05-25 19:11:51 +0100 | [diff] [blame] | 313 | state->mode_blob = NULL; |
| 314 | |
Daniel Stone | 819364d | 2015-05-26 14:36:48 +0100 | [diff] [blame] | 315 | if (mode) { |
Daniel Stone | 99cf4a2 | 2015-05-25 19:11:51 +0100 | [diff] [blame] | 316 | drm_mode_convert_to_umode(&umode, mode); |
| 317 | state->mode_blob = |
| 318 | drm_property_create_blob(state->crtc->dev, |
| 319 | sizeof(umode), |
| 320 | &umode); |
| 321 | if (IS_ERR(state->mode_blob)) |
| 322 | return PTR_ERR(state->mode_blob); |
| 323 | |
Daniel Stone | 819364d | 2015-05-26 14:36:48 +0100 | [diff] [blame] | 324 | drm_mode_copy(&state->mode, mode); |
| 325 | state->enable = true; |
| 326 | DRM_DEBUG_ATOMIC("Set [MODE:%s] for CRTC state %p\n", |
| 327 | mode->name, state); |
| 328 | } else { |
| 329 | memset(&state->mode, 0, sizeof(state->mode)); |
| 330 | state->enable = false; |
| 331 | DRM_DEBUG_ATOMIC("Set [NOMODE] for CRTC state %p\n", |
| 332 | state); |
| 333 | } |
| 334 | |
| 335 | return 0; |
| 336 | } |
| 337 | EXPORT_SYMBOL(drm_atomic_set_mode_for_crtc); |
| 338 | |
Daniel Stone | 819364d | 2015-05-26 14:36:48 +0100 | [diff] [blame] | 339 | /** |
Daniel Stone | 955f3c3 | 2015-05-25 19:11:52 +0100 | [diff] [blame] | 340 | * drm_atomic_set_mode_prop_for_crtc - set mode for CRTC |
| 341 | * @state: the CRTC whose incoming state to update |
| 342 | * @blob: pointer to blob property to use for mode |
| 343 | * |
| 344 | * Set a mode (originating from a blob property) on the desired CRTC state. |
| 345 | * This function will take a reference on the blob property for the CRTC state, |
| 346 | * and release the reference held on the state's existing mode property, if any |
| 347 | * was set. |
| 348 | * |
| 349 | * RETURNS: |
| 350 | * Zero on success, error code on failure. Cannot return -EDEADLK. |
| 351 | */ |
| 352 | int drm_atomic_set_mode_prop_for_crtc(struct drm_crtc_state *state, |
| 353 | struct drm_property_blob *blob) |
| 354 | { |
| 355 | if (blob == state->mode_blob) |
| 356 | return 0; |
| 357 | |
Markus Elfring | 5f91190 | 2015-11-06 12:03:46 +0100 | [diff] [blame] | 358 | drm_property_unreference_blob(state->mode_blob); |
Daniel Stone | 955f3c3 | 2015-05-25 19:11:52 +0100 | [diff] [blame] | 359 | state->mode_blob = NULL; |
| 360 | |
Tomi Valkeinen | 6709887 | 2016-05-31 15:03:17 +0300 | [diff] [blame] | 361 | memset(&state->mode, 0, sizeof(state->mode)); |
| 362 | |
Daniel Stone | 955f3c3 | 2015-05-25 19:11:52 +0100 | [diff] [blame] | 363 | if (blob) { |
| 364 | if (blob->length != sizeof(struct drm_mode_modeinfo) || |
| 365 | drm_mode_convert_umode(&state->mode, |
| 366 | (const struct drm_mode_modeinfo *) |
| 367 | blob->data)) |
| 368 | return -EINVAL; |
| 369 | |
| 370 | state->mode_blob = drm_property_reference_blob(blob); |
| 371 | state->enable = true; |
| 372 | DRM_DEBUG_ATOMIC("Set [MODE:%s] for CRTC state %p\n", |
| 373 | state->mode.name, state); |
| 374 | } else { |
Daniel Stone | 955f3c3 | 2015-05-25 19:11:52 +0100 | [diff] [blame] | 375 | state->enable = false; |
| 376 | DRM_DEBUG_ATOMIC("Set [NOMODE] for CRTC state %p\n", |
| 377 | state); |
| 378 | } |
| 379 | |
| 380 | return 0; |
| 381 | } |
| 382 | EXPORT_SYMBOL(drm_atomic_set_mode_prop_for_crtc); |
| 383 | |
| 384 | /** |
Lionel Landwerlin | 5488dc1 | 2016-02-26 17:05:00 +0000 | [diff] [blame] | 385 | * drm_atomic_replace_property_blob - replace a blob property |
| 386 | * @blob: a pointer to the member blob to be replaced |
| 387 | * @new_blob: the new blob to replace with |
Lionel Landwerlin | 5488dc1 | 2016-02-26 17:05:00 +0000 | [diff] [blame] | 388 | * @replaced: whether the blob has been replaced |
| 389 | * |
| 390 | * RETURNS: |
| 391 | * Zero on success, error code on failure |
| 392 | */ |
| 393 | static void |
| 394 | drm_atomic_replace_property_blob(struct drm_property_blob **blob, |
| 395 | struct drm_property_blob *new_blob, |
| 396 | bool *replaced) |
| 397 | { |
| 398 | struct drm_property_blob *old_blob = *blob; |
| 399 | |
| 400 | if (old_blob == new_blob) |
| 401 | return; |
| 402 | |
Markus Elfring | f35cbe6 | 2016-07-20 17:54:32 +0200 | [diff] [blame] | 403 | drm_property_unreference_blob(old_blob); |
Lionel Landwerlin | 5488dc1 | 2016-02-26 17:05:00 +0000 | [diff] [blame] | 404 | if (new_blob) |
| 405 | drm_property_reference_blob(new_blob); |
| 406 | *blob = new_blob; |
| 407 | *replaced = true; |
| 408 | |
| 409 | return; |
| 410 | } |
| 411 | |
| 412 | static int |
| 413 | drm_atomic_replace_property_blob_from_id(struct drm_crtc *crtc, |
| 414 | struct drm_property_blob **blob, |
| 415 | uint64_t blob_id, |
| 416 | ssize_t expected_size, |
| 417 | bool *replaced) |
| 418 | { |
| 419 | struct drm_device *dev = crtc->dev; |
| 420 | struct drm_property_blob *new_blob = NULL; |
| 421 | |
| 422 | if (blob_id != 0) { |
| 423 | new_blob = drm_property_lookup_blob(dev, blob_id); |
| 424 | if (new_blob == NULL) |
| 425 | return -EINVAL; |
| 426 | if (expected_size > 0 && expected_size != new_blob->length) |
| 427 | return -EINVAL; |
| 428 | } |
| 429 | |
| 430 | drm_atomic_replace_property_blob(blob, new_blob, replaced); |
| 431 | |
| 432 | return 0; |
| 433 | } |
| 434 | |
| 435 | /** |
Rob Clark | 40ecc69 | 2014-12-18 16:01:46 -0500 | [diff] [blame] | 436 | * drm_atomic_crtc_set_property - set property on CRTC |
| 437 | * @crtc: the drm CRTC to set a property on |
| 438 | * @state: the state object to update with the new property value |
| 439 | * @property: the property to set |
| 440 | * @val: the new property value |
| 441 | * |
| 442 | * Use this instead of calling crtc->atomic_set_property directly. |
| 443 | * This function handles generic/core properties and calls out to |
| 444 | * driver's ->atomic_set_property() for driver properties. To ensure |
| 445 | * consistent behavior you must call this function rather than the |
| 446 | * driver hook directly. |
| 447 | * |
| 448 | * RETURNS: |
| 449 | * Zero on success, error code on failure |
| 450 | */ |
| 451 | int drm_atomic_crtc_set_property(struct drm_crtc *crtc, |
| 452 | struct drm_crtc_state *state, struct drm_property *property, |
| 453 | uint64_t val) |
| 454 | { |
Daniel Vetter | eab3bbe | 2015-01-22 16:36:21 +0100 | [diff] [blame] | 455 | struct drm_device *dev = crtc->dev; |
| 456 | struct drm_mode_config *config = &dev->mode_config; |
Lionel Landwerlin | 5488dc1 | 2016-02-26 17:05:00 +0000 | [diff] [blame] | 457 | bool replaced = false; |
Daniel Stone | 955f3c3 | 2015-05-25 19:11:52 +0100 | [diff] [blame] | 458 | int ret; |
Daniel Vetter | eab3bbe | 2015-01-22 16:36:21 +0100 | [diff] [blame] | 459 | |
Daniel Stone | 2779836 | 2015-03-19 04:33:26 +0000 | [diff] [blame] | 460 | if (property == config->prop_active) |
Daniel Vetter | eab3bbe | 2015-01-22 16:36:21 +0100 | [diff] [blame] | 461 | state->active = val; |
Daniel Stone | 955f3c3 | 2015-05-25 19:11:52 +0100 | [diff] [blame] | 462 | else if (property == config->prop_mode_id) { |
| 463 | struct drm_property_blob *mode = |
| 464 | drm_property_lookup_blob(dev, val); |
| 465 | ret = drm_atomic_set_mode_prop_for_crtc(state, mode); |
Markus Elfring | 5f91190 | 2015-11-06 12:03:46 +0100 | [diff] [blame] | 466 | drm_property_unreference_blob(mode); |
Daniel Stone | 955f3c3 | 2015-05-25 19:11:52 +0100 | [diff] [blame] | 467 | return ret; |
Lionel Landwerlin | 5488dc1 | 2016-02-26 17:05:00 +0000 | [diff] [blame] | 468 | } else if (property == config->degamma_lut_property) { |
| 469 | ret = drm_atomic_replace_property_blob_from_id(crtc, |
| 470 | &state->degamma_lut, |
| 471 | val, |
| 472 | -1, |
| 473 | &replaced); |
Mario Kleiner | add1fa7 | 2016-08-27 01:02:28 +0200 | [diff] [blame] | 474 | state->color_mgmt_changed |= replaced; |
Lionel Landwerlin | 5488dc1 | 2016-02-26 17:05:00 +0000 | [diff] [blame] | 475 | return ret; |
| 476 | } else if (property == config->ctm_property) { |
| 477 | ret = drm_atomic_replace_property_blob_from_id(crtc, |
| 478 | &state->ctm, |
| 479 | val, |
| 480 | sizeof(struct drm_color_ctm), |
| 481 | &replaced); |
Mario Kleiner | add1fa7 | 2016-08-27 01:02:28 +0200 | [diff] [blame] | 482 | state->color_mgmt_changed |= replaced; |
Lionel Landwerlin | 5488dc1 | 2016-02-26 17:05:00 +0000 | [diff] [blame] | 483 | return ret; |
| 484 | } else if (property == config->gamma_lut_property) { |
| 485 | ret = drm_atomic_replace_property_blob_from_id(crtc, |
| 486 | &state->gamma_lut, |
| 487 | val, |
| 488 | -1, |
| 489 | &replaced); |
Mario Kleiner | add1fa7 | 2016-08-27 01:02:28 +0200 | [diff] [blame] | 490 | state->color_mgmt_changed |= replaced; |
Lionel Landwerlin | 5488dc1 | 2016-02-26 17:05:00 +0000 | [diff] [blame] | 491 | return ret; |
| 492 | } else if (crtc->funcs->atomic_set_property) |
Rob Clark | 40ecc69 | 2014-12-18 16:01:46 -0500 | [diff] [blame] | 493 | return crtc->funcs->atomic_set_property(crtc, state, property, val); |
Daniel Stone | 2779836 | 2015-03-19 04:33:26 +0000 | [diff] [blame] | 494 | else |
| 495 | return -EINVAL; |
| 496 | |
| 497 | return 0; |
Rob Clark | 40ecc69 | 2014-12-18 16:01:46 -0500 | [diff] [blame] | 498 | } |
| 499 | EXPORT_SYMBOL(drm_atomic_crtc_set_property); |
| 500 | |
Daniel Vetter | c0714fc | 2015-12-04 09:45:57 +0100 | [diff] [blame] | 501 | /** |
| 502 | * drm_atomic_crtc_get_property - get property value from CRTC state |
| 503 | * @crtc: the drm CRTC to set a property on |
| 504 | * @state: the state object to get the property value from |
| 505 | * @property: the property to set |
| 506 | * @val: return location for the property value |
| 507 | * |
Rob Clark | ac9c925 | 2014-12-18 16:01:47 -0500 | [diff] [blame] | 508 | * This function handles generic/core properties and calls out to |
| 509 | * driver's ->atomic_get_property() for driver properties. To ensure |
| 510 | * consistent behavior you must call this function rather than the |
| 511 | * driver hook directly. |
Daniel Vetter | c0714fc | 2015-12-04 09:45:57 +0100 | [diff] [blame] | 512 | * |
| 513 | * RETURNS: |
| 514 | * Zero on success, error code on failure |
Rob Clark | ac9c925 | 2014-12-18 16:01:47 -0500 | [diff] [blame] | 515 | */ |
Geliang Tang | bf22f3b | 2015-09-24 03:01:03 -0700 | [diff] [blame] | 516 | static int |
| 517 | drm_atomic_crtc_get_property(struct drm_crtc *crtc, |
Rob Clark | ac9c925 | 2014-12-18 16:01:47 -0500 | [diff] [blame] | 518 | const struct drm_crtc_state *state, |
| 519 | struct drm_property *property, uint64_t *val) |
| 520 | { |
Daniel Stone | 8f164ce | 2015-03-19 04:33:25 +0000 | [diff] [blame] | 521 | struct drm_device *dev = crtc->dev; |
| 522 | struct drm_mode_config *config = &dev->mode_config; |
| 523 | |
| 524 | if (property == config->prop_active) |
| 525 | *val = state->active; |
Daniel Stone | 955f3c3 | 2015-05-25 19:11:52 +0100 | [diff] [blame] | 526 | else if (property == config->prop_mode_id) |
| 527 | *val = (state->mode_blob) ? state->mode_blob->base.id : 0; |
Lionel Landwerlin | 5488dc1 | 2016-02-26 17:05:00 +0000 | [diff] [blame] | 528 | else if (property == config->degamma_lut_property) |
| 529 | *val = (state->degamma_lut) ? state->degamma_lut->base.id : 0; |
| 530 | else if (property == config->ctm_property) |
| 531 | *val = (state->ctm) ? state->ctm->base.id : 0; |
| 532 | else if (property == config->gamma_lut_property) |
| 533 | *val = (state->gamma_lut) ? state->gamma_lut->base.id : 0; |
Daniel Stone | 8f164ce | 2015-03-19 04:33:25 +0000 | [diff] [blame] | 534 | else if (crtc->funcs->atomic_get_property) |
Rob Clark | ac9c925 | 2014-12-18 16:01:47 -0500 | [diff] [blame] | 535 | return crtc->funcs->atomic_get_property(crtc, state, property, val); |
Daniel Stone | 8f164ce | 2015-03-19 04:33:25 +0000 | [diff] [blame] | 536 | else |
| 537 | return -EINVAL; |
| 538 | |
| 539 | return 0; |
Rob Clark | ac9c925 | 2014-12-18 16:01:47 -0500 | [diff] [blame] | 540 | } |
Rob Clark | ac9c925 | 2014-12-18 16:01:47 -0500 | [diff] [blame] | 541 | |
| 542 | /** |
Rob Clark | 5e74373 | 2014-12-18 16:01:51 -0500 | [diff] [blame] | 543 | * drm_atomic_crtc_check - check crtc state |
| 544 | * @crtc: crtc to check |
| 545 | * @state: crtc state to check |
| 546 | * |
| 547 | * Provides core sanity checks for crtc state. |
| 548 | * |
| 549 | * RETURNS: |
| 550 | * Zero on success, error code on failure |
| 551 | */ |
| 552 | static int drm_atomic_crtc_check(struct drm_crtc *crtc, |
| 553 | struct drm_crtc_state *state) |
| 554 | { |
| 555 | /* NOTE: we explicitly don't enforce constraints such as primary |
| 556 | * layer covering entire screen, since that is something we want |
| 557 | * to allow (on hw that supports it). For hw that does not, it |
| 558 | * should be checked in driver's crtc->atomic_check() vfunc. |
| 559 | * |
| 560 | * TODO: Add generic modeset state checks once we support those. |
| 561 | */ |
Daniel Vetter | eab3bbe | 2015-01-22 16:36:21 +0100 | [diff] [blame] | 562 | |
| 563 | if (state->active && !state->enable) { |
Ville Syrjälä | fa3ab4c | 2015-12-08 18:41:53 +0200 | [diff] [blame] | 564 | DRM_DEBUG_ATOMIC("[CRTC:%d:%s] active without enabled\n", |
| 565 | crtc->base.id, crtc->name); |
Daniel Vetter | eab3bbe | 2015-01-22 16:36:21 +0100 | [diff] [blame] | 566 | return -EINVAL; |
| 567 | } |
| 568 | |
Daniel Stone | 99cf4a2 | 2015-05-25 19:11:51 +0100 | [diff] [blame] | 569 | /* The state->enable vs. state->mode_blob checks can be WARN_ON, |
| 570 | * as this is a kernel-internal detail that userspace should never |
| 571 | * be able to trigger. */ |
| 572 | if (drm_core_check_feature(crtc->dev, DRIVER_ATOMIC) && |
| 573 | WARN_ON(state->enable && !state->mode_blob)) { |
Ville Syrjälä | fa3ab4c | 2015-12-08 18:41:53 +0200 | [diff] [blame] | 574 | DRM_DEBUG_ATOMIC("[CRTC:%d:%s] enabled without mode blob\n", |
| 575 | crtc->base.id, crtc->name); |
Daniel Stone | 99cf4a2 | 2015-05-25 19:11:51 +0100 | [diff] [blame] | 576 | return -EINVAL; |
| 577 | } |
| 578 | |
| 579 | if (drm_core_check_feature(crtc->dev, DRIVER_ATOMIC) && |
| 580 | WARN_ON(!state->enable && state->mode_blob)) { |
Ville Syrjälä | fa3ab4c | 2015-12-08 18:41:53 +0200 | [diff] [blame] | 581 | DRM_DEBUG_ATOMIC("[CRTC:%d:%s] disabled with mode blob\n", |
| 582 | crtc->base.id, crtc->name); |
Daniel Stone | 99cf4a2 | 2015-05-25 19:11:51 +0100 | [diff] [blame] | 583 | return -EINVAL; |
| 584 | } |
| 585 | |
Daniel Vetter | 4cba685 | 2015-12-08 09:49:20 +0100 | [diff] [blame] | 586 | /* |
| 587 | * Reject event generation for when a CRTC is off and stays off. |
| 588 | * It wouldn't be hard to implement this, but userspace has a track |
| 589 | * record of happily burning through 100% cpu (or worse, crash) when the |
| 590 | * display pipe is suspended. To avoid all that fun just reject updates |
| 591 | * that ask for events since likely that indicates a bug in the |
| 592 | * compositor's drawing loop. This is consistent with the vblank IOCTL |
| 593 | * and legacy page_flip IOCTL which also reject service on a disabled |
| 594 | * pipe. |
| 595 | */ |
| 596 | if (state->event && !state->active && !crtc->state->active) { |
| 597 | DRM_DEBUG_ATOMIC("[CRTC:%d] requesting event but off\n", |
| 598 | crtc->base.id); |
| 599 | return -EINVAL; |
| 600 | } |
| 601 | |
Rob Clark | 5e74373 | 2014-12-18 16:01:51 -0500 | [diff] [blame] | 602 | return 0; |
| 603 | } |
| 604 | |
| 605 | /** |
Daniel Vetter | cc4ceb4 | 2014-07-25 21:30:38 +0200 | [diff] [blame] | 606 | * drm_atomic_get_plane_state - get plane state |
| 607 | * @state: global atomic state object |
| 608 | * @plane: plane to get state object for |
| 609 | * |
| 610 | * This function returns the plane state for the given plane, allocating it if |
| 611 | * needed. It will also grab the relevant plane lock to make sure that the state |
| 612 | * is consistent. |
| 613 | * |
| 614 | * Returns: |
| 615 | * |
| 616 | * Either the allocated state or the error code encoded into the pointer. When |
| 617 | * the error is EDEADLK then the w/w mutex code has detected a deadlock and the |
| 618 | * entire atomic sequence must be restarted. All other errors are fatal. |
| 619 | */ |
| 620 | struct drm_plane_state * |
| 621 | drm_atomic_get_plane_state(struct drm_atomic_state *state, |
| 622 | struct drm_plane *plane) |
| 623 | { |
Maarten Lankhorst | 1b26a5e | 2015-05-13 10:37:25 +0200 | [diff] [blame] | 624 | int ret, index = drm_plane_index(plane); |
Daniel Vetter | cc4ceb4 | 2014-07-25 21:30:38 +0200 | [diff] [blame] | 625 | struct drm_plane_state *plane_state; |
| 626 | |
Maarten Lankhorst | 7f4eaa89 | 2016-05-03 11:12:31 +0200 | [diff] [blame] | 627 | WARN_ON(!state->acquire_ctx); |
| 628 | |
Maarten Lankhorst | 1b26a5e | 2015-05-13 10:37:25 +0200 | [diff] [blame] | 629 | plane_state = drm_atomic_get_existing_plane_state(state, plane); |
| 630 | if (plane_state) |
| 631 | return plane_state; |
Daniel Vetter | cc4ceb4 | 2014-07-25 21:30:38 +0200 | [diff] [blame] | 632 | |
Daniel Vetter | 4d02e2d | 2014-11-11 10:12:00 +0100 | [diff] [blame] | 633 | ret = drm_modeset_lock(&plane->mutex, state->acquire_ctx); |
Daniel Vetter | cc4ceb4 | 2014-07-25 21:30:38 +0200 | [diff] [blame] | 634 | if (ret) |
| 635 | return ERR_PTR(ret); |
| 636 | |
| 637 | plane_state = plane->funcs->atomic_duplicate_state(plane); |
| 638 | if (!plane_state) |
| 639 | return ERR_PTR(-ENOMEM); |
| 640 | |
Daniel Vetter | b8b5342 | 2016-06-02 00:06:33 +0200 | [diff] [blame] | 641 | state->planes[index].state = plane_state; |
| 642 | state->planes[index].ptr = plane; |
Daniel Vetter | cc4ceb4 | 2014-07-25 21:30:38 +0200 | [diff] [blame] | 643 | plane_state->state = state; |
| 644 | |
Ville Syrjälä | 9f4c97a | 2015-12-08 18:41:54 +0200 | [diff] [blame] | 645 | DRM_DEBUG_ATOMIC("Added [PLANE:%d:%s] %p state to %p\n", |
| 646 | plane->base.id, plane->name, plane_state, state); |
Daniel Vetter | cc4ceb4 | 2014-07-25 21:30:38 +0200 | [diff] [blame] | 647 | |
| 648 | if (plane_state->crtc) { |
| 649 | struct drm_crtc_state *crtc_state; |
| 650 | |
| 651 | crtc_state = drm_atomic_get_crtc_state(state, |
| 652 | plane_state->crtc); |
| 653 | if (IS_ERR(crtc_state)) |
| 654 | return ERR_CAST(crtc_state); |
| 655 | } |
| 656 | |
| 657 | return plane_state; |
| 658 | } |
| 659 | EXPORT_SYMBOL(drm_atomic_get_plane_state); |
| 660 | |
| 661 | /** |
Rob Clark | 40ecc69 | 2014-12-18 16:01:46 -0500 | [diff] [blame] | 662 | * drm_atomic_plane_set_property - set property on plane |
| 663 | * @plane: the drm plane to set a property on |
| 664 | * @state: the state object to update with the new property value |
| 665 | * @property: the property to set |
| 666 | * @val: the new property value |
| 667 | * |
| 668 | * Use this instead of calling plane->atomic_set_property directly. |
| 669 | * This function handles generic/core properties and calls out to |
| 670 | * driver's ->atomic_set_property() for driver properties. To ensure |
| 671 | * consistent behavior you must call this function rather than the |
| 672 | * driver hook directly. |
| 673 | * |
| 674 | * RETURNS: |
| 675 | * Zero on success, error code on failure |
| 676 | */ |
| 677 | int drm_atomic_plane_set_property(struct drm_plane *plane, |
| 678 | struct drm_plane_state *state, struct drm_property *property, |
| 679 | uint64_t val) |
| 680 | { |
Rob Clark | 6b4959f | 2014-12-18 16:01:53 -0500 | [diff] [blame] | 681 | struct drm_device *dev = plane->dev; |
| 682 | struct drm_mode_config *config = &dev->mode_config; |
| 683 | |
| 684 | if (property == config->prop_fb_id) { |
| 685 | struct drm_framebuffer *fb = drm_framebuffer_lookup(dev, val); |
| 686 | drm_atomic_set_fb_for_plane(state, fb); |
| 687 | if (fb) |
| 688 | drm_framebuffer_unreference(fb); |
| 689 | } else if (property == config->prop_crtc_id) { |
| 690 | struct drm_crtc *crtc = drm_crtc_find(dev, val); |
| 691 | return drm_atomic_set_crtc_for_plane(state, crtc); |
| 692 | } else if (property == config->prop_crtc_x) { |
| 693 | state->crtc_x = U642I64(val); |
| 694 | } else if (property == config->prop_crtc_y) { |
| 695 | state->crtc_y = U642I64(val); |
| 696 | } else if (property == config->prop_crtc_w) { |
| 697 | state->crtc_w = val; |
| 698 | } else if (property == config->prop_crtc_h) { |
| 699 | state->crtc_h = val; |
| 700 | } else if (property == config->prop_src_x) { |
| 701 | state->src_x = val; |
| 702 | } else if (property == config->prop_src_y) { |
| 703 | state->src_y = val; |
| 704 | } else if (property == config->prop_src_w) { |
| 705 | state->src_w = val; |
| 706 | } else if (property == config->prop_src_h) { |
| 707 | state->src_h = val; |
Matt Roper | 1da3062 | 2015-01-21 16:35:40 -0800 | [diff] [blame] | 708 | } else if (property == config->rotation_property) { |
Ville Syrjälä | 6e0c7c3 | 2016-09-26 19:30:47 +0300 | [diff] [blame^] | 709 | if (!is_power_of_2(val & DRM_ROTATE_MASK)) |
| 710 | return -EINVAL; |
Matt Roper | 1da3062 | 2015-01-21 16:35:40 -0800 | [diff] [blame] | 711 | state->rotation = val; |
Marek Szyprowski | 44d1240d | 2016-06-13 11:11:26 +0200 | [diff] [blame] | 712 | } else if (property == plane->zpos_property) { |
| 713 | state->zpos = val; |
Rob Clark | 6b4959f | 2014-12-18 16:01:53 -0500 | [diff] [blame] | 714 | } else if (plane->funcs->atomic_set_property) { |
| 715 | return plane->funcs->atomic_set_property(plane, state, |
| 716 | property, val); |
| 717 | } else { |
| 718 | return -EINVAL; |
| 719 | } |
| 720 | |
| 721 | return 0; |
Rob Clark | 40ecc69 | 2014-12-18 16:01:46 -0500 | [diff] [blame] | 722 | } |
| 723 | EXPORT_SYMBOL(drm_atomic_plane_set_property); |
| 724 | |
Daniel Vetter | c0714fc | 2015-12-04 09:45:57 +0100 | [diff] [blame] | 725 | /** |
| 726 | * drm_atomic_plane_get_property - get property value from plane state |
| 727 | * @plane: the drm plane to set a property on |
| 728 | * @state: the state object to get the property value from |
| 729 | * @property: the property to set |
| 730 | * @val: return location for the property value |
| 731 | * |
Rob Clark | ac9c925 | 2014-12-18 16:01:47 -0500 | [diff] [blame] | 732 | * This function handles generic/core properties and calls out to |
| 733 | * driver's ->atomic_get_property() for driver properties. To ensure |
| 734 | * consistent behavior you must call this function rather than the |
| 735 | * driver hook directly. |
Daniel Vetter | c0714fc | 2015-12-04 09:45:57 +0100 | [diff] [blame] | 736 | * |
| 737 | * RETURNS: |
| 738 | * Zero on success, error code on failure |
Rob Clark | ac9c925 | 2014-12-18 16:01:47 -0500 | [diff] [blame] | 739 | */ |
Daniel Vetter | a97df1c | 2014-12-18 22:49:02 +0100 | [diff] [blame] | 740 | static int |
| 741 | drm_atomic_plane_get_property(struct drm_plane *plane, |
Rob Clark | ac9c925 | 2014-12-18 16:01:47 -0500 | [diff] [blame] | 742 | const struct drm_plane_state *state, |
| 743 | struct drm_property *property, uint64_t *val) |
| 744 | { |
Rob Clark | 6b4959f | 2014-12-18 16:01:53 -0500 | [diff] [blame] | 745 | struct drm_device *dev = plane->dev; |
| 746 | struct drm_mode_config *config = &dev->mode_config; |
| 747 | |
| 748 | if (property == config->prop_fb_id) { |
| 749 | *val = (state->fb) ? state->fb->base.id : 0; |
| 750 | } else if (property == config->prop_crtc_id) { |
| 751 | *val = (state->crtc) ? state->crtc->base.id : 0; |
| 752 | } else if (property == config->prop_crtc_x) { |
| 753 | *val = I642U64(state->crtc_x); |
| 754 | } else if (property == config->prop_crtc_y) { |
| 755 | *val = I642U64(state->crtc_y); |
| 756 | } else if (property == config->prop_crtc_w) { |
| 757 | *val = state->crtc_w; |
| 758 | } else if (property == config->prop_crtc_h) { |
| 759 | *val = state->crtc_h; |
| 760 | } else if (property == config->prop_src_x) { |
| 761 | *val = state->src_x; |
| 762 | } else if (property == config->prop_src_y) { |
| 763 | *val = state->src_y; |
| 764 | } else if (property == config->prop_src_w) { |
| 765 | *val = state->src_w; |
| 766 | } else if (property == config->prop_src_h) { |
| 767 | *val = state->src_h; |
Tvrtko Ursulin | 4cda09c | 2015-02-26 13:49:17 +0000 | [diff] [blame] | 768 | } else if (property == config->rotation_property) { |
| 769 | *val = state->rotation; |
Marek Szyprowski | 44d1240d | 2016-06-13 11:11:26 +0200 | [diff] [blame] | 770 | } else if (property == plane->zpos_property) { |
| 771 | *val = state->zpos; |
Rob Clark | 6b4959f | 2014-12-18 16:01:53 -0500 | [diff] [blame] | 772 | } else if (plane->funcs->atomic_get_property) { |
Rob Clark | ac9c925 | 2014-12-18 16:01:47 -0500 | [diff] [blame] | 773 | return plane->funcs->atomic_get_property(plane, state, property, val); |
Rob Clark | 6b4959f | 2014-12-18 16:01:53 -0500 | [diff] [blame] | 774 | } else { |
| 775 | return -EINVAL; |
| 776 | } |
| 777 | |
| 778 | return 0; |
Rob Clark | ac9c925 | 2014-12-18 16:01:47 -0500 | [diff] [blame] | 779 | } |
Rob Clark | ac9c925 | 2014-12-18 16:01:47 -0500 | [diff] [blame] | 780 | |
Daniel Vetter | f8aeb41 | 2015-08-26 21:49:42 +0200 | [diff] [blame] | 781 | static bool |
| 782 | plane_switching_crtc(struct drm_atomic_state *state, |
| 783 | struct drm_plane *plane, |
| 784 | struct drm_plane_state *plane_state) |
| 785 | { |
| 786 | if (!plane->state->crtc || !plane_state->crtc) |
| 787 | return false; |
| 788 | |
| 789 | if (plane->state->crtc == plane_state->crtc) |
| 790 | return false; |
| 791 | |
| 792 | /* This could be refined, but currently there's no helper or driver code |
| 793 | * to implement direct switching of active planes nor userspace to take |
| 794 | * advantage of more direct plane switching without the intermediate |
| 795 | * full OFF state. |
| 796 | */ |
| 797 | return true; |
| 798 | } |
| 799 | |
Rob Clark | ac9c925 | 2014-12-18 16:01:47 -0500 | [diff] [blame] | 800 | /** |
Rob Clark | 5e74373 | 2014-12-18 16:01:51 -0500 | [diff] [blame] | 801 | * drm_atomic_plane_check - check plane state |
| 802 | * @plane: plane to check |
| 803 | * @state: plane state to check |
| 804 | * |
| 805 | * Provides core sanity checks for plane state. |
| 806 | * |
| 807 | * RETURNS: |
| 808 | * Zero on success, error code on failure |
| 809 | */ |
| 810 | static int drm_atomic_plane_check(struct drm_plane *plane, |
| 811 | struct drm_plane_state *state) |
| 812 | { |
| 813 | unsigned int fb_width, fb_height; |
Laurent Pinchart | ead8610 | 2015-03-05 02:25:43 +0200 | [diff] [blame] | 814 | int ret; |
Rob Clark | 5e74373 | 2014-12-18 16:01:51 -0500 | [diff] [blame] | 815 | |
| 816 | /* either *both* CRTC and FB must be set, or neither */ |
| 817 | if (WARN_ON(state->crtc && !state->fb)) { |
Daniel Vetter | 17a38d9 | 2015-02-22 12:24:16 +0100 | [diff] [blame] | 818 | DRM_DEBUG_ATOMIC("CRTC set but no FB\n"); |
Rob Clark | 5e74373 | 2014-12-18 16:01:51 -0500 | [diff] [blame] | 819 | return -EINVAL; |
| 820 | } else if (WARN_ON(state->fb && !state->crtc)) { |
Daniel Vetter | 17a38d9 | 2015-02-22 12:24:16 +0100 | [diff] [blame] | 821 | DRM_DEBUG_ATOMIC("FB set but no CRTC\n"); |
Rob Clark | 5e74373 | 2014-12-18 16:01:51 -0500 | [diff] [blame] | 822 | return -EINVAL; |
| 823 | } |
| 824 | |
| 825 | /* if disabled, we don't care about the rest of the state: */ |
| 826 | if (!state->crtc) |
| 827 | return 0; |
| 828 | |
| 829 | /* Check whether this plane is usable on this CRTC */ |
| 830 | if (!(plane->possible_crtcs & drm_crtc_mask(state->crtc))) { |
Daniel Vetter | 17a38d9 | 2015-02-22 12:24:16 +0100 | [diff] [blame] | 831 | DRM_DEBUG_ATOMIC("Invalid crtc for plane\n"); |
Rob Clark | 5e74373 | 2014-12-18 16:01:51 -0500 | [diff] [blame] | 832 | return -EINVAL; |
| 833 | } |
| 834 | |
| 835 | /* Check whether this plane supports the fb pixel format. */ |
Laurent Pinchart | ead8610 | 2015-03-05 02:25:43 +0200 | [diff] [blame] | 836 | ret = drm_plane_check_pixel_format(plane, state->fb->pixel_format); |
| 837 | if (ret) { |
Eric Engestrom | d382814 | 2016-08-15 16:29:55 +0100 | [diff] [blame] | 838 | char *format_name = drm_get_format_name(state->fb->pixel_format); |
Eric Engestrom | 90844f0 | 2016-08-15 01:02:38 +0100 | [diff] [blame] | 839 | DRM_DEBUG_ATOMIC("Invalid pixel format %s\n", format_name); |
| 840 | kfree(format_name); |
Laurent Pinchart | ead8610 | 2015-03-05 02:25:43 +0200 | [diff] [blame] | 841 | return ret; |
Rob Clark | 5e74373 | 2014-12-18 16:01:51 -0500 | [diff] [blame] | 842 | } |
| 843 | |
| 844 | /* Give drivers some help against integer overflows */ |
| 845 | if (state->crtc_w > INT_MAX || |
| 846 | state->crtc_x > INT_MAX - (int32_t) state->crtc_w || |
| 847 | state->crtc_h > INT_MAX || |
| 848 | state->crtc_y > INT_MAX - (int32_t) state->crtc_h) { |
Daniel Vetter | 17a38d9 | 2015-02-22 12:24:16 +0100 | [diff] [blame] | 849 | DRM_DEBUG_ATOMIC("Invalid CRTC coordinates %ux%u+%d+%d\n", |
| 850 | state->crtc_w, state->crtc_h, |
| 851 | state->crtc_x, state->crtc_y); |
Rob Clark | 5e74373 | 2014-12-18 16:01:51 -0500 | [diff] [blame] | 852 | return -ERANGE; |
| 853 | } |
| 854 | |
| 855 | fb_width = state->fb->width << 16; |
| 856 | fb_height = state->fb->height << 16; |
| 857 | |
| 858 | /* Make sure source coordinates are inside the fb. */ |
| 859 | if (state->src_w > fb_width || |
| 860 | state->src_x > fb_width - state->src_w || |
| 861 | state->src_h > fb_height || |
| 862 | state->src_y > fb_height - state->src_h) { |
Daniel Vetter | 17a38d9 | 2015-02-22 12:24:16 +0100 | [diff] [blame] | 863 | DRM_DEBUG_ATOMIC("Invalid source coordinates " |
| 864 | "%u.%06ux%u.%06u+%u.%06u+%u.%06u\n", |
| 865 | state->src_w >> 16, ((state->src_w & 0xffff) * 15625) >> 10, |
| 866 | state->src_h >> 16, ((state->src_h & 0xffff) * 15625) >> 10, |
| 867 | state->src_x >> 16, ((state->src_x & 0xffff) * 15625) >> 10, |
| 868 | state->src_y >> 16, ((state->src_y & 0xffff) * 15625) >> 10); |
Rob Clark | 5e74373 | 2014-12-18 16:01:51 -0500 | [diff] [blame] | 869 | return -ENOSPC; |
| 870 | } |
| 871 | |
Daniel Vetter | f8aeb41 | 2015-08-26 21:49:42 +0200 | [diff] [blame] | 872 | if (plane_switching_crtc(state->state, plane, state)) { |
Ville Syrjälä | 9f4c97a | 2015-12-08 18:41:54 +0200 | [diff] [blame] | 873 | DRM_DEBUG_ATOMIC("[PLANE:%d:%s] switching CRTC directly\n", |
| 874 | plane->base.id, plane->name); |
Daniel Vetter | f8aeb41 | 2015-08-26 21:49:42 +0200 | [diff] [blame] | 875 | return -EINVAL; |
| 876 | } |
| 877 | |
Rob Clark | 5e74373 | 2014-12-18 16:01:51 -0500 | [diff] [blame] | 878 | return 0; |
| 879 | } |
| 880 | |
| 881 | /** |
Daniel Vetter | cc4ceb4 | 2014-07-25 21:30:38 +0200 | [diff] [blame] | 882 | * drm_atomic_get_connector_state - get connector state |
| 883 | * @state: global atomic state object |
| 884 | * @connector: connector to get state object for |
| 885 | * |
| 886 | * This function returns the connector state for the given connector, |
| 887 | * allocating it if needed. It will also grab the relevant connector lock to |
| 888 | * make sure that the state is consistent. |
| 889 | * |
| 890 | * Returns: |
| 891 | * |
| 892 | * Either the allocated state or the error code encoded into the pointer. When |
| 893 | * the error is EDEADLK then the w/w mutex code has detected a deadlock and the |
| 894 | * entire atomic sequence must be restarted. All other errors are fatal. |
| 895 | */ |
| 896 | struct drm_connector_state * |
| 897 | drm_atomic_get_connector_state(struct drm_atomic_state *state, |
| 898 | struct drm_connector *connector) |
| 899 | { |
| 900 | int ret, index; |
| 901 | struct drm_mode_config *config = &connector->dev->mode_config; |
| 902 | struct drm_connector_state *connector_state; |
| 903 | |
Maarten Lankhorst | 7f4eaa89 | 2016-05-03 11:12:31 +0200 | [diff] [blame] | 904 | WARN_ON(!state->acquire_ctx); |
| 905 | |
Daniel Vetter | c7eb76f | 2014-11-19 18:38:06 +0100 | [diff] [blame] | 906 | ret = drm_modeset_lock(&config->connection_mutex, state->acquire_ctx); |
| 907 | if (ret) |
| 908 | return ERR_PTR(ret); |
| 909 | |
Daniel Vetter | cc4ceb4 | 2014-07-25 21:30:38 +0200 | [diff] [blame] | 910 | index = drm_connector_index(connector); |
| 911 | |
Daniel Vetter | f52b69f1 | 2014-11-19 18:38:08 +0100 | [diff] [blame] | 912 | if (index >= state->num_connector) { |
Daniel Vetter | 63e83c1 | 2016-06-02 00:06:32 +0200 | [diff] [blame] | 913 | struct __drm_connnectors_state *c; |
Maarten Lankhorst | 5fff80b | 2016-02-17 08:32:05 +0100 | [diff] [blame] | 914 | int alloc = max(index + 1, config->num_connector); |
| 915 | |
| 916 | c = krealloc(state->connectors, alloc * sizeof(*state->connectors), GFP_KERNEL); |
| 917 | if (!c) |
| 918 | return ERR_PTR(-ENOMEM); |
| 919 | |
| 920 | state->connectors = c; |
| 921 | memset(&state->connectors[state->num_connector], 0, |
| 922 | sizeof(*state->connectors) * (alloc - state->num_connector)); |
| 923 | |
Maarten Lankhorst | 5fff80b | 2016-02-17 08:32:05 +0100 | [diff] [blame] | 924 | state->num_connector = alloc; |
Daniel Vetter | f52b69f1 | 2014-11-19 18:38:08 +0100 | [diff] [blame] | 925 | } |
| 926 | |
Daniel Vetter | 63e83c1 | 2016-06-02 00:06:32 +0200 | [diff] [blame] | 927 | if (state->connectors[index].state) |
| 928 | return state->connectors[index].state; |
Daniel Vetter | cc4ceb4 | 2014-07-25 21:30:38 +0200 | [diff] [blame] | 929 | |
Daniel Vetter | cc4ceb4 | 2014-07-25 21:30:38 +0200 | [diff] [blame] | 930 | connector_state = connector->funcs->atomic_duplicate_state(connector); |
| 931 | if (!connector_state) |
| 932 | return ERR_PTR(-ENOMEM); |
| 933 | |
Dave Airlie | b164d31 | 2016-04-27 11:10:09 +1000 | [diff] [blame] | 934 | drm_connector_reference(connector); |
Daniel Vetter | 63e83c1 | 2016-06-02 00:06:32 +0200 | [diff] [blame] | 935 | state->connectors[index].state = connector_state; |
| 936 | state->connectors[index].ptr = connector; |
Daniel Vetter | cc4ceb4 | 2014-07-25 21:30:38 +0200 | [diff] [blame] | 937 | connector_state->state = state; |
| 938 | |
Daniel Vetter | 17a38d9 | 2015-02-22 12:24:16 +0100 | [diff] [blame] | 939 | DRM_DEBUG_ATOMIC("Added [CONNECTOR:%d] %p state to %p\n", |
| 940 | connector->base.id, connector_state, state); |
Daniel Vetter | cc4ceb4 | 2014-07-25 21:30:38 +0200 | [diff] [blame] | 941 | |
| 942 | if (connector_state->crtc) { |
| 943 | struct drm_crtc_state *crtc_state; |
| 944 | |
| 945 | crtc_state = drm_atomic_get_crtc_state(state, |
| 946 | connector_state->crtc); |
| 947 | if (IS_ERR(crtc_state)) |
| 948 | return ERR_CAST(crtc_state); |
| 949 | } |
| 950 | |
| 951 | return connector_state; |
| 952 | } |
| 953 | EXPORT_SYMBOL(drm_atomic_get_connector_state); |
| 954 | |
| 955 | /** |
Rob Clark | 40ecc69 | 2014-12-18 16:01:46 -0500 | [diff] [blame] | 956 | * drm_atomic_connector_set_property - set property on connector. |
| 957 | * @connector: the drm connector to set a property on |
| 958 | * @state: the state object to update with the new property value |
| 959 | * @property: the property to set |
| 960 | * @val: the new property value |
| 961 | * |
| 962 | * Use this instead of calling connector->atomic_set_property directly. |
| 963 | * This function handles generic/core properties and calls out to |
| 964 | * driver's ->atomic_set_property() for driver properties. To ensure |
| 965 | * consistent behavior you must call this function rather than the |
| 966 | * driver hook directly. |
| 967 | * |
| 968 | * RETURNS: |
| 969 | * Zero on success, error code on failure |
| 970 | */ |
| 971 | int drm_atomic_connector_set_property(struct drm_connector *connector, |
| 972 | struct drm_connector_state *state, struct drm_property *property, |
| 973 | uint64_t val) |
| 974 | { |
| 975 | struct drm_device *dev = connector->dev; |
| 976 | struct drm_mode_config *config = &dev->mode_config; |
| 977 | |
Rob Clark | ae16c59 | 2014-12-18 16:01:54 -0500 | [diff] [blame] | 978 | if (property == config->prop_crtc_id) { |
| 979 | struct drm_crtc *crtc = drm_crtc_find(dev, val); |
| 980 | return drm_atomic_set_crtc_for_connector(state, crtc); |
| 981 | } else if (property == config->dpms_property) { |
Rob Clark | 40ecc69 | 2014-12-18 16:01:46 -0500 | [diff] [blame] | 982 | /* setting DPMS property requires special handling, which |
| 983 | * is done in legacy setprop path for us. Disallow (for |
| 984 | * now?) atomic writes to DPMS property: |
| 985 | */ |
| 986 | return -EINVAL; |
| 987 | } else if (connector->funcs->atomic_set_property) { |
| 988 | return connector->funcs->atomic_set_property(connector, |
| 989 | state, property, val); |
| 990 | } else { |
| 991 | return -EINVAL; |
| 992 | } |
| 993 | } |
| 994 | EXPORT_SYMBOL(drm_atomic_connector_set_property); |
| 995 | |
Daniel Vetter | c0714fc | 2015-12-04 09:45:57 +0100 | [diff] [blame] | 996 | /** |
| 997 | * drm_atomic_connector_get_property - get property value from connector state |
| 998 | * @connector: the drm connector to set a property on |
| 999 | * @state: the state object to get the property value from |
| 1000 | * @property: the property to set |
| 1001 | * @val: return location for the property value |
| 1002 | * |
Rob Clark | ac9c925 | 2014-12-18 16:01:47 -0500 | [diff] [blame] | 1003 | * This function handles generic/core properties and calls out to |
| 1004 | * driver's ->atomic_get_property() for driver properties. To ensure |
| 1005 | * consistent behavior you must call this function rather than the |
| 1006 | * driver hook directly. |
Daniel Vetter | c0714fc | 2015-12-04 09:45:57 +0100 | [diff] [blame] | 1007 | * |
| 1008 | * RETURNS: |
| 1009 | * Zero on success, error code on failure |
Rob Clark | ac9c925 | 2014-12-18 16:01:47 -0500 | [diff] [blame] | 1010 | */ |
Daniel Vetter | a97df1c | 2014-12-18 22:49:02 +0100 | [diff] [blame] | 1011 | static int |
| 1012 | drm_atomic_connector_get_property(struct drm_connector *connector, |
Rob Clark | ac9c925 | 2014-12-18 16:01:47 -0500 | [diff] [blame] | 1013 | const struct drm_connector_state *state, |
| 1014 | struct drm_property *property, uint64_t *val) |
| 1015 | { |
| 1016 | struct drm_device *dev = connector->dev; |
| 1017 | struct drm_mode_config *config = &dev->mode_config; |
| 1018 | |
Rob Clark | ae16c59 | 2014-12-18 16:01:54 -0500 | [diff] [blame] | 1019 | if (property == config->prop_crtc_id) { |
| 1020 | *val = (state->crtc) ? state->crtc->base.id : 0; |
| 1021 | } else if (property == config->dpms_property) { |
Rob Clark | ac9c925 | 2014-12-18 16:01:47 -0500 | [diff] [blame] | 1022 | *val = connector->dpms; |
| 1023 | } else if (connector->funcs->atomic_get_property) { |
| 1024 | return connector->funcs->atomic_get_property(connector, |
| 1025 | state, property, val); |
| 1026 | } else { |
| 1027 | return -EINVAL; |
| 1028 | } |
| 1029 | |
| 1030 | return 0; |
| 1031 | } |
Rob Clark | ac9c925 | 2014-12-18 16:01:47 -0500 | [diff] [blame] | 1032 | |
Rob Clark | 88a48e2 | 2014-12-18 16:01:50 -0500 | [diff] [blame] | 1033 | int drm_atomic_get_property(struct drm_mode_object *obj, |
| 1034 | struct drm_property *property, uint64_t *val) |
| 1035 | { |
| 1036 | struct drm_device *dev = property->dev; |
| 1037 | int ret; |
| 1038 | |
| 1039 | switch (obj->type) { |
| 1040 | case DRM_MODE_OBJECT_CONNECTOR: { |
| 1041 | struct drm_connector *connector = obj_to_connector(obj); |
| 1042 | WARN_ON(!drm_modeset_is_locked(&dev->mode_config.connection_mutex)); |
| 1043 | ret = drm_atomic_connector_get_property(connector, |
| 1044 | connector->state, property, val); |
| 1045 | break; |
| 1046 | } |
| 1047 | case DRM_MODE_OBJECT_CRTC: { |
| 1048 | struct drm_crtc *crtc = obj_to_crtc(obj); |
| 1049 | WARN_ON(!drm_modeset_is_locked(&crtc->mutex)); |
| 1050 | ret = drm_atomic_crtc_get_property(crtc, |
| 1051 | crtc->state, property, val); |
| 1052 | break; |
| 1053 | } |
| 1054 | case DRM_MODE_OBJECT_PLANE: { |
| 1055 | struct drm_plane *plane = obj_to_plane(obj); |
| 1056 | WARN_ON(!drm_modeset_is_locked(&plane->mutex)); |
| 1057 | ret = drm_atomic_plane_get_property(plane, |
| 1058 | plane->state, property, val); |
| 1059 | break; |
| 1060 | } |
| 1061 | default: |
| 1062 | ret = -EINVAL; |
| 1063 | break; |
| 1064 | } |
| 1065 | |
| 1066 | return ret; |
| 1067 | } |
| 1068 | |
| 1069 | /** |
Daniel Vetter | cc4ceb4 | 2014-07-25 21:30:38 +0200 | [diff] [blame] | 1070 | * drm_atomic_set_crtc_for_plane - set crtc for plane |
Daniel Vetter | 07cc0ef | 2014-11-27 15:49:39 +0100 | [diff] [blame] | 1071 | * @plane_state: the plane whose incoming state to update |
Daniel Vetter | cc4ceb4 | 2014-07-25 21:30:38 +0200 | [diff] [blame] | 1072 | * @crtc: crtc to use for the plane |
| 1073 | * |
| 1074 | * Changing the assigned crtc for a plane requires us to grab the lock and state |
| 1075 | * for the new crtc, as needed. This function takes care of all these details |
| 1076 | * besides updating the pointer in the state object itself. |
| 1077 | * |
| 1078 | * Returns: |
| 1079 | * 0 on success or can fail with -EDEADLK or -ENOMEM. When the error is EDEADLK |
| 1080 | * then the w/w mutex code has detected a deadlock and the entire atomic |
| 1081 | * sequence must be restarted. All other errors are fatal. |
| 1082 | */ |
| 1083 | int |
Daniel Vetter | 07cc0ef | 2014-11-27 15:49:39 +0100 | [diff] [blame] | 1084 | drm_atomic_set_crtc_for_plane(struct drm_plane_state *plane_state, |
| 1085 | struct drm_crtc *crtc) |
Daniel Vetter | cc4ceb4 | 2014-07-25 21:30:38 +0200 | [diff] [blame] | 1086 | { |
Daniel Vetter | 07cc0ef | 2014-11-27 15:49:39 +0100 | [diff] [blame] | 1087 | struct drm_plane *plane = plane_state->plane; |
Daniel Vetter | cc4ceb4 | 2014-07-25 21:30:38 +0200 | [diff] [blame] | 1088 | struct drm_crtc_state *crtc_state; |
| 1089 | |
Rob Clark | 6ddd388 | 2014-11-21 15:28:31 -0500 | [diff] [blame] | 1090 | if (plane_state->crtc) { |
| 1091 | crtc_state = drm_atomic_get_crtc_state(plane_state->state, |
| 1092 | plane_state->crtc); |
| 1093 | if (WARN_ON(IS_ERR(crtc_state))) |
| 1094 | return PTR_ERR(crtc_state); |
| 1095 | |
| 1096 | crtc_state->plane_mask &= ~(1 << drm_plane_index(plane)); |
| 1097 | } |
| 1098 | |
| 1099 | plane_state->crtc = crtc; |
| 1100 | |
Daniel Vetter | cc4ceb4 | 2014-07-25 21:30:38 +0200 | [diff] [blame] | 1101 | if (crtc) { |
| 1102 | crtc_state = drm_atomic_get_crtc_state(plane_state->state, |
| 1103 | crtc); |
| 1104 | if (IS_ERR(crtc_state)) |
| 1105 | return PTR_ERR(crtc_state); |
Rob Clark | 6ddd388 | 2014-11-21 15:28:31 -0500 | [diff] [blame] | 1106 | crtc_state->plane_mask |= (1 << drm_plane_index(plane)); |
Daniel Vetter | cc4ceb4 | 2014-07-25 21:30:38 +0200 | [diff] [blame] | 1107 | } |
| 1108 | |
Daniel Vetter | cc4ceb4 | 2014-07-25 21:30:38 +0200 | [diff] [blame] | 1109 | if (crtc) |
Ville Syrjälä | fa3ab4c | 2015-12-08 18:41:53 +0200 | [diff] [blame] | 1110 | DRM_DEBUG_ATOMIC("Link plane state %p to [CRTC:%d:%s]\n", |
| 1111 | plane_state, crtc->base.id, crtc->name); |
Daniel Vetter | cc4ceb4 | 2014-07-25 21:30:38 +0200 | [diff] [blame] | 1112 | else |
Daniel Vetter | 17a38d9 | 2015-02-22 12:24:16 +0100 | [diff] [blame] | 1113 | DRM_DEBUG_ATOMIC("Link plane state %p to [NOCRTC]\n", |
| 1114 | plane_state); |
Daniel Vetter | cc4ceb4 | 2014-07-25 21:30:38 +0200 | [diff] [blame] | 1115 | |
| 1116 | return 0; |
| 1117 | } |
| 1118 | EXPORT_SYMBOL(drm_atomic_set_crtc_for_plane); |
| 1119 | |
| 1120 | /** |
John Hunter | 16d78bc2e | 2015-04-07 19:38:50 +0800 | [diff] [blame] | 1121 | * drm_atomic_set_fb_for_plane - set framebuffer for plane |
Daniel Vetter | 321ebf0 | 2014-11-04 22:57:27 +0100 | [diff] [blame] | 1122 | * @plane_state: atomic state object for the plane |
| 1123 | * @fb: fb to use for the plane |
| 1124 | * |
| 1125 | * Changing the assigned framebuffer for a plane requires us to grab a reference |
| 1126 | * to the new fb and drop the reference to the old fb, if there is one. This |
| 1127 | * function takes care of all these details besides updating the pointer in the |
| 1128 | * state object itself. |
| 1129 | */ |
| 1130 | void |
| 1131 | drm_atomic_set_fb_for_plane(struct drm_plane_state *plane_state, |
| 1132 | struct drm_framebuffer *fb) |
| 1133 | { |
| 1134 | if (plane_state->fb) |
| 1135 | drm_framebuffer_unreference(plane_state->fb); |
| 1136 | if (fb) |
| 1137 | drm_framebuffer_reference(fb); |
| 1138 | plane_state->fb = fb; |
| 1139 | |
| 1140 | if (fb) |
Daniel Vetter | 17a38d9 | 2015-02-22 12:24:16 +0100 | [diff] [blame] | 1141 | DRM_DEBUG_ATOMIC("Set [FB:%d] for plane state %p\n", |
| 1142 | fb->base.id, plane_state); |
Daniel Vetter | 321ebf0 | 2014-11-04 22:57:27 +0100 | [diff] [blame] | 1143 | else |
Daniel Vetter | 17a38d9 | 2015-02-22 12:24:16 +0100 | [diff] [blame] | 1144 | DRM_DEBUG_ATOMIC("Set [NOFB] for plane state %p\n", |
| 1145 | plane_state); |
Daniel Vetter | 321ebf0 | 2014-11-04 22:57:27 +0100 | [diff] [blame] | 1146 | } |
| 1147 | EXPORT_SYMBOL(drm_atomic_set_fb_for_plane); |
| 1148 | |
| 1149 | /** |
Daniel Vetter | cc4ceb4 | 2014-07-25 21:30:38 +0200 | [diff] [blame] | 1150 | * drm_atomic_set_crtc_for_connector - set crtc for connector |
| 1151 | * @conn_state: atomic state object for the connector |
| 1152 | * @crtc: crtc to use for the connector |
| 1153 | * |
| 1154 | * Changing the assigned crtc for a connector requires us to grab the lock and |
| 1155 | * state for the new crtc, as needed. This function takes care of all these |
| 1156 | * details besides updating the pointer in the state object itself. |
| 1157 | * |
| 1158 | * Returns: |
| 1159 | * 0 on success or can fail with -EDEADLK or -ENOMEM. When the error is EDEADLK |
| 1160 | * then the w/w mutex code has detected a deadlock and the entire atomic |
| 1161 | * sequence must be restarted. All other errors are fatal. |
| 1162 | */ |
| 1163 | int |
| 1164 | drm_atomic_set_crtc_for_connector(struct drm_connector_state *conn_state, |
| 1165 | struct drm_crtc *crtc) |
| 1166 | { |
| 1167 | struct drm_crtc_state *crtc_state; |
| 1168 | |
Chris Wilson | e2d800a | 2016-05-06 12:47:45 +0100 | [diff] [blame] | 1169 | if (conn_state->crtc == crtc) |
| 1170 | return 0; |
| 1171 | |
| 1172 | if (conn_state->crtc) { |
Maarten Lankhorst | 4cd9fa5 | 2016-01-04 12:53:18 +0100 | [diff] [blame] | 1173 | crtc_state = drm_atomic_get_existing_crtc_state(conn_state->state, |
| 1174 | conn_state->crtc); |
| 1175 | |
| 1176 | crtc_state->connector_mask &= |
| 1177 | ~(1 << drm_connector_index(conn_state->connector)); |
Chris Wilson | e2d800a | 2016-05-06 12:47:45 +0100 | [diff] [blame] | 1178 | |
| 1179 | drm_connector_unreference(conn_state->connector); |
| 1180 | conn_state->crtc = NULL; |
Maarten Lankhorst | 4cd9fa5 | 2016-01-04 12:53:18 +0100 | [diff] [blame] | 1181 | } |
| 1182 | |
Daniel Vetter | cc4ceb4 | 2014-07-25 21:30:38 +0200 | [diff] [blame] | 1183 | if (crtc) { |
| 1184 | crtc_state = drm_atomic_get_crtc_state(conn_state->state, crtc); |
| 1185 | if (IS_ERR(crtc_state)) |
| 1186 | return PTR_ERR(crtc_state); |
Maarten Lankhorst | 4cd9fa5 | 2016-01-04 12:53:18 +0100 | [diff] [blame] | 1187 | |
| 1188 | crtc_state->connector_mask |= |
| 1189 | 1 << drm_connector_index(conn_state->connector); |
Daniel Vetter | cc4ceb4 | 2014-07-25 21:30:38 +0200 | [diff] [blame] | 1190 | |
Chris Wilson | e2d800a | 2016-05-06 12:47:45 +0100 | [diff] [blame] | 1191 | drm_connector_reference(conn_state->connector); |
| 1192 | conn_state->crtc = crtc; |
Daniel Vetter | cc4ceb4 | 2014-07-25 21:30:38 +0200 | [diff] [blame] | 1193 | |
Ville Syrjälä | fa3ab4c | 2015-12-08 18:41:53 +0200 | [diff] [blame] | 1194 | DRM_DEBUG_ATOMIC("Link connector state %p to [CRTC:%d:%s]\n", |
| 1195 | conn_state, crtc->base.id, crtc->name); |
Chris Wilson | e2d800a | 2016-05-06 12:47:45 +0100 | [diff] [blame] | 1196 | } else { |
Daniel Vetter | 17a38d9 | 2015-02-22 12:24:16 +0100 | [diff] [blame] | 1197 | DRM_DEBUG_ATOMIC("Link connector state %p to [NOCRTC]\n", |
| 1198 | conn_state); |
Chris Wilson | e2d800a | 2016-05-06 12:47:45 +0100 | [diff] [blame] | 1199 | } |
Daniel Vetter | cc4ceb4 | 2014-07-25 21:30:38 +0200 | [diff] [blame] | 1200 | |
| 1201 | return 0; |
| 1202 | } |
| 1203 | EXPORT_SYMBOL(drm_atomic_set_crtc_for_connector); |
| 1204 | |
| 1205 | /** |
| 1206 | * drm_atomic_add_affected_connectors - add connectors for crtc |
| 1207 | * @state: atomic state |
| 1208 | * @crtc: DRM crtc |
| 1209 | * |
| 1210 | * This function walks the current configuration and adds all connectors |
| 1211 | * currently using @crtc to the atomic configuration @state. Note that this |
| 1212 | * function must acquire the connection mutex. This can potentially cause |
| 1213 | * unneeded seralization if the update is just for the planes on one crtc. Hence |
| 1214 | * drivers and helpers should only call this when really needed (e.g. when a |
| 1215 | * full modeset needs to happen due to some change). |
| 1216 | * |
| 1217 | * Returns: |
| 1218 | * 0 on success or can fail with -EDEADLK or -ENOMEM. When the error is EDEADLK |
| 1219 | * then the w/w mutex code has detected a deadlock and the entire atomic |
| 1220 | * sequence must be restarted. All other errors are fatal. |
| 1221 | */ |
| 1222 | int |
| 1223 | drm_atomic_add_affected_connectors(struct drm_atomic_state *state, |
| 1224 | struct drm_crtc *crtc) |
| 1225 | { |
| 1226 | struct drm_mode_config *config = &state->dev->mode_config; |
| 1227 | struct drm_connector *connector; |
| 1228 | struct drm_connector_state *conn_state; |
| 1229 | int ret; |
| 1230 | |
| 1231 | ret = drm_modeset_lock(&config->connection_mutex, state->acquire_ctx); |
| 1232 | if (ret) |
| 1233 | return ret; |
| 1234 | |
Ville Syrjälä | fa3ab4c | 2015-12-08 18:41:53 +0200 | [diff] [blame] | 1235 | DRM_DEBUG_ATOMIC("Adding all current connectors for [CRTC:%d:%s] to %p\n", |
| 1236 | crtc->base.id, crtc->name, state); |
Daniel Vetter | cc4ceb4 | 2014-07-25 21:30:38 +0200 | [diff] [blame] | 1237 | |
| 1238 | /* |
| 1239 | * Changed connectors are already in @state, so only need to look at the |
| 1240 | * current configuration. |
| 1241 | */ |
Daniel Vetter | 9a9f5ce | 2015-07-09 23:44:34 +0200 | [diff] [blame] | 1242 | drm_for_each_connector(connector, state->dev) { |
Daniel Vetter | cc4ceb4 | 2014-07-25 21:30:38 +0200 | [diff] [blame] | 1243 | if (connector->state->crtc != crtc) |
| 1244 | continue; |
| 1245 | |
| 1246 | conn_state = drm_atomic_get_connector_state(state, connector); |
| 1247 | if (IS_ERR(conn_state)) |
| 1248 | return PTR_ERR(conn_state); |
| 1249 | } |
| 1250 | |
| 1251 | return 0; |
| 1252 | } |
| 1253 | EXPORT_SYMBOL(drm_atomic_add_affected_connectors); |
| 1254 | |
| 1255 | /** |
Maarten Lankhorst | e01e9f7 | 2015-05-19 16:41:02 +0200 | [diff] [blame] | 1256 | * drm_atomic_add_affected_planes - add planes for crtc |
| 1257 | * @state: atomic state |
| 1258 | * @crtc: DRM crtc |
| 1259 | * |
| 1260 | * This function walks the current configuration and adds all planes |
| 1261 | * currently used by @crtc to the atomic configuration @state. This is useful |
| 1262 | * when an atomic commit also needs to check all currently enabled plane on |
| 1263 | * @crtc, e.g. when changing the mode. It's also useful when re-enabling a CRTC |
| 1264 | * to avoid special code to force-enable all planes. |
| 1265 | * |
| 1266 | * Since acquiring a plane state will always also acquire the w/w mutex of the |
| 1267 | * current CRTC for that plane (if there is any) adding all the plane states for |
| 1268 | * a CRTC will not reduce parallism of atomic updates. |
| 1269 | * |
| 1270 | * Returns: |
| 1271 | * 0 on success or can fail with -EDEADLK or -ENOMEM. When the error is EDEADLK |
| 1272 | * then the w/w mutex code has detected a deadlock and the entire atomic |
| 1273 | * sequence must be restarted. All other errors are fatal. |
| 1274 | */ |
| 1275 | int |
| 1276 | drm_atomic_add_affected_planes(struct drm_atomic_state *state, |
| 1277 | struct drm_crtc *crtc) |
| 1278 | { |
| 1279 | struct drm_plane *plane; |
| 1280 | |
| 1281 | WARN_ON(!drm_atomic_get_existing_crtc_state(state, crtc)); |
| 1282 | |
| 1283 | drm_for_each_plane_mask(plane, state->dev, crtc->state->plane_mask) { |
| 1284 | struct drm_plane_state *plane_state = |
| 1285 | drm_atomic_get_plane_state(state, plane); |
| 1286 | |
| 1287 | if (IS_ERR(plane_state)) |
| 1288 | return PTR_ERR(plane_state); |
| 1289 | } |
| 1290 | return 0; |
| 1291 | } |
| 1292 | EXPORT_SYMBOL(drm_atomic_add_affected_planes); |
| 1293 | |
| 1294 | /** |
Daniel Vetter | cc4ceb4 | 2014-07-25 21:30:38 +0200 | [diff] [blame] | 1295 | * drm_atomic_legacy_backoff - locking backoff for legacy ioctls |
| 1296 | * @state: atomic state |
| 1297 | * |
| 1298 | * This function should be used by legacy entry points which don't understand |
| 1299 | * -EDEADLK semantics. For simplicity this one will grab all modeset locks after |
John Hunter | 16d78bc2e | 2015-04-07 19:38:50 +0800 | [diff] [blame] | 1300 | * the slowpath completed. |
Daniel Vetter | cc4ceb4 | 2014-07-25 21:30:38 +0200 | [diff] [blame] | 1301 | */ |
| 1302 | void drm_atomic_legacy_backoff(struct drm_atomic_state *state) |
| 1303 | { |
Maarten Lankhorst | 81e257e | 2016-06-23 13:45:06 +0200 | [diff] [blame] | 1304 | struct drm_device *dev = state->dev; |
| 1305 | unsigned crtc_mask = 0; |
| 1306 | struct drm_crtc *crtc; |
Daniel Vetter | cc4ceb4 | 2014-07-25 21:30:38 +0200 | [diff] [blame] | 1307 | int ret; |
Maarten Lankhorst | 81e257e | 2016-06-23 13:45:06 +0200 | [diff] [blame] | 1308 | bool global = false; |
| 1309 | |
| 1310 | drm_for_each_crtc(crtc, dev) { |
| 1311 | if (crtc->acquire_ctx != state->acquire_ctx) |
| 1312 | continue; |
| 1313 | |
| 1314 | crtc_mask |= drm_crtc_mask(crtc); |
| 1315 | crtc->acquire_ctx = NULL; |
| 1316 | } |
| 1317 | |
| 1318 | if (WARN_ON(dev->mode_config.acquire_ctx == state->acquire_ctx)) { |
| 1319 | global = true; |
| 1320 | |
| 1321 | dev->mode_config.acquire_ctx = NULL; |
| 1322 | } |
Daniel Vetter | cc4ceb4 | 2014-07-25 21:30:38 +0200 | [diff] [blame] | 1323 | |
| 1324 | retry: |
| 1325 | drm_modeset_backoff(state->acquire_ctx); |
| 1326 | |
Maarten Lankhorst | 81e257e | 2016-06-23 13:45:06 +0200 | [diff] [blame] | 1327 | ret = drm_modeset_lock_all_ctx(dev, state->acquire_ctx); |
Daniel Vetter | cc4ceb4 | 2014-07-25 21:30:38 +0200 | [diff] [blame] | 1328 | if (ret) |
| 1329 | goto retry; |
Maarten Lankhorst | 81e257e | 2016-06-23 13:45:06 +0200 | [diff] [blame] | 1330 | |
| 1331 | drm_for_each_crtc(crtc, dev) |
| 1332 | if (drm_crtc_mask(crtc) & crtc_mask) |
| 1333 | crtc->acquire_ctx = state->acquire_ctx; |
| 1334 | |
| 1335 | if (global) |
| 1336 | dev->mode_config.acquire_ctx = state->acquire_ctx; |
Daniel Vetter | cc4ceb4 | 2014-07-25 21:30:38 +0200 | [diff] [blame] | 1337 | } |
| 1338 | EXPORT_SYMBOL(drm_atomic_legacy_backoff); |
| 1339 | |
| 1340 | /** |
| 1341 | * drm_atomic_check_only - check whether a given config would work |
| 1342 | * @state: atomic configuration to check |
| 1343 | * |
| 1344 | * Note that this function can return -EDEADLK if the driver needed to acquire |
| 1345 | * more locks but encountered a deadlock. The caller must then do the usual w/w |
| 1346 | * backoff dance and restart. All other errors are fatal. |
| 1347 | * |
| 1348 | * Returns: |
| 1349 | * 0 on success, negative error code on failure. |
| 1350 | */ |
| 1351 | int drm_atomic_check_only(struct drm_atomic_state *state) |
| 1352 | { |
Rob Clark | 5e74373 | 2014-12-18 16:01:51 -0500 | [diff] [blame] | 1353 | struct drm_device *dev = state->dev; |
| 1354 | struct drm_mode_config *config = &dev->mode_config; |
Ander Conselvan de Oliveira | df63b99 | 2015-04-10 14:58:39 +0300 | [diff] [blame] | 1355 | struct drm_plane *plane; |
| 1356 | struct drm_plane_state *plane_state; |
| 1357 | struct drm_crtc *crtc; |
| 1358 | struct drm_crtc_state *crtc_state; |
Rob Clark | 5e74373 | 2014-12-18 16:01:51 -0500 | [diff] [blame] | 1359 | int i, ret = 0; |
Daniel Vetter | cc4ceb4 | 2014-07-25 21:30:38 +0200 | [diff] [blame] | 1360 | |
Daniel Vetter | 17a38d9 | 2015-02-22 12:24:16 +0100 | [diff] [blame] | 1361 | DRM_DEBUG_ATOMIC("checking %p\n", state); |
Daniel Vetter | cc4ceb4 | 2014-07-25 21:30:38 +0200 | [diff] [blame] | 1362 | |
Ander Conselvan de Oliveira | df63b99 | 2015-04-10 14:58:39 +0300 | [diff] [blame] | 1363 | for_each_plane_in_state(state, plane, plane_state, i) { |
| 1364 | ret = drm_atomic_plane_check(plane, plane_state); |
Rob Clark | 5e74373 | 2014-12-18 16:01:51 -0500 | [diff] [blame] | 1365 | if (ret) { |
Ville Syrjälä | 9f4c97a | 2015-12-08 18:41:54 +0200 | [diff] [blame] | 1366 | DRM_DEBUG_ATOMIC("[PLANE:%d:%s] atomic core check failed\n", |
| 1367 | plane->base.id, plane->name); |
Rob Clark | 5e74373 | 2014-12-18 16:01:51 -0500 | [diff] [blame] | 1368 | return ret; |
| 1369 | } |
| 1370 | } |
| 1371 | |
Ander Conselvan de Oliveira | df63b99 | 2015-04-10 14:58:39 +0300 | [diff] [blame] | 1372 | for_each_crtc_in_state(state, crtc, crtc_state, i) { |
| 1373 | ret = drm_atomic_crtc_check(crtc, crtc_state); |
Rob Clark | 5e74373 | 2014-12-18 16:01:51 -0500 | [diff] [blame] | 1374 | if (ret) { |
Ville Syrjälä | fa3ab4c | 2015-12-08 18:41:53 +0200 | [diff] [blame] | 1375 | DRM_DEBUG_ATOMIC("[CRTC:%d:%s] atomic core check failed\n", |
| 1376 | crtc->base.id, crtc->name); |
Rob Clark | 5e74373 | 2014-12-18 16:01:51 -0500 | [diff] [blame] | 1377 | return ret; |
| 1378 | } |
| 1379 | } |
| 1380 | |
Daniel Vetter | cc4ceb4 | 2014-07-25 21:30:38 +0200 | [diff] [blame] | 1381 | if (config->funcs->atomic_check) |
Rob Clark | 5e74373 | 2014-12-18 16:01:51 -0500 | [diff] [blame] | 1382 | ret = config->funcs->atomic_check(state->dev, state); |
| 1383 | |
Rob Clark | d34f20d | 2014-12-18 16:01:56 -0500 | [diff] [blame] | 1384 | if (!state->allow_modeset) { |
Ander Conselvan de Oliveira | df63b99 | 2015-04-10 14:58:39 +0300 | [diff] [blame] | 1385 | for_each_crtc_in_state(state, crtc, crtc_state, i) { |
Daniel Vetter | 2465ff6 | 2015-06-18 09:58:55 +0200 | [diff] [blame] | 1386 | if (drm_atomic_crtc_needs_modeset(crtc_state)) { |
Ville Syrjälä | fa3ab4c | 2015-12-08 18:41:53 +0200 | [diff] [blame] | 1387 | DRM_DEBUG_ATOMIC("[CRTC:%d:%s] requires full modeset\n", |
| 1388 | crtc->base.id, crtc->name); |
Rob Clark | d34f20d | 2014-12-18 16:01:56 -0500 | [diff] [blame] | 1389 | return -EINVAL; |
| 1390 | } |
| 1391 | } |
| 1392 | } |
| 1393 | |
Rob Clark | 5e74373 | 2014-12-18 16:01:51 -0500 | [diff] [blame] | 1394 | return ret; |
Daniel Vetter | cc4ceb4 | 2014-07-25 21:30:38 +0200 | [diff] [blame] | 1395 | } |
| 1396 | EXPORT_SYMBOL(drm_atomic_check_only); |
| 1397 | |
| 1398 | /** |
| 1399 | * drm_atomic_commit - commit configuration atomically |
| 1400 | * @state: atomic configuration to check |
| 1401 | * |
| 1402 | * Note that this function can return -EDEADLK if the driver needed to acquire |
| 1403 | * more locks but encountered a deadlock. The caller must then do the usual w/w |
| 1404 | * backoff dance and restart. All other errors are fatal. |
| 1405 | * |
| 1406 | * Also note that on successful execution ownership of @state is transferred |
| 1407 | * from the caller of this function to the function itself. The caller must not |
| 1408 | * free or in any other way access @state. If the function fails then the caller |
| 1409 | * must clean up @state itself. |
| 1410 | * |
| 1411 | * Returns: |
| 1412 | * 0 on success, negative error code on failure. |
| 1413 | */ |
| 1414 | int drm_atomic_commit(struct drm_atomic_state *state) |
| 1415 | { |
| 1416 | struct drm_mode_config *config = &state->dev->mode_config; |
| 1417 | int ret; |
| 1418 | |
| 1419 | ret = drm_atomic_check_only(state); |
| 1420 | if (ret) |
| 1421 | return ret; |
| 1422 | |
Daniel Vetter | 17a38d9 | 2015-02-22 12:24:16 +0100 | [diff] [blame] | 1423 | DRM_DEBUG_ATOMIC("commiting %p\n", state); |
Daniel Vetter | cc4ceb4 | 2014-07-25 21:30:38 +0200 | [diff] [blame] | 1424 | |
| 1425 | return config->funcs->atomic_commit(state->dev, state, false); |
| 1426 | } |
| 1427 | EXPORT_SYMBOL(drm_atomic_commit); |
| 1428 | |
| 1429 | /** |
Maarten Lankhorst | b837ba0 | 2016-04-26 16:11:35 +0200 | [diff] [blame] | 1430 | * drm_atomic_nonblocking_commit - atomic&nonblocking configuration commit |
Daniel Vetter | cc4ceb4 | 2014-07-25 21:30:38 +0200 | [diff] [blame] | 1431 | * @state: atomic configuration to check |
| 1432 | * |
| 1433 | * Note that this function can return -EDEADLK if the driver needed to acquire |
| 1434 | * more locks but encountered a deadlock. The caller must then do the usual w/w |
| 1435 | * backoff dance and restart. All other errors are fatal. |
| 1436 | * |
| 1437 | * Also note that on successful execution ownership of @state is transferred |
| 1438 | * from the caller of this function to the function itself. The caller must not |
| 1439 | * free or in any other way access @state. If the function fails then the caller |
| 1440 | * must clean up @state itself. |
| 1441 | * |
| 1442 | * Returns: |
| 1443 | * 0 on success, negative error code on failure. |
| 1444 | */ |
Maarten Lankhorst | b837ba0 | 2016-04-26 16:11:35 +0200 | [diff] [blame] | 1445 | int drm_atomic_nonblocking_commit(struct drm_atomic_state *state) |
Daniel Vetter | cc4ceb4 | 2014-07-25 21:30:38 +0200 | [diff] [blame] | 1446 | { |
| 1447 | struct drm_mode_config *config = &state->dev->mode_config; |
| 1448 | int ret; |
| 1449 | |
| 1450 | ret = drm_atomic_check_only(state); |
| 1451 | if (ret) |
| 1452 | return ret; |
| 1453 | |
Maarten Lankhorst | b837ba0 | 2016-04-26 16:11:35 +0200 | [diff] [blame] | 1454 | DRM_DEBUG_ATOMIC("commiting %p nonblocking\n", state); |
Daniel Vetter | cc4ceb4 | 2014-07-25 21:30:38 +0200 | [diff] [blame] | 1455 | |
| 1456 | return config->funcs->atomic_commit(state->dev, state, true); |
| 1457 | } |
Maarten Lankhorst | b837ba0 | 2016-04-26 16:11:35 +0200 | [diff] [blame] | 1458 | EXPORT_SYMBOL(drm_atomic_nonblocking_commit); |
Rob Clark | d34f20d | 2014-12-18 16:01:56 -0500 | [diff] [blame] | 1459 | |
| 1460 | /* |
| 1461 | * The big monstor ioctl |
| 1462 | */ |
| 1463 | |
| 1464 | static struct drm_pending_vblank_event *create_vblank_event( |
Gustavo Padovan | 1b47aaf | 2016-06-02 00:06:35 +0200 | [diff] [blame] | 1465 | struct drm_device *dev, struct drm_file *file_priv, |
| 1466 | struct fence *fence, uint64_t user_data) |
Rob Clark | d34f20d | 2014-12-18 16:01:56 -0500 | [diff] [blame] | 1467 | { |
| 1468 | struct drm_pending_vblank_event *e = NULL; |
Daniel Vetter | 2dd500f | 2016-01-11 22:40:56 +0100 | [diff] [blame] | 1469 | int ret; |
Rob Clark | d34f20d | 2014-12-18 16:01:56 -0500 | [diff] [blame] | 1470 | |
| 1471 | e = kzalloc(sizeof *e, GFP_KERNEL); |
Daniel Vetter | 2dd500f | 2016-01-11 22:40:56 +0100 | [diff] [blame] | 1472 | if (!e) |
| 1473 | return NULL; |
Rob Clark | d34f20d | 2014-12-18 16:01:56 -0500 | [diff] [blame] | 1474 | |
| 1475 | e->event.base.type = DRM_EVENT_FLIP_COMPLETE; |
Daniel Vetter | 2dd500f | 2016-01-11 22:40:56 +0100 | [diff] [blame] | 1476 | e->event.base.length = sizeof(e->event); |
Rob Clark | d34f20d | 2014-12-18 16:01:56 -0500 | [diff] [blame] | 1477 | e->event.user_data = user_data; |
Rob Clark | d34f20d | 2014-12-18 16:01:56 -0500 | [diff] [blame] | 1478 | |
Gustavo Padovan | 1b47aaf | 2016-06-02 00:06:35 +0200 | [diff] [blame] | 1479 | if (file_priv) { |
| 1480 | ret = drm_event_reserve_init(dev, file_priv, &e->base, |
| 1481 | &e->event.base); |
| 1482 | if (ret) { |
| 1483 | kfree(e); |
| 1484 | return NULL; |
| 1485 | } |
Daniel Vetter | 2dd500f | 2016-01-11 22:40:56 +0100 | [diff] [blame] | 1486 | } |
| 1487 | |
Gustavo Padovan | 1b47aaf | 2016-06-02 00:06:35 +0200 | [diff] [blame] | 1488 | e->base.fence = fence; |
| 1489 | |
Rob Clark | d34f20d | 2014-12-18 16:01:56 -0500 | [diff] [blame] | 1490 | return e; |
| 1491 | } |
| 1492 | |
Rob Clark | d34f20d | 2014-12-18 16:01:56 -0500 | [diff] [blame] | 1493 | static int atomic_set_prop(struct drm_atomic_state *state, |
| 1494 | struct drm_mode_object *obj, struct drm_property *prop, |
| 1495 | uint64_t prop_value) |
| 1496 | { |
| 1497 | struct drm_mode_object *ref; |
| 1498 | int ret; |
| 1499 | |
| 1500 | if (!drm_property_change_valid_get(prop, prop_value, &ref)) |
| 1501 | return -EINVAL; |
| 1502 | |
| 1503 | switch (obj->type) { |
| 1504 | case DRM_MODE_OBJECT_CONNECTOR: { |
| 1505 | struct drm_connector *connector = obj_to_connector(obj); |
| 1506 | struct drm_connector_state *connector_state; |
| 1507 | |
| 1508 | connector_state = drm_atomic_get_connector_state(state, connector); |
| 1509 | if (IS_ERR(connector_state)) { |
| 1510 | ret = PTR_ERR(connector_state); |
| 1511 | break; |
| 1512 | } |
| 1513 | |
| 1514 | ret = drm_atomic_connector_set_property(connector, |
| 1515 | connector_state, prop, prop_value); |
| 1516 | break; |
| 1517 | } |
| 1518 | case DRM_MODE_OBJECT_CRTC: { |
| 1519 | struct drm_crtc *crtc = obj_to_crtc(obj); |
| 1520 | struct drm_crtc_state *crtc_state; |
| 1521 | |
| 1522 | crtc_state = drm_atomic_get_crtc_state(state, crtc); |
| 1523 | if (IS_ERR(crtc_state)) { |
| 1524 | ret = PTR_ERR(crtc_state); |
| 1525 | break; |
| 1526 | } |
| 1527 | |
| 1528 | ret = drm_atomic_crtc_set_property(crtc, |
| 1529 | crtc_state, prop, prop_value); |
| 1530 | break; |
| 1531 | } |
| 1532 | case DRM_MODE_OBJECT_PLANE: { |
| 1533 | struct drm_plane *plane = obj_to_plane(obj); |
| 1534 | struct drm_plane_state *plane_state; |
| 1535 | |
| 1536 | plane_state = drm_atomic_get_plane_state(state, plane); |
| 1537 | if (IS_ERR(plane_state)) { |
| 1538 | ret = PTR_ERR(plane_state); |
| 1539 | break; |
| 1540 | } |
| 1541 | |
| 1542 | ret = drm_atomic_plane_set_property(plane, |
| 1543 | plane_state, prop, prop_value); |
| 1544 | break; |
| 1545 | } |
| 1546 | default: |
| 1547 | ret = -EINVAL; |
| 1548 | break; |
| 1549 | } |
| 1550 | |
| 1551 | drm_property_change_valid_put(prop, ref); |
| 1552 | return ret; |
| 1553 | } |
| 1554 | |
Maarten Lankhorst | 0f45c26 | 2015-11-11 11:29:09 +0100 | [diff] [blame] | 1555 | /** |
Maarten Lankhorst | 9744bf4 | 2015-11-24 10:34:34 +0100 | [diff] [blame] | 1556 | * drm_atomic_clean_old_fb -- Unset old_fb pointers and set plane->fb pointers. |
Maarten Lankhorst | 0f45c26 | 2015-11-11 11:29:09 +0100 | [diff] [blame] | 1557 | * |
| 1558 | * @dev: drm device to check. |
| 1559 | * @plane_mask: plane mask for planes that were updated. |
| 1560 | * @ret: return value, can be -EDEADLK for a retry. |
| 1561 | * |
| 1562 | * Before doing an update plane->old_fb is set to plane->fb, |
| 1563 | * but before dropping the locks old_fb needs to be set to NULL |
| 1564 | * and plane->fb updated. This is a common operation for each |
| 1565 | * atomic update, so this call is split off as a helper. |
| 1566 | */ |
| 1567 | void drm_atomic_clean_old_fb(struct drm_device *dev, |
| 1568 | unsigned plane_mask, |
| 1569 | int ret) |
| 1570 | { |
| 1571 | struct drm_plane *plane; |
| 1572 | |
| 1573 | /* if succeeded, fixup legacy plane crtc/fb ptrs before dropping |
| 1574 | * locks (ie. while it is still safe to deref plane->state). We |
| 1575 | * need to do this here because the driver entry points cannot |
| 1576 | * distinguish between legacy and atomic ioctls. |
| 1577 | */ |
| 1578 | drm_for_each_plane_mask(plane, dev, plane_mask) { |
| 1579 | if (ret == 0) { |
| 1580 | struct drm_framebuffer *new_fb = plane->state->fb; |
| 1581 | if (new_fb) |
| 1582 | drm_framebuffer_reference(new_fb); |
| 1583 | plane->fb = new_fb; |
| 1584 | plane->crtc = plane->state->crtc; |
| 1585 | |
| 1586 | if (plane->old_fb) |
| 1587 | drm_framebuffer_unreference(plane->old_fb); |
| 1588 | } |
| 1589 | plane->old_fb = NULL; |
| 1590 | } |
| 1591 | } |
| 1592 | EXPORT_SYMBOL(drm_atomic_clean_old_fb); |
| 1593 | |
Rob Clark | d34f20d | 2014-12-18 16:01:56 -0500 | [diff] [blame] | 1594 | int drm_mode_atomic_ioctl(struct drm_device *dev, |
| 1595 | void *data, struct drm_file *file_priv) |
| 1596 | { |
| 1597 | struct drm_mode_atomic *arg = data; |
| 1598 | uint32_t __user *objs_ptr = (uint32_t __user *)(unsigned long)(arg->objs_ptr); |
| 1599 | uint32_t __user *count_props_ptr = (uint32_t __user *)(unsigned long)(arg->count_props_ptr); |
| 1600 | uint32_t __user *props_ptr = (uint32_t __user *)(unsigned long)(arg->props_ptr); |
| 1601 | uint64_t __user *prop_values_ptr = (uint64_t __user *)(unsigned long)(arg->prop_values_ptr); |
| 1602 | unsigned int copied_objs, copied_props; |
| 1603 | struct drm_atomic_state *state; |
| 1604 | struct drm_modeset_acquire_ctx ctx; |
| 1605 | struct drm_plane *plane; |
Ander Conselvan de Oliveira | df63b99 | 2015-04-10 14:58:39 +0300 | [diff] [blame] | 1606 | struct drm_crtc *crtc; |
| 1607 | struct drm_crtc_state *crtc_state; |
Maarten Lankhorst | 4572372 | 2015-11-11 11:29:08 +0100 | [diff] [blame] | 1608 | unsigned plane_mask; |
Rob Clark | d34f20d | 2014-12-18 16:01:56 -0500 | [diff] [blame] | 1609 | int ret = 0; |
Maarten Lankhorst | f92f053 | 2016-09-08 12:30:01 +0200 | [diff] [blame] | 1610 | unsigned int i, j; |
Rob Clark | d34f20d | 2014-12-18 16:01:56 -0500 | [diff] [blame] | 1611 | |
| 1612 | /* disallow for drivers not supporting atomic: */ |
| 1613 | if (!drm_core_check_feature(dev, DRIVER_ATOMIC)) |
| 1614 | return -EINVAL; |
| 1615 | |
| 1616 | /* disallow for userspace that has not enabled atomic cap (even |
| 1617 | * though this may be a bit overkill, since legacy userspace |
| 1618 | * wouldn't know how to call this ioctl) |
| 1619 | */ |
| 1620 | if (!file_priv->atomic) |
| 1621 | return -EINVAL; |
| 1622 | |
| 1623 | if (arg->flags & ~DRM_MODE_ATOMIC_FLAGS) |
| 1624 | return -EINVAL; |
| 1625 | |
| 1626 | if (arg->reserved) |
| 1627 | return -EINVAL; |
| 1628 | |
| 1629 | if ((arg->flags & DRM_MODE_PAGE_FLIP_ASYNC) && |
| 1630 | !dev->mode_config.async_page_flip) |
| 1631 | return -EINVAL; |
| 1632 | |
| 1633 | /* can't test and expect an event at the same time. */ |
| 1634 | if ((arg->flags & DRM_MODE_ATOMIC_TEST_ONLY) && |
| 1635 | (arg->flags & DRM_MODE_PAGE_FLIP_EVENT)) |
| 1636 | return -EINVAL; |
| 1637 | |
| 1638 | drm_modeset_acquire_init(&ctx, 0); |
| 1639 | |
| 1640 | state = drm_atomic_state_alloc(dev); |
| 1641 | if (!state) |
| 1642 | return -ENOMEM; |
| 1643 | |
| 1644 | state->acquire_ctx = &ctx; |
| 1645 | state->allow_modeset = !!(arg->flags & DRM_MODE_ATOMIC_ALLOW_MODESET); |
| 1646 | |
| 1647 | retry: |
Maarten Lankhorst | 4572372 | 2015-11-11 11:29:08 +0100 | [diff] [blame] | 1648 | plane_mask = 0; |
Rob Clark | d34f20d | 2014-12-18 16:01:56 -0500 | [diff] [blame] | 1649 | copied_objs = 0; |
| 1650 | copied_props = 0; |
| 1651 | |
| 1652 | for (i = 0; i < arg->count_objs; i++) { |
| 1653 | uint32_t obj_id, count_props; |
| 1654 | struct drm_mode_object *obj; |
| 1655 | |
| 1656 | if (get_user(obj_id, objs_ptr + copied_objs)) { |
| 1657 | ret = -EFAULT; |
Maarten Lankhorst | ec9f932 | 2015-06-24 08:59:25 +0200 | [diff] [blame] | 1658 | goto out; |
Rob Clark | d34f20d | 2014-12-18 16:01:56 -0500 | [diff] [blame] | 1659 | } |
| 1660 | |
| 1661 | obj = drm_mode_object_find(dev, obj_id, DRM_MODE_OBJECT_ANY); |
Dave Airlie | b164d31 | 2016-04-27 11:10:09 +1000 | [diff] [blame] | 1662 | if (!obj) { |
| 1663 | ret = -ENOENT; |
| 1664 | goto out; |
| 1665 | } |
| 1666 | |
| 1667 | if (!obj->properties) { |
| 1668 | drm_mode_object_unreference(obj); |
Rob Clark | d34f20d | 2014-12-18 16:01:56 -0500 | [diff] [blame] | 1669 | ret = -ENOENT; |
Maarten Lankhorst | ec9f932 | 2015-06-24 08:59:25 +0200 | [diff] [blame] | 1670 | goto out; |
Rob Clark | d34f20d | 2014-12-18 16:01:56 -0500 | [diff] [blame] | 1671 | } |
| 1672 | |
Rob Clark | d34f20d | 2014-12-18 16:01:56 -0500 | [diff] [blame] | 1673 | if (get_user(count_props, count_props_ptr + copied_objs)) { |
Dave Airlie | b164d31 | 2016-04-27 11:10:09 +1000 | [diff] [blame] | 1674 | drm_mode_object_unreference(obj); |
Rob Clark | d34f20d | 2014-12-18 16:01:56 -0500 | [diff] [blame] | 1675 | ret = -EFAULT; |
Maarten Lankhorst | ec9f932 | 2015-06-24 08:59:25 +0200 | [diff] [blame] | 1676 | goto out; |
Rob Clark | d34f20d | 2014-12-18 16:01:56 -0500 | [diff] [blame] | 1677 | } |
| 1678 | |
| 1679 | copied_objs++; |
| 1680 | |
| 1681 | for (j = 0; j < count_props; j++) { |
| 1682 | uint32_t prop_id; |
| 1683 | uint64_t prop_value; |
| 1684 | struct drm_property *prop; |
| 1685 | |
| 1686 | if (get_user(prop_id, props_ptr + copied_props)) { |
Dave Airlie | b164d31 | 2016-04-27 11:10:09 +1000 | [diff] [blame] | 1687 | drm_mode_object_unreference(obj); |
Rob Clark | d34f20d | 2014-12-18 16:01:56 -0500 | [diff] [blame] | 1688 | ret = -EFAULT; |
Maarten Lankhorst | ec9f932 | 2015-06-24 08:59:25 +0200 | [diff] [blame] | 1689 | goto out; |
Rob Clark | d34f20d | 2014-12-18 16:01:56 -0500 | [diff] [blame] | 1690 | } |
| 1691 | |
Maarten Lankhorst | f92f053 | 2016-09-08 12:30:01 +0200 | [diff] [blame] | 1692 | prop = drm_mode_obj_find_prop_id(obj, prop_id); |
Rob Clark | d34f20d | 2014-12-18 16:01:56 -0500 | [diff] [blame] | 1693 | if (!prop) { |
Dave Airlie | b164d31 | 2016-04-27 11:10:09 +1000 | [diff] [blame] | 1694 | drm_mode_object_unreference(obj); |
Rob Clark | d34f20d | 2014-12-18 16:01:56 -0500 | [diff] [blame] | 1695 | ret = -ENOENT; |
Maarten Lankhorst | ec9f932 | 2015-06-24 08:59:25 +0200 | [diff] [blame] | 1696 | goto out; |
Rob Clark | d34f20d | 2014-12-18 16:01:56 -0500 | [diff] [blame] | 1697 | } |
| 1698 | |
Guenter Roeck | 42c5814 | 2015-01-12 21:12:17 -0800 | [diff] [blame] | 1699 | if (copy_from_user(&prop_value, |
| 1700 | prop_values_ptr + copied_props, |
| 1701 | sizeof(prop_value))) { |
Dave Airlie | b164d31 | 2016-04-27 11:10:09 +1000 | [diff] [blame] | 1702 | drm_mode_object_unreference(obj); |
Rob Clark | d34f20d | 2014-12-18 16:01:56 -0500 | [diff] [blame] | 1703 | ret = -EFAULT; |
Maarten Lankhorst | ec9f932 | 2015-06-24 08:59:25 +0200 | [diff] [blame] | 1704 | goto out; |
Rob Clark | d34f20d | 2014-12-18 16:01:56 -0500 | [diff] [blame] | 1705 | } |
| 1706 | |
| 1707 | ret = atomic_set_prop(state, obj, prop, prop_value); |
Dave Airlie | b164d31 | 2016-04-27 11:10:09 +1000 | [diff] [blame] | 1708 | if (ret) { |
| 1709 | drm_mode_object_unreference(obj); |
Maarten Lankhorst | ec9f932 | 2015-06-24 08:59:25 +0200 | [diff] [blame] | 1710 | goto out; |
Dave Airlie | b164d31 | 2016-04-27 11:10:09 +1000 | [diff] [blame] | 1711 | } |
Rob Clark | d34f20d | 2014-12-18 16:01:56 -0500 | [diff] [blame] | 1712 | |
| 1713 | copied_props++; |
| 1714 | } |
Maarten Lankhorst | a9cc54e | 2015-06-24 08:59:24 +0200 | [diff] [blame] | 1715 | |
Maarten Lankhorst | c4749c9 | 2015-08-31 12:25:04 +0200 | [diff] [blame] | 1716 | if (obj->type == DRM_MODE_OBJECT_PLANE && count_props && |
| 1717 | !(arg->flags & DRM_MODE_ATOMIC_TEST_ONLY)) { |
Maarten Lankhorst | a9cc54e | 2015-06-24 08:59:24 +0200 | [diff] [blame] | 1718 | plane = obj_to_plane(obj); |
| 1719 | plane_mask |= (1 << drm_plane_index(plane)); |
| 1720 | plane->old_fb = plane->fb; |
| 1721 | } |
Dave Airlie | b164d31 | 2016-04-27 11:10:09 +1000 | [diff] [blame] | 1722 | drm_mode_object_unreference(obj); |
Rob Clark | d34f20d | 2014-12-18 16:01:56 -0500 | [diff] [blame] | 1723 | } |
| 1724 | |
| 1725 | if (arg->flags & DRM_MODE_PAGE_FLIP_EVENT) { |
Ander Conselvan de Oliveira | df63b99 | 2015-04-10 14:58:39 +0300 | [diff] [blame] | 1726 | for_each_crtc_in_state(state, crtc, crtc_state, i) { |
Rob Clark | d34f20d | 2014-12-18 16:01:56 -0500 | [diff] [blame] | 1727 | struct drm_pending_vblank_event *e; |
| 1728 | |
Gustavo Padovan | 1b47aaf | 2016-06-02 00:06:35 +0200 | [diff] [blame] | 1729 | e = create_vblank_event(dev, file_priv, NULL, |
| 1730 | arg->user_data); |
Rob Clark | d34f20d | 2014-12-18 16:01:56 -0500 | [diff] [blame] | 1731 | if (!e) { |
| 1732 | ret = -ENOMEM; |
Maarten Lankhorst | ec9f932 | 2015-06-24 08:59:25 +0200 | [diff] [blame] | 1733 | goto out; |
Rob Clark | d34f20d | 2014-12-18 16:01:56 -0500 | [diff] [blame] | 1734 | } |
| 1735 | |
| 1736 | crtc_state->event = e; |
| 1737 | } |
| 1738 | } |
| 1739 | |
| 1740 | if (arg->flags & DRM_MODE_ATOMIC_TEST_ONLY) { |
Maarten Lankhorst | c4749c9 | 2015-08-31 12:25:04 +0200 | [diff] [blame] | 1741 | /* |
| 1742 | * Unlike commit, check_only does not clean up state. |
Chris Wilson | 0853695 | 2016-10-14 13:18:18 +0100 | [diff] [blame] | 1743 | * Below we call drm_atomic_state_put for it. |
Maarten Lankhorst | c4749c9 | 2015-08-31 12:25:04 +0200 | [diff] [blame] | 1744 | */ |
Rob Clark | d34f20d | 2014-12-18 16:01:56 -0500 | [diff] [blame] | 1745 | ret = drm_atomic_check_only(state); |
Rob Clark | d34f20d | 2014-12-18 16:01:56 -0500 | [diff] [blame] | 1746 | } else if (arg->flags & DRM_MODE_ATOMIC_NONBLOCK) { |
Maarten Lankhorst | b837ba0 | 2016-04-26 16:11:35 +0200 | [diff] [blame] | 1747 | ret = drm_atomic_nonblocking_commit(state); |
Rob Clark | d34f20d | 2014-12-18 16:01:56 -0500 | [diff] [blame] | 1748 | } else { |
| 1749 | ret = drm_atomic_commit(state); |
| 1750 | } |
| 1751 | |
Maarten Lankhorst | ec9f932 | 2015-06-24 08:59:25 +0200 | [diff] [blame] | 1752 | out: |
Maarten Lankhorst | 0f45c26 | 2015-11-11 11:29:09 +0100 | [diff] [blame] | 1753 | drm_atomic_clean_old_fb(dev, plane_mask, ret); |
Maarten Lankhorst | ec9f932 | 2015-06-24 08:59:25 +0200 | [diff] [blame] | 1754 | |
Maarten Lankhorst | c4749c9 | 2015-08-31 12:25:04 +0200 | [diff] [blame] | 1755 | if (ret && arg->flags & DRM_MODE_PAGE_FLIP_EVENT) { |
| 1756 | /* |
| 1757 | * TEST_ONLY and PAGE_FLIP_EVENT are mutually exclusive, |
| 1758 | * if they weren't, this code should be called on success |
| 1759 | * for TEST_ONLY too. |
| 1760 | */ |
| 1761 | |
| 1762 | for_each_crtc_in_state(state, crtc, crtc_state, i) { |
| 1763 | if (!crtc_state->event) |
| 1764 | continue; |
| 1765 | |
Daniel Vetter | 2dd500f | 2016-01-11 22:40:56 +0100 | [diff] [blame] | 1766 | drm_event_cancel_free(dev, &crtc_state->event->base); |
Maarten Lankhorst | c4749c9 | 2015-08-31 12:25:04 +0200 | [diff] [blame] | 1767 | } |
| 1768 | } |
| 1769 | |
Maarten Lankhorst | ec9f932 | 2015-06-24 08:59:25 +0200 | [diff] [blame] | 1770 | if (ret == -EDEADLK) { |
| 1771 | drm_atomic_state_clear(state); |
| 1772 | drm_modeset_backoff(&ctx); |
| 1773 | goto retry; |
| 1774 | } |
| 1775 | |
Chris Wilson | 0853695 | 2016-10-14 13:18:18 +0100 | [diff] [blame] | 1776 | drm_atomic_state_put(state); |
Rob Clark | d34f20d | 2014-12-18 16:01:56 -0500 | [diff] [blame] | 1777 | |
| 1778 | drm_modeset_drop_locks(&ctx); |
| 1779 | drm_modeset_acquire_fini(&ctx); |
| 1780 | |
| 1781 | return ret; |
Rob Clark | d34f20d | 2014-12-18 16:01:56 -0500 | [diff] [blame] | 1782 | } |