Re: [PATCH v2 0/9] TTM shrinker helpers and xe buffer object shrinker

2024-04-16 Thread Christian König
Am 16.04.24 um 15:08 schrieb Thomas Hellström: Hi, Christian, On Tue, 2024-04-16 at 13:55 +0200, Christian König wrote: While patches 1-4 look good from a high level I still think it needs some prerequisite and re-ordering. First of all make all the cleanups separate patches. In other words

Re: [PATCH v2 0/9] TTM shrinker helpers and xe buffer object shrinker

2024-04-16 Thread Thomas Hellström
Hi, Christian, On Tue, 2024-04-16 at 13:55 +0200, Christian König wrote: > While patches 1-4 look good from a high level I still think it needs > some prerequisite and re-ordering. > > First of all make all the cleanups separate patches. In other words > that > ttm_resource_manager_next()

Re: [PATCH v2 0/9] TTM shrinker helpers and xe buffer object shrinker

2024-04-16 Thread Christian König
While patches 1-4 look good from a high level I still think it needs some prerequisite and re-ordering. First of all make all the cleanups separate patches. In other words that ttm_resource_manager_next() takes only the cursor as argument, adding

[PATCH v2 0/9] TTM shrinker helpers and xe buffer object shrinker

2024-04-16 Thread Thomas Hellström
This series implements TTM shrinker / eviction helpers and an xe bo shrinker. It builds on two previous series, *and obsoletes these*. First https://www.mail-archive.com/dri-devel@lists.freedesktop.org/msg484425.html for patch 1-4, which IMO still could be reviewed and pushed as a separate