Skip to content

Commit

Permalink
drm/i915/hangcheck: Track context changes
Browse files Browse the repository at this point in the history
Given sufficient preemption, we may see a busy system that doesn't
advance seqno while performing work across multiple contexts, and given
sufficient pathology not even notice a change in ACTHD. What does change
between the preempting contexts is their RING, so take note of that and
treat a change in the ring address as being an indication of forward
progress.

Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
Cc: Mika Kuoppala <mika.kuoppala@linux.intel.com>
Reviewed-by: Tvrtko Ursulin <tvrtko.ursulin@intel.com>
Link: https://patchwork.freedesktop.org/patch/msgid/20190501114541.10077-1-chris@chris-wilson.co.uk
  • Loading branch information
Chris Wilson committed May 3, 2019
1 parent d69ebf4 commit f410776
Show file tree
Hide file tree
Showing 2 changed files with 10 additions and 3 deletions.
1 change: 1 addition & 0 deletions drivers/gpu/drm/i915/gt/intel_engine_types.h
Original file line number Diff line number Diff line change
Expand Up @@ -53,6 +53,7 @@ struct intel_instdone {

struct intel_engine_hangcheck {
u64 acthd;
u32 last_ring;
u32 last_seqno;
u32 next_seqno;
unsigned long action_timestamp;
Expand Down
12 changes: 9 additions & 3 deletions drivers/gpu/drm/i915/gt/intel_hangcheck.c
Original file line number Diff line number Diff line change
Expand Up @@ -27,6 +27,7 @@

struct hangcheck {
u64 acthd;
u32 ring;
u32 seqno;
enum intel_engine_hangcheck_action action;
unsigned long action_timestamp;
Expand Down Expand Up @@ -134,25 +135,30 @@ static void hangcheck_load_sample(struct intel_engine_cs *engine,
{
hc->acthd = intel_engine_get_active_head(engine);
hc->seqno = intel_engine_get_hangcheck_seqno(engine);
hc->ring = ENGINE_READ(engine, RING_START);
}

static void hangcheck_store_sample(struct intel_engine_cs *engine,
const struct hangcheck *hc)
{
engine->hangcheck.acthd = hc->acthd;
engine->hangcheck.last_seqno = hc->seqno;
engine->hangcheck.last_ring = hc->ring;
}

static enum intel_engine_hangcheck_action
hangcheck_get_action(struct intel_engine_cs *engine,
const struct hangcheck *hc)
{
if (engine->hangcheck.last_seqno != hc->seqno)
return ENGINE_ACTIVE_SEQNO;

if (intel_engine_is_idle(engine))
return ENGINE_IDLE;

if (engine->hangcheck.last_ring != hc->ring)
return ENGINE_ACTIVE_SEQNO;

if (engine->hangcheck.last_seqno != hc->seqno)
return ENGINE_ACTIVE_SEQNO;

return engine_stuck(engine, hc->acthd);
}

Expand Down

0 comments on commit f410776

Please sign in to comment.