diff options
author | Chris Wilson <chris@chris-wilson.co.uk> | 2009-02-11 09:26:31 -0500 |
---|---|---|
committer | Dave Airlie <airlied@redhat.com> | 2009-02-19 21:21:11 -0500 |
commit | 13af10627676879d1b20ee3cdba9a28f0906dd98 (patch) | |
tree | e169cc0980db0a8db76688386dee6c869d6c2c22 /drivers | |
parent | 2ebed176a7ee126448d34fc336afb2ea0238c280 (diff) |
drm/i915: Release and unlock on mmap_gtt error path.
We failed to unlock the mutex after failing to create the mmap offset.
Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
Signed-off-by: Eric Anholt <eric@anholt.net>
Signed-off-by: Dave Airlie <airlied@linux.ie>
Diffstat (limited to 'drivers')
-rw-r--r-- | drivers/gpu/drm/i915/i915_gem.c | 5 |
1 files changed, 4 insertions, 1 deletions
diff --git a/drivers/gpu/drm/i915/i915_gem.c b/drivers/gpu/drm/i915/i915_gem.c index ff0d94dd3550..f565b6afe414 100644 --- a/drivers/gpu/drm/i915/i915_gem.c +++ b/drivers/gpu/drm/i915/i915_gem.c | |||
@@ -758,8 +758,11 @@ i915_gem_mmap_gtt_ioctl(struct drm_device *dev, void *data, | |||
758 | 758 | ||
759 | if (!obj_priv->mmap_offset) { | 759 | if (!obj_priv->mmap_offset) { |
760 | ret = i915_gem_create_mmap_offset(obj); | 760 | ret = i915_gem_create_mmap_offset(obj); |
761 | if (ret) | 761 | if (ret) { |
762 | drm_gem_object_unreference(obj); | ||
763 | mutex_unlock(&dev->struct_mutex); | ||
762 | return ret; | 764 | return ret; |
765 | } | ||
763 | } | 766 | } |
764 | 767 | ||
765 | args->offset = obj_priv->mmap_offset; | 768 | args->offset = obj_priv->mmap_offset; |