Re: [PATCH] drm/ttm: make the pool shrinker lock a mutex

2021-01-12 Thread Daniel Vetter
On Tue, Jan 12, 2021 at 3:06 PM Christian König wrote: > > Am 12.01.21 um 15:03 schrieb Daniel Vetter: > > On Mon, Jan 11, 2021 at 2:57 PM Christian König > > wrote: > >> set_pages_wb() might sleep and so we can't do this in an atomic context. > >> > >> Signed-off-by: Christian König > >> Report

Re: [PATCH] drm/ttm: make the pool shrinker lock a mutex

2021-01-12 Thread Christian König
Am 12.01.21 um 15:03 schrieb Daniel Vetter: On Mon, Jan 11, 2021 at 2:57 PM Christian König wrote: set_pages_wb() might sleep and so we can't do this in an atomic context. Signed-off-by: Christian König Reported-by: Mikhail Gavrilov Fixes: d099fc8f540a ("drm/ttm: new TT backend allocation po

Re: [PATCH] drm/ttm: make the pool shrinker lock a mutex

2021-01-12 Thread Daniel Vetter
On Mon, Jan 11, 2021 at 2:57 PM Christian König wrote: > > set_pages_wb() might sleep and so we can't do this in an atomic context. > > Signed-off-by: Christian König > Reported-by: Mikhail Gavrilov > Fixes: d099fc8f540a ("drm/ttm: new TT backend allocation pool v3") Hm I guess long term proper

Re: [PATCH] drm/ttm: make the pool shrinker lock a mutex

2021-01-12 Thread Huang Rui
On Tue, Jan 12, 2021 at 06:49:18PM +0800, Christian König wrote: > Ping? Ray can I get an acked-by? It's an important bug fix. > > Thanks, > Christian. > > Am 11.01.21 um 14:57 schrieb Christian König: > > set_pages_wb() might sleep and so we can't do this in an atomic context. > > > > Signed-off

Re: [PATCH] drm/ttm: make the pool shrinker lock a mutex

2021-01-12 Thread Christian König
Ping? Ray can I get an acked-by? It's an important bug fix. Thanks, Christian. Am 11.01.21 um 14:57 schrieb Christian König: set_pages_wb() might sleep and so we can't do this in an atomic context. Signed-off-by: Christian König Reported-by: Mikhail Gavrilov Fixes: d099fc8f540a ("drm/ttm: ne

[PATCH] drm/ttm: make the pool shrinker lock a mutex

2021-01-11 Thread Christian König
set_pages_wb() might sleep and so we can't do this in an atomic context. Signed-off-by: Christian König Reported-by: Mikhail Gavrilov Fixes: d099fc8f540a ("drm/ttm: new TT backend allocation pool v3") --- drivers/gpu/drm/ttm/ttm_pool.c | 20 ++-- 1 file changed, 10 insertions(+)