Bug 218652 - acpitz sensors regression on Linux 6.7+ on Framework 16
Summary: acpitz sensors regression on Linux 6.7+ on Framework 16
Status: RESOLVED DUPLICATE of bug 218586
Alias: None
Product: ACPI
Classification: Unclassified
Component: Power-Thermal (show other bugs)
Hardware: All Linux
: P3 normal
Assignee: acpi_power-thermal
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2024-03-28 04:47 UTC by Quentin Smith
Modified: 2024-03-30 14:16 UTC (History)
2 users (show)

See Also:
Kernel Version:
Subsystem:
Regression: Yes
Bisected commit-id:


Attachments
possible patch (1.42 KB, patch)
2024-03-29 15:17 UTC, Mario Limonciello (AMD)
Details | Diff

Description Quentin Smith 2024-03-28 04:47:57 UTC
After upgrading from 6.1.78 to 6.8.1, I discovered that the `acpitz-*` sensors had gone missing. I believe that was caused by this commit:

https://github.com/torvalds/linux/commit/f04256a8f7de2c13619b636cec1109e596804229

because the kernel is now logging at boot:

```
ACPI: thermal: [Firmware Bug]: Invalid critical threshold (-274000)
ACPI: thermal: [Firmware Bug]: No valid trip points!
ACPI: thermal: [Firmware Bug]: Invalid critical threshold (-274000)
ACPI: thermal: [Firmware Bug]: No valid trip points!
ACPI: thermal: [Firmware Bug]: Invalid critical threshold (-274000)
ACPI: thermal: [Firmware Bug]: No valid trip points!
ACPI: thermal: [Firmware Bug]: Invalid critical threshold (-274000)
ACPI: thermal: [Firmware Bug]: No valid trip points!
```

If Iā€™m reading the laptop's DSDT correctly, the `_HOT` threshold is reported as 0x1218 (463.2 degrees) and the `_CRT` threshold is 0x12E0 (483.2 degrees). The old code would always create the thermal zones, but after that commit the kernel ignores any threshold above 448 degrees as implausibly high, and if there were no valid thresholds it ignores the whole thermal zone.

Even if there are no plausible thresholds, the thermal zone should still be created so the sensor values can be read.

I reported this in the Framework forum at https://community.frame.work/t/firmware-bug-acpi-table-error-causes-missing-sensors-on-linux-6-7/47767, but Mario Limonciello from AMD asked me to file an upstream bug since the commit message says "No intentional functional impact."
Comment 1 Artem S. Tashkinov 2024-03-29 09:04:17 UTC
> I believe that was caused by this commit:

Can you confirm reverting this commit fixes the issue for you?
Comment 2 Mario Limonciello (AMD) 2024-03-29 15:17:47 UTC
Created attachment 306058 [details]
possible patch

Can you see if this patch helps?
Comment 3 Mario Limonciello (AMD) 2024-03-30 14:16:08 UTC

*** This bug has been marked as a duplicate of bug 218586 ***

Note You need to log in before you can comment on or make changes to this bug.