Re: [PATCH 1/2] drm/ttm: Don't evict SG BOs

2021-04-28 Thread Felix Kuehling
Am 2021-04-28 um 12:58 p.m. schrieb Christian König: > Am 28.04.21 um 18:49 schrieb Felix Kuehling: >> Am 2021-04-28 um 12:33 p.m. schrieb Christian König: >>> Am 28.04.21 um 17:19 schrieb Felix Kuehling: >>> [SNIP] >> Failing that, I'd probably have to abandon userptr BOs altogether >> and

Re: [PATCH 1/2] drm/ttm: Don't evict SG BOs

2021-04-28 Thread Christian König
Am 28.04.21 um 18:49 schrieb Felix Kuehling: Am 2021-04-28 um 12:33 p.m. schrieb Christian König: Am 28.04.21 um 17:19 schrieb Felix Kuehling: [SNIP] Failing that, I'd probably have to abandon userptr BOs altogether and switch system memory mappings over to using the new SVM API on systems wher

Re: [PATCH 1/2] drm/ttm: Don't evict SG BOs

2021-04-28 Thread Felix Kuehling
Am 2021-04-28 um 12:33 p.m. schrieb Christian König: > Am 28.04.21 um 17:19 schrieb Felix Kuehling: >> Am 2021-04-28 um 5:05 a.m. schrieb Christian König: >> [SNIP] >> Hmm, I was missing something. The amdgpu_gtt_mgr doesn't actually >> allocate space for many BOs: >> >> if (!place->lpfn)

Re: [PATCH 1/2] drm/ttm: Don't evict SG BOs

2021-04-28 Thread Christian König
Am 28.04.21 um 17:19 schrieb Felix Kuehling: Am 2021-04-28 um 5:05 a.m. schrieb Christian König: [SNIP] Hmm, I was missing something. The amdgpu_gtt_mgr doesn't actually allocate space for many BOs: if (!place->lpfn) { mem->mm_node = NULL; mem->start =

Re: [PATCH 1/2] drm/ttm: Don't evict SG BOs

2021-04-28 Thread Felix Kuehling
Am 2021-04-28 um 5:05 a.m. schrieb Christian König: > Am 28.04.21 um 09:49 schrieb Felix Kuehling: >> Am 2021-04-28 um 3:04 a.m. schrieb Christian König: >>> Am 28.04.21 um 07:33 schrieb Felix Kuehling: SG BOs do not occupy space that is managed by TTM. So do not evict them. Thi

Re: [PATCH 1/2] drm/ttm: Don't evict SG BOs

2021-04-28 Thread Christian König
Am 28.04.21 um 09:49 schrieb Felix Kuehling: Am 2021-04-28 um 3:04 a.m. schrieb Christian König: Am 28.04.21 um 07:33 schrieb Felix Kuehling: SG BOs do not occupy space that is managed by TTM. So do not evict them. This fixes unexpected evictions of KFD's userptr BOs. KFD only expects userptr

Re: [PATCH 1/2] drm/ttm: Don't evict SG BOs

2021-04-28 Thread Felix Kuehling
Am 2021-04-28 um 3:04 a.m. schrieb Christian König: > Am 28.04.21 um 07:33 schrieb Felix Kuehling: >> SG BOs do not occupy space that is managed by TTM. So do not evict them. >> >> This fixes unexpected evictions of KFD's userptr BOs. KFD only expects >> userptr "evictions" in the form of MMU noti

Re: [PATCH 1/2] drm/ttm: Don't evict SG BOs

2021-04-28 Thread Christian König
Am 28.04.21 um 07:33 schrieb Felix Kuehling: SG BOs do not occupy space that is managed by TTM. So do not evict them. This fixes unexpected evictions of KFD's userptr BOs. KFD only expects userptr "evictions" in the form of MMU notifiers. NAK, SG BOs also account for the memory the GPU can cur

[PATCH 1/2] drm/ttm: Don't evict SG BOs

2021-04-27 Thread Felix Kuehling
SG BOs do not occupy space that is managed by TTM. So do not evict them. This fixes unexpected evictions of KFD's userptr BOs. KFD only expects userptr "evictions" in the form of MMU notifiers. Signed-off-by: Felix Kuehling --- drivers/gpu/drm/ttm/ttm_bo.c | 4 1 file changed, 4 insertions