Re: [PATCH] drm/xe: Remove debug message from migrate_clear()

[Matthew Brost <matthew.brost@intel.com>]

On Fri, Apr 05, 2024 at 08:38:49AM -0700, José Roberto de Souza wrote:
> This messages is printed a lot and from my understanding it do not
> bring any value, so here dropping it.
> 
> Signed-off-by: José Roberto de Souza <jose.souza@intel.com>

We want to drop the similar messages in xe_migrate_copy too but this LGTM.

With that:
Reviewed-by: Matthew Brost <matthew.brost@intel.com>

> ---
>  drivers/gpu/drm/xe/xe_migrate.c | 3 ---
>  1 file changed, 3 deletions(-)
> 
> diff --git a/drivers/gpu/drm/xe/xe_migrate.c b/drivers/gpu/drm/xe/xe_migrate.c
> index 524b0198fcae3..2e37cbe35d16e 100644
> --- a/drivers/gpu/drm/xe/xe_migrate.c
> +++ b/drivers/gpu/drm/xe/xe_migrate.c
> @@ -984,7 +984,6 @@ struct dma_fence *xe_migrate_clear(struct xe_migrate *m,
>  	struct xe_res_cursor src_it;
>  	struct ttm_resource *src = dst;
>  	int err;
> -	int pass = 0;
>  
>  	if (!clear_vram)
>  		xe_res_first_sg(xe_bo_sg(bo), 0, bo->size, &src_it);
> @@ -1005,8 +1004,6 @@ struct dma_fence *xe_migrate_clear(struct xe_migrate *m,
>  
>  		clear_L0 = xe_migrate_res_sizes(m, &src_it);
>  
> -		drm_dbg(&xe->drm, "Pass %u, size: %llu\n", pass++, clear_L0);
> -
>  		/* Calculate final sizes and batch size.. */
>  		batch_size = 2 +
>  			pte_update_size(m, clear_vram, src, &src_it,
> -- 
> 2.44.0
>