@@ -63,7 +63,9 @@ static int __get_platform_enable_guc(struct drm_i915_private *i915)
if (intel_uc_fw_is_selected(huc_fw))
enable_guc |= ENABLE_GUC_LOAD_HUC;
- /* Any platform specific fine-tuning can be done here */
+ /* Don't allow GuC submission on pre-Gen11 */
+ if (INTEL_GEN(i915) < 11)
+ enable_guc &= ~ENABLE_GUC_SUBMISSION;
return enable_guc;
}
@@ -116,6 +118,13 @@ static void sanitize_options_early(struct drm_i915_private *i915)
yesno(intel_uc_is_using_guc_submission(i915)),
yesno(intel_uc_is_using_huc(i915)));
+ /* Verify GuC submission support */
+ if (intel_uc_is_using_guc_submission(i915) && INTEL_GEN(i915) < 11) {
+ DRM_WARN("Incompatible option detected: %s=%d, %s!\n",
+ "enable_guc", i915_modparams.enable_guc,
+ "submission not supported");
+ }
+
/* Verify GuC firmware availability */
if (intel_uc_is_using_guc(i915) && !intel_uc_fw_is_selected(guc_fw)) {
DRM_WARN("Incompatible option detected: %s=%d, %s!\n",
@@ -288,6 +297,10 @@ int intel_uc_init(struct drm_i915_private *i915)
if (!HAS_GUC(i915))
return -ENODEV;
+ /* Don't allow GuC submission on pre-Gen11 */
+ if (USES_GUC_SUBMISSION(i915) && INTEL_GEN(i915) < 11)
+ return -EIO;
+
ret = intel_guc_init(guc);
if (ret)
return ret;
Due to the GuC interface changes, new firmware releases will stop support GuC submission mode for pre-Gen11 platforms. Sanitize the enable_guc option so that only HuC authentication would be possible. Signed-off-by: Michal Wajdeczko <michal.wajdeczko@intel.com> Cc: Joonas Lahtinen <joonas.lahtinen@linux.intel.com> Cc: Rodrigo Vivi <rodrigo.vivi@intel.com> Cc: Daniele Ceraolo Spurio <daniele.ceraolospurio@intel.com> Cc: John Spotswood <john.a.spotswood@intel.com> Cc: Vinay Belgaumkar <vinay.belgaumkar@intel.com> Cc: Tony Ye <tony.ye@intel.com> Cc: Anusha Srivatsa <anusha.srivatsa@intel.com> Cc: Jeff Mcgee <jeff.mcgee@intel.com> Cc: Antonio Argenziano <antonio.argenziano@intel.com> Cc: Sujaritha Sundaresan <sujaritha.sundaresan@intel.com> --- drivers/gpu/drm/i915/intel_uc.c | 15 ++++++++++++++- 1 file changed, 14 insertions(+), 1 deletion(-)