diff options
| author | Ben Widawsky | 2012-09-27 01:15:20 +0200 |
|---|---|---|
| committer | Daniel Vetter | 2012-09-27 08:45:11 +0200 |
| commit | 3bc2913e2cb74aac8af90b46fa251dfb8d854665 (patch) | |
| tree | 888e254ccaf8390dbbd554b30c685c420098913f /drivers/gpu/drm/i915/i915_gem.c | |
| parent | drm/i915: use adjusted_mode instead of mode for checking the 6bpc force flag (diff) | |
| download | kernel-qcow2-linux-3bc2913e2cb74aac8af90b46fa251dfb8d854665.tar.gz kernel-qcow2-linux-3bc2913e2cb74aac8af90b46fa251dfb8d854665.tar.xz kernel-qcow2-linux-3bc2913e2cb74aac8af90b46fa251dfb8d854665.zip | |
drm/i915: Fix set_caching locking
On the EINVAL case we don't release struct_mutex. It should be safe to
grab the lock after checking the parameters, which also resolves the
issues.
Signed-off-by: Ben Widawsky <ben@bwidawsk.net>
Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
Diffstat (limited to 'drivers/gpu/drm/i915/i915_gem.c')
| -rw-r--r-- | drivers/gpu/drm/i915/i915_gem.c | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/drivers/gpu/drm/i915/i915_gem.c b/drivers/gpu/drm/i915/i915_gem.c index 365a7dc8a4a8..76bbb379d389 100644 --- a/drivers/gpu/drm/i915/i915_gem.c +++ b/drivers/gpu/drm/i915/i915_gem.c @@ -3218,10 +3218,6 @@ int i915_gem_set_caching_ioctl(struct drm_device *dev, void *data, enum i915_cache_level level; int ret; - ret = i915_mutex_lock_interruptible(dev); - if (ret) - return ret; - switch (args->caching) { case I915_CACHING_NONE: level = I915_CACHE_NONE; @@ -3233,6 +3229,10 @@ int i915_gem_set_caching_ioctl(struct drm_device *dev, void *data, return -EINVAL; } + ret = i915_mutex_lock_interruptible(dev); + if (ret) + return ret; + obj = to_intel_bo(drm_gem_object_lookup(dev, file, args->handle)); if (&obj->base == NULL) { ret = -ENOENT; |