Quoting Tvrtko Ursulin (2019-06-10 14:01:10) > From: Tvrtko Ursulin <[email protected]> > > It seems intel_engine_get_instdone is able to get instdone for all engines > but intel_hangcheck.c/subunits_stuck decides to ignore it for non render. > > We can just drop the check in subunits_stuck since the checks on > unavailable fields will always return stuck, which when bitwise and with > the potential unstuck instdone is harmless. > > Signed-off-by: Tvrtko Ursulin <[email protected]> > Cc: Chris Wilson <[email protected]> > Cc: Mika Kuoppala <[email protected]> > --- > Would actually using the available data improve hang detection?
I got hung up on this question, and didn't address the patch on its merits. Reviewed-by: Chris Wilson <[email protected]> -Chris _______________________________________________ Intel-gfx mailing list [email protected] https://lists.freedesktop.org/mailman/listinfo/intel-gfx
