Re: [PATCH 1/1] drm/xe: enable lite restore

["Souza, Jose" <jose.souza@intel.com>]

On Wed, 2024-10-16 at 11:57 -0700, fei.yang@intel.com wrote:
> From: Fei Yang <fei.yang@intel.com>
> 
> The lite restore is a performance improvement feature which avoids
> unnecessary full context save and restore if the task been switched
> in is the same as the one been switched out. This is supposed to be
> enabled by default.

in the same what? engine?
please append that to commit message.

> 
> Signed-off-by: Fei Yang <fei.yang@intel.com>
> Cc: John Harrison <John.C.Harrison@Intel.com>
> ---
>  drivers/gpu/drm/xe/xe_guc.c      | 2 +-
>  drivers/gpu/drm/xe/xe_guc_fwif.h | 1 +
>  2 files changed, 2 insertions(+), 1 deletion(-)
> 
> diff --git a/drivers/gpu/drm/xe/xe_guc.c b/drivers/gpu/drm/xe/xe_guc.c
> index 8570b1218287..fed30f2872f7 100644
> --- a/drivers/gpu/drm/xe/xe_guc.c
> +++ b/drivers/gpu/drm/xe/xe_guc.c
> @@ -70,7 +70,7 @@ static u32 guc_ctl_debug_flags(struct xe_guc *guc)
>  
>  static u32 guc_ctl_feature_flags(struct xe_guc *guc)
>  {
> -	u32 flags = 0;
> +	u32 flags = GUC_CTL_ENABLE_LITE_RESTORE;
>  
>  	if (!guc_to_xe(guc)->info.skip_guc_pc)
>  		flags |= GUC_CTL_ENABLE_SLPC;
> diff --git a/drivers/gpu/drm/xe/xe_guc_fwif.h b/drivers/gpu/drm/xe/xe_guc_fwif.h
> index 01e3ab590c3a..08ffe59f22fa 100644
> --- a/drivers/gpu/drm/xe/xe_guc_fwif.h
> +++ b/drivers/gpu/drm/xe/xe_guc_fwif.h
> @@ -105,6 +105,7 @@ struct guc_update_exec_queue_policy {
>  
>  #define GUC_CTL_FEATURE			2
>  #define   GUC_CTL_ENABLE_SLPC		BIT(2)
> +#define   GUC_CTL_ENABLE_LITE_RESTORE	BIT(4)
>  #define   GUC_CTL_DISABLE_SCHEDULER	BIT(14)
>  
>  #define GUC_CTL_DEBUG			3