xref: /openbsd-src/sys/dev/pci/drm/drm_lock.c (revision ad8b1aafbcc34f7eb86e4ebfd874be286017954b)
1c349dbc7Sjsg /*
2c349dbc7Sjsg  * \file drm_lock.c
3c349dbc7Sjsg  * IOCTLs for locking
4c349dbc7Sjsg  *
5c349dbc7Sjsg  * \author Rickard E. (Rik) Faith <faith@valinux.com>
6c349dbc7Sjsg  * \author Gareth Hughes <gareth@valinux.com>
7c349dbc7Sjsg  */
8c349dbc7Sjsg 
9c349dbc7Sjsg /*
10c349dbc7Sjsg  * Created: Tue Feb  2 08:37:54 1999 by faith@valinux.com
11c349dbc7Sjsg  *
12c349dbc7Sjsg  * Copyright 1999 Precision Insight, Inc., Cedar Park, Texas.
13c349dbc7Sjsg  * Copyright 2000 VA Linux Systems, Inc., Sunnyvale, California.
14c349dbc7Sjsg  * All Rights Reserved.
15c349dbc7Sjsg  *
16c349dbc7Sjsg  * Permission is hereby granted, free of charge, to any person obtaining a
17c349dbc7Sjsg  * copy of this software and associated documentation files (the "Software"),
18c349dbc7Sjsg  * to deal in the Software without restriction, including without limitation
19c349dbc7Sjsg  * the rights to use, copy, modify, merge, publish, distribute, sublicense,
20c349dbc7Sjsg  * and/or sell copies of the Software, and to permit persons to whom the
21c349dbc7Sjsg  * Software is furnished to do so, subject to the following conditions:
22c349dbc7Sjsg  *
23c349dbc7Sjsg  * The above copyright notice and this permission notice (including the next
24c349dbc7Sjsg  * paragraph) shall be included in all copies or substantial portions of the
25c349dbc7Sjsg  * Software.
26c349dbc7Sjsg  *
27c349dbc7Sjsg  * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
28c349dbc7Sjsg  * IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
29c349dbc7Sjsg  * FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT.  IN NO EVENT SHALL
30c349dbc7Sjsg  * VA LINUX SYSTEMS AND/OR ITS SUPPLIERS BE LIABLE FOR ANY CLAIM, DAMAGES OR
31c349dbc7Sjsg  * OTHER LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE,
32c349dbc7Sjsg  * ARISING FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR
33c349dbc7Sjsg  * OTHER DEALINGS IN THE SOFTWARE.
34c349dbc7Sjsg  */
35c349dbc7Sjsg 
36c349dbc7Sjsg #include <linux/export.h>
37c349dbc7Sjsg #include <linux/sched/signal.h>
38c349dbc7Sjsg 
39c349dbc7Sjsg #include <drm/drm.h>
40c349dbc7Sjsg #include <drm/drm_drv.h>
41c349dbc7Sjsg #include <drm/drm_file.h>
42c349dbc7Sjsg #include <drm/drm_print.h>
43c349dbc7Sjsg 
44c349dbc7Sjsg #include "drm_internal.h"
45c349dbc7Sjsg #include "drm_legacy.h"
46c349dbc7Sjsg 
47c349dbc7Sjsg static int drm_lock_take(struct drm_lock_data *lock_data, unsigned int context);
48c349dbc7Sjsg 
49c349dbc7Sjsg /*
50c349dbc7Sjsg  * Take the heavyweight lock.
51c349dbc7Sjsg  *
52c349dbc7Sjsg  * \param lock lock pointer.
53c349dbc7Sjsg  * \param context locking context.
54c349dbc7Sjsg  * \return one if the lock is held, or zero otherwise.
55c349dbc7Sjsg  *
56c349dbc7Sjsg  * Attempt to mark the lock as held by the given context, via the \p cmpxchg instruction.
57c349dbc7Sjsg  */
58c349dbc7Sjsg static
drm_lock_take(struct drm_lock_data * lock_data,unsigned int context)59c349dbc7Sjsg int drm_lock_take(struct drm_lock_data *lock_data,
60c349dbc7Sjsg 		  unsigned int context)
61c349dbc7Sjsg {
62c349dbc7Sjsg 	unsigned int old, new, prev;
63c349dbc7Sjsg 	volatile unsigned int *lock = &lock_data->hw_lock->lock;
64c349dbc7Sjsg 
65c349dbc7Sjsg 	spin_lock_bh(&lock_data->spinlock);
66c349dbc7Sjsg 	do {
67c349dbc7Sjsg 		old = *lock;
68c349dbc7Sjsg 		if (old & _DRM_LOCK_HELD)
69c349dbc7Sjsg 			new = old | _DRM_LOCK_CONT;
70c349dbc7Sjsg 		else {
71c349dbc7Sjsg 			new = context | _DRM_LOCK_HELD |
72c349dbc7Sjsg 				((lock_data->user_waiters + lock_data->kernel_waiters > 1) ?
73c349dbc7Sjsg 				 _DRM_LOCK_CONT : 0);
74c349dbc7Sjsg 		}
75c349dbc7Sjsg 		prev = cmpxchg(lock, old, new);
76c349dbc7Sjsg 	} while (prev != old);
77c349dbc7Sjsg 	spin_unlock_bh(&lock_data->spinlock);
78c349dbc7Sjsg 
79c349dbc7Sjsg 	if (_DRM_LOCKING_CONTEXT(old) == context) {
80c349dbc7Sjsg 		if (old & _DRM_LOCK_HELD) {
81c349dbc7Sjsg 			if (context != DRM_KERNEL_CONTEXT) {
82c349dbc7Sjsg 				DRM_ERROR("%d holds heavyweight lock\n",
83c349dbc7Sjsg 					  context);
84c349dbc7Sjsg 			}
85c349dbc7Sjsg 			return 0;
86c349dbc7Sjsg 		}
87c349dbc7Sjsg 	}
88c349dbc7Sjsg 
89c349dbc7Sjsg 	if ((_DRM_LOCKING_CONTEXT(new)) == context && (new & _DRM_LOCK_HELD)) {
90c349dbc7Sjsg 		/* Have lock */
91c349dbc7Sjsg 		return 1;
92c349dbc7Sjsg 	}
93c349dbc7Sjsg 	return 0;
94c349dbc7Sjsg }
95c349dbc7Sjsg 
96c349dbc7Sjsg /*
97c349dbc7Sjsg  * This takes a lock forcibly and hands it to context.	Should ONLY be used
98c349dbc7Sjsg  * inside *_unlock to give lock to kernel before calling *_dma_schedule.
99c349dbc7Sjsg  *
100c349dbc7Sjsg  * \param dev DRM device.
101c349dbc7Sjsg  * \param lock lock pointer.
102c349dbc7Sjsg  * \param context locking context.
103c349dbc7Sjsg  * \return always one.
104c349dbc7Sjsg  *
105c349dbc7Sjsg  * Resets the lock file pointer.
106c349dbc7Sjsg  * Marks the lock as held by the given context, via the \p cmpxchg instruction.
107c349dbc7Sjsg  */
drm_lock_transfer(struct drm_lock_data * lock_data,unsigned int context)108c349dbc7Sjsg static int drm_lock_transfer(struct drm_lock_data *lock_data,
109c349dbc7Sjsg 			     unsigned int context)
110c349dbc7Sjsg {
111c349dbc7Sjsg 	unsigned int old, new, prev;
112c349dbc7Sjsg 	volatile unsigned int *lock = &lock_data->hw_lock->lock;
113c349dbc7Sjsg 
114c349dbc7Sjsg 	lock_data->file_priv = NULL;
115c349dbc7Sjsg 	do {
116c349dbc7Sjsg 		old = *lock;
117c349dbc7Sjsg 		new = context | _DRM_LOCK_HELD;
118c349dbc7Sjsg 		prev = cmpxchg(lock, old, new);
119c349dbc7Sjsg 	} while (prev != old);
120c349dbc7Sjsg 	return 1;
121c349dbc7Sjsg }
122c349dbc7Sjsg 
drm_legacy_lock_free(struct drm_lock_data * lock_data,unsigned int context)123c349dbc7Sjsg static int drm_legacy_lock_free(struct drm_lock_data *lock_data,
124c349dbc7Sjsg 				unsigned int context)
125c349dbc7Sjsg {
126c349dbc7Sjsg 	unsigned int old, new, prev;
127c349dbc7Sjsg 	volatile unsigned int *lock = &lock_data->hw_lock->lock;
128c349dbc7Sjsg 
129c349dbc7Sjsg 	spin_lock_bh(&lock_data->spinlock);
130c349dbc7Sjsg 	if (lock_data->kernel_waiters != 0) {
131c349dbc7Sjsg 		drm_lock_transfer(lock_data, 0);
132c349dbc7Sjsg 		lock_data->idle_has_lock = 1;
133c349dbc7Sjsg 		spin_unlock_bh(&lock_data->spinlock);
134c349dbc7Sjsg 		return 1;
135c349dbc7Sjsg 	}
136c349dbc7Sjsg 	spin_unlock_bh(&lock_data->spinlock);
137c349dbc7Sjsg 
138c349dbc7Sjsg 	do {
139c349dbc7Sjsg 		old = *lock;
140c349dbc7Sjsg 		new = _DRM_LOCKING_CONTEXT(old);
141c349dbc7Sjsg 		prev = cmpxchg(lock, old, new);
142c349dbc7Sjsg 	} while (prev != old);
143c349dbc7Sjsg 
144c349dbc7Sjsg 	if (_DRM_LOCK_IS_HELD(old) && _DRM_LOCKING_CONTEXT(old) != context) {
145c349dbc7Sjsg 		DRM_ERROR("%d freed heavyweight lock held by %d\n",
146c349dbc7Sjsg 			  context, _DRM_LOCKING_CONTEXT(old));
147c349dbc7Sjsg 		return 1;
148c349dbc7Sjsg 	}
149c349dbc7Sjsg 	wake_up_interruptible(&lock_data->lock_queue);
150c349dbc7Sjsg 	return 0;
151c349dbc7Sjsg }
152c349dbc7Sjsg 
153c349dbc7Sjsg /*
154c349dbc7Sjsg  * Lock ioctl.
155c349dbc7Sjsg  *
156c349dbc7Sjsg  * \param inode device inode.
157c349dbc7Sjsg  * \param file_priv DRM file private.
158c349dbc7Sjsg  * \param cmd command.
159c349dbc7Sjsg  * \param arg user argument, pointing to a drm_lock structure.
160c349dbc7Sjsg  * \return zero on success or negative number on failure.
161c349dbc7Sjsg  *
162c349dbc7Sjsg  * Add the current task to the lock wait queue, and attempt to take to lock.
163c349dbc7Sjsg  */
drm_legacy_lock(struct drm_device * dev,void * data,struct drm_file * file_priv)164c349dbc7Sjsg int drm_legacy_lock(struct drm_device *dev, void *data,
165c349dbc7Sjsg 		    struct drm_file *file_priv)
166c349dbc7Sjsg {
167c349dbc7Sjsg 	DECLARE_WAITQUEUE(entry, current);
168c349dbc7Sjsg 	struct drm_lock *lock = data;
169c349dbc7Sjsg 	struct drm_master *master = file_priv->master;
170c349dbc7Sjsg 	int ret = 0;
171c349dbc7Sjsg 
172c349dbc7Sjsg 	if (!drm_core_check_feature(dev, DRIVER_LEGACY))
173c349dbc7Sjsg 		return -EOPNOTSUPP;
174c349dbc7Sjsg 
175c349dbc7Sjsg 	++file_priv->lock_count;
176c349dbc7Sjsg 
177c349dbc7Sjsg 	if (lock->context == DRM_KERNEL_CONTEXT) {
178c349dbc7Sjsg 		DRM_ERROR("Process %d using kernel context %d\n",
179c349dbc7Sjsg 			  task_pid_nr(current), lock->context);
180c349dbc7Sjsg 		return -EINVAL;
181c349dbc7Sjsg 	}
182c349dbc7Sjsg 
183c349dbc7Sjsg 	DRM_DEBUG("%d (pid %d) requests lock (0x%08x), flags = 0x%08x\n",
184c349dbc7Sjsg 		  lock->context, task_pid_nr(current),
185c349dbc7Sjsg 		  master->lock.hw_lock ? master->lock.hw_lock->lock : -1,
186c349dbc7Sjsg 		  lock->flags);
187c349dbc7Sjsg 
188c349dbc7Sjsg 	add_wait_queue(&master->lock.lock_queue, &entry);
189c349dbc7Sjsg 	spin_lock_bh(&master->lock.spinlock);
190c349dbc7Sjsg 	master->lock.user_waiters++;
191c349dbc7Sjsg 	spin_unlock_bh(&master->lock.spinlock);
192c349dbc7Sjsg 
193c349dbc7Sjsg 	for (;;) {
194c349dbc7Sjsg 		__set_current_state(TASK_INTERRUPTIBLE);
195c349dbc7Sjsg 		if (!master->lock.hw_lock) {
196c349dbc7Sjsg 			/* Device has been unregistered */
197c349dbc7Sjsg 			send_sig(SIGTERM, current, 0);
198c349dbc7Sjsg 			ret = -EINTR;
199c349dbc7Sjsg 			break;
200c349dbc7Sjsg 		}
201c349dbc7Sjsg 		if (drm_lock_take(&master->lock, lock->context)) {
202c349dbc7Sjsg 			master->lock.file_priv = file_priv;
203c349dbc7Sjsg 			master->lock.lock_time = jiffies;
204c349dbc7Sjsg 			break;	/* Got lock */
205c349dbc7Sjsg 		}
206c349dbc7Sjsg 
207c349dbc7Sjsg 		/* Contention */
208c349dbc7Sjsg 		mutex_unlock(&drm_global_mutex);
209c349dbc7Sjsg 		schedule();
210c349dbc7Sjsg 		mutex_lock(&drm_global_mutex);
211c349dbc7Sjsg 		if (signal_pending(current)) {
212c349dbc7Sjsg 			ret = -EINTR;
213c349dbc7Sjsg 			break;
214c349dbc7Sjsg 		}
215c349dbc7Sjsg 	}
216c349dbc7Sjsg 	spin_lock_bh(&master->lock.spinlock);
217c349dbc7Sjsg 	master->lock.user_waiters--;
218c349dbc7Sjsg 	spin_unlock_bh(&master->lock.spinlock);
219c349dbc7Sjsg 	__set_current_state(TASK_RUNNING);
220c349dbc7Sjsg 	remove_wait_queue(&master->lock.lock_queue, &entry);
221c349dbc7Sjsg 
222c349dbc7Sjsg 	DRM_DEBUG("%d %s\n", lock->context,
223c349dbc7Sjsg 		  ret ? "interrupted" : "has lock");
224c349dbc7Sjsg 	if (ret) return ret;
225c349dbc7Sjsg 
226c349dbc7Sjsg 	/* don't set the block all signals on the master process for now
227c349dbc7Sjsg 	 * really probably not the correct answer but lets us debug xkb
228c349dbc7Sjsg  	 * xserver for now */
229c349dbc7Sjsg 	if (!drm_is_current_master(file_priv)) {
230c349dbc7Sjsg 		dev->sigdata.context = lock->context;
231c349dbc7Sjsg 		dev->sigdata.lock = master->lock.hw_lock;
232c349dbc7Sjsg 	}
233c349dbc7Sjsg 
234c349dbc7Sjsg 	if (dev->driver->dma_quiescent && (lock->flags & _DRM_LOCK_QUIESCENT))
235c349dbc7Sjsg 	{
236c349dbc7Sjsg 		if (dev->driver->dma_quiescent(dev)) {
237c349dbc7Sjsg 			DRM_DEBUG("%d waiting for DMA quiescent\n",
238c349dbc7Sjsg 				  lock->context);
239c349dbc7Sjsg 			return -EBUSY;
240c349dbc7Sjsg 		}
241c349dbc7Sjsg 	}
242c349dbc7Sjsg 
243c349dbc7Sjsg 	return 0;
244c349dbc7Sjsg }
245c349dbc7Sjsg 
246c349dbc7Sjsg /*
247c349dbc7Sjsg  * Unlock ioctl.
248c349dbc7Sjsg  *
249c349dbc7Sjsg  * \param inode device inode.
250c349dbc7Sjsg  * \param file_priv DRM file private.
251c349dbc7Sjsg  * \param cmd command.
252c349dbc7Sjsg  * \param arg user argument, pointing to a drm_lock structure.
253c349dbc7Sjsg  * \return zero on success or negative number on failure.
254c349dbc7Sjsg  *
255c349dbc7Sjsg  * Transfer and free the lock.
256c349dbc7Sjsg  */
drm_legacy_unlock(struct drm_device * dev,void * data,struct drm_file * file_priv)257c349dbc7Sjsg int drm_legacy_unlock(struct drm_device *dev, void *data, struct drm_file *file_priv)
258c349dbc7Sjsg {
259c349dbc7Sjsg 	struct drm_lock *lock = data;
260c349dbc7Sjsg 	struct drm_master *master = file_priv->master;
261c349dbc7Sjsg 
262c349dbc7Sjsg 	if (!drm_core_check_feature(dev, DRIVER_LEGACY))
263c349dbc7Sjsg 		return -EOPNOTSUPP;
264c349dbc7Sjsg 
265c349dbc7Sjsg 	if (lock->context == DRM_KERNEL_CONTEXT) {
266c349dbc7Sjsg 		DRM_ERROR("Process %d using kernel context %d\n",
267c349dbc7Sjsg 			  task_pid_nr(current), lock->context);
268c349dbc7Sjsg 		return -EINVAL;
269c349dbc7Sjsg 	}
270c349dbc7Sjsg 
271c349dbc7Sjsg 	if (drm_legacy_lock_free(&master->lock, lock->context)) {
272c349dbc7Sjsg 		/* FIXME: Should really bail out here. */
273c349dbc7Sjsg 	}
274c349dbc7Sjsg 
275c349dbc7Sjsg 	return 0;
276c349dbc7Sjsg }
277c349dbc7Sjsg 
278c349dbc7Sjsg /*
279c349dbc7Sjsg  * This function returns immediately and takes the hw lock
280c349dbc7Sjsg  * with the kernel context if it is free, otherwise it gets the highest priority when and if
281c349dbc7Sjsg  * it is eventually released.
282c349dbc7Sjsg  *
283c349dbc7Sjsg  * This guarantees that the kernel will _eventually_ have the lock _unless_ it is held
284c349dbc7Sjsg  * by a blocked process. (In the latter case an explicit wait for the hardware lock would cause
285c349dbc7Sjsg  * a deadlock, which is why the "idlelock" was invented).
286c349dbc7Sjsg  *
287c349dbc7Sjsg  * This should be sufficient to wait for GPU idle without
288c349dbc7Sjsg  * having to worry about starvation.
289c349dbc7Sjsg  */
drm_legacy_idlelock_take(struct drm_lock_data * lock_data)290c349dbc7Sjsg void drm_legacy_idlelock_take(struct drm_lock_data *lock_data)
291c349dbc7Sjsg {
292c349dbc7Sjsg 	int ret;
293c349dbc7Sjsg 
294c349dbc7Sjsg 	spin_lock_bh(&lock_data->spinlock);
295c349dbc7Sjsg 	lock_data->kernel_waiters++;
296c349dbc7Sjsg 	if (!lock_data->idle_has_lock) {
297c349dbc7Sjsg 
298c349dbc7Sjsg 		spin_unlock_bh(&lock_data->spinlock);
299c349dbc7Sjsg 		ret = drm_lock_take(lock_data, DRM_KERNEL_CONTEXT);
300c349dbc7Sjsg 		spin_lock_bh(&lock_data->spinlock);
301c349dbc7Sjsg 
302c349dbc7Sjsg 		if (ret == 1)
303c349dbc7Sjsg 			lock_data->idle_has_lock = 1;
304c349dbc7Sjsg 	}
305c349dbc7Sjsg 	spin_unlock_bh(&lock_data->spinlock);
306c349dbc7Sjsg }
307c349dbc7Sjsg EXPORT_SYMBOL(drm_legacy_idlelock_take);
308c349dbc7Sjsg 
drm_legacy_idlelock_release(struct drm_lock_data * lock_data)309c349dbc7Sjsg void drm_legacy_idlelock_release(struct drm_lock_data *lock_data)
310c349dbc7Sjsg {
311c349dbc7Sjsg 	unsigned int old, prev;
312c349dbc7Sjsg 	volatile unsigned int *lock = &lock_data->hw_lock->lock;
313c349dbc7Sjsg 
314c349dbc7Sjsg 	spin_lock_bh(&lock_data->spinlock);
315c349dbc7Sjsg 	if (--lock_data->kernel_waiters == 0) {
316c349dbc7Sjsg 		if (lock_data->idle_has_lock) {
317c349dbc7Sjsg 			do {
318c349dbc7Sjsg 				old = *lock;
319c349dbc7Sjsg 				prev = cmpxchg(lock, old, DRM_KERNEL_CONTEXT);
320c349dbc7Sjsg 			} while (prev != old);
321c349dbc7Sjsg 			wake_up_interruptible(&lock_data->lock_queue);
322c349dbc7Sjsg 			lock_data->idle_has_lock = 0;
323c349dbc7Sjsg 		}
324c349dbc7Sjsg 	}
325c349dbc7Sjsg 	spin_unlock_bh(&lock_data->spinlock);
326c349dbc7Sjsg }
327c349dbc7Sjsg EXPORT_SYMBOL(drm_legacy_idlelock_release);
328c349dbc7Sjsg 
drm_legacy_i_have_hw_lock(struct drm_device * dev,struct drm_file * file_priv)329c349dbc7Sjsg static int drm_legacy_i_have_hw_lock(struct drm_device *dev,
330c349dbc7Sjsg 				     struct drm_file *file_priv)
331c349dbc7Sjsg {
332c349dbc7Sjsg 	struct drm_master *master = file_priv->master;
333*ad8b1aafSjsg 
334c349dbc7Sjsg 	return (file_priv->lock_count && master->lock.hw_lock &&
335c349dbc7Sjsg 		_DRM_LOCK_IS_HELD(master->lock.hw_lock->lock) &&
336c349dbc7Sjsg 		master->lock.file_priv == file_priv);
337c349dbc7Sjsg }
338c349dbc7Sjsg 
drm_legacy_lock_release(struct drm_device * dev,struct file * filp)339c349dbc7Sjsg void drm_legacy_lock_release(struct drm_device *dev, struct file *filp)
340c349dbc7Sjsg {
341c349dbc7Sjsg 	struct drm_file *file_priv = filp->private_data;
342c349dbc7Sjsg 
343c349dbc7Sjsg 	/* if the master has gone away we can't do anything with the lock */
344c349dbc7Sjsg 	if (!dev->master)
345c349dbc7Sjsg 		return;
346c349dbc7Sjsg 
347c349dbc7Sjsg 	if (drm_legacy_i_have_hw_lock(dev, file_priv)) {
348c349dbc7Sjsg 		DRM_DEBUG("File %p released, freeing lock for context %d\n",
349c349dbc7Sjsg 			  filp, _DRM_LOCKING_CONTEXT(file_priv->master->lock.hw_lock->lock));
350c349dbc7Sjsg 		drm_legacy_lock_free(&file_priv->master->lock,
351c349dbc7Sjsg 				     _DRM_LOCKING_CONTEXT(file_priv->master->lock.hw_lock->lock));
352c349dbc7Sjsg 	}
353c349dbc7Sjsg }
354c349dbc7Sjsg 
drm_legacy_lock_master_cleanup(struct drm_device * dev,struct drm_master * master)355c349dbc7Sjsg void drm_legacy_lock_master_cleanup(struct drm_device *dev, struct drm_master *master)
356c349dbc7Sjsg {
357c349dbc7Sjsg 	if (!drm_core_check_feature(dev, DRIVER_LEGACY))
358c349dbc7Sjsg 		return;
359c349dbc7Sjsg 
360c349dbc7Sjsg 	/*
361c349dbc7Sjsg 	 * Since the master is disappearing, so is the
362c349dbc7Sjsg 	 * possibility to lock.
363c349dbc7Sjsg 	 */
364c349dbc7Sjsg 	mutex_lock(&dev->struct_mutex);
365c349dbc7Sjsg 	if (master->lock.hw_lock) {
366c349dbc7Sjsg 		if (dev->sigdata.lock == master->lock.hw_lock)
367c349dbc7Sjsg 			dev->sigdata.lock = NULL;
368c349dbc7Sjsg 		master->lock.hw_lock = NULL;
369c349dbc7Sjsg 		master->lock.file_priv = NULL;
370c349dbc7Sjsg 		wake_up_interruptible_all(&master->lock.lock_queue);
371c349dbc7Sjsg 	}
372c349dbc7Sjsg 	mutex_unlock(&dev->struct_mutex);
373c349dbc7Sjsg }
374