Re: [PATCH 1/2] drm/amdgpu: fix a potential deadlock in amdgpu_bo_create_restricted()

2017-02-08 Thread Michel Dänzer
On 09/02/17 08:35 AM, Samuel Pitoiset wrote: > When ttm_bo_init() fails, the reservation mutex should be unlocked. > > In debug build, the kernel reported "possible recursive locking > detected" in this codepath. For debugging purposes, I also added > a "WARN_ON(ww_mutex_is_locked())" when

[PATCH 1/2] drm/amdgpu: fix a potential deadlock in amdgpu_bo_create_restricted()

2017-02-08 Thread Samuel Pitoiset
When ttm_bo_init() fails, the reservation mutex should be unlocked. In debug build, the kernel reported "possible recursive locking detected" in this codepath. For debugging purposes, I also added a "WARN_ON(ww_mutex_is_locked())" when ttm_bo_init() fails and the mutex was locked as expected.