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

Yang, Fei fei.yang at intel.com
Wed Oct 16 22:39:34 UTC 2024


> On Wed, 2024-10-16 at 11:57 -0700, fei.yang at intel.com wrote:
>> From: Fei Yang <fei.yang at 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.

Sorry, not sure what exactly you are looking for, but anyway I sent the
patch again with updated commit message, hope that would be a bit more
clear.

-Fei

>>
>> Signed-off-by: Fei Yang <fei.yang at intel.com>
>> Cc: John Harrison <John.C.Harrison at 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


More information about the Intel-xe mailing list