Can you check and make sure this doesn't break polaris or some other generation? Alex On Wed, Jul 15, 2020 at 1:12 AM Quan, Evan <Evan.Quan@xxxxxxx> wrote: > > [AMD Official Use Only - Internal Distribution Only] > > Acked-by: Evan Quan <evan.quan@xxxxxxx> > > -----Original Message----- > From: Nirmoy Das <nirmoy.aiemd@xxxxxxxxx> > Sent: Friday, July 10, 2020 10:51 PM > To: amd-gfx@xxxxxxxxxxxxxxxxxxxxx > Cc: Deucher, Alexander <Alexander.Deucher@xxxxxxx>; Quan, Evan <Evan.Quan@xxxxxxx>; Das, Nirmoy <Nirmoy.Das@xxxxxxx> > Subject: [PATCH] drm/amdgpu: do not disable SMU on vm reboot > > For passthrough device, we do baco reset after 1st vm boot so > if we disable SMU on 1st VM shutdown baco reset will fail for > 2nd vm boot. > > Signed-off-by: Nirmoy Das <nirmoy.das@xxxxxxx> > --- > drivers/gpu/drm/amd/amdgpu/amdgpu_drv.c | 3 ++- > 1 file changed, 2 insertions(+), 1 deletion(-) > > diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_drv.c b/drivers/gpu/drm/amd/amdgpu/amdgpu_drv.c > index d1adbc45d37b..07be61ce969a 100644 > --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_drv.c > +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_drv.c > @@ -1178,7 +1178,8 @@ amdgpu_pci_shutdown(struct pci_dev *pdev) > * unfortunately we can't detect certain > * hypervisors so just do this all the time. > */ > -adev->mp1_state = PP_MP1_STATE_UNLOAD; > +if (!amdgpu_passthrough(adev)) > +adev->mp1_state = PP_MP1_STATE_UNLOAD; > amdgpu_device_ip_suspend(adev); > adev->mp1_state = PP_MP1_STATE_NONE; > } > -- > 2.27.0 > > _______________________________________________ > amd-gfx mailing list > amd-gfx@xxxxxxxxxxxxxxxxxxxxx > https://lists.freedesktop.org/mailman/listinfo/amd-gfx _______________________________________________ amd-gfx mailing list amd-gfx@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/amd-gfx
