On Mon, Oct 20, 2025 at 6:53 PM Mario Limonciello (AMD) <[email protected]> wrote: > > From: Mario Limonciello <[email protected]> > > The PM core should be notified that thaw was skipped for the device > so that if it's tried to be resumed (such as an aborted hibernate) > that it gets another chance to resume. > > Cc: Muhammad Usama Anjum <[email protected]> > Signed-off-by: Mario Limonciello <[email protected]> > --- > drivers/gpu/drm/amd/amdgpu/amdgpu_drv.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_drv.c > b/drivers/gpu/drm/amd/amdgpu/amdgpu_drv.c > index 61268aa82df4d..d40af069f24dd 100644 > --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_drv.c > +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_drv.c > @@ -2681,7 +2681,7 @@ static int amdgpu_pmops_thaw(struct device *dev) > > /* do not resume device if it's normal hibernation */ > if (!pm_hibernate_is_recovering() && > !pm_hibernation_mode_is_suspend()) > - return 0; > + return -EBUSY;
So that's why you need the special handling of -EBUSY in the previous patch. I think that you need to save some state in this driver and then use it in subsequent callbacks instead of hacking the core to do what you want. > return amdgpu_device_resume(drm_dev, true); > } > --
