drm/i915: Only unpin the default ctx object if it exists
authorChris Wilson <chris@chris-wilson.co.uk>
Fri, 16 May 2014 17:59:00 +0000 (18:59 +0100)
committerDaniel Vetter <daniel.vetter@ffwll.ch>
Fri, 16 May 2014 19:41:12 +0000 (21:41 +0200)
commitd3b448d9917a3d6531e499d88bfb13ea5e31e4ad
tree5b734d8d90c333038b08653cd330774f18a6f080
parent823c690958ca3792f5450e8e3167409f4ea7ba7c
drm/i915: Only unpin the default ctx object if it exists

Since commit 691e6415c891b8b2b082a120b896b443531c4d45
Author: Chris Wilson <chris@chris-wilson.co.uk>
Date:   Wed Apr 9 09:07:36 2014 +0100

    drm/i915: Always use kref tracking for all contexts.

we have contexts everywhere, and so we must be careful to distinguish
fake contexts, which do not have an associated bo, and real ones, which
do. In particular, we now need to be careful not to dereference NULL
pointers.

This is one such example, as the commit highlighted above failed to move
the unpinning of the default ctx object into the real-context-only
branch.

Reported-by: Daniel Vetter <daniel.vetter@ffwll.ch>
Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=78792
Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
Cc: Daniel Vetter <daniel.vetter@ffwll.ch>
Cc: Ben Widawsky <benjamin.widawsky@intel.com>
Cc: Mika Kuoppala <mika.kuoppala@intel.com>
Cc: Jani Nikula <jani.nikula@intel.com>
Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
drivers/gpu/drm/i915/i915_gem_context.c