[PATCH] drm/xe/gsc: Make GSCCS disabling message less alarming
John Harrison
john.c.harrison at intel.com
Thu Dec 19 01:51:25 UTC 2024
On 12/18/2024 17:09, Daniele Ceraolo Spurio wrote:
> The GSCCS is only used to transport messages from the driver to the GSC
> FW; therefore, we can disable it if we're not using the FW, which is
> the case on both BMG and PTL. However, the current wording of the logged
> message could be interpreted as a problem, so reword to make it clearer
> it is not an error and lower it debug verbosity as users don't really
> need to know about it.
>
> Reported-by: Eero Tamminen <eero.t.tamminen at intel.com>
> Closes: https://gitlab.freedesktop.org/drm/xe/kernel/-/issues/3866
> Signed-off-by: Daniele Ceraolo Spurio <daniele.ceraolospurio at intel.com>
> ---
> drivers/gpu/drm/xe/xe_hw_engine.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/gpu/drm/xe/xe_hw_engine.c b/drivers/gpu/drm/xe/xe_hw_engine.c
> index 4294aa600192..ac9c666a9652 100644
> --- a/drivers/gpu/drm/xe/xe_hw_engine.c
> +++ b/drivers/gpu/drm/xe/xe_hw_engine.c
> @@ -775,7 +775,7 @@ static void check_gsc_availability(struct xe_gt *gt)
> xe_mmio_write32(>->mmio, GUNIT_GSC_INTR_ENABLE, 0);
> xe_mmio_write32(>->mmio, GUNIT_GSC_INTR_MASK, ~0);
>
> - drm_info(&xe->drm, "gsccs disabled due to lack of FW\n");
> + drm_dbg(&xe->drm, "GSC FW not used, disabling gsccs\n");
This message would also be hit in the case where the firmware is
supposed to be present but isn't. E.g. on a LNL but without the right
firmware package installed? But presumably in that case, there would
already have been an earlier message about missing firmware files?
John.
> }
> }
>
More information about the Intel-xe
mailing list