Skip site navigation (1)Skip section navigation (2)
Date:      Wed, 13 May 2026 12:39:56 +0000
From:      Olivier Certner <olce@FreeBSD.org>
To:        src-committers@FreeBSD.org, dev-commits-src-all@FreeBSD.org, dev-commits-src-main@FreeBSD.org
Subject:   git: 5ba100a35cc5 - main - acpi_spmc(4): Global message on constraints parsing failure
Message-ID:  <6a04711c.18646.5b796bdb@gitrepo.freebsd.org>

index | next in thread | raw e-mail

The branch main has been updated by olce:

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

commit 5ba100a35cc5999d86a975fea9383d279f12e177
Author:     Olivier Certner <olce@FreeBSD.org>
AuthorDate: 2026-05-04 19:49:32 +0000
Commit:     Olivier Certner <olce@FreeBSD.org>
CommitDate: 2026-05-13 12:38:23 +0000

    acpi_spmc(4): Global message on constraints parsing failure
    
    ...in order to indicate to users that power state constraints will not
    be checked at all.
    
    Reviewed by:    imp, obiwac
    Sponsored by:   The FreeBSD Foundation
    Differential Revision:  https://reviews.freebsd.org/D56816
---
 sys/dev/acpica/acpi_spmc.c | 8 +++++++-
 1 file changed, 7 insertions(+), 1 deletion(-)

diff --git a/sys/dev/acpica/acpi_spmc.c b/sys/dev/acpica/acpi_spmc.c
index 5aed9ad3e5e4..751b4c8f0641 100644
--- a/sys/dev/acpica/acpi_spmc.c
+++ b/sys/dev/acpica/acpi_spmc.c
@@ -367,6 +367,7 @@ acpi_spmc_attach(device_t dev)
 	struct acpi_spmc_softc *const sc = device_get_softc(dev);
 	const ACPI_HANDLE handle = acpi_get_handle(dev);
 	char buf[32];
+	int error;
 
 	/*
 	 * ACPI_ID_PROBE() in acpi_spmc_probe() cannot succeed without a handle.
@@ -399,7 +400,12 @@ acpi_spmc_attach(device_t dev)
 			acpi_spmc_dsm_print_functions(sc, dsms[i]);
 
 	/* Get device constraints. We can only call this once so do this now. */
-	acpi_spmc_get_constraints(sc);
+	error = acpi_spmc_get_constraints(sc);
+	if (error != 0)
+		/* acpi_spmc_get_constraints() takes care of cleaning up. */
+		device_printf(dev,
+		    "Could not parse power state constraints (%d), "
+		    "will not check for them before suspend\n", error);
 
 	sc->eh_suspend = EVENTHANDLER_REGISTER(acpi_post_dev_suspend,
 	    acpi_spmc_suspend, dev, 0);


home | help

Want to link to this message? Use this
URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?6a04711c.18646.5b796bdb>