Re: [PATCH] drm/ttm: Skip swapout if ttm object is not populated

[Christian König <ckoenig.leichtzumerken@xxxxxxxxx>]

Am 21.05.21 um 10:31 schrieb xinhui pan:
> Swapping a ttm object which has no backend pages makes no sense.
>
> Suggested-by: Christian König <christian.koenig@xxxxxxx>
> Signed-off-by: xinhui pan <xinhui.pan@xxxxxxx>

Reviewed-by: Christian König <christian.koenig@xxxxxxx>

Going to add a CC: stable and pushing that to drm-misc-fixes in a minute.

> ---
>   drivers/gpu/drm/ttm/ttm_device.c | 2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/gpu/drm/ttm/ttm_device.c b/drivers/gpu/drm/ttm/ttm_device.c
> index 510e3e001dab..a1dcf7d55c90 100644
> --- a/drivers/gpu/drm/ttm/ttm_device.c
> +++ b/drivers/gpu/drm/ttm/ttm_device.c
> @@ -145,7 +145,7 @@ int ttm_device_swapout(struct ttm_device *bdev, struct ttm_operation_ctx *ctx,
>   			list_for_each_entry(bo, &man->lru[j], lru) {
>   				uint32_t num_pages;
>   
> -				if (!bo->ttm ||
> +				if (!bo->ttm || !ttm_tt_is_populated(bo->ttm) ||
>   				    bo->ttm->page_flags & TTM_PAGE_FLAG_SG ||
>   				    bo->ttm->page_flags & TTM_PAGE_FLAG_SWAPPED)
>   					continue;