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