[Intel-gfx] [PATCH 10/34] drm/i915: Remove GPU reset dependence on struct_mutex
Chris Wilson
chris at chris-wilson.co.uk
Thu Jan 24 14:10:40 UTC 2019
Quoting Chris Wilson (2019-01-24 12:50:30)
> Quoting Mika Kuoppala (2019-01-24 12:06:01)
> > Chris Wilson <chris at chris-wilson.co.uk> writes:
> > > - /*
> > > - * We want to perform per-engine reset from atomic context (e.g.
> > > - * softirq), which imposes the constraint that we cannot sleep.
> > > - * However, experience suggests that spending a bit of time waiting
> > > - * for a reset helps in various cases, so for a full-device reset
> > > - * we apply the opposite rule and wait if we want to. As we should
> > > - * always follow up a failed per-engine reset with a full device reset,
> > > - * being a little faster, stricter and more error prone for the
> > > - * atomic case seems an acceptable compromise.
> > > - *
> > > - * Unfortunately this leads to a bimodal routine, when the goal was
> > > - * to have a single reset function that worked for resetting any
> > > - * number of engines simultaneously.
> > > - */
> > > - might_sleep_if(engine_mask == ALL_ENGINES);
> >
> > Oh here it is. I was after this on atomic resets.
>
> I was saying it didn't make sense to lift the restriction until we
> relied upon. Just because the code became safe doesn't mean it was then
> part of the API :)
Hmm, set-wedged is meant to be magical more or less now. That gives more
weight to the argument of making intel_gpu_reset() magical and removing
this comment earlier.
-Chris
More information about the Intel-gfx
mailing list