blob: 51f534db91070772e59265d16b3115a4883f2afe [file] [log] [blame]
Rob Clark51fd3712013-11-19 12:10:12 -05001/*
2 * Copyright (C) 2014 Red Hat
3 * Author: Rob Clark <robdclark@gmail.com>
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
24#include <drm/drmP.h>
25#include <drm/drm_crtc.h>
26#include <drm/drm_modeset_lock.h>
27
28/**
29 * DOC: kms locking
30 *
31 * As KMS moves toward more fine grained locking, and atomic ioctl where
32 * userspace can indirectly control locking order, it becomes necessary
Daniel Vetter0645de52016-05-30 11:10:49 +020033 * to use &ww_mutex and acquire-contexts to avoid deadlocks. But because
Rob Clark51fd3712013-11-19 12:10:12 -050034 * the locking is more distributed around the driver code, we want a bit
35 * of extra utility/tracking out of our acquire-ctx. This is provided
Daniel Vetterd5745282017-01-25 07:26:45 +010036 * by &struct drm_modeset_lock and &struct drm_modeset_acquire_ctx.
Rob Clark51fd3712013-11-19 12:10:12 -050037 *
Daniel Vetter0645de52016-05-30 11:10:49 +020038 * For basic principles of &ww_mutex, see: Documentation/locking/ww-mutex-design.txt
Rob Clark51fd3712013-11-19 12:10:12 -050039 *
Daniel Vetterda5335b2016-05-31 22:55:13 +020040 * The basic usage pattern is to::
Rob Clark51fd3712013-11-19 12:10:12 -050041 *
Maarten Lankhorst6f8bcc72017-09-12 15:37:44 +020042 * drm_modeset_acquire_init(ctx, DRM_MODESET_ACQUIRE_INTERRUPTIBLE)
Danilo Cesar Lemes de Paulaf03d8ed2015-11-25 18:07:55 +010043 * retry:
Rob Clark51fd3712013-11-19 12:10:12 -050044 * foreach (lock in random_ordered_set_of_locks) {
Maarten Lankhorst6f8bcc72017-09-12 15:37:44 +020045 * ret = drm_modeset_lock(lock, ctx)
Danilo Cesar Lemes de Paulaf03d8ed2015-11-25 18:07:55 +010046 * if (ret == -EDEADLK) {
Maarten Lankhorst6f8bcc72017-09-12 15:37:44 +020047 * ret = drm_modeset_backoff(ctx);
48 * if (!ret)
49 * goto retry;
Danilo Cesar Lemes de Paulaf03d8ed2015-11-25 18:07:55 +010050 * }
Maarten Lankhorst6f8bcc72017-09-12 15:37:44 +020051 * if (ret)
52 * goto out;
Rob Clark51fd3712013-11-19 12:10:12 -050053 * }
Rob Clark51fd3712013-11-19 12:10:12 -050054 * ... do stuff ...
Maarten Lankhorst6f8bcc72017-09-12 15:37:44 +020055 * out:
56 * drm_modeset_drop_locks(ctx);
57 * drm_modeset_acquire_fini(ctx);
Daniel Vetter0645de52016-05-30 11:10:49 +020058 *
Sean Paulb7ea04d2018-11-29 10:04:17 -050059 * For convenience this control flow is implemented in
60 * DRM_MODESET_LOCK_ALL_BEGIN() and DRM_MODESET_LOCK_ALL_END() for the case
61 * where all modeset locks need to be taken through drm_modeset_lock_all_ctx().
62 *
Liviu Dudaufc2157b2017-07-20 17:07:48 +010063 * If all that is needed is a single modeset lock, then the &struct
64 * drm_modeset_acquire_ctx is not needed and the locking can be simplified
Maarten Lankhorst6f8bcc72017-09-12 15:37:44 +020065 * by passing a NULL instead of ctx in the drm_modeset_lock() call or
66 * calling drm_modeset_lock_single_interruptible(). To unlock afterwards
67 * call drm_modeset_unlock().
Liviu Dudaufc2157b2017-07-20 17:07:48 +010068 *
69 * On top of these per-object locks using &ww_mutex there's also an overall
Daniel Vetterd5745282017-01-25 07:26:45 +010070 * &drm_mode_config.mutex, for protecting everything else. Mostly this means
Daniel Vetterb07c4252016-11-29 10:24:40 +010071 * probe state of connectors, and preventing hotplug add/removal of connectors.
Daniel Vetter0645de52016-05-30 11:10:49 +020072 *
Daniel Vetterb07c4252016-11-29 10:24:40 +010073 * Finally there's a bunch of dedicated locks to protect drm core internal
74 * lists and lookup data structures.
Rob Clark51fd3712013-11-19 12:10:12 -050075 */
76
Rob Clark35cf0352016-11-14 17:40:57 -050077static DEFINE_WW_CLASS(crtc_ww_class);
78
Rob Clark51fd3712013-11-19 12:10:12 -050079/**
Daniel Vetterbf9e37b2015-07-28 13:18:42 +020080 * drm_modeset_lock_all - take all modeset locks
Thierry Reding06eaae42015-12-02 17:50:03 +010081 * @dev: DRM device
Daniel Vettera6a8bb82014-07-25 17:47:18 +020082 *
Daniel Vetterbf9e37b2015-07-28 13:18:42 +020083 * This function takes all modeset locks, suitable where a more fine-grained
Thierry Reding06eaae42015-12-02 17:50:03 +010084 * scheme isn't (yet) implemented. Locks must be dropped by calling the
85 * drm_modeset_unlock_all() function.
86 *
87 * This function is deprecated. It allocates a lock acquisition context and
Daniel Vetterd5745282017-01-25 07:26:45 +010088 * stores it in &drm_device.mode_config. This facilitate conversion of
Thierry Reding06eaae42015-12-02 17:50:03 +010089 * existing code because it removes the need to manually deal with the
90 * acquisition context, but it is also brittle because the context is global
91 * and care must be taken not to nest calls. New code should use the
92 * drm_modeset_lock_all_ctx() function and pass in the context explicitly.
Daniel Vettera6a8bb82014-07-25 17:47:18 +020093 */
Daniel Vetterbf9e37b2015-07-28 13:18:42 +020094void drm_modeset_lock_all(struct drm_device *dev)
Daniel Vettera6a8bb82014-07-25 17:47:18 +020095{
96 struct drm_mode_config *config = &dev->mode_config;
97 struct drm_modeset_acquire_ctx *ctx;
98 int ret;
99
Chris Wilsond18d1a52017-10-31 11:55:35 +0000100 ctx = kzalloc(sizeof(*ctx), GFP_KERNEL | __GFP_NOFAIL);
Daniel Vetterbf9e37b2015-07-28 13:18:42 +0200101 if (WARN_ON(!ctx))
102 return;
Daniel Vettera6a8bb82014-07-25 17:47:18 +0200103
Daniel Vetterbf9e37b2015-07-28 13:18:42 +0200104 mutex_lock(&config->mutex);
Daniel Vettera6a8bb82014-07-25 17:47:18 +0200105
106 drm_modeset_acquire_init(ctx, 0);
107
108retry:
Thierry Reding06eaae42015-12-02 17:50:03 +0100109 ret = drm_modeset_lock_all_ctx(dev, ctx);
110 if (ret < 0) {
111 if (ret == -EDEADLK) {
112 drm_modeset_backoff(ctx);
113 goto retry;
114 }
115
116 drm_modeset_acquire_fini(ctx);
117 kfree(ctx);
118 return;
119 }
Maarten Lankhorst5fcdc9d2018-02-21 16:23:31 +0100120 ww_acquire_done(&ctx->ww_ctx);
Daniel Vettera6a8bb82014-07-25 17:47:18 +0200121
122 WARN_ON(config->acquire_ctx);
123
Thierry Reding06eaae42015-12-02 17:50:03 +0100124 /*
125 * We hold the locks now, so it is safe to stash the acquisition
126 * context for drm_modeset_unlock_all().
Daniel Vettera6a8bb82014-07-25 17:47:18 +0200127 */
128 config->acquire_ctx = ctx;
129
130 drm_warn_on_modeset_not_all_locked(dev);
Daniel Vettera6a8bb82014-07-25 17:47:18 +0200131}
132EXPORT_SYMBOL(drm_modeset_lock_all);
133
134/**
135 * drm_modeset_unlock_all - drop all modeset locks
Thierry Reding06eaae42015-12-02 17:50:03 +0100136 * @dev: DRM device
Daniel Vettera6a8bb82014-07-25 17:47:18 +0200137 *
Thierry Reding06eaae42015-12-02 17:50:03 +0100138 * This function drops all modeset locks taken by a previous call to the
139 * drm_modeset_lock_all() function.
140 *
141 * This function is deprecated. It uses the lock acquisition context stored
Daniel Vetterd5745282017-01-25 07:26:45 +0100142 * in &drm_device.mode_config. This facilitates conversion of existing
Thierry Reding06eaae42015-12-02 17:50:03 +0100143 * code because it removes the need to manually deal with the acquisition
144 * context, but it is also brittle because the context is global and care must
145 * be taken not to nest calls. New code should pass the acquisition context
146 * directly to the drm_modeset_drop_locks() function.
Daniel Vettera6a8bb82014-07-25 17:47:18 +0200147 */
148void drm_modeset_unlock_all(struct drm_device *dev)
149{
150 struct drm_mode_config *config = &dev->mode_config;
151 struct drm_modeset_acquire_ctx *ctx = config->acquire_ctx;
152
153 if (WARN_ON(!ctx))
154 return;
155
156 config->acquire_ctx = NULL;
157 drm_modeset_drop_locks(ctx);
158 drm_modeset_acquire_fini(ctx);
159
160 kfree(ctx);
161
162 mutex_unlock(&dev->mode_config.mutex);
163}
164EXPORT_SYMBOL(drm_modeset_unlock_all);
165
Daniel Vetterd059f652014-07-25 18:07:40 +0200166/**
Daniel Vettera6a8bb82014-07-25 17:47:18 +0200167 * drm_warn_on_modeset_not_all_locked - check that all modeset locks are locked
168 * @dev: device
169 *
170 * Useful as a debug assert.
171 */
172void drm_warn_on_modeset_not_all_locked(struct drm_device *dev)
173{
174 struct drm_crtc *crtc;
175
176 /* Locking is currently fubar in the panic handler. */
177 if (oops_in_progress)
178 return;
179
Daniel Vettere4f62542015-07-09 23:44:35 +0200180 drm_for_each_crtc(crtc, dev)
Daniel Vettera6a8bb82014-07-25 17:47:18 +0200181 WARN_ON(!drm_modeset_is_locked(&crtc->mutex));
182
183 WARN_ON(!drm_modeset_is_locked(&dev->mode_config.connection_mutex));
184 WARN_ON(!mutex_is_locked(&dev->mode_config.mutex));
185}
186EXPORT_SYMBOL(drm_warn_on_modeset_not_all_locked);
187
188/**
Rob Clark51fd3712013-11-19 12:10:12 -0500189 * drm_modeset_acquire_init - initialize acquire context
190 * @ctx: the acquire context
Maarten Lankhorst6f8bcc72017-09-12 15:37:44 +0200191 * @flags: 0 or %DRM_MODESET_ACQUIRE_INTERRUPTIBLE
192 *
193 * When passing %DRM_MODESET_ACQUIRE_INTERRUPTIBLE to @flags,
194 * all calls to drm_modeset_lock() will perform an interruptible
195 * wait.
Rob Clark51fd3712013-11-19 12:10:12 -0500196 */
197void drm_modeset_acquire_init(struct drm_modeset_acquire_ctx *ctx,
198 uint32_t flags)
199{
Rob Clarkfb549182014-06-07 10:55:39 -0400200 memset(ctx, 0, sizeof(*ctx));
Rob Clark51fd3712013-11-19 12:10:12 -0500201 ww_acquire_init(&ctx->ww_ctx, &crtc_ww_class);
202 INIT_LIST_HEAD(&ctx->locked);
Maarten Lankhorst6f8bcc72017-09-12 15:37:44 +0200203
204 if (flags & DRM_MODESET_ACQUIRE_INTERRUPTIBLE)
205 ctx->interruptible = true;
Rob Clark51fd3712013-11-19 12:10:12 -0500206}
207EXPORT_SYMBOL(drm_modeset_acquire_init);
208
209/**
210 * drm_modeset_acquire_fini - cleanup acquire context
211 * @ctx: the acquire context
212 */
213void drm_modeset_acquire_fini(struct drm_modeset_acquire_ctx *ctx)
214{
215 ww_acquire_fini(&ctx->ww_ctx);
216}
217EXPORT_SYMBOL(drm_modeset_acquire_fini);
218
219/**
220 * drm_modeset_drop_locks - drop all locks
221 * @ctx: the acquire context
222 *
223 * Drop all locks currently held against this acquire context.
224 */
225void drm_modeset_drop_locks(struct drm_modeset_acquire_ctx *ctx)
226{
227 WARN_ON(ctx->contended);
228 while (!list_empty(&ctx->locked)) {
229 struct drm_modeset_lock *lock;
230
231 lock = list_first_entry(&ctx->locked,
232 struct drm_modeset_lock, head);
233
234 drm_modeset_unlock(lock);
235 }
236}
237EXPORT_SYMBOL(drm_modeset_drop_locks);
238
239static inline int modeset_lock(struct drm_modeset_lock *lock,
240 struct drm_modeset_acquire_ctx *ctx,
241 bool interruptible, bool slow)
242{
243 int ret;
244
245 WARN_ON(ctx->contended);
246
Daniel Vettercb597bb2014-07-27 19:09:33 +0200247 if (ctx->trylock_only) {
Maarten Lankhorst825926d2015-08-27 13:58:09 +0200248 lockdep_assert_held(&ctx->ww_ctx);
249
Daniel Vettercb597bb2014-07-27 19:09:33 +0200250 if (!ww_mutex_trylock(&lock->mutex))
251 return -EBUSY;
252 else
253 return 0;
254 } else if (interruptible && slow) {
Rob Clark51fd3712013-11-19 12:10:12 -0500255 ret = ww_mutex_lock_slow_interruptible(&lock->mutex, &ctx->ww_ctx);
256 } else if (interruptible) {
257 ret = ww_mutex_lock_interruptible(&lock->mutex, &ctx->ww_ctx);
258 } else if (slow) {
259 ww_mutex_lock_slow(&lock->mutex, &ctx->ww_ctx);
260 ret = 0;
261 } else {
262 ret = ww_mutex_lock(&lock->mutex, &ctx->ww_ctx);
263 }
264 if (!ret) {
265 WARN_ON(!list_empty(&lock->head));
266 list_add(&lock->head, &ctx->locked);
267 } else if (ret == -EALREADY) {
268 /* we already hold the lock.. this is fine. For atomic
269 * we will need to be able to drm_modeset_lock() things
270 * without having to keep track of what is already locked
271 * or not.
272 */
273 ret = 0;
274 } else if (ret == -EDEADLK) {
275 ctx->contended = lock;
276 }
277
278 return ret;
279}
280
Maarten Lankhorst6f8bcc72017-09-12 15:37:44 +0200281/**
282 * drm_modeset_backoff - deadlock avoidance backoff
283 * @ctx: the acquire context
284 *
285 * If deadlock is detected (ie. drm_modeset_lock() returns -EDEADLK),
286 * you must call this function to drop all currently held locks and
287 * block until the contended lock becomes available.
288 *
289 * This function returns 0 on success, or -ERESTARTSYS if this context
290 * is initialized with %DRM_MODESET_ACQUIRE_INTERRUPTIBLE and the
291 * wait has been interrupted.
292 */
293int drm_modeset_backoff(struct drm_modeset_acquire_ctx *ctx)
Rob Clark51fd3712013-11-19 12:10:12 -0500294{
295 struct drm_modeset_lock *contended = ctx->contended;
296
297 ctx->contended = NULL;
298
299 if (WARN_ON(!contended))
300 return 0;
301
302 drm_modeset_drop_locks(ctx);
303
Maarten Lankhorst6f8bcc72017-09-12 15:37:44 +0200304 return modeset_lock(contended, ctx, ctx->interruptible, true);
Rob Clark51fd3712013-11-19 12:10:12 -0500305}
306EXPORT_SYMBOL(drm_modeset_backoff);
307
308/**
Rob Clark35cf0352016-11-14 17:40:57 -0500309 * drm_modeset_lock_init - initialize lock
310 * @lock: lock to init
311 */
312void drm_modeset_lock_init(struct drm_modeset_lock *lock)
313{
314 ww_mutex_init(&lock->mutex, &crtc_ww_class);
315 INIT_LIST_HEAD(&lock->head);
316}
317EXPORT_SYMBOL(drm_modeset_lock_init);
318
319/**
Rob Clark51fd3712013-11-19 12:10:12 -0500320 * drm_modeset_lock - take modeset lock
321 * @lock: lock to take
322 * @ctx: acquire ctx
323 *
Liviu Dudaufc2157b2017-07-20 17:07:48 +0100324 * If @ctx is not NULL, then its ww acquire context is used and the
Rob Clark51fd3712013-11-19 12:10:12 -0500325 * lock will be tracked by the context and can be released by calling
326 * drm_modeset_drop_locks(). If -EDEADLK is returned, this means a
327 * deadlock scenario has been detected and it is an error to attempt
328 * to take any more locks without first calling drm_modeset_backoff().
Liviu Dudaufc2157b2017-07-20 17:07:48 +0100329 *
Maarten Lankhorst6f8bcc72017-09-12 15:37:44 +0200330 * If the @ctx is not NULL and initialized with
331 * %DRM_MODESET_ACQUIRE_INTERRUPTIBLE, this function will fail with
332 * -ERESTARTSYS when interrupted.
333 *
Liviu Dudaufc2157b2017-07-20 17:07:48 +0100334 * If @ctx is NULL then the function call behaves like a normal,
Maarten Lankhorst6f8bcc72017-09-12 15:37:44 +0200335 * uninterruptible non-nesting mutex_lock() call.
Rob Clark51fd3712013-11-19 12:10:12 -0500336 */
337int drm_modeset_lock(struct drm_modeset_lock *lock,
338 struct drm_modeset_acquire_ctx *ctx)
339{
340 if (ctx)
Maarten Lankhorst6f8bcc72017-09-12 15:37:44 +0200341 return modeset_lock(lock, ctx, ctx->interruptible, false);
Rob Clark51fd3712013-11-19 12:10:12 -0500342
343 ww_mutex_lock(&lock->mutex, NULL);
344 return 0;
345}
346EXPORT_SYMBOL(drm_modeset_lock);
347
348/**
Maarten Lankhorst6f8bcc72017-09-12 15:37:44 +0200349 * drm_modeset_lock_single_interruptible - take a single modeset lock
Rob Clark51fd3712013-11-19 12:10:12 -0500350 * @lock: lock to take
Rob Clark51fd3712013-11-19 12:10:12 -0500351 *
Maarten Lankhorst6f8bcc72017-09-12 15:37:44 +0200352 * This function behaves as drm_modeset_lock() with a NULL context,
353 * but performs interruptible waits.
354 *
355 * This function returns 0 on success, or -ERESTARTSYS when interrupted.
Rob Clark51fd3712013-11-19 12:10:12 -0500356 */
Maarten Lankhorst6f8bcc72017-09-12 15:37:44 +0200357int drm_modeset_lock_single_interruptible(struct drm_modeset_lock *lock)
Rob Clark51fd3712013-11-19 12:10:12 -0500358{
Rob Clark51fd3712013-11-19 12:10:12 -0500359 return ww_mutex_lock_interruptible(&lock->mutex, NULL);
360}
Maarten Lankhorst6f8bcc72017-09-12 15:37:44 +0200361EXPORT_SYMBOL(drm_modeset_lock_single_interruptible);
Rob Clark51fd3712013-11-19 12:10:12 -0500362
363/**
364 * drm_modeset_unlock - drop modeset lock
365 * @lock: lock to release
366 */
367void drm_modeset_unlock(struct drm_modeset_lock *lock)
368{
369 list_del_init(&lock->head);
370 ww_mutex_unlock(&lock->mutex);
371}
372EXPORT_SYMBOL(drm_modeset_unlock);
373
Thierry Reding06eaae42015-12-02 17:50:03 +0100374/**
375 * drm_modeset_lock_all_ctx - take all modeset locks
376 * @dev: DRM device
377 * @ctx: lock acquisition context
378 *
379 * This function takes all modeset locks, suitable where a more fine-grained
380 * scheme isn't (yet) implemented.
381 *
Daniel Vetterd5745282017-01-25 07:26:45 +0100382 * Unlike drm_modeset_lock_all(), it doesn't take the &drm_mode_config.mutex
Thierry Reding06eaae42015-12-02 17:50:03 +0100383 * since that lock isn't required for modeset state changes. Callers which
384 * need to grab that lock too need to do so outside of the acquire context
385 * @ctx.
386 *
387 * Locks acquired with this function should be released by calling the
388 * drm_modeset_drop_locks() function on @ctx.
389 *
Sean Paulb7ea04d2018-11-29 10:04:17 -0500390 * See also: DRM_MODESET_LOCK_ALL_BEGIN() and DRM_MODESET_LOCK_ALL_END()
391 *
Thierry Reding06eaae42015-12-02 17:50:03 +0100392 * Returns: 0 on success or a negative error-code on failure.
393 */
394int drm_modeset_lock_all_ctx(struct drm_device *dev,
395 struct drm_modeset_acquire_ctx *ctx)
Rob Clark51fd3712013-11-19 12:10:12 -0500396{
Rob Clark51fd3712013-11-19 12:10:12 -0500397 struct drm_crtc *crtc;
Daniel Vetter4d02e2d2014-11-11 10:12:00 +0100398 struct drm_plane *plane;
Thierry Reding06eaae42015-12-02 17:50:03 +0100399 int ret;
400
401 ret = drm_modeset_lock(&dev->mode_config.connection_mutex, ctx);
402 if (ret)
403 return ret;
Rob Clark51fd3712013-11-19 12:10:12 -0500404
Daniel Vettere4f62542015-07-09 23:44:35 +0200405 drm_for_each_crtc(crtc, dev) {
Rob Clark51fd3712013-11-19 12:10:12 -0500406 ret = drm_modeset_lock(&crtc->mutex, ctx);
407 if (ret)
408 return ret;
409 }
410
Daniel Vettere4f62542015-07-09 23:44:35 +0200411 drm_for_each_plane(plane, dev) {
Daniel Vetter4d02e2d2014-11-11 10:12:00 +0100412 ret = drm_modeset_lock(&plane->mutex, ctx);
413 if (ret)
414 return ret;
415 }
416
Rob Clark51fd3712013-11-19 12:10:12 -0500417 return 0;
418}
Thierry Reding06eaae42015-12-02 17:50:03 +0100419EXPORT_SYMBOL(drm_modeset_lock_all_ctx);