diff options
Diffstat (limited to 'drivers/gpu/drm/drm_fops.c')
| -rw-r--r-- | drivers/gpu/drm/drm_fops.c | 23 |
1 files changed, 11 insertions, 12 deletions
diff --git a/drivers/gpu/drm/drm_fops.c b/drivers/gpu/drm/drm_fops.c index e7aace20981f..2ca8df8b6102 100644 --- a/drivers/gpu/drm/drm_fops.c +++ b/drivers/gpu/drm/drm_fops.c | |||
| @@ -39,6 +39,9 @@ | |||
| 39 | #include <linux/slab.h> | 39 | #include <linux/slab.h> |
| 40 | #include <linux/smp_lock.h> | 40 | #include <linux/smp_lock.h> |
| 41 | 41 | ||
| 42 | /* from BKL pushdown: note that nothing else serializes idr_find() */ | ||
| 43 | DEFINE_MUTEX(drm_global_mutex); | ||
| 44 | |||
| 42 | static int drm_open_helper(struct inode *inode, struct file *filp, | 45 | static int drm_open_helper(struct inode *inode, struct file *filp, |
| 43 | struct drm_device * dev); | 46 | struct drm_device * dev); |
| 44 | 47 | ||
| @@ -175,8 +178,7 @@ int drm_stub_open(struct inode *inode, struct file *filp) | |||
| 175 | 178 | ||
| 176 | DRM_DEBUG("\n"); | 179 | DRM_DEBUG("\n"); |
| 177 | 180 | ||
| 178 | /* BKL pushdown: note that nothing else serializes idr_find() */ | 181 | mutex_lock(&drm_global_mutex); |
| 179 | lock_kernel(); | ||
| 180 | minor = idr_find(&drm_minors_idr, minor_id); | 182 | minor = idr_find(&drm_minors_idr, minor_id); |
| 181 | if (!minor) | 183 | if (!minor) |
| 182 | goto out; | 184 | goto out; |
| @@ -197,7 +199,7 @@ int drm_stub_open(struct inode *inode, struct file *filp) | |||
| 197 | fops_put(old_fops); | 199 | fops_put(old_fops); |
| 198 | 200 | ||
| 199 | out: | 201 | out: |
| 200 | unlock_kernel(); | 202 | mutex_unlock(&drm_global_mutex); |
| 201 | return err; | 203 | return err; |
| 202 | } | 204 | } |
| 203 | 205 | ||
| @@ -472,7 +474,7 @@ int drm_release(struct inode *inode, struct file *filp) | |||
| 472 | struct drm_device *dev = file_priv->minor->dev; | 474 | struct drm_device *dev = file_priv->minor->dev; |
| 473 | int retcode = 0; | 475 | int retcode = 0; |
| 474 | 476 | ||
| 475 | lock_kernel(); | 477 | mutex_lock(&drm_global_mutex); |
| 476 | 478 | ||
| 477 | DRM_DEBUG("open_count = %d\n", dev->open_count); | 479 | DRM_DEBUG("open_count = %d\n", dev->open_count); |
| 478 | 480 | ||
| @@ -573,17 +575,14 @@ int drm_release(struct inode *inode, struct file *filp) | |||
| 573 | if (atomic_read(&dev->ioctl_count)) { | 575 | if (atomic_read(&dev->ioctl_count)) { |
| 574 | DRM_ERROR("Device busy: %d\n", | 576 | DRM_ERROR("Device busy: %d\n", |
| 575 | atomic_read(&dev->ioctl_count)); | 577 | atomic_read(&dev->ioctl_count)); |
| 576 | spin_unlock(&dev->count_lock); | 578 | retcode = -EBUSY; |
| 577 | unlock_kernel(); | 579 | goto out; |
| 578 | return -EBUSY; | ||
| 579 | } | 580 | } |
| 580 | spin_unlock(&dev->count_lock); | 581 | retcode = drm_lastclose(dev); |
| 581 | unlock_kernel(); | ||
| 582 | return drm_lastclose(dev); | ||
| 583 | } | 582 | } |
| 583 | out: | ||
| 584 | spin_unlock(&dev->count_lock); | 584 | spin_unlock(&dev->count_lock); |
| 585 | 585 | mutex_unlock(&drm_global_mutex); | |
| 586 | unlock_kernel(); | ||
| 587 | 586 | ||
| 588 | return retcode; | 587 | return retcode; |
| 589 | } | 588 | } |
