Merge ~baconyao/checkbox-limerick/+git/checkbox-provider-limerick:fix-thermal into ~checkbox-dev/checkbox-limerick/+git/checkbox-provider-limerick:master

Proposed by PeiYao Chang
Status: Merged
Approved by: PeiYao Chang
Approved revision: b26e59c353c915a896014b19f961e562557aa61c
Merged at revision: 8c250c9c770fe66a375416065c386912d418209b
Proposed branch: ~baconyao/checkbox-limerick/+git/checkbox-provider-limerick:fix-thermal
Merge into: ~checkbox-dev/checkbox-limerick/+git/checkbox-provider-limerick:master
Diff against target: 21 lines (+2/-2)
1 file modified
units/thermal-sensor/jobs.pxu (+2/-2)
Reviewer Review Type Date Requested Status
Patrick Liu (community) Approve
Vic Liu Approve
Vincent Liao Pending
StanleyHuang Pending
Rick Wu Pending
Review via email: mp+449077@code.launchpad.net

Commit message

    Fix: Increasing the time of stress-ng for thermal

    On KD240 platform, the thermal is not close to CPU, so the increasing
    temperature after doing stress-ng is not huge or even lower than before.

    So, I increase the stress-ng time from 50 seconds to 5 minutes, and it
    works.

    Ref: LP#2025211

To post a comment you must log in.
Revision history for this message
Vic Liu (zongminl) wrote :

+1

review: Approve
Revision history for this message
Patrick Liu (patliu) wrote :

+1

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1diff --git a/units/thermal-sensor/jobs.pxu b/units/thermal-sensor/jobs.pxu
2index 2521ab4..2fb0f2b 100644
3--- a/units/thermal-sensor/jobs.pxu
4+++ b/units/thermal-sensor/jobs.pxu
5@@ -3,14 +3,14 @@ _summary: Check thermal sensor - hwmon0 is activate
6 plugin: shell
7 user: root
8 command:
9- echo "Sleep 10 seconds than start stress-ng for 50 seconds"
10+ echo "Sleep 10 seconds than start stress-ng for 5 minutes"
11 sleep 10
12 FILE=/sys/class/hwmon/hwmon0/temp1_input
13 if [ -f "$FILE" ]
14 then
15 cur=$(cat "$FILE")
16 printf "Before stress: %0.3f°C\n" "$(bc -l <<< "$cur/1000")"
17- stress-ng --cpu 8 --io 4 --vm 2 --vm-bytes 128M --fork 4 --timeout 50s
18+ stress-ng --cpu 8 --io 4 --vm 2 --vm-bytes 128M --fork 4 --timeout 300s
19 new=$(cat "$FILE")
20 printf "After stress: %0.3f°C\n" "$(bc -l <<< "$new/1000")"
21 [[ $new -gt $cur ]]

Subscribers

People subscribed via source and target branches