thermald: Add support for Kabylake and Broadwell-GT processor models

Bug #1683829 reported by Colin Ian King on 2017-04-18
12
This bug affects 1 person
Affects Status Importance Assigned to Milestone
thermald (Ubuntu)
Medium
Colin Ian King
Xenial
Undecided
Unassigned
Yakkety
Undecided
Unassigned
Zesty
Medium
Colin Ian King

Bug Description

[SRU][XENIAL][YAKKETY][ZESTY]

Newer x86 processor families are supported with thermald upstream commit 405dcc0a6464d92e656789a5a76bb23f479975ea ("Add Kabylake and Broadwell-GT processor models"). It makes sense to add these CPU IDs to supported versions of thermald.

[REGRESSION POTENTIAL]
This fix adds new CPU IDs to the table of supported processors. This fix has the potential of only changing behaviour of CPUs that have been added to the list. The regression potential for existing CPUs is close to zero as this is a table driven check.

[TESTCASE]
Modern CPUs with IDs will be supported with this fix:
     6, 0x47 (Broadwell-GT3E)
     6, 0x8e (Kabylake)
     6, 0x9e (Kabylake)

..so thermald should be able to now detect this with the fix. Without the rix, the thermald log will report "Need Linux PowerCap sysfs", with the fix, this message will not be shown.

Changed in thermald (Ubuntu):
status: New → In Progress
importance: Undecided → Low
assignee: nobody → Colin Ian King (colin-king)
description: updated
Changed in thermald (Ubuntu):
importance: Low → Medium

An upload of thermald to zesty-proposed has been rejected from the upload queue for the following reason: "Contains .pc junk, will be reuploaded.".

Hello Colin, or anyone else affected,

Accepted thermald into zesty-proposed. The package will build now and be available at https://launchpad.net/ubuntu/+source/thermald/1.5.4-4ubuntu1 in a few hours, and then in the -proposed repository.

Please help us by testing this new package. See https://wiki.ubuntu.com/Testing/EnableProposed for documentation on how to enable and use -proposed.Your feedback will aid us getting this update out to other Ubuntu users.

If this package fixes the bug for you, please add a comment to this bug, mentioning the version of the package you tested, and change the tag from verification-needed to verification-done. If it does not fix the bug for you, please add a comment stating that, and change the tag to verification-failed. In either case, details of your testing will help us make a better decision.

Further information regarding the verification process can be found at https://wiki.ubuntu.com/QATeam/PerformingSRUVerification . Thank you in advance!

Changed in thermald (Ubuntu Zesty):
status: In Progress → Fix Committed
tags: added: verification-needed
Changed in thermald (Ubuntu Yakkety):
status: New → Fix Committed
Andy Whitcroft (apw) wrote :

Hello Colin, or anyone else affected,

Accepted thermald into yakkety-proposed. The package will build now and be available at https://launchpad.net/ubuntu/+source/thermald/1.5.3-4ubuntu2 in a few hours, and then in the -proposed repository.

Please help us by testing this new package. See https://wiki.ubuntu.com/Testing/EnableProposed for documentation on how to enable and use -proposed.Your feedback will aid us getting this update out to other Ubuntu users.

If this package fixes the bug for you, please add a comment to this bug, mentioning the version of the package you tested, and change the tag from verification-needed to verification-done. If it does not fix the bug for you, please add a comment stating that, and change the tag to verification-failed. In either case, details of your testing will help us make a better decision.

Further information regarding the verification process can be found at https://wiki.ubuntu.com/QATeam/PerformingSRUVerification . Thank you in advance!

Changed in thermald (Ubuntu Xenial):
status: New → Fix Committed
Andy Whitcroft (apw) wrote :

Hello Colin, or anyone else affected,

Accepted thermald into xenial-proposed. The package will build now and be available at https://launchpad.net/ubuntu/+source/thermald/1.5-2ubuntu4 in a few hours, and then in the -proposed repository.

Please help us by testing this new package. See https://wiki.ubuntu.com/Testing/EnableProposed for documentation on how to enable and use -proposed.Your feedback will aid us getting this update out to other Ubuntu users.

If this package fixes the bug for you, please add a comment to this bug, mentioning the version of the package you tested, and change the tag from verification-needed to verification-done. If it does not fix the bug for you, please add a comment stating that, and change the tag to verification-failed. In either case, details of your testing will help us make a better decision.

Further information regarding the verification process can be found at https://wiki.ubuntu.com/QATeam/PerformingSRUVerification . Thank you in advance!

Kai-Heng Feng (kaihengfeng) wrote :

thermald in -proposed works on KBL now. Thanks.

tags: added: verification-done
removed: verification-needed
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package thermald - 1.5.4-4ubuntu1

---------------
thermald (1.5.4-4ubuntu1) zesty; urgency=medium

   * Add missing support for Kabylake and Broadwell-GT (LP: #1683829)
    - upstream commit 405dcc0a6464d92, add some more CPU IDs
   * Fix energy preference default (LP: #1683825)
    - 37a25879239466f ("set default preference"); set the default
      energy preference to ENERGY_CONSERVE rather than a garbage value

 -- Colin King <email address hidden> Tue, 18 Apr 2017 15:42:01 +0100

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

Other bug subscribers