Quoting Michał Winiarski (2018-07-19 10:10:22) > On Tue, Jul 17, 2018 at 09:29:32PM +0100, Chris Wilson wrote: > > We make a decision at module load whether to use the GuC backend or not, > > but lose that setup across set-wedge. Currently, the guc doesn't > > override the engine->set_default_submission hook letting execlists sneak > > back in temporarily on unwedging leading to an unbalanced park/unpark. > > > > v2: Remove comment about switching back temporarily to execlists on > > guc_submission_disable(). We currently only call disable on shutdown, > > and plan to also call disable before suspend and reset, in which case we > > will either restore guc submission or mark the driver as wedged, making > > the reset back to execlists pointless. > > v3: Move reset.prepare across > > > > Testcase: igt/drv_module_reload/basic-reload-inject > > Testcase: igt/gem_eio > > Signed-off-by: Chris Wilson <chris@xxxxxxxxxxxxxxxxxx> > > Cc: Michał Winiarski <michal.winiarski@xxxxxxxxx> > > Cc: Michal Wajdeczko <michal.wajdeczko@xxxxxxxxx> > > Reviewed-by: Michał Winiarski <michal.winiarski@xxxxxxxxx> Ta, and with everything that purports to fix the elusive basic-reload-inject, I now need to find an igt patch and victim. -Chris _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/intel-gfx
