Patch "ACPICA: executer/exsystem: Don't nag user about every Stall() violating the spec" has been added to the 6.1-stable tree

[Sasha Levin <sashal@xxxxxxxxxx>]

This is a note to let you know that I've just added the patch titled

    ACPICA: executer/exsystem: Don't nag user about every Stall() violating the spec

to the 6.1-stable tree which can be found at:
    http://www.kernel.org/git/?p=linux/kernel/git/stable/stable-queue.git;a=summary

The filename of the patch is:
     acpica-executer-exsystem-don-t-nag-user-about-every-.patch
and it can be found in the queue-6.1 subdirectory.

If you, or anyone else, feels it should not be added to the stable tree,
please let <stable@xxxxxxxxxxxxxxx> know about it.



commit 806f68a60e9794e99ab0a80ee102ac7786c74416
Author: Vasily Khoruzhick <anarsoul@xxxxxxxxx>
Date:   Tue Apr 2 21:12:40 2024 -0700

    ACPICA: executer/exsystem: Don't nag user about every Stall() violating the spec
    
    [ Upstream commit c82c507126c9c9db350be28f14c83fad1c7969ae ]
    
    ACPICA commit 129b75516fc49fe1fd6b8c5798f86c13854630b3
    
    Stop nagging user about every Stall() that violates the spec
    
    On my Dell XPS 15 7590 I get hundreds of these warnings after few hours of
    uptime:
    
    $ dmesg | grep "fix the firmware" | wc -l
    261
    
    I cannot fix the firmware and I doubt that Dell cares about 4 year old
    laptop either
    
    Fixes: ace8f1c54a02 ("ACPICA: executer/exsystem: Inform users about ACPI spec violation")
    Link: https://github.com/acpica/acpica/commit/129b7551
    Signed-off-by: Vasily Khoruzhick <anarsoul@xxxxxxxxx>
    Signed-off-by: Rafael J. Wysocki <rafael.j.wysocki@xxxxxxxxx>
    Signed-off-by: Sasha Levin <sashal@xxxxxxxxxx>

diff --git a/drivers/acpi/acpica/exsystem.c b/drivers/acpi/acpica/exsystem.c
index 7b5470f404f3f..104e6e96c31ea 100644
--- a/drivers/acpi/acpica/exsystem.c
+++ b/drivers/acpi/acpica/exsystem.c
@@ -133,14 +133,15 @@ acpi_status acpi_ex_system_do_stall(u32 how_long_us)
 		 * (ACPI specifies 100 usec as max, but this gives some slack in
 		 * order to support existing BIOSs)
 		 */
-		ACPI_ERROR((AE_INFO,
-			    "Time parameter is too large (%u)", how_long_us));
+		ACPI_ERROR_ONCE((AE_INFO,
+				 "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_WARNING_ONCE((AE_INFO,
+					   "Time parameter %u us > 100 us violating ACPI spec, please fix the firmware.",
+					   how_long_us));
 		}
 		acpi_os_stall(how_long_us);
 	}