Re: [RFC PATCH 2/2] drm/ttm: skip swapout when ttm has no backend page.

Am 19.05.21 um 04:28 schrieb xinhui pan:

Signed-off-by: xinhui pan xinhui.pan@amd.com

drivers/gpu/drm/ttm/ttm_device.c | 1 + 1 file changed, 1 insertion(+)

diff --git a/drivers/gpu/drm/ttm/ttm_device.c b/drivers/gpu/drm/ttm/ttm_device.c index 510e3e001dab..a9772fcc8f9c 100644 --- a/drivers/gpu/drm/ttm/ttm_device.c +++ b/drivers/gpu/drm/ttm/ttm_device.c @@ -146,6 +146,7 @@ int ttm_device_swapout(struct ttm_device *bdev, struct ttm_operation_ctx *ctx, uint32_t num_pages;

		if (!bo->ttm ||

• 		    !bo->ttm->pages || !bo->ttm->pages[0] ||
  

This should call ttm_tt_is_populated() instead.

But apart from that it sounds like the right approach to fix this.

Thanks, Christian.

		    bo->ttm->page_flags & TTM_PAGE_FLAG_SG ||
		    bo->ttm->page_flags & TTM_PAGE_FLAG_SWAPPED)
			continue;