[PATCH 1/2] drm/ttm: fix the warning for hit_low and evict_low

Tvrtko Ursulin tvrtko.ursulin at igalia.com
Thu Apr 17 11:31:19 UTC 2025


On 17/04/2025 10:34, Sunil Khatri wrote:
> fix the below warning messages:
> ttm/ttm_bo.c:1098: warning: Function parameter or struct member 'hit_low' not described in 'ttm_bo_swapout_walk'
> ttm/ttm_bo.c:1098: warning: Function parameter or struct member 'evict_low' not described in 'ttm_bo_swapout_walk'
> 
> Signed-off-by: Sunil Khatri <sunil.khatri at amd.com>
> ---
>   drivers/gpu/drm/ttm/ttm_bo.c | 3 ++-
>   1 file changed, 2 insertions(+), 1 deletion(-)
> 
> diff --git a/drivers/gpu/drm/ttm/ttm_bo.c b/drivers/gpu/drm/ttm/ttm_bo.c
> index 95b86003c50d..33fefdcc6323 100644
> --- a/drivers/gpu/drm/ttm/ttm_bo.c
> +++ b/drivers/gpu/drm/ttm/ttm_bo.c
> @@ -1093,7 +1093,8 @@ struct ttm_bo_swapout_walk {
>   	struct ttm_lru_walk walk;
>   	/** @gfp_flags: The gfp flags to use for ttm_tt_swapout() */
>   	gfp_t gfp_flags;
> -
> +	/** @hit_low: flag to mark low */
> +	/** @evict_low:  flag for low eviction */
>   	bool hit_low, evict_low;
>   };
>   

Hm, could you check if you could copy and adapt more detailed comments 
from struct ttm_bo_evict_walk?

	/** @try_low: Whether we should attempt to evict BO's with low 
watermark threshold */
	bool try_low;
	/** @hit_low: If we cannot evict a bo when @try_low is false (first 
pass) */
	bool hit_low;
};

Regards,

Tvrtko



More information about the amd-gfx mailing list