Re: [PATCH 3/4] drm/xe: Mark G2H work queue with WQ_MEM_RECLAIM

["Nilawar, Badal" <badal.nilawar@intel.com>]

On 22-10-2024 10:36, Ghimiray, Himal Prasad wrote:
> 
> 
> On 21-10-2024 23:27, Matthew Brost wrote:
>> G2H work queue can be used to free memory thus we should allow this work
>> queue to run during reclaim. Mark with G2H work queue with
>> WQ_MEM_RECLAIM appropriately.
>>
>> Signed-off-by: Matthew Brost <matthew.brost@intel.com>
>> ---
>>   drivers/gpu/drm/xe/xe_guc_ct.c | 2 +-
>>   1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> diff --git a/drivers/gpu/drm/xe/xe_guc_ct.c b/drivers/gpu/drm/xe/ 
>> xe_guc_ct.c
>> index c260d8840990..1b5d8fb1033a 100644
>> --- a/drivers/gpu/drm/xe/xe_guc_ct.c
>> +++ b/drivers/gpu/drm/xe/xe_guc_ct.c
>> @@ -213,7 +213,7 @@ int xe_guc_ct_init(struct xe_guc_ct *ct)
>>       xe_gt_assert(gt, !(guc_ct_size() % PAGE_SIZE));
>> -    ct->g2h_wq = alloc_ordered_workqueue("xe-g2h-wq", 0);
>> +    ct->g2h_wq = alloc_ordered_workqueue("xe-g2h-wq", WQ_MEM_RECLAIM);
> 
> LGTM
> Reviewed-by: Himal Prasad Ghimiray <himal.prasad.ghimiray@intel.com>

Some how this patch is not reflecting in my mailbox, so replying here.
Reviewed-by: Badal Nilawar <badal.nilawar@intel.com>

Regards,
Badal

> 
> 
>>       if (!ct->g2h_wq)
>>           return -ENOMEM;
>