diff options
author | Chris Wilson <chris@chris-wilson.co.uk> | 2018-02-07 06:15:45 -0500 |
---|---|---|
committer | Chris Wilson <chris@chris-wilson.co.uk> | 2018-02-07 08:12:32 -0500 |
commit | 8177e1125235a40c1af110e37febf6bb1e7b3f81 (patch) | |
tree | 39befba028f87149c7c081b944e7079286128115 /drivers/gpu/drm/i915/intel_ringbuffer.c | |
parent | c22b355ff7fd60378c55d1401530287fea4e3a96 (diff) |
drm/i915: Tidy up some error messages around reset failure
On blb and pnv, we are seeing sporadic
i915 0000:00:02.0: Resetting chip after gpu hang
[drm:intel_gpu_reset [i915]] rcs0: timed out on STOP_RING
[drm:i915_reset [i915]] *ERROR* Failed hw init on reset -5
which notably lack the actual root cause of the error. Ostensibly it
should be the init_ring_common() that failed, but it's error paths are
covered by DRM_ERROR.
Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
Reviewed-by: Tvrtko Ursulin <tvrtko.ursulin@intel.com>
Link: https://patchwork.freedesktop.org/patch/msgid/20180207111545.17078-1-chris@chris-wilson.co.uk
Diffstat (limited to 'drivers/gpu/drm/i915/intel_ringbuffer.c')
-rw-r--r-- | drivers/gpu/drm/i915/intel_ringbuffer.c | 18 |
1 files changed, 9 insertions, 9 deletions
diff --git a/drivers/gpu/drm/i915/intel_ringbuffer.c b/drivers/gpu/drm/i915/intel_ringbuffer.c index e2085820b586..f93e4532371a 100644 --- a/drivers/gpu/drm/i915/intel_ringbuffer.c +++ b/drivers/gpu/drm/i915/intel_ringbuffer.c | |||
@@ -453,13 +453,13 @@ static int init_ring_common(struct intel_engine_cs *engine) | |||
453 | 453 | ||
454 | if (!stop_ring(engine)) { | 454 | if (!stop_ring(engine)) { |
455 | /* G45 ring initialization often fails to reset head to zero */ | 455 | /* G45 ring initialization often fails to reset head to zero */ |
456 | DRM_DEBUG_KMS("%s head not reset to zero " | 456 | DRM_DEBUG_DRIVER("%s head not reset to zero " |
457 | "ctl %08x head %08x tail %08x start %08x\n", | 457 | "ctl %08x head %08x tail %08x start %08x\n", |
458 | engine->name, | 458 | engine->name, |
459 | I915_READ_CTL(engine), | 459 | I915_READ_CTL(engine), |
460 | I915_READ_HEAD(engine), | 460 | I915_READ_HEAD(engine), |
461 | I915_READ_TAIL(engine), | 461 | I915_READ_TAIL(engine), |
462 | I915_READ_START(engine)); | 462 | I915_READ_START(engine)); |
463 | 463 | ||
464 | if (!stop_ring(engine)) { | 464 | if (!stop_ring(engine)) { |
465 | DRM_ERROR("failed to set %s head to zero " | 465 | DRM_ERROR("failed to set %s head to zero " |
@@ -492,8 +492,8 @@ static int init_ring_common(struct intel_engine_cs *engine) | |||
492 | 492 | ||
493 | /* WaClearRingBufHeadRegAtInit:ctg,elk */ | 493 | /* WaClearRingBufHeadRegAtInit:ctg,elk */ |
494 | if (I915_READ_HEAD(engine)) | 494 | if (I915_READ_HEAD(engine)) |
495 | DRM_DEBUG("%s initialization failed [head=%08x], fudging\n", | 495 | DRM_DEBUG_DRIVER("%s initialization failed [head=%08x], fudging\n", |
496 | engine->name, I915_READ_HEAD(engine)); | 496 | engine->name, I915_READ_HEAD(engine)); |
497 | 497 | ||
498 | intel_ring_update_space(ring); | 498 | intel_ring_update_space(ring); |
499 | I915_WRITE_HEAD(engine, ring->head); | 499 | I915_WRITE_HEAD(engine, ring->head); |