Re: [PATCH AUTOSEL 5.5 408/542] drm/amdgpu: add the lost mutex_init back

[Sasha Levin]

On Fri, Feb 14, 2020 at 11:22:27AM -0500, Alex Deucher wrote:
> On Fri, Feb 14, 2020 at 10:57 AM Sasha Levin <sashal@xxxxxxxxxx> wrote:
> > From: "Pan, Xinhui" <Xinhui.Pan@xxxxxxx>
> >
> > [ Upstream commit bd0522112332663e386df1b8642052463ea9b3b9 ]
> >
> > Initialize notifier_lock.
> >
> > Bug: https://gitlab.freedesktop.org/drm/amd/issues/1016
> > Reviewed-by: Feifei Xu <Feifei.Xu@xxxxxxx>
> > Reviewed-by: Christian König <christian.koenig@xxxxxxx>
> > Signed-off-by: xinhui pan <xinhui.pan@xxxxxxx>
> > Signed-off-by: Alex Deucher <alexander.deucher@xxxxxxx>
> > Signed-off-by: Sasha Levin <sashal@xxxxxxxxxx>
> > ---
> >  drivers/gpu/drm/amd/amdgpu/amdgpu_device.c | 1 +
> >  1 file changed, 1 insertion(+)
> >
> > diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_device.c b/drivers/gpu/drm/amd/amdgpu/amdgpu_device.c
> > index 332b9c24a2cd0..a2f788ad7e1c6 100644
> > --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_device.c
> > +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_device.c
> > @@ -2797,6 +2797,7 @@ int amdgpu_device_init(struct amdgpu_device *adev,
> >         mutex_init(&adev->notifier_lock);
> >         mutex_init(&adev->virt.dpm_mutex);
> >         mutex_init(&adev->psp.mutex);
> > +       mutex_init(&adev->notifier_lock);
>
> This patch is not relevant here.  The same mutex is already
> initialized 3 lines above.

Now dropped, thank you.

--
Thanks,
Sasha