diff options
| author | Paul Menzel | 2022-04-11 20:59:00 +0200 |
|---|---|---|
| committer | Rafael J. Wysocki | 2022-04-13 20:24:58 +0200 |
| commit | ace8f1c54a02b96036b50defa9d842c10292b6bb (patch) | |
| tree | 2ac2d980a0cec3c659f9121bcf4c1f7a15940bb9 /drivers/acpi/acpica | |
| parent | 1c5d62f5dd979dbe7f4b16a5bd3c64fbecb7d92a (diff) | |
ACPICA: executer/exsystem: Inform users about ACPI spec violation
ACPICA commit 05ba545ce7859392250b18c10081db25c90ed8d7
Values greater than 100 microseconds violate the ACPI specification, so
warn users about it.
From ACPI Specification version 6.2 Errata A, 19.6.128 *Stall (Stall for
a Short Time)*:
> The implementation of Stall is OS-specific, but must not relinquish
> control of the processor. Because of this, delays longer than 100
> microseconds must use Sleep instead of Stall.
Link: https://github.com/acpica/acpica/commit/05ba545c
Signed-off-by: Bob Moore <robert.moore@intel.com>
Signed-off-by: Rafael J. Wysocki <rafael.j.wysocki@intel.com>
Reported-by: kernel test robot <lkp@intel.com>
Diffstat (limited to 'drivers/acpi/acpica')
| -rw-r--r-- | drivers/acpi/acpica/exsystem.c | 5 |
1 files changed, 5 insertions, 0 deletions
diff --git a/drivers/acpi/acpica/exsystem.c b/drivers/acpi/acpica/exsystem.c index 7d19f2398b51..754cf289fc01 100644 --- a/drivers/acpi/acpica/exsystem.c +++ b/drivers/acpi/acpica/exsystem.c @@ -137,6 +137,11 @@ acpi_status acpi_ex_system_do_stall(u32 how_long_us) "Time parameter is too large (%u)", how_long_us)); status = AE_AML_OPERAND_VALUE; } else { + if (how_long_US > 100) { + ACPI_WARNING((AE_INFO, + "Time parameter %u us > 100 us violating ACPI spec, please fix the firmware.", + how_long_us)); + } acpi_os_stall(how_long_us); } |