Re: [PATCH] drm/xe: Disable garbage collector work item on SVM close

["Thomas Hellström" <thomas.hellstrom@linux.intel.com>]

On Thu, 2026-02-26 at 17:52 -0800, Matthew Brost wrote:
> When an SVM is closed, the garbage collector work item must be
> stopped
> synchronously and any future queuing must be prevented. Replace
> flush_work() with disable_work_sync() to ensure both conditions are
> met.
> 
> Fixes: 63f6e480d115 ("drm/xe: Add SVM garbage collector")
> Cc: stable@vger.kernel.org
> Signed-off-by: Matthew Brost <matthew.brost@intel.com>

Reviewed-by: Thomas Hellström <thomas.hellstrom@linux.intel.com>


> ---
>  drivers/gpu/drm/xe/xe_svm.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/drivers/gpu/drm/xe/xe_svm.c
> b/drivers/gpu/drm/xe/xe_svm.c
> index 002b6c22ad3f..f8b57eca76fd 100644
> --- a/drivers/gpu/drm/xe/xe_svm.c
> +++ b/drivers/gpu/drm/xe/xe_svm.c
> @@ -903,7 +903,7 @@ int xe_svm_init(struct xe_vm *vm)
>  void xe_svm_close(struct xe_vm *vm)
>  {
>  	xe_assert(vm->xe, xe_vm_is_closed(vm));
> -	flush_work(&vm->svm.garbage_collector.work);
> +	disable_work_sync(&vm->svm.garbage_collector.work);
>  	xe_svm_put_pagemaps(vm);
>  	drm_pagemap_release_owner(&vm->svm.peer);
>  }