Applied. Thanks!
On Tue, May 14, 2024 at 3:27 AM Christian König <[email protected]> wrote: > > Am 14.05.24 um 09:06 schrieb Friedrich Vock: > > The special case for VM passthrough doesn't check adev->nbio.funcs > > before dereferencing it. If GPUs that don't have an NBIO block are > > passed through, this leads to a NULL pointer dereference on startup. > > > > Signed-off-by: Friedrich Vock <[email protected]> > > Acked-by: Christian König <[email protected]> > > > > > Fixes: 1bece222eab ("drm/amdgpu: Clear doorbell interrupt status for Sienna > > Cichlid") > > Cc: Alex Deucher <[email protected]> > > Cc: Christian König <[email protected]> > > --- > > drivers/gpu/drm/amd/amdgpu/amdgpu_device.c | 2 +- > > 1 file changed, 1 insertion(+), 1 deletion(-) > > > > diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_device.c > > b/drivers/gpu/drm/amd/amdgpu/amdgpu_device.c > > index 861ccff78af95..83c4533ee75c8 100644 > > --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_device.c > > +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_device.c > > @@ -6165,7 +6165,7 @@ int amdgpu_device_baco_exit(struct drm_device *dev) > > adev->nbio.funcs->enable_doorbell_interrupt) > > adev->nbio.funcs->enable_doorbell_interrupt(adev, true); > > > > - if (amdgpu_passthrough(adev) && > > + if (amdgpu_passthrough(adev) && adev->nbio.funcs && > > adev->nbio.funcs->clear_doorbell_interrupt) > > adev->nbio.funcs->clear_doorbell_interrupt(adev); > > > > -- > > 2.45.0 > > >
