diff options
| author | Daniel Vetter <[email protected]> | 2013-08-08 07:10:37 +0000 |
|---|---|---|
| committer | Dave Airlie <[email protected]> | 2013-08-19 03:24:54 +0000 |
| commit | eb91626ac4b9af3d5602a7db888b8bc4cb23eb3b (patch) | |
| tree | b7edc496796b7789b912944ef3f766a20f1a6e4a /drivers/pci/hotplug/ibmphp_hpc.c | |
| parent | Merge branch 'drm-fixes-3.11' of git://people.freedesktop.org/~agd5f/linux (diff) | |
| download | kernel-eb91626ac4b9af3d5602a7db888b8bc4cb23eb3b.tar.gz kernel-eb91626ac4b9af3d5602a7db888b8bc4cb23eb3b.zip | |
drm/i915: unpin backing storage in dmabuf_unmap
This fixes a WARN in i915_gem_free_object when the
obj->pages_pin_count isn't 0.
v2: Add locking to unmap, noticed by Chris Wilson. Note that even
though we call unmap with our own dev->struct_mutex held that won't
result in an immediate deadlock since we never go through the dma_buf
interfaces for our own, reimported buffers. But it's still easy to
blow up and anger lockdep, but that's already the case with our ->map
implementation. Fixing this for real will involve per dma-buf ww mutex
locking by the callers. And lots of fun. So go with the duct-tape
approach for now.
Cc: Chris Wilson <[email protected]>
Reported-by: Maarten Lankhorst <[email protected]>
Cc: Maarten Lankhorst <[email protected]>
Tested-by: Armin K. <[email protected]> (v1)
Tested-by: Dave Airlie <[email protected]>
Acked-by: Maarten Lankhorst <[email protected]>
Signed-off-by: Daniel Vetter <[email protected]>
Signed-off-by: Dave Airlie <[email protected]>
Diffstat (limited to 'drivers/pci/hotplug/ibmphp_hpc.c')
0 files changed, 0 insertions, 0 deletions