Re: [PATCH 0/4] TTM unlockable restartable LRU list iteration

2024-03-01 Thread Thomas Hellström
On Fri, 2024-03-01 at 15:20 +0100, Christian König wrote: > Am 01.03.24 um 14:45 schrieb Thomas Hellström: > > On Thu, 2024-02-29 at 18:34 +0100, Thomas Hellström wrote: > > > Hi, Christian. > > > > > > Thanks for having a look. > > > > > > On Thu, 2024-02-29 at 16:08 +0100, Christian König wrote

Re: [PATCH 0/4] TTM unlockable restartable LRU list iteration

2024-03-01 Thread Christian König
Am 01.03.24 um 14:45 schrieb Thomas Hellström: On Thu, 2024-02-29 at 18:34 +0100, Thomas Hellström wrote: Hi, Christian. Thanks for having a look. On Thu, 2024-02-29 at 16:08 +0100, Christian König wrote: Am 16.02.24 um 15:20 schrieb Thomas Hellström: [SNIP] My approach was basically to not

Re: [PATCH 0/4] TTM unlockable restartable LRU list iteration

2024-03-01 Thread Thomas Hellström
On Thu, 2024-02-29 at 18:34 +0100, Thomas Hellström wrote: > Hi, Christian. > > Thanks for having a look. > > On Thu, 2024-02-29 at 16:08 +0100, Christian König wrote: > > Am 16.02.24 um 15:20 schrieb Thomas Hellström: > > [SNIP] > > > > My approach was basically to not only lock the current BO,

Re: [PATCH 0/4] TTM unlockable restartable LRU list iteration

2024-02-29 Thread Thomas Hellström
Hi, Christian. Thanks for having a look. On Thu, 2024-02-29 at 16:08 +0100, Christian König wrote: > Am 16.02.24 um 15:20 schrieb Thomas Hellström: > [SNIP] > > > My approach was basically to not only lock the current BO, but > > > also > > > the > > > next one. Since only a locked BO can move on

Re: [PATCH 0/4] TTM unlockable restartable LRU list iteration

2024-02-29 Thread Christian König
Am 16.02.24 um 15:20 schrieb Thomas Hellström: [SNIP] My approach was basically to not only lock the current BO, but also the next one. Since only a locked BO can move on the LRU we effectively created an anchor. Before I dig into the code a couple of questions: These are described in the patch

Re: [PATCH 0/4] TTM unlockable restartable LRU list iteration

2024-02-16 Thread Thomas Hellström
On Fri, 2024-02-16 at 15:00 +0100, Christian König wrote: > Am 16.02.24 um 14:13 schrieb Thomas Hellström: > > This patch-set is a prerequisite for a standalone TTM shrinker > > and for exhaustive TTM eviction using sleeping dma_resv locks, > > which is the motivation it. > > > > Currently when un

Re: [PATCH 0/4] TTM unlockable restartable LRU list iteration

2024-02-16 Thread Christian König
Am 16.02.24 um 14:13 schrieb Thomas Hellström: This patch-set is a prerequisite for a standalone TTM shrinker and for exhaustive TTM eviction using sleeping dma_resv locks, which is the motivation it. Currently when unlocking the TTM lru list lock, iteration needs to be restarted from the beginn

[PATCH 0/4] TTM unlockable restartable LRU list iteration

2024-02-16 Thread Thomas Hellström
This patch-set is a prerequisite for a standalone TTM shrinker and for exhaustive TTM eviction using sleeping dma_resv locks, which is the motivation it. Currently when unlocking the TTM lru list lock, iteration needs to be restarted from the beginning, rather from the next LRU list node. This can