xref: /openbmc/linux/drivers/gpu/drm/drm_lock.c (revision 948de842)
145b2fda3SQian Cai /*
2c0e09200SDave Airlie  * \file drm_lock.c
3c0e09200SDave Airlie  * IOCTLs for locking
4c0e09200SDave Airlie  *
5c0e09200SDave Airlie  * \author Rickard E. (Rik) Faith <faith@valinux.com>
6c0e09200SDave Airlie  * \author Gareth Hughes <gareth@valinux.com>
7c0e09200SDave Airlie  */
8c0e09200SDave Airlie 
9c0e09200SDave Airlie /*
10c0e09200SDave Airlie  * Created: Tue Feb  2 08:37:54 1999 by faith@valinux.com
11c0e09200SDave Airlie  *
12c0e09200SDave Airlie  * Copyright 1999 Precision Insight, Inc., Cedar Park, Texas.
13c0e09200SDave Airlie  * Copyright 2000 VA Linux Systems, Inc., Sunnyvale, California.
14c0e09200SDave Airlie  * All Rights Reserved.
15c0e09200SDave Airlie  *
16c0e09200SDave Airlie  * Permission is hereby granted, free of charge, to any person obtaining a
17c0e09200SDave Airlie  * copy of this software and associated documentation files (the "Software"),
18c0e09200SDave Airlie  * to deal in the Software without restriction, including without limitation
19c0e09200SDave Airlie  * the rights to use, copy, modify, merge, publish, distribute, sublicense,
20c0e09200SDave Airlie  * and/or sell copies of the Software, and to permit persons to whom the
21c0e09200SDave Airlie  * Software is furnished to do so, subject to the following conditions:
22c0e09200SDave Airlie  *
23c0e09200SDave Airlie  * The above copyright notice and this permission notice (including the next
24c0e09200SDave Airlie  * paragraph) shall be included in all copies or substantial portions of the
25c0e09200SDave Airlie  * Software.
26c0e09200SDave Airlie  *
27c0e09200SDave Airlie  * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
28c0e09200SDave Airlie  * IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
29c0e09200SDave Airlie  * FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT.  IN NO EVENT SHALL
30c0e09200SDave Airlie  * VA LINUX SYSTEMS AND/OR ITS SUPPLIERS BE LIABLE FOR ANY CLAIM, DAMAGES OR
31c0e09200SDave Airlie  * OTHER LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE,
32c0e09200SDave Airlie  * ARISING FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR
33c0e09200SDave Airlie  * OTHER DEALINGS IN THE SOFTWARE.
34c0e09200SDave Airlie  */
35c0e09200SDave Airlie 
365c2a5ce6SDave Airlie #include <linux/export.h>
373f07c014SIngo Molnar #include <linux/sched/signal.h>
383f07c014SIngo Molnar 
390500c04eSSam Ravnborg #include <drm/drm.h>
400500c04eSSam Ravnborg #include <drm/drm_drv.h>
410500c04eSSam Ravnborg #include <drm/drm_file.h>
420500c04eSSam Ravnborg #include <drm/drm_print.h>
430500c04eSSam Ravnborg 
4444af3f5cSDaniel Vetter #include "drm_internal.h"
450500c04eSSam Ravnborg #include "drm_legacy.h"
46c0e09200SDave Airlie 
474ac5ec40SDaniel Vetter static int drm_lock_take(struct drm_lock_data *lock_data, unsigned int context);
484ac5ec40SDaniel Vetter 
49bd50d4a2SBenjamin Gaignard /*
501a75a222SDaniel Vetter  * Take the heavyweight lock.
511a75a222SDaniel Vetter  *
521a75a222SDaniel Vetter  * \param lock lock pointer.
531a75a222SDaniel Vetter  * \param context locking context.
541a75a222SDaniel Vetter  * \return one if the lock is held, or zero otherwise.
551a75a222SDaniel Vetter  *
561a75a222SDaniel Vetter  * Attempt to mark the lock as held by the given context, via the \p cmpxchg instruction.
571a75a222SDaniel Vetter  */
581a75a222SDaniel Vetter static
drm_lock_take(struct drm_lock_data * lock_data,unsigned int context)591a75a222SDaniel Vetter int drm_lock_take(struct drm_lock_data *lock_data,
601a75a222SDaniel Vetter 		  unsigned int context)
611a75a222SDaniel Vetter {
621a75a222SDaniel Vetter 	unsigned int old, new, prev;
631a75a222SDaniel Vetter 	volatile unsigned int *lock = &lock_data->hw_lock->lock;
641a75a222SDaniel Vetter 
651a75a222SDaniel Vetter 	spin_lock_bh(&lock_data->spinlock);
661a75a222SDaniel Vetter 	do {
671a75a222SDaniel Vetter 		old = *lock;
681a75a222SDaniel Vetter 		if (old & _DRM_LOCK_HELD)
691a75a222SDaniel Vetter 			new = old | _DRM_LOCK_CONT;
701a75a222SDaniel Vetter 		else {
711a75a222SDaniel Vetter 			new = context | _DRM_LOCK_HELD |
721a75a222SDaniel Vetter 				((lock_data->user_waiters + lock_data->kernel_waiters > 1) ?
731a75a222SDaniel Vetter 				 _DRM_LOCK_CONT : 0);
741a75a222SDaniel Vetter 		}
751a75a222SDaniel Vetter 		prev = cmpxchg(lock, old, new);
761a75a222SDaniel Vetter 	} while (prev != old);
771a75a222SDaniel Vetter 	spin_unlock_bh(&lock_data->spinlock);
781a75a222SDaniel Vetter 
791a75a222SDaniel Vetter 	if (_DRM_LOCKING_CONTEXT(old) == context) {
801a75a222SDaniel Vetter 		if (old & _DRM_LOCK_HELD) {
811a75a222SDaniel Vetter 			if (context != DRM_KERNEL_CONTEXT) {
821a75a222SDaniel Vetter 				DRM_ERROR("%d holds heavyweight lock\n",
831a75a222SDaniel Vetter 					  context);
841a75a222SDaniel Vetter 			}
851a75a222SDaniel Vetter 			return 0;
861a75a222SDaniel Vetter 		}
871a75a222SDaniel Vetter 	}
881a75a222SDaniel Vetter 
891a75a222SDaniel Vetter 	if ((_DRM_LOCKING_CONTEXT(new)) == context && (new & _DRM_LOCK_HELD)) {
901a75a222SDaniel Vetter 		/* Have lock */
911a75a222SDaniel Vetter 		return 1;
921a75a222SDaniel Vetter 	}
931a75a222SDaniel Vetter 	return 0;
941a75a222SDaniel Vetter }
951a75a222SDaniel Vetter 
96bd50d4a2SBenjamin Gaignard /*
971a75a222SDaniel Vetter  * This takes a lock forcibly and hands it to context.	Should ONLY be used
981a75a222SDaniel Vetter  * inside *_unlock to give lock to kernel before calling *_dma_schedule.
991a75a222SDaniel Vetter  *
1001a75a222SDaniel Vetter  * \param dev DRM device.
1011a75a222SDaniel Vetter  * \param lock lock pointer.
1021a75a222SDaniel Vetter  * \param context locking context.
1031a75a222SDaniel Vetter  * \return always one.
1041a75a222SDaniel Vetter  *
1051a75a222SDaniel Vetter  * Resets the lock file pointer.
1061a75a222SDaniel Vetter  * Marks the lock as held by the given context, via the \p cmpxchg instruction.
1071a75a222SDaniel Vetter  */
drm_lock_transfer(struct drm_lock_data * lock_data,unsigned int context)1081a75a222SDaniel Vetter static int drm_lock_transfer(struct drm_lock_data *lock_data,
1091a75a222SDaniel Vetter 			     unsigned int context)
1101a75a222SDaniel Vetter {
1111a75a222SDaniel Vetter 	unsigned int old, new, prev;
1121a75a222SDaniel Vetter 	volatile unsigned int *lock = &lock_data->hw_lock->lock;
1131a75a222SDaniel Vetter 
1141a75a222SDaniel Vetter 	lock_data->file_priv = NULL;
1151a75a222SDaniel Vetter 	do {
1161a75a222SDaniel Vetter 		old = *lock;
1171a75a222SDaniel Vetter 		new = context | _DRM_LOCK_HELD;
1181a75a222SDaniel Vetter 		prev = cmpxchg(lock, old, new);
1191a75a222SDaniel Vetter 	} while (prev != old);
1201a75a222SDaniel Vetter 	return 1;
1211a75a222SDaniel Vetter }
1221a75a222SDaniel Vetter 
drm_legacy_lock_free(struct drm_lock_data * lock_data,unsigned int context)1231a75a222SDaniel Vetter static int drm_legacy_lock_free(struct drm_lock_data *lock_data,
1241a75a222SDaniel Vetter 				unsigned int context)
1251a75a222SDaniel Vetter {
1261a75a222SDaniel Vetter 	unsigned int old, new, prev;
1271a75a222SDaniel Vetter 	volatile unsigned int *lock = &lock_data->hw_lock->lock;
1281a75a222SDaniel Vetter 
1291a75a222SDaniel Vetter 	spin_lock_bh(&lock_data->spinlock);
1301a75a222SDaniel Vetter 	if (lock_data->kernel_waiters != 0) {
1311a75a222SDaniel Vetter 		drm_lock_transfer(lock_data, 0);
1321a75a222SDaniel Vetter 		lock_data->idle_has_lock = 1;
1331a75a222SDaniel Vetter 		spin_unlock_bh(&lock_data->spinlock);
1341a75a222SDaniel Vetter 		return 1;
1351a75a222SDaniel Vetter 	}
1361a75a222SDaniel Vetter 	spin_unlock_bh(&lock_data->spinlock);
1371a75a222SDaniel Vetter 
1381a75a222SDaniel Vetter 	do {
1391a75a222SDaniel Vetter 		old = *lock;
1401a75a222SDaniel Vetter 		new = _DRM_LOCKING_CONTEXT(old);
1411a75a222SDaniel Vetter 		prev = cmpxchg(lock, old, new);
1421a75a222SDaniel Vetter 	} while (prev != old);
1431a75a222SDaniel Vetter 
1441a75a222SDaniel Vetter 	if (_DRM_LOCK_IS_HELD(old) && _DRM_LOCKING_CONTEXT(old) != context) {
1451a75a222SDaniel Vetter 		DRM_ERROR("%d freed heavyweight lock held by %d\n",
1461a75a222SDaniel Vetter 			  context, _DRM_LOCKING_CONTEXT(old));
1471a75a222SDaniel Vetter 		return 1;
1481a75a222SDaniel Vetter 	}
1491a75a222SDaniel Vetter 	wake_up_interruptible(&lock_data->lock_queue);
1501a75a222SDaniel Vetter 	return 0;
1511a75a222SDaniel Vetter }
1521a75a222SDaniel Vetter 
153bd50d4a2SBenjamin Gaignard /*
154c0e09200SDave Airlie  * Lock ioctl.
155c0e09200SDave Airlie  *
156c0e09200SDave Airlie  * \param inode device inode.
157c0e09200SDave Airlie  * \param file_priv DRM file private.
158c0e09200SDave Airlie  * \param cmd command.
159c0e09200SDave Airlie  * \param arg user argument, pointing to a drm_lock structure.
160c0e09200SDave Airlie  * \return zero on success or negative number on failure.
161c0e09200SDave Airlie  *
162c0e09200SDave Airlie  * Add the current task to the lock wait queue, and attempt to take to lock.
163c0e09200SDave Airlie  */
drm_legacy_lock(struct drm_device * dev,void * data,struct drm_file * file_priv)164bb6d822eSDavid Herrmann int drm_legacy_lock(struct drm_device *dev, void *data,
165bb6d822eSDavid Herrmann 		    struct drm_file *file_priv)
166c0e09200SDave Airlie {
167c0e09200SDave Airlie 	DECLARE_WAITQUEUE(entry, current);
168c0e09200SDave Airlie 	struct drm_lock *lock = data;
1697c1c2871SDave Airlie 	struct drm_master *master = file_priv->master;
170c0e09200SDave Airlie 	int ret = 0;
171c0e09200SDave Airlie 
172fa538645SDaniel Vetter 	if (!drm_core_check_feature(dev, DRIVER_LEGACY))
17369fdf420SChris Wilson 		return -EOPNOTSUPP;
174da168d81SDaniel Vetter 
175c0e09200SDave Airlie 	++file_priv->lock_count;
176c0e09200SDave Airlie 
177c0e09200SDave Airlie 	if (lock->context == DRM_KERNEL_CONTEXT) {
178c0e09200SDave Airlie 		DRM_ERROR("Process %d using kernel context %d\n",
179c0e09200SDave Airlie 			  task_pid_nr(current), lock->context);
180c0e09200SDave Airlie 		return -EINVAL;
181c0e09200SDave Airlie 	}
182c0e09200SDave Airlie 
183c0e09200SDave Airlie 	DRM_DEBUG("%d (pid %d) requests lock (0x%08x), flags = 0x%08x\n",
184c0e09200SDave Airlie 		  lock->context, task_pid_nr(current),
185a2ec1c13SChris Wilson 		  master->lock.hw_lock ? master->lock.hw_lock->lock : -1,
186a2ec1c13SChris Wilson 		  lock->flags);
187c0e09200SDave Airlie 
1887c1c2871SDave Airlie 	add_wait_queue(&master->lock.lock_queue, &entry);
1897c1c2871SDave Airlie 	spin_lock_bh(&master->lock.spinlock);
1907c1c2871SDave Airlie 	master->lock.user_waiters++;
1917c1c2871SDave Airlie 	spin_unlock_bh(&master->lock.spinlock);
1927c1c2871SDave Airlie 
193c0e09200SDave Airlie 	for (;;) {
194c0e09200SDave Airlie 		__set_current_state(TASK_INTERRUPTIBLE);
1957c1c2871SDave Airlie 		if (!master->lock.hw_lock) {
196c0e09200SDave Airlie 			/* Device has been unregistered */
197fda714c2SThomas Hellstrom 			send_sig(SIGTERM, current, 0);
198c0e09200SDave Airlie 			ret = -EINTR;
199c0e09200SDave Airlie 			break;
200c0e09200SDave Airlie 		}
2017c1c2871SDave Airlie 		if (drm_lock_take(&master->lock, lock->context)) {
2027c1c2871SDave Airlie 			master->lock.file_priv = file_priv;
2037c1c2871SDave Airlie 			master->lock.lock_time = jiffies;
204c0e09200SDave Airlie 			break;	/* Got lock */
205c0e09200SDave Airlie 		}
206c0e09200SDave Airlie 
207c0e09200SDave Airlie 		/* Contention */
20808f2e669SArnd Bergmann 		mutex_unlock(&drm_global_mutex);
209c0e09200SDave Airlie 		schedule();
21008f2e669SArnd Bergmann 		mutex_lock(&drm_global_mutex);
211c0e09200SDave Airlie 		if (signal_pending(current)) {
2124d77c88eSThomas Hellstrom 			ret = -EINTR;
213c0e09200SDave Airlie 			break;
214c0e09200SDave Airlie 		}
215c0e09200SDave Airlie 	}
2167c1c2871SDave Airlie 	spin_lock_bh(&master->lock.spinlock);
2177c1c2871SDave Airlie 	master->lock.user_waiters--;
2187c1c2871SDave Airlie 	spin_unlock_bh(&master->lock.spinlock);
219c0e09200SDave Airlie 	__set_current_state(TASK_RUNNING);
2207c1c2871SDave Airlie 	remove_wait_queue(&master->lock.lock_queue, &entry);
221c0e09200SDave Airlie 
222c0e09200SDave Airlie 	DRM_DEBUG("%d %s\n", lock->context,
223c0e09200SDave Airlie 		  ret ? "interrupted" : "has lock");
224c0e09200SDave Airlie 	if (ret) return ret;
225c0e09200SDave Airlie 
2263e5fc80aSDave Airlie 	/* don't set the block all signals on the master process for now
2273e5fc80aSDave Airlie 	 * really probably not the correct answer but lets us debug xkb
2283e5fc80aSDave Airlie  	 * xserver for now */
229b3ac9f25SDaniel Vetter 	if (!drm_is_current_master(file_priv)) {
230c0e09200SDave Airlie 		dev->sigdata.context = lock->context;
2317c1c2871SDave Airlie 		dev->sigdata.lock = master->lock.hw_lock;
2323e5fc80aSDave Airlie 	}
233c0e09200SDave Airlie 
234c0e09200SDave Airlie 	if (dev->driver->dma_quiescent && (lock->flags & _DRM_LOCK_QUIESCENT))
235c0e09200SDave Airlie 	{
236c0e09200SDave Airlie 		if (dev->driver->dma_quiescent(dev)) {
237c0e09200SDave Airlie 			DRM_DEBUG("%d waiting for DMA quiescent\n",
238c0e09200SDave Airlie 				  lock->context);
239c0e09200SDave Airlie 			return -EBUSY;
240c0e09200SDave Airlie 		}
241c0e09200SDave Airlie 	}
242c0e09200SDave Airlie 
243c0e09200SDave Airlie 	return 0;
244c0e09200SDave Airlie }
245c0e09200SDave Airlie 
246bd50d4a2SBenjamin Gaignard /*
247c0e09200SDave Airlie  * Unlock ioctl.
248c0e09200SDave Airlie  *
249c0e09200SDave Airlie  * \param inode device inode.
250c0e09200SDave Airlie  * \param file_priv DRM file private.
251c0e09200SDave Airlie  * \param cmd command.
252c0e09200SDave Airlie  * \param arg user argument, pointing to a drm_lock structure.
253c0e09200SDave Airlie  * \return zero on success or negative number on failure.
254c0e09200SDave Airlie  *
255c0e09200SDave Airlie  * Transfer and free the lock.
256c0e09200SDave Airlie  */
drm_legacy_unlock(struct drm_device * dev,void * data,struct drm_file * file_priv)257bb6d822eSDavid Herrmann int drm_legacy_unlock(struct drm_device *dev, void *data, struct drm_file *file_priv)
258c0e09200SDave Airlie {
259c0e09200SDave Airlie 	struct drm_lock *lock = data;
26045ff46c5SDaniel Vetter 	struct drm_master *master = file_priv->master;
261c0e09200SDave Airlie 
262fa538645SDaniel Vetter 	if (!drm_core_check_feature(dev, DRIVER_LEGACY))
26369fdf420SChris Wilson 		return -EOPNOTSUPP;
264da168d81SDaniel Vetter 
265c0e09200SDave Airlie 	if (lock->context == DRM_KERNEL_CONTEXT) {
266c0e09200SDave Airlie 		DRM_ERROR("Process %d using kernel context %d\n",
267c0e09200SDave Airlie 			  task_pid_nr(current), lock->context);
268c0e09200SDave Airlie 		return -EINVAL;
269c0e09200SDave Airlie 	}
270c0e09200SDave Airlie 
271bb6d822eSDavid Herrmann 	if (drm_legacy_lock_free(&master->lock, lock->context)) {
27245ff46c5SDaniel Vetter 		/* FIXME: Should really bail out here. */
27345ff46c5SDaniel Vetter 	}
27445ff46c5SDaniel Vetter 
275c0e09200SDave Airlie 	return 0;
276c0e09200SDave Airlie }
277c0e09200SDave Airlie 
278bd50d4a2SBenjamin Gaignard /*
279c0e09200SDave Airlie  * This function returns immediately and takes the hw lock
280c0e09200SDave Airlie  * with the kernel context if it is free, otherwise it gets the highest priority when and if
281c0e09200SDave Airlie  * it is eventually released.
282c0e09200SDave Airlie  *
283c0e09200SDave Airlie  * This guarantees that the kernel will _eventually_ have the lock _unless_ it is held
284c0e09200SDave Airlie  * by a blocked process. (In the latter case an explicit wait for the hardware lock would cause
285c0e09200SDave Airlie  * a deadlock, which is why the "idlelock" was invented).
286c0e09200SDave Airlie  *
287c0e09200SDave Airlie  * This should be sufficient to wait for GPU idle without
288c0e09200SDave Airlie  * having to worry about starvation.
289c0e09200SDave Airlie  */
drm_legacy_idlelock_take(struct drm_lock_data * lock_data)290bb6d822eSDavid Herrmann void drm_legacy_idlelock_take(struct drm_lock_data *lock_data)
291c0e09200SDave Airlie {
2924a1b0714SLaurent Pinchart 	int ret;
293c0e09200SDave Airlie 
294c0e09200SDave Airlie 	spin_lock_bh(&lock_data->spinlock);
295c0e09200SDave Airlie 	lock_data->kernel_waiters++;
296c0e09200SDave Airlie 	if (!lock_data->idle_has_lock) {
297c0e09200SDave Airlie 
298c0e09200SDave Airlie 		spin_unlock_bh(&lock_data->spinlock);
299c0e09200SDave Airlie 		ret = drm_lock_take(lock_data, DRM_KERNEL_CONTEXT);
300c0e09200SDave Airlie 		spin_lock_bh(&lock_data->spinlock);
301c0e09200SDave Airlie 
302c0e09200SDave Airlie 		if (ret == 1)
303c0e09200SDave Airlie 			lock_data->idle_has_lock = 1;
304c0e09200SDave Airlie 	}
305c0e09200SDave Airlie 	spin_unlock_bh(&lock_data->spinlock);
306c0e09200SDave Airlie }
307bb6d822eSDavid Herrmann EXPORT_SYMBOL(drm_legacy_idlelock_take);
308c0e09200SDave Airlie 
drm_legacy_idlelock_release(struct drm_lock_data * lock_data)309bb6d822eSDavid Herrmann void drm_legacy_idlelock_release(struct drm_lock_data *lock_data)
310c0e09200SDave Airlie {
311c0e09200SDave Airlie 	unsigned int old, prev;
312c0e09200SDave Airlie 	volatile unsigned int *lock = &lock_data->hw_lock->lock;
313c0e09200SDave Airlie 
314c0e09200SDave Airlie 	spin_lock_bh(&lock_data->spinlock);
315c0e09200SDave Airlie 	if (--lock_data->kernel_waiters == 0) {
316c0e09200SDave Airlie 		if (lock_data->idle_has_lock) {
317c0e09200SDave Airlie 			do {
318c0e09200SDave Airlie 				old = *lock;
319c0e09200SDave Airlie 				prev = cmpxchg(lock, old, DRM_KERNEL_CONTEXT);
320c0e09200SDave Airlie 			} while (prev != old);
321c0e09200SDave Airlie 			wake_up_interruptible(&lock_data->lock_queue);
322c0e09200SDave Airlie 			lock_data->idle_has_lock = 0;
323c0e09200SDave Airlie 		}
324c0e09200SDave Airlie 	}
325c0e09200SDave Airlie 	spin_unlock_bh(&lock_data->spinlock);
326c0e09200SDave Airlie }
327bb6d822eSDavid Herrmann EXPORT_SYMBOL(drm_legacy_idlelock_release);
328c0e09200SDave Airlie 
drm_legacy_i_have_hw_lock(struct drm_device * dev,struct drm_file * file_priv)3291a75a222SDaniel Vetter static int drm_legacy_i_have_hw_lock(struct drm_device *dev,
330bb6d822eSDavid Herrmann 				     struct drm_file *file_priv)
331c0e09200SDave Airlie {
3327c1c2871SDave Airlie 	struct drm_master *master = file_priv->master;
333948de842SSuraj Upadhyay 
3347c1c2871SDave Airlie 	return (file_priv->lock_count && master->lock.hw_lock &&
3357c1c2871SDave Airlie 		_DRM_LOCK_IS_HELD(master->lock.hw_lock->lock) &&
3367c1c2871SDave Airlie 		master->lock.file_priv == file_priv);
337c0e09200SDave Airlie }
3381a75a222SDaniel Vetter 
drm_legacy_lock_release(struct drm_device * dev,struct file * filp)3391a75a222SDaniel Vetter void drm_legacy_lock_release(struct drm_device *dev, struct file *filp)
3401a75a222SDaniel Vetter {
3411a75a222SDaniel Vetter 	struct drm_file *file_priv = filp->private_data;
3421a75a222SDaniel Vetter 
3431a75a222SDaniel Vetter 	/* if the master has gone away we can't do anything with the lock */
34495c081c1SDaniel Vetter 	if (!dev->master)
3451a75a222SDaniel Vetter 		return;
3461a75a222SDaniel Vetter 
3471a75a222SDaniel Vetter 	if (drm_legacy_i_have_hw_lock(dev, file_priv)) {
3481a75a222SDaniel Vetter 		DRM_DEBUG("File %p released, freeing lock for context %d\n",
3491a75a222SDaniel Vetter 			  filp, _DRM_LOCKING_CONTEXT(file_priv->master->lock.hw_lock->lock));
3501a75a222SDaniel Vetter 		drm_legacy_lock_free(&file_priv->master->lock,
3511a75a222SDaniel Vetter 				     _DRM_LOCKING_CONTEXT(file_priv->master->lock.hw_lock->lock));
3521a75a222SDaniel Vetter 	}
3531a75a222SDaniel Vetter }
354058ca50cSDave Airlie 
drm_legacy_lock_master_cleanup(struct drm_device * dev,struct drm_master * master)355058ca50cSDave Airlie void drm_legacy_lock_master_cleanup(struct drm_device *dev, struct drm_master *master)
356058ca50cSDave Airlie {
357058ca50cSDave Airlie 	if (!drm_core_check_feature(dev, DRIVER_LEGACY))
358058ca50cSDave Airlie 		return;
359058ca50cSDave Airlie 
360058ca50cSDave Airlie 	/*
361058ca50cSDave Airlie 	 * Since the master is disappearing, so is the
362058ca50cSDave Airlie 	 * possibility to lock.
3635b99cad6SDan Carpenter 	 */
3645b99cad6SDan Carpenter 	mutex_lock(&dev->struct_mutex);
365058ca50cSDave Airlie 	if (master->lock.hw_lock) {
366058ca50cSDave Airlie 		if (dev->sigdata.lock == master->lock.hw_lock)
367058ca50cSDave Airlie 			dev->sigdata.lock = NULL;
368058ca50cSDave Airlie 		master->lock.hw_lock = NULL;
369058ca50cSDave Airlie 		master->lock.file_priv = NULL;
370058ca50cSDave Airlie 		wake_up_interruptible_all(&master->lock.lock_queue);
371058ca50cSDave Airlie 	}
372058ca50cSDave Airlie 	mutex_unlock(&dev->struct_mutex);
373058ca50cSDave Airlie }
374