Chris Wilson <ch...@chris-wilson.co.uk> writes: > On Tue, Dec 01, 2015 at 05:56:12PM +0200, Mika Kuoppala wrote: >> If head seems stuck and engine in question is rcs, >> inspect subunit state transitions from undone to done, >> before deciding that this really is a hang instead of limited >> progress. Only account the transitions of subunits from >> undone to done once, to prevent unstable subunit states >> to keep us falsely active. >> >> As this adds one extra steps to hangcheck heuristics, >> before hang is declared, it adds 1500ms to to detect hang >> for render ring to a total of 7500ms. We could sample >> the subunit states on first head stuck condition but >> decide not to do so only in order to mimic old behaviour. This >> way the check order of promotion from seqno > atchd > instdone >> is consistently done. >> >> v2: Deal with unstable done states (Arun) >> Clear instdone progress on head and seqno movement (Chris) >> Report raw and accumulated instdone's in in debugfs (Chris) >> Return HANGCHECK_ACTIVE on undone->done >> >> References: https://bugs.freedesktop.org/show_bug.cgi?id=93029 >> Cc: Chris Wilson <ch...@chris-wilson.co.uk> >> Cc: Dave Gordon <david.s.gor...@intel.com> >> Cc: Daniel Vetter <dan...@ffwll.ch> >> Cc: Arun Siluvery <arun.siluv...@linux.intel.com> >> Signed-off-by: Mika Kuoppala <mika.kuopp...@intel.com> > > I feel slightly dubious in discarding the 1->0 transitions (as it just > means that a shared function that was previously idle is now in use > again), but if they truly do fluctuate randomly? then accumulating > should mean we eventually escape.
Atleast with a tight bb start loop inside one batch one bit fluctulated. We could improve further by having maximum amount of transitions per bit? > > Reviewed-by: Chris Wilson <ch...@chris-wilson.co.uk> > -Chris Thanks, -Mika > > -- > Chris Wilson, Intel Open Source Technology Centre _______________________________________________ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/intel-gfx
