ACK: [SRU v2] [G] [PATCH 0/6] Prevent thermal shutdown during boot process

[Kleber Souza]

On 27.01.21 04:05, Kai-Heng Feng wrote:
> BugLink: https://bugs.launchpad.net/bugs/1906168
> 
> [Impact]
> Surprising thermal shutdown at boot on Intel based mobile workstations.
> 
> [Fix]
> Since these thermal devcies are not in ACPI ThermalZone, OS shouldn't
> shutdown the system.
> 
> These critial temperatures are for usespace to handle, so let kernel
> know it shouldn't handle it.
> 
> For Groovy, a patch that removes .notify callback is dropped.
> 
> [Test]
> Use reboot stress as a reproducer. 5% chance to see a surprising
> shutdown at boot.
> 
> With the fix applied, the thermal shutdown is no longer reproducible.
> 
> [Where problems could occur]
> For ACPI based platforms, we still have "acpitz" to protect systems from
> overheating. If these acpitz sensors don't work, then the system could
> face real overheating issue.
> 
> Daniel Lezcano (4):
>    thermal/core: Emit a warning if the thermal zone is updated without
>      ops
>    thermal/core: Add critical and hot ops
>    thermal/drivers/acpi: Use hot and critical ops
>    thermal/drivers/rcar: Remove notification usage
> 
> Kai-Heng Feng (2):
>    thermal: int340x: Fix unexpected shutdown at critical temperature
>    thermal: intel: pch: Fix unexpected shutdown at critical temperature
> 
>   drivers/acpi/thermal.c                        | 30 ++++++------
>   .../int340x_thermal/int340x_thermal_zone.c    |  6 +++
>   drivers/thermal/intel/intel_pch_thermal.c     |  6 +++
>   drivers/thermal/rcar_thermal.c                | 19 --------
>   drivers/thermal/thermal_core.c                | 46 ++++++++++++-------
>   include/linux/thermal.h                       |  3 ++
>   6 files changed, 58 insertions(+), 52 deletions(-)
> 

Acked-by: Kleber Sacilotto de Souza <kleber.souza at canonical.com>