diff options
| author | Chris Wilson | 2017-12-17 14:28:50 +0100 |
|---|---|---|
| committer | Chris Wilson | 2017-12-18 12:16:07 +0100 |
| commit | a31d73c3a7218539bc25fc298ac6cd5b66d96b86 (patch) | |
| tree | a83328828a391c76b73208f3e592f3a9ce251d01 /drivers/gpu/drm/i915/i915_drv.c | |
| parent | drm/i915: Unifying debugfs return codes for unsupported features (diff) | |
| download | kernel-qcow2-linux-a31d73c3a7218539bc25fc298ac6cd5b66d96b86.tar.gz kernel-qcow2-linux-a31d73c3a7218539bc25fc298ac6cd5b66d96b86.tar.xz kernel-qcow2-linux-a31d73c3a7218539bc25fc298ac6cd5b66d96b86.zip | |
drm/i915: Re-enable GGTT earlier after GPU reset
Inside i915_gem_reset(), we start touching the HW and so require the
low-level HW to be re-enabled, in particular the PCI BARs.
Fixes: 7b6da818d86f ("drm/i915: Restore the kernel context after a GPU reset on an idle engine")
References: 0db8c9612091 ("drm/i915: Re-enable GTT following a device reset")
Testcase: igt/drv_hangman #i915g/i915gm
Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
Cc: Mika Kuoppala <mika.kuoppala@linux.intel.com>
Cc: Joonas Lahtinen <joonas.lahtinen@linux.intel.com>
Cc: Michel Thierry <michel.thierry@intel.com>
Link: https://patchwork.freedesktop.org/patch/msgid/20171217132852.30642-1-chris@chris-wilson.co.uk
Reviewed-by: Tvrtko Ursulin <tvrtko.ursulin@intel.com>
Diffstat (limited to 'drivers/gpu/drm/i915/i915_drv.c')
| -rw-r--r-- | drivers/gpu/drm/i915/i915_drv.c | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/drivers/gpu/drm/i915/i915_drv.c b/drivers/gpu/drm/i915/i915_drv.c index 6d39fdf2b604..72bea281edb7 100644 --- a/drivers/gpu/drm/i915/i915_drv.c +++ b/drivers/gpu/drm/i915/i915_drv.c @@ -1924,9 +1924,6 @@ void i915_reset(struct drm_i915_private *i915, unsigned int flags) goto taint; } - i915_gem_reset(i915); - intel_overlay_reset(i915); - /* Ok, now get things going again... */ /* @@ -1939,6 +1936,9 @@ void i915_reset(struct drm_i915_private *i915, unsigned int flags) goto error; } + i915_gem_reset(i915); + intel_overlay_reset(i915); + /* * Next we need to restore the context, but we don't use those * yet either... |