not use /etc/thermald/thermal-conf.xml by default

Bug #1799389 reported by Alex Tu
12
This bug affects 2 people
Affects Status Importance Assigned to Milestone
OEM Priority Project
Won't Fix
High
Unassigned
thermald (Ubuntu)
Expired
Medium
Unassigned

Bug Description

from man thermal-conf.xml, thermald should use /etc/thermald/thermal-conf.xml by default.
So, that user can control the behavior.

"
- If the ACPI data is not optimized or buggy. In this case thermal-conf.xml can be used to correct the behavior without change in BIOS.
"

But, current thermald will always use the one auto generated:
/var/run/thermald/thermal-conf.xml.auto
---
ProblemType: Bug
ApportVersion: 2.20.9-0ubuntu7.2
Architecture: amd64
DistributionChannelDescriptor:
 # This is the distribution channel descriptor for the OEM CDs
 # For more information see http://wiki.ubuntu.com/DistributionChannelDescriptor
 canonical-oem-somerville-bionic-amd64-20180608-47+bionic-master+X32
DistroRelease: Ubuntu 18.04
InstallationDate: Installed on 2018-10-23 (0 days ago)
InstallationMedia: Ubuntu 18.04 "Bionic" - Build amd64 LIVE Binary 20180608-09:38
Package: thermald 1.7.0-5ubuntu1
PackageArchitecture: amd64
ProcVersionSignature: Ubuntu 4.15.0-1024.29-oem 4.15.18
Tags: bionic
Uname: Linux 4.15.0-1024-oem x86_64
UpgradeStatus: No upgrade log present (probably fresh install)
UserGroups: adm cdrom dip lpadmin plugdev sambashare sudo
_MarkForUpload: True

Revision history for this message
Alex Tu (alextu) wrote : Dependencies.txt

apport information

tags: added: apport-collected bionic
description: updated
Revision history for this message
Alex Tu (alextu) wrote : ProcCpuinfoMinimal.txt

apport information

Revision history for this message
Alex Tu (alextu) wrote : ProcEnviron.txt

apport information

Revision history for this message
Srinivas Pandruvada (srinivas-pandruvada) wrote :

Good observation. We need to update man page.
I am looking for volunteer for updating man page.

Changed in oem-priority:
status: New → Confirmed
importance: Undecided → High
tags: added: somerville
Revision history for this message
Alex Tu (alextu) wrote :

Hi srinivas-pandruvada,
It's a workaround to update man to avoid confusing, but not really help the case which user would like to correct the behavior without change in BIOS.

So, I propose to rename original /etc/thermald/thermal-conf.xml to /etc/thermald/thermal-conf.xml.template then change the code of thermald to read /etc/thermald/thermal-conf.xml first and fall back to /var/run/thermald/thermal-conf.xml.auto if /etc/thermald/thermal-conf.xml is not there.

welcome any idea.

Rex Tsai (chihchun)
tags: added: oem-priority
Revision history for this message
Launchpad Janitor (janitor) wrote :

Status changed to 'Confirmed' because the bug affects multiple users.

Changed in thermald (Ubuntu):
status: New → Confirmed
Revision history for this message
Daniele Dellafiore (ildella) wrote :

Not sure is the same issue, but in boot logs as per `journalctl -b` I always have this error in Ubuntu 19.10:

thermald[1079]: [WARN]error: could not parse file /etc/thermald/thermal-conf.xml

Revision history for this message
Colin Ian King (colin-king) wrote :

Is this still an issue?

Changed in thermald (Ubuntu):
status: Confirmed → Incomplete
importance: Undecided → Medium
Revision history for this message
Yuan-Chen Cheng (ycheng-twn) wrote :

close as no activity

Changed in oem-priority:
status: Confirmed → Won't Fix
Revision history for this message
Launchpad Janitor (janitor) wrote :

[Expired for thermald (Ubuntu) because there has been no activity for 60 days.]

Changed in thermald (Ubuntu):
status: Incomplete → Expired
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.