diff options
| author |   Dave Airlie <airlied@redhat.com> | 2010-10-07 14:01:17 +1000 |
|---|---|---|
| committer | Dave Airlie <airlied@redhat.com> | 2010-10-07 14:01:17 +1000 |
| commit | dab8dcfa3c8e3b021a138bee7c17791b4991ba55 (patch) | |
| tree | a18d922cd613beb3ef8dc375124a81c5396f50d6 /drivers/gpu/drm/nouveau/nouveau_notifier.c | |
| parent | drm/ttm: Fix two race conditions + fix busy codepaths (diff) | |
| download | linux-dev-dab8dcfa3c8e3b021a138bee7c17791b4991ba55.tar.xz linux-dev-dab8dcfa3c8e3b021a138bee7c17791b4991ba55.zip | |
drm: don't drop handle reference on unload
since the handle references are all tied to a file_priv, and when it disappears
all the handle refs go with it.
The fbcon ones we'd only notice on unload, but the nouveau notifier one
would would happen on reboot.
nouveau: Reported-by: Marc Dionne <marc.c.dionne@gmail.com>
nouveau: Tested-by: Marc Dionne <marc.c.dionne@gmail.com>
i915 unload: Reported-by: Keith Packard <keithp@keithp.com>
Acked-by: Ben Skeggs <bskeggs@redhat.com>
Signed-off-by: Dave Airlie <airlied@redhat.com>
Diffstat (limited to 'drivers/gpu/drm/nouveau/nouveau_notifier.c')
| -rw-r--r-- | drivers/gpu/drm/nouveau/nouveau_notifier.c | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/drivers/gpu/drm/nouveau/nouveau_notifier.c b/drivers/gpu/drm/nouveau/nouveau_notifier.c index 3c9964a8fbad..3ec181ff50ce 100644 --- a/drivers/gpu/drm/nouveau/nouveau_notifier.c +++ b/drivers/gpu/drm/nouveau/nouveau_notifier.c @@ -79,7 +79,6 @@ nouveau_notifier_takedown_channel(struct nouveau_channel *chan) mutex_lock(&dev->struct_mutex); nouveau_bo_unpin(chan->notifier_bo); mutex_unlock(&dev->struct_mutex); - drm_gem_object_handle_unreference_unlocked(chan->notifier_bo->gem); drm_gem_object_unreference_unlocked(chan->notifier_bo->gem); drm_mm_takedown(&chan->notifier_heap); } |