drm/i915: Use GEM suspend when aborting initialisation
authorChris Wilson <chris@chris-wilson.co.uk>
Wed, 6 Jun 2018 14:54:41 +0000 (15:54 +0100)
committerChris Wilson <chris@chris-wilson.co.uk>
Thu, 7 Jun 2018 07:42:36 +0000 (08:42 +0100)
commit8571a05a9dd0986a788f03e4de1a42fc9235959f
tree24269dd59133f51bcec3fd9593869ca642b2bc0e
parent14c3f8425080a1ff97df7b81f7c339bf42c427a3
drm/i915: Use GEM suspend when aborting initialisation

As part of our GEM initialisation now, we send a request to the hardware
in order to record the initial GPU state. This coupled with deferred
idle workers, makes aborting on error tricky. We already have the
mechanism in place to wait on the GPU and cancel all the deferred
workers for suspend, so let's reuse it during the error teardown. It is
already used in places for later init error handling, but doing so at
this point is slightly ugly due to the mutex dance (it's ok, the module
load is still single threaded).

Testcase: igt/drv_module_reload/basic-reload-inject
Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
Cc: Michał Winiarski <michal.winiarski@intel.com>
Reviewed-by: Michał Winiarski <michal.winiarski@intel.com>
Link: https://patchwork.freedesktop.org/patch/msgid/20180606145441.4460-1-chris@chris-wilson.co.uk
drivers/gpu/drm/i915/i915_gem.c