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

["Ghimiray, Himal Prasad" <himal.prasad.ghimiray@intel.com>]

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>


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