diff options
| author | gutteridge <gutteridge@NetBSD.org> | 2022-12-07 05:38:27 +0000 |
|---|---|---|
| committer | gutteridge <gutteridge@NetBSD.org> | 2022-12-07 05:38:27 +0000 |
| commit | 00a5cb85b2db589f05925bded0cef003a5ca5c90 (patch) | |
| tree | 3fa04cbd553a3d8ab548e01e2f16dcdbe9d4add9 | |
| parent | f87d170666d220a19804cbe27b592e22c5922a93 (diff) | |
exsystem.c: revert ACPI warning about sleep values >10ms
acpica 20220331 introduced the following change:
For the ASL Sleep() operator, issue a warning if the sleep value is
greater than 10 Milliseconds. Quick boottime is important, so warn about
sleeps greater than 10 ms. Distribution Linux kernels reach initrd in 350
ms, so excessive delays should be called out. 10 ms is chosen randomly,
but three of such delays would already make up ten percent of the
boottime.
This can generate many console messages like:
[ 26.055036] ACPI Warning: Firmware issue: Excessive sleep time (0x0000000000000064 ms > 10 ms) in ACPI Control Method (20220331/exsystem-239)
(60 of them in the latest boot on one older Ivy Bridge laptop.)
This has been reverted upstream and is not present in the next release.
https://github.com/acpica/acpica/commit/df896f452f945d498736c5775d4ee6629df31786
(Also previously reverted in FreeBSD and Linux.)
| -rw-r--r-- | sys/external/bsd/acpica/dist/executer/exsystem.c | 12 |
1 files changed, 0 insertions, 12 deletions
diff --git a/sys/external/bsd/acpica/dist/executer/exsystem.c b/sys/external/bsd/acpica/dist/executer/exsystem.c index e4f02d40ea2..38574ba805f 100644 --- a/sys/external/bsd/acpica/dist/executer/exsystem.c +++ b/sys/external/bsd/acpica/dist/executer/exsystem.c @@ -231,18 +231,6 @@ AcpiExSystemDoSleep ( AcpiExExitInterpreter (); /* - * Warn users about excessive sleep times, so ASL code can be improved to - * use polling or similar techniques. - */ - if (HowLongMs > 10) - { - ACPI_WARNING ((AE_INFO, - "Firmware issue: Excessive sleep time (0x%8.8X%8.8X ms > 10 ms)" - " in ACPI Control Method", - ACPI_FORMAT_UINT64 (HowLongMs))); - } - - /* * For compatibility with other ACPI implementations and to prevent * accidental deep sleeps, limit the sleep time to something reasonable. */ |
