Re: [PATCH 3/3] drm/ttm: switch to per device LRU lock

2021-03-22 Thread Huang Rui
On Fri, Mar 19, 2021 at 08:10:21PM +0800, Christian König wrote: > Am 19.03.21 um 05:32 schrieb Huang Rui: > > On Thu, Mar 18, 2021 at 08:47:19PM +0800, Christian König wrote: > >> Instead of having a global lock. > >> > >> Signed-off-by: Christian König > >> --- > >>

Re: [PATCH 3/3] drm/ttm: switch to per device LRU lock

2021-03-19 Thread Christian König
Am 19.03.21 um 05:32 schrieb Huang Rui: On Thu, Mar 18, 2021 at 08:47:19PM +0800, Christian König wrote: Instead of having a global lock. Signed-off-by: Christian König --- drivers/gpu/drm/amd/amdgpu/amdgpu_vm.c | 8 ++--- drivers/gpu/drm/qxl/qxl_release.c | 5 +--

Re: [PATCH 3/3] drm/ttm: switch to per device LRU lock

2021-03-18 Thread Huang Rui
On Thu, Mar 18, 2021 at 08:47:19PM +0800, Christian König wrote: > Instead of having a global lock. > > Signed-off-by: Christian König > --- > drivers/gpu/drm/amd/amdgpu/amdgpu_vm.c | 8 ++--- > drivers/gpu/drm/qxl/qxl_release.c | 5 +-- > drivers/gpu/drm/ttm/ttm_bo.c | 49

[PATCH 3/3] drm/ttm: switch to per device LRU lock

2021-03-18 Thread Christian König
Instead of having a global lock. Signed-off-by: Christian König --- drivers/gpu/drm/amd/amdgpu/amdgpu_vm.c | 8 ++--- drivers/gpu/drm/qxl/qxl_release.c | 5 +-- drivers/gpu/drm/ttm/ttm_bo.c | 49 -- drivers/gpu/drm/ttm/ttm_device.c | 12 +++

Re: [PATCH 3/3] drm/ttm: switch to per device LRU lock

2021-03-16 Thread Christian König
Am 16.03.21 um 13:05 schrieb Daniel Vetter: On Tue, Mar 16, 2021 at 1:03 PM Christian König wrote: Am 16.03.21 um 10:35 schrieb Daniel Vetter: On Mon, Mar 15, 2021 at 05:04:22PM +0100, Christian König wrote: Instead of having a global lock. Signed-off-by: Christian König I guess per

Re: [PATCH 3/3] drm/ttm: switch to per device LRU lock

2021-03-16 Thread Daniel Vetter
On Tue, Mar 16, 2021 at 1:03 PM Christian König wrote: > > Am 16.03.21 um 10:35 schrieb Daniel Vetter: > > On Mon, Mar 15, 2021 at 05:04:22PM +0100, Christian König wrote: > >> Instead of having a global lock. > >> > >> Signed-off-by: Christian König > > I guess per zone lru lock is a lot more

Re: [PATCH 3/3] drm/ttm: switch to per device LRU lock

2021-03-16 Thread Christian König
Am 16.03.21 um 10:35 schrieb Daniel Vetter: On Mon, Mar 15, 2021 at 05:04:22PM +0100, Christian König wrote: Instead of having a global lock. Signed-off-by: Christian König I guess per zone lru lock is a lot more work since then we need to handle ordering and ABBA deadlocks? s/zone/mem

Re: [PATCH 3/3] drm/ttm: switch to per device LRU lock

2021-03-16 Thread Daniel Vetter
On Mon, Mar 15, 2021 at 05:04:22PM +0100, Christian König wrote: > Instead of having a global lock. > > Signed-off-by: Christian König I guess per zone lru lock is a lot more work since then we need to handle ordering and ABBA deadlocks? s/zone/mem region/ I think is the proper ttm lingo.

Re: [PATCH 3/3] drm/ttm: switch to per device LRU lock

2021-03-15 Thread kernel test robot
Hi "Christian, I love your patch! Perhaps something to improve: [auto build test WARNING on drm-tip/drm-tip] [cannot apply to drm-intel/for-linux-next drm-exynos/exynos-drm-next tegra-drm/drm/tegra/for-next linus/master drm/drm-next v5.12-rc3 next-20210315] [If your patch is applied to the

[PATCH 3/3] drm/ttm: switch to per device LRU lock

2021-03-15 Thread Christian König
Instead of having a global lock. Signed-off-by: Christian König --- drivers/gpu/drm/amd/amdgpu/amdgpu_vm.c | 8 ++--- drivers/gpu/drm/qxl/qxl_release.c | 5 +-- drivers/gpu/drm/ttm/ttm_bo.c | 49 -- drivers/gpu/drm/ttm/ttm_device.c | 12 +++

[PATCH 3/3] drm/ttm: switch to per device LRU lock

2021-02-11 Thread Christian König
Instead of having a global lock. Signed-off-by: Christian König --- drivers/gpu/drm/amd/amdgpu/amdgpu_vm.c | 8 ++--- drivers/gpu/drm/qxl/qxl_release.c | 5 +-- drivers/gpu/drm/ttm/ttm_bo.c | 49 -- drivers/gpu/drm/ttm/ttm_device.c | 12 +++

[PATCH 3/3] drm/ttm: switch to per device LRU lock

2021-02-10 Thread Christian König
Instead of having a global lock. Signed-off-by: Christian König --- drivers/gpu/drm/amd/amdgpu/amdgpu_vm.c | 8 ++--- drivers/gpu/drm/qxl/qxl_release.c | 5 +-- drivers/gpu/drm/ttm/ttm_bo.c | 49 -- drivers/gpu/drm/ttm/ttm_device.c | 12 +++