diff options
| author | Dave Gordon <david.s.gordon@intel.com> | 2016-07-21 20:39:38 +0300 |
|---|---|---|
| committer | Chris Wilson <chris@chris-wilson.co.uk> | 2016-07-21 21:08:59 +0300 |
| commit | 13f17b22a720c4d98c3df1ebafcf9e3d3ac1c463 (patch) | |
| tree | 06b41d7ecfb4491631faf9139546d828b27d99d9 /drivers/gpu/drm/drm_crtc_internal.h | |
| parent | b12e0ee2080c093a08243726f0f3c57fddf954a5 (diff) | |
| download | linux-13f17b22a720c4d98c3df1ebafcf9e3d3ac1c463.tar.xz | |
drm/i915: use i915_gem_object_put_unlocked() after releasing mutex
The exit path in intel_overlay_put_image_ioctl() first unlocks the
struct_mutex, then drops its reference to 'new_bo' by calling
i915_gem_object_put(). As it isn't holding the mutex at this point,
this should be i915_gem_object_put_unlocked().
This was previously correct but got splatted in the recent
s/drm_gem_object_unreference/i915_gem_object_put/
where the _unlocked suffix was lost in this one case.
v2: don't bother fixing whitespace glitch [Chris Wilson]
Chris can do it next time he touches gem_evict.c ;)
Fixes: f8c417cd drm/i915: Rename drm_gem_object_unreference in preparation ...
Signed-off-by: Dave Gordon <david.s.gordon@intel.com>
Reviewed-by: Chris Wilson <chris@chris-wilson.co.uk>
Link: http://patchwork.freedesktop.org/patch/msgid/1469122778-14416-1-git-send-email-david.s.gordon@intel.com
Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
Diffstat (limited to 'drivers/gpu/drm/drm_crtc_internal.h')
0 files changed, 0 insertions, 0 deletions