Re: [PATCH v2] drm/xe: Don't trigger rebind on initial dma-buf validation

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

On Mon, 2025-08-25 at 08:28 -0700, Matthew Brost wrote:
> On the first validate of an imported dma-buf (initial bind), the
> device
> has no GPU mappings, so a rebind is unnecessary. Rebinding here is
> harmful in multi-GPU setups and for VMs using preempt-fence mode, as
> it
> would evict in-flight GPU work.
> 
> v2:
>  - Drop dma_buf_validated, check for XE_PL_SYSTEM (Thomas)
> 
> Signed-off-by: Matthee Brost <matthew.brost@intel.com>
> ---
>  drivers/gpu/drm/xe/xe_bo.c | 3 ++-
>  1 file changed, 2 insertions(+), 1 deletion(-)
> 
> diff --git a/drivers/gpu/drm/xe/xe_bo.c b/drivers/gpu/drm/xe/xe_bo.c
> index 6fea39842e1e..a0630e120cd2 100644
> --- a/drivers/gpu/drm/xe/xe_bo.c
> +++ b/drivers/gpu/drm/xe/xe_bo.c
> @@ -816,7 +816,8 @@ static int xe_bo_move(struct ttm_buffer_object
> *ttm_bo, bool evict,
>  	}
>  
>  	if (ttm_bo->type == ttm_bo_type_sg) {
> -		ret = xe_bo_move_notify(bo, ctx);
> +		if (new_mem->mem_type == XE_PL_SYSTEM)
> +			ret = xe_bo_move_notify(bo, ctx);
>  		if (!ret)
>  			ret = xe_bo_move_dmabuf(ttm_bo, new_mem);
>  		return ret;

LGTM. If CI is happy,

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