Re: [PATCH 1/6] drm/amdgpu: rework VM state machine lock handling v2

2018-05-22 Thread Christian König
top.org Subject: Re: [PATCH 1/6] drm/amdgpu: rework VM state machine lock handling v2 2, 3, 4, 5 are Reviewed-by: Junwei Zhang Patch 1: could you show the reserving VM? Patch 6: I could read that code, but not sure the purpose. Jerry On 05/17/2018 05:49 PM, Christian König wrote: Only the move

Re: [PATCH 1/6] drm/amdgpu: rework VM state machine lock handling v2

2018-05-19 Thread Zhang, Jerry
[PATCH 1/6] drm/amdgpu: rework VM state machine lock handling v2 2, 3, 4, 5 are Reviewed-by: Junwei Zhang Patch 1: could you show the reserving VM? Patch 6: I could read that code, but not sure the purpose. Jerry On 05/17/2018 05:49 PM, Christian König wrote: > Only the moved state needs a

Re: [PATCH 1/6] drm/amdgpu: rework VM state machine lock handling v2

2018-05-18 Thread Zhang, Jerry (Junwei)
2, 3, 4, 5 are Reviewed-by: Junwei Zhang Patch 1: could you show the reserving VM? Patch 6: I could read that code, but not sure the purpose. Jerry On 05/17/2018 05:49 PM, Christian König wrote: Only the moved state needs a separate spin lock protection. All other states are protected by res

[PATCH 1/6] drm/amdgpu: rework VM state machine lock handling v2

2018-05-17 Thread Christian König
Only the moved state needs a separate spin lock protection. All other states are protected by reserving the VM anyway. v2: fix some more incorrect cases Signed-off-by: Christian König --- drivers/gpu/drm/amd/amdgpu/amdgpu_vm.c | 66 +++--- drivers/gpu/drm/amd/amdgpu/