summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorChris Wilson <chris@chris-wilson.co.uk>2017-12-11 23:40:40 +0300
committerChris Wilson <chris@chris-wilson.co.uk>2017-12-13 00:07:40 +0300
commit3ef98f50ca7cef5cc9b9bf89cd46830afbf4df68 (patch)
tree0f5bae5db395feb9e959ab7e981aab98638d054a
parent0132a1a5d44d2cd32a249dbe999a88c2134a6bd1 (diff)
downloadlinux-3ef98f50ca7cef5cc9b9bf89cd46830afbf4df68.tar.xz
drm/i915: Print an error message for a reset failure on gen2
Since on gen2, we do not universally have a GPU reset implementation, we fail i915_reset() at intel_has_gpu_reset(). However, this is also intentionally disabled for CI testing and so it only has a debug message. Promote that debug message to a user-facing error message that should explain why their machine became unusable following the GPU hang. Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk> Cc: Ville Syrjälä <ville.syrjala@linux.intel.com> Reviewed-by: Ville Syrjälä <ville.syrjala@linux.intel.com> Link: https://patchwork.freedesktop.org/patch/msgid/20171211204040.22858-1-chris@chris-wilson.co.uk
-rw-r--r--drivers/gpu/drm/i915/i915_drv.c5
1 files changed, 4 insertions, 1 deletions
diff --git a/drivers/gpu/drm/i915/i915_drv.c b/drivers/gpu/drm/i915/i915_drv.c
index 54a8fca7e7b2..721ccce1832f 100644
--- a/drivers/gpu/drm/i915/i915_drv.c
+++ b/drivers/gpu/drm/i915/i915_drv.c
@@ -1903,7 +1903,10 @@ void i915_reset(struct drm_i915_private *i915, unsigned int flags)
}
if (!intel_has_gpu_reset(i915)) {
- DRM_DEBUG_DRIVER("GPU reset disabled\n");
+ if (i915_modparams.reset)
+ dev_err(i915->drm.dev, "GPU reset not supported\n");
+ else
+ DRM_DEBUG_DRIVER("GPU reset disabled\n");
goto error;
}