Re: 回复: [PATCH] drm/ttm: Put BO in its memory manager's lru list

2022-01-11 Thread Christian König
ct: Re: 回复: [PATCH] drm/ttm: Put BO in its memory manager's lru list IIRC we have completely dropped this patch in favor of a check at a different place. Regards, Christian. Am 11.01.22 um 09:47 schrieb Chen, Guchun: [Public] Hi Christian, Looks this patch still missed in 5.16 kernel. Is

RE: 回复: [PATCH] drm/ttm: Put BO in its memory manager's lru list

2022-01-11 Thread Chen, Guchun
- From: Christian König Sent: Tuesday, January 11, 2022 7:27 PM To: Chen, Guchun ; Pan, Xinhui ; Koenig, Christian ; amd-...@lists.freedesktop.org Cc: dri-devel@lists.freedesktop.org Subject: Re: 回复: [PATCH] drm/ttm: Put BO in its memory manager's lru list IIRC we have completely droppe

Re: 回复: [PATCH] drm/ttm: Put BO in its memory manager's lru list

2022-01-11 Thread Christian König
__ 发件人: Koenig, Christian 发送时间: 2021年11月9日 20:35 收件人: Pan, Xinhui; amd-...@lists.freedesktop.org 抄送: dri-devel@lists.freedesktop.org 主题: Re: 回复: [PATCH] drm/ttm: Put BO in its memory manager's lru list Mhm, I'm not sure what the rational behind that is. Not moving the BO would make things les

RE: 回复: [PATCH] drm/ttm: Put BO in its memory manager's lru list

2022-01-11 Thread Chen, Guchun
, November 9, 2021 9:16 PM To: Koenig, Christian ; amd-...@lists.freedesktop.org Cc: dri-devel@lists.freedesktop.org Subject: 回复: 回复: [PATCH] drm/ttm: Put BO in its memory manager's lru list [AMD Official Use Only] [AMD Official Use Only] Actually this patch does not totally fix the mismat

Re: 回复: 回复: 回复: [PATCH] drm/ttm: Put BO in its memory manager's lru list

2021-11-09 Thread Christian König
g, Christian 发送时间: 2021年11月9日 21:18 收件人: Pan, Xinhui; amd-...@lists.freedesktop.org 抄送: dri-devel@lists.freedesktop.org 主题: Re: 回复: 回复: [PATCH] drm/ttm: Put BO in its memory manager's lru list Exactly that's the reason why we should have the double check in TTM I've mentioned in t

回复: 回复: 回复: [PATCH] drm/ttm: Put BO in its memory manager's lru list

2021-11-09 Thread Pan, Xinhui
Christian 发送时间: 2021年11月9日 21:18 收件人: Pan, Xinhui; amd-...@lists.freedesktop.org 抄送: dri-devel@lists.freedesktop.org 主题: Re: 回复: 回复: [PATCH] drm/ttm: Put BO in its memory manager's lru list Exactly that's the reason why we should have the double check in TTM I've mentioned in t

Re: 回复: 回复: [PATCH] drm/ttm: Put BO in its memory manager's lru list

2021-11-09 Thread Christian König
hristian; amd-...@lists.freedesktop.org 抄送: dri-devel@lists.freedesktop.org 主题: 回复: 回复: [PATCH] drm/ttm: Put BO in its memory manager's lru list Yes, a stable tag is needed. vulkan guys say 5.14 hit this issue too. I think that amdgpu_bo_move() does support copy from sysMem to sysMem correctl

Re: 回复: 回复: [PATCH] drm/ttm: Put BO in its memory manager's lru list

2021-11-09 Thread Christian König
__ 发件人: Koenig, Christian 发送时间: 2021年11月9日 20:35 收件人: Pan, Xinhui; amd-...@lists.freedesktop.org 抄送: dri-devel@lists.freedesktop.org 主题: Re: 回复: [PATCH] drm/ttm: Put BO in its memory manager's lru list Mhm, I'm not sure what the rational behind that is. Not moving the BO wou

回复: 回复: [PATCH] drm/ttm: Put BO in its memory manager's lru list

2021-11-09 Thread Pan, Xinhui
ist is on vram domain) to sMem. 发件人: Pan, Xinhui 发送时间: 2021年11月9日 21:05 收件人: Koenig, Christian; amd-...@lists.freedesktop.org 抄送: dri-devel@lists.freedesktop.org 主题: 回复: 回复: [PATCH] drm/ttm: Put BO in its memory manager's lru list Yes, a stable tag i

回复: 回复: [PATCH] drm/ttm: Put BO in its memory manager's lru list

2021-11-09 Thread Pan, Xinhui
发送时间: 2021年11月9日 20:35 收件人: Pan, Xinhui; amd-...@lists.freedesktop.org 抄送: dri-devel@lists.freedesktop.org 主题: Re: 回复: [PATCH] drm/ttm: Put BO in its memory manager's lru list Mhm, I'm not sure what the rational behind that is. Not moving the BO would make things less efficient, but should never c

Re: 回复: [PATCH] drm/ttm: Put BO in its memory manager's lru list

2021-11-09 Thread Christian König
: Pan, Xinhui; amd-...@lists.freedesktop.org 抄送: dri-devel@lists.freedesktop.org 主题: Re: [PATCH] drm/ttm: Put BO in its memory manager's lru list Am 09.11.21 um 12:19 schrieb xinhui pan: After we move BO to a new memory region, we should put it to the new memory manager's lru list regar

回复: [PATCH] drm/ttm: Put BO in its memory manager's lru list

2021-11-09 Thread Pan, Xinhui
, Christian 发送时间: 2021年11月9日 20:20 收件人: Pan, Xinhui; amd-...@lists.freedesktop.org 抄送: dri-devel@lists.freedesktop.org 主题: Re: [PATCH] drm/ttm: Put BO in its memory manager's lru list Am 09.11.21 um 12:19 schrieb xinhui pan: > After we move BO to a new memory region, we should put it to > the

Re: [PATCH] drm/ttm: Put BO in its memory manager's lru list

2021-11-09 Thread Christian König
Am 09.11.21 um 12:19 schrieb xinhui pan: After we move BO to a new memory region, we should put it to the new memory manager's lru list regardless we unlock the resv or not. Signed-off-by: xinhui pan Interesting find, did you trigger that somehow or did you just stumbled over it by reading t

[PATCH] drm/ttm: Put BO in its memory manager's lru list

2021-11-09 Thread xinhui pan
After we move BO to a new memory region, we should put it to the new memory manager's lru list regardless we unlock the resv or not. Signed-off-by: xinhui pan --- 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