On Tue, Feb 23, 2021 at 10:28 PM xinhui pan xinhui.pan@amd.com wrote:

BO would be added into swap list if it is validated into system domain. If BO is validated again into non-system domain, say, VRAM domain. It actually should not be in the swap list.

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

Acked-by: Alex Deucher alexander.deucher@amd.com

---

drivers/gpu/drm/ttm/ttm_bo.c | 2 ++ 1 file changed, 2 insertions(+)

diff --git a/drivers/gpu/drm/ttm/ttm_bo.c b/drivers/gpu/drm/ttm/ttm_bo.c index a97d41f4ce3c..3a10bebb75d6 100644 --- a/drivers/gpu/drm/ttm/ttm_bo.c +++ b/drivers/gpu/drm/ttm/ttm_bo.c @@ -111,6 +111,8 @@ void ttm_bo_move_to_lru_tail(struct ttm_buffer_object *bo,

            swap = &ttm_glob.swap_lru[bo->priority];
            list_move_tail(&bo->swap, swap);

•   } else {
  

•           list_del_init(&bo->swap);
    }
  
    if (bdev->funcs->del_from_lru_notify)
  

-- 2.25.1

---

dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel

Am 24.02.21 um 04:28 schrieb xinhui pan:

BO would be added into swap list if it is validated into system domain. If BO is validated again into non-system domain, say, VRAM domain. It actually should not be in the swap list.

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

Reviewed-by: Christian König christian.koenig@amd.com

Going to push that to drm-misc-fixes tomorrow.

Thanks, Christian.

---

drivers/gpu/drm/ttm/ttm_bo.c | 2 ++ 1 file changed, 2 insertions(+)

diff --git a/drivers/gpu/drm/ttm/ttm_bo.c b/drivers/gpu/drm/ttm/ttm_bo.c index a97d41f4ce3c..3a10bebb75d6 100644 --- a/drivers/gpu/drm/ttm/ttm_bo.c +++ b/drivers/gpu/drm/ttm/ttm_bo.c @@ -111,6 +111,8 @@ void ttm_bo_move_to_lru_tail(struct ttm_buffer_object *bo,

swap = &ttm_glob.swap_lru[bo->priority];
list_move_tail(&bo->swap, swap);

• } else {

• list_del_init(&bo->swap);
  

  }

  if (bdev->funcs->del_from_lru_notify)