Re: [PATCH] drm/ttm: Remove pinned bos from LRU in ttm_bo_move_to_lru_tail() v2

2021-01-11 Thread Christian König
Am 08.01.21 um 19:49 schrieb Mike Lothian: Hi This breaks things for me on my Prime system https://gitlab.freedesktop.org/drm/misc/-/issues/23 This is most likely not correct. The patch just fixes another bug which would break prime before it hits the issue in the i915 driver. Christian.

Re: [PATCH] drm/ttm: Remove pinned bos from LRU in ttm_bo_move_to_lru_tail() v2

2021-01-08 Thread Mike Lothian
Hi This breaks things for me on my Prime system https://gitlab.freedesktop.org/drm/misc/-/issues/23 Cheers Mike ___ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel

Re: [PATCH] drm/ttm: Remove pinned bos from LRU in ttm_bo_move_to_lru_tail() v2

2021-01-05 Thread Christian König
Am 05.01.21 um 19:12 schrieb Lyude Paul: Reviewed-by: Lyude Paul Guessing it's fine if I push this with your sob and review added? Works for me, just take care that you pick the right branch. I always seem to push my stuff into the wrong one. Christian. On Tue, 2021-01-05 at 12:45

Re: [PATCH] drm/ttm: Remove pinned bos from LRU in ttm_bo_move_to_lru_tail() v2

2021-01-05 Thread Lyude Paul
Reviewed-by: Lyude Paul Guessing it's fine if I push this with your sob and review added? On Tue, 2021-01-05 at 12:45 +0100, Christian König wrote: > From: Lyude Paul > > Recently a regression was introduced which caused TTM's buffer eviction to > attempt to evict already-pinned BOs, causing

Re: [PATCH] drm/ttm: Remove pinned bos from LRU in ttm_bo_move_to_lru_tail()

2021-01-05 Thread Christian König
Am 04.01.21 um 22:06 schrieb Christian König: Am 05.01.21 um 00:13 schrieb Lyude Paul: Recently a regression was introduced which caused TTM's buffer eviction to attempt to evict already-pinned BOs, causing issues with buffer eviction under memory pressure along with suspend/resume:   

[PATCH] drm/ttm: Remove pinned bos from LRU in ttm_bo_move_to_lru_tail() v2

2021-01-05 Thread Christian König
From: Lyude Paul Recently a regression was introduced which caused TTM's buffer eviction to attempt to evict already-pinned BOs, causing issues with buffer eviction under memory pressure along with suspend/resume: nouveau :1f:00.0: DRM: evicting buffers... nouveau :1f:00.0: DRM:

Re: [PATCH] drm/ttm: Remove pinned bos from LRU in ttm_bo_move_to_lru_tail()

2021-01-04 Thread Christian König
Am 05.01.21 um 00:13 schrieb Lyude Paul: Recently a regression was introduced which caused TTM's buffer eviction to attempt to evict already-pinned BOs, causing issues with buffer eviction under memory pressure along with suspend/resume: nouveau :1f:00.0: DRM: evicting buffers...

[PATCH] drm/ttm: Remove pinned bos from LRU in ttm_bo_move_to_lru_tail()

2021-01-04 Thread Lyude Paul
Recently a regression was introduced which caused TTM's buffer eviction to attempt to evict already-pinned BOs, causing issues with buffer eviction under memory pressure along with suspend/resume: nouveau :1f:00.0: DRM: evicting buffers... nouveau :1f:00.0: DRM: Moving pinned object