diff options
author | Jesse Barnes <jbarnes@virtuousgeek.org> | 2009-02-11 16:28:14 -0500 |
---|---|---|
committer | Dave Airlie <airlied@redhat.com> | 2009-02-19 21:21:13 -0500 |
commit | 496818f08a78476abdb307e241911536221239fc (patch) | |
tree | 0d1fd0c3723ddcc5220d6ee8dff161e74f9d59d2 /drivers/gpu/drm/i915/intel_display.c | |
parent | 43565a0648e664744ac9201c199681451355edcc (diff) |
drm/i915: take struct mutex around fb unref
Need to do this in case the unref ends up doing a free.
Signed-off-by: Jesse Barnes <jbarnes@virtuousgeek.org>
Signed-off-by: Eric Anholt <eric@anholt.net>
Signed-off-by: Dave Airlie <airlied@redhat.com>
Diffstat (limited to 'drivers/gpu/drm/i915/intel_display.c')
-rw-r--r-- | drivers/gpu/drm/i915/intel_display.c | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/drivers/gpu/drm/i915/intel_display.c b/drivers/gpu/drm/i915/intel_display.c index 13f9b6667c94..4d2baf7b00be 100644 --- a/drivers/gpu/drm/i915/intel_display.c +++ b/drivers/gpu/drm/i915/intel_display.c | |||
@@ -1606,7 +1606,9 @@ intel_user_framebuffer_create(struct drm_device *dev, | |||
1606 | 1606 | ||
1607 | ret = intel_framebuffer_create(dev, mode_cmd, &fb, obj); | 1607 | ret = intel_framebuffer_create(dev, mode_cmd, &fb, obj); |
1608 | if (ret) { | 1608 | if (ret) { |
1609 | mutex_lock(&dev->struct_mutex); | ||
1609 | drm_gem_object_unreference(obj); | 1610 | drm_gem_object_unreference(obj); |
1611 | mutex_unlock(&dev->struct_mutex); | ||
1610 | return NULL; | 1612 | return NULL; |
1611 | } | 1613 | } |
1612 | 1614 | ||