Reviewed-by: Andrey Grodzovsky <andrey.grodzovsky at amd.com> Thanks, Andrey On 01/19/2018 02:00 PM, mikita.lipski at amd.com wrote: > From: Mikita Lipski <mikita.lipski at amd.com> > > Inside amdgpu_gtt_mgr_fini add a missing lock to maintain > locking balance > > Signed-off-by: Mikita Lipski <mikita.lipski at amd.com> > --- > drivers/gpu/drm/amd/amdgpu/amdgpu_gtt_mgr.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_gtt_mgr.c b/drivers/gpu/drm/amd/amdgpu/amdgpu_gtt_mgr.c > index 7abc1f3..da7b1b9 100644 > --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_gtt_mgr.c > +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_gtt_mgr.c > @@ -75,7 +75,7 @@ static int amdgpu_gtt_mgr_init(struct ttm_mem_type_manager *man, > static int amdgpu_gtt_mgr_fini(struct ttm_mem_type_manager *man) > { > struct amdgpu_gtt_mgr *mgr = man->priv; > - > + spin_lock(&mgr->lock); > drm_mm_takedown(&mgr->mm); > spin_unlock(&mgr->lock); > kfree(mgr);