git: c9bfc37b61d2 - main - acpi: Don't check suspend-to-idle if suspend failed

From: Aymeric Wibo <obiwac_at_FreeBSD.org>
Date: Fri, 18 Sep 2026 10:59:22 UTC
The branch main has been updated by obiwac:

URL: https://cgit.FreeBSD.org/src/commit/?id=c9bfc37b61d299f6de4b6b9799db95181b3bd1d0

commit c9bfc37b61d299f6de4b6b9799db95181b3bd1d0
Author:     Aymeric Wibo <obiwac@FreeBSD.org>
AuthorDate: 2026-09-16 17:07:19 +0000
Commit:     Aymeric Wibo <obiwac@FreeBSD.org>
CommitDate: 2026-09-18 10:59:12 +0000

    acpi: Don't check suspend-to-idle if suspend failed
    
    If we e.g. failed to suspend a device and suspend bounced because of
    that, then we're not expected to have entered a deep sleep state in the
    first place. In this situation, don't overload the user with irrelevant
    information.
    
    Reviewed by:    olce
    Fixes:          5f68acc931a4 ("acpi: Warn if no amdsmu(4) loaded after suspend-to-idle resume")
    Sponsored by:   The FreeBSD Foundation
    Differential Revision:  https://reviews.freebsd.org/D59731
---
 sys/dev/acpica/acpi.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/sys/dev/acpica/acpi.c b/sys/dev/acpica/acpi.c
index 2ca70b2f0ed4..e2bc335d98e9 100644
--- a/sys/dev/acpica/acpi.c
+++ b/sys/dev/acpica/acpi.c
@@ -3908,7 +3908,7 @@ backout:
 
     EVENTHANDLER_INVOKE(power_resume, stype);
 
-    if (stype == POWER_STYPE_SUSPEND_TO_IDLE)
+    if (stype == POWER_STYPE_SUSPEND_TO_IDLE && ACPI_SUCCESS(status))
 	check_post_suspend_to_idle(sc->acpi_dev);
 
     /* Allow another sleep request after a while. */