[LTP] [PATCH v2] thermal: add new test group
Cyril Hrubis
chrubis@suse.cz
Tue Nov 18 19:18:14 CET 2025
Hi!
> Signed-off-by: Piotr Kubaj <piotr.kubaj@intel.com>
> ---
> runtest/thermal | 2 +
> scenario_groups/default | 1 +
> testcases/kernel/Makefile | 1 +
> testcases/kernel/thermal/Makefile | 11 +++
> testcases/kernel/thermal/thermal01.sh | 101 ++++++++++++++++++++++++++
> 5 files changed, 116 insertions(+)
> create mode 100644 runtest/thermal
> create mode 100644 testcases/kernel/thermal/Makefile
> create mode 100755 testcases/kernel/thermal/thermal01.sh
>
> diff --git a/runtest/thermal b/runtest/thermal
> new file mode 100644
> index 000000000..804ef7d79
> --- /dev/null
> +++ b/runtest/thermal
> @@ -0,0 +1,2 @@
> +#THERMAL
> +thermal_interrupt_events thermal01.sh
> diff --git a/scenario_groups/default b/scenario_groups/default
> index 0e76b2bee..ffdd7ff25 100644
> --- a/scenario_groups/default
> +++ b/scenario_groups/default
> @@ -26,3 +26,4 @@ crypto
> kernel_misc
> uevent
> watchqueue
> +thermal
> diff --git a/testcases/kernel/Makefile b/testcases/kernel/Makefile
> index 98fd45a9d..ac816e4e8 100644
> --- a/testcases/kernel/Makefile
> +++ b/testcases/kernel/Makefile
> @@ -36,6 +36,7 @@ SUBDIRS += connectors \
> sched \
> security \
> sound \
> + thermal \
> tracing \
> uevents \
> watchqueue \
> diff --git a/testcases/kernel/thermal/Makefile b/testcases/kernel/thermal/Makefile
> new file mode 100644
> index 000000000..789db430d
> --- /dev/null
> +++ b/testcases/kernel/thermal/Makefile
> @@ -0,0 +1,11 @@
> +# SPDX-License-Identifier: GPL-2.0-or-later
> +# Copyright (c) 2025, Intel Corporation. All rights reserved.
> +# Author:Piotr Kubaj <piotr.kubaj@intel.com>
> +
> +top_srcdir ?= ../../..
> +
> +include $(top_srcdir)/include/mk/env_pre.mk
> +
> +INSTALL_TARGETS := thermal01.sh
> +
> +include $(top_srcdir)/include/mk/generic_leaf_target.mk
> diff --git a/testcases/kernel/thermal/thermal01.sh b/testcases/kernel/thermal/thermal01.sh
> new file mode 100755
> index 000000000..138c30ee7
> --- /dev/null
> +++ b/testcases/kernel/thermal/thermal01.sh
> @@ -0,0 +1,101 @@
> +#!/usr/bin/env bash
> +# SPDX-License-Identifier: GPL-2.0-or-later
> +# Copyright (C) 2025 Intel - http://www.intel.com/
> +#
> +# ---
> +# doc
> +# Tests the CPU package thermal sensor interface for Intel platforms.
> +#
> +# It works by checking the initial count of thermal interrupts. Then it
> +# decreases the threshold for sending a thermal interrupt to just above
> +# the current temperature and runs a workload on the CPU. Finally, it restores
> +# the original thermal threshold and checks whether the number of thermal
> +# interrupts increased.
> +# ---
> +#
> +# ---
> +# env
> +# {
> +# "needs_root": true,
> +# "supported_archs": ["x86", "x86_64"],
> +# "needs_cmds": ["stress-ng"]
> +# }
> +# ---
> +
> +export LTP_TIMEOUT_MUL=6
Instead of this the test needs to set up runtime in the environment
properly. I will send a patch for the tst_run_shell.c that adds support
for runtime to the JSON env tomorrow.
> +. tst_loader.sh
> +
> +thermal_zone_numbers=""
> +temp=""
> +temp_high=""
> +status=0
> +
> +tst_test()
> +{
> + line=$(grep "Thermal event interrupts" /proc/interrupts)
> + if [ $? -eq 0 ]; then
> + interrupt_array_init=$(echo "$line" | tr -d "a-zA-Z:" | awk '{$1=$1;print}')
> + echo "Initial values of thermal interrupt counters: $interrupt_array_init"
All these echo should be tst_res TINFO "..." or tst_res TDEBUG "..."
(not shown by default) messages instead, so that the test verbosity can
be properly controlled.
> + num=$(nproc)
> + echo "Number of logical cores: $num"
> +
> + else
> + tst_brk TCONF "Thermal event interrupts is not found."
> + fi
> +
> + # Below we check for the thermal_zone which uses x86_pkg_temp driver
> + thermal_zone_numbers=$(grep -l x86_pkg_temp /sys/class/thermal/thermal_zone*/type | sed 's/[^0-9]//g' | tr -t '\n' ' ')
> + echo "x86_pkg_temp thermal zones: $thermal_zone_numbers"
> +
> + if [ -z $thermal_zone_numbers ]; then
> + tst_brk TCONF "No x86_pkg_temp thermal zones found"
> + fi
> + for i in $thermal_zone_numbers; do
> + echo "Currently testing x86_pkg_temp thermal_zone$i"
> + TEMP=/sys/class/thermal/thermal_zone$i/temp
> + temp=$(cat "$TEMP")
> + echo "thermal_zone$i current temperature is $temp"
> + if [ "$(echo "$temp <= 0" | bc)" -eq 1 ]; then
bc package is not installed by default on certain systems, so we either
need to request it explicitly or not use it.
What about:
case "$TEMP" in
[0-9]*) ;;
*)
tst_brk TBROK "Unexpected zone temperature value $temp";;
esac
> + tst_brk TBROK "Unexpected zone temperature value $temp"
> + fi
> + trip=$(cat /sys/class/thermal/thermal_zone$i/trip_point_1_temp)
> + # Setting trip_point_1_temp for termal_zone$i to $temp + 10 (0.001°C)
> + temp_high=$(( temp + 10 ))
> + echo $temp_high > /sys/class/thermal/thermal_zone$i/trip_point_1_temp
> + run_time=30
> + sleep_time=10
> + while [ $sleep_time -gt 0 ]; do
> + stress-ng --matrix 0 --timeout $run_time
> + temp_cur=$(cat "$TEMP")
> + echo "temp_cur: $temp_cur"
> + [ $temp_cur -gt $temp_high ] && break
> + sleep $sleep_time
> + run_time=$(( run_time - 3 ))
> + sleep_time=$(( sleep_time - 1 ))
> + done
> + [ $temp_cur -gt $temp_high ] || tst_res TFAIL "Zone temperature is not rising as expected"
> +
> + # Restore the original trip_point_1_temp value
> + echo $trip > /sys/class/thermal/thermal_zone$i/trip_point_1_temp
> +
> + # Check whether thermal interrupts count actually increased
> + interrupt_array_later=$(grep "Thermal event interrupts" /proc/interrupts | \
> + tr -d "a-zA-Z:" | awk '{$1=$1;print}')
> + echo "Current values of thermal interrupt counters: $interrupt_array_later"
> + for j in $(seq 1 "$num"); do
> + interrupt_later=$(echo "$interrupt_array_later" | cut -d " " -f "$j")
> + interrupt_init=$(echo "$interrupt_array_init" | cut -d " " -f "$j")
> + if [ $interrupt_later -le $interrupt_init ]; then
> + status=1
> + fi
> + done
> + done
> +
> + if [ $status -eq 0 ]; then
> + tst_res TPASS "x86 package thermal interrupt triggered"
> + else
> + tst_res TFAIL "x86 package thermal interrupt did not trigger"
> + fi
> +}
> +
> +. tst_run.sh
> --
> 2.47.3
>
> ---------------------------------------------------------------------
> Intel Technology Poland sp. z o.o.
> ul. Slowackiego 173 | 80-298 Gdansk | Sad Rejonowy Gdansk Polnoc | VII Wydzial Gospodarczy Krajowego Rejestru Sadowego - KRS 101882 | NIP 957-07-52-316 | Kapital zakladowy 200.000 PLN.
> Spolka oswiadcza, ze posiada status duzego przedsiebiorcy w rozumieniu ustawy z dnia 8 marca 2013 r. o przeciwdzialaniu nadmiernym opoznieniom w transakcjach handlowych.
>
> Ta wiadomosc wraz z zalacznikami jest przeznaczona dla okreslonego adresata i moze zawierac informacje poufne. W razie przypadkowego otrzymania tej wiadomosci, prosimy o powiadomienie nadawcy oraz trwale jej usuniecie; jakiekolwiek przegladanie lub rozpowszechnianie jest zabronione.
> This e-mail and any attachments may contain confidential material for the sole use of the intended recipient(s). If you are not the intended recipient, please contact the sender and delete all copies; any review or distribution by others is strictly prohibited.
>
> --
> Mailing list info: https://lists.linux.it/listinfo/ltp
--
Cyril Hrubis
chrubis@suse.cz
More information about the ltp
mailing list