> -----Original Message----- > From: Dan Carpenter <[email protected]> > Sent: Wednesday, May 15, 2019 5:52 PM > To: Deucher, Alexander <[email protected]>; Wang, Kevin(Yang) > <[email protected]> > Cc: Koenig, Christian <[email protected]>; Zhou, David(ChunMing) > <[email protected]>; David Airlie <[email protected]>; Daniel Vetter > <[email protected]>; Huang, Ray <[email protected]>; Gao, Likun > <[email protected]>; Gui, Jack <[email protected]>; amd- > [email protected]; [email protected]; linux- > [email protected]; [email protected] > Subject: [PATCH] drm/amd/powerplay: fix locking in > smu_feature_set_supported() > > There is a typo so the code unlocks twice instead of taking the lock and then > releasing it. > > Fixes: f14a323db5b0 ("drm/amd/powerplay: implement update enabled > feature state to smc for smu11") > Signed-off-by: Dan Carpenter <[email protected]> >
Thanks! Reviewed-by: Huang Rui <[email protected]> Will apply it. --- > drivers/gpu/drm/amd/powerplay/amdgpu_smu.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/gpu/drm/amd/powerplay/amdgpu_smu.c > b/drivers/gpu/drm/amd/powerplay/amdgpu_smu.c > index 52d919a8b70a..85ac29af5363 100644 > --- a/drivers/gpu/drm/amd/powerplay/amdgpu_smu.c > +++ b/drivers/gpu/drm/amd/powerplay/amdgpu_smu.c > @@ -310,7 +310,7 @@ int smu_feature_set_supported(struct smu_context > *smu, int feature_id, > > WARN_ON(feature_id > feature->feature_num); > > - mutex_unlock(&feature->mutex); > + mutex_lock(&feature->mutex); > if (enable) > test_and_set_bit(feature_id, feature->supported); > else > -- > 2.20.1
