Re: [PATCH v1] drm/xe/guc: Increase GuC log sizes in debug builds

[Daniele Ceraolo Spurio <daniele.ceraolospurio@intel.com>]

On 2/13/2026 6:00 AM, Tomasz Lis wrote:
> Increase event log size for GuC debug to 16MB, and for general debug
> to 8MB. This allows for useful debug even if performance-affecting
> DRM_XE_DEBUG_GUC is not enabled.
>
> Without this change, GuC logs gathered by CI are useless for debug
> due to limited size, which translates to time frame not even able
> to cover cleanup after test.
>
> Signed-off-by: Tomasz Lis <tomasz.lis@intel.com>
> Cc: Michal Wajdeczko <michal.wajdeczko@intel.com>
> Cc: Daniele Ceraolo Spurio <daniele.ceraolospurio@intel.com>
> Cc: Matthew Brost <matthew.brost@intel.com>
> Cc: Matt Roper <matthew.d.roper@intel.com>
> ---
>   drivers/gpu/drm/xe/xe_guc_log.h | 6 +++++-
>   1 file changed, 5 insertions(+), 1 deletion(-)
>
> diff --git a/drivers/gpu/drm/xe/xe_guc_log.h b/drivers/gpu/drm/xe/xe_guc_log.h
> index 1b05bb60c1c7..4649a260755e 100644
> --- a/drivers/gpu/drm/xe/xe_guc_log.h
> +++ b/drivers/gpu/drm/xe/xe_guc_log.h
> @@ -13,9 +13,13 @@ struct drm_printer;
>   struct xe_device;
>   
>   #if IS_ENABLED(CONFIG_DRM_XE_DEBUG_GUC)
> -#define XE_GUC_LOG_EVENT_DATA_BUFFER_SIZE	SZ_8M
> +#define XE_GUC_LOG_EVENT_DATA_BUFFER_SIZE	SZ_16M
>   #define XE_GUC_LOG_CRASH_DUMP_BUFFER_SIZE	SZ_1M
>   #define XE_GUC_LOG_STATE_CAPTURE_BUFFER_SIZE	SZ_2M
> +#elif IS_ENABLED(CONFIG_DRM_XE_DEBUG)
> +#define XE_GUC_LOG_EVENT_DATA_BUFFER_SIZE	SZ_8M

Not sure if this is too big if we ever need to dump the guc log in dmesg 
in CI (AFAIK CI has a 2MB limit for dmesg, but we compress the log 
before dumping). We can tune it down later if we ever have issues, so 
I'm ok with going with 8MB for now.

Reviewed-by: Daniele Ceraolo Spurio <daniele.ceraolospurio@intel.com>

Daniele

> +#define XE_GUC_LOG_CRASH_DUMP_BUFFER_SIZE	SZ_1M
> +#define XE_GUC_LOG_STATE_CAPTURE_BUFFER_SIZE	SZ_1M
>   #else
>   #define XE_GUC_LOG_EVENT_DATA_BUFFER_SIZE	SZ_64K
>   #define XE_GUC_LOG_CRASH_DUMP_BUFFER_SIZE	SZ_16K