The third patch in the series attempts to downgrade the check for is_guc_running to is_fw_available within intel_uc_fini_hw. We cannot rely on is_guc_running because we will completely skip our attempt to fini uC hw during unload. However, this exposes a new set of problems due to the driver being able to continue after GuC initialization failures: we can now try to disable submission or communication that was never enabled in the first place.
The first two patches attempt to resolve those issues by adding (hopefully stronger) checks around submission_disable and communication_disable. Thanks, Fernando Cc: Chris Wilson <ch...@chris-wilson.co.uk> Cc: Michal Wajdeczko <michal.wajdec...@intel.com> Cc: Daniele Ceraolo Spurio <daniele.ceraolospu...@intel.com> Fernando Pacheco (3): drm/i915/uc: Extract common code from GuC stop/disable comm drm/i915/uc: Disable GuC submission only if currently enabled drm/i915/uc: Fini hw even if GuC is not running .../gpu/drm/i915/gt/uc/intel_guc_submission.c | 23 +++++++++++++ .../gpu/drm/i915/gt/uc/intel_guc_submission.h | 1 + drivers/gpu/drm/i915/gt/uc/intel_uc.c | 34 +++++++++++-------- 3 files changed, 44 insertions(+), 14 deletions(-) -- 2.23.0 _______________________________________________ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/intel-gfx
