Skip to content

system_monitor_service: add Tctl to the list of valid CPU temp labels#3002

Closed
LionHeartP wants to merge 1 commit into
noctalia-dev:mainfrom
LionHeartP:main
Closed

system_monitor_service: add Tctl to the list of valid CPU temp labels#3002
LionHeartP wants to merge 1 commit into
noctalia-dev:mainfrom
LionHeartP:main

Conversation

@LionHeartP

Copy link
Copy Markdown
Contributor

Summary

Adds Tctl to the list of valid CPU temp labels

Motivation

On my system, I have no Tdie sensor. The fallback to any "CPU" label that exists can poll the wrong sensor resulting in 0C or invalid readings overall

❯ grep "" /sys/class/hwmon/hwmon*/temp*_label
/sys/class/hwmon/hwmon0/temp1_label:Composite
/sys/class/hwmon/hwmon1/temp1_label:Composite
/sys/class/hwmon/hwmon2/temp1_label:CPU
/sys/class/hwmon/hwmon2/temp2_label:System
/sys/class/hwmon/hwmon2/temp3_label:VRM MOS
/sys/class/hwmon/hwmon2/temp4_label:PCH
/sys/class/hwmon/hwmon2/temp5_label:CPU Socket
/sys/class/hwmon/hwmon2/temp6_label:PCIe x1
/sys/class/hwmon/hwmon2/temp7_label:M2_1
/sys/class/hwmon/hwmon3/temp1_label:edge
/sys/class/hwmon/hwmon3/temp2_label:junction
/sys/class/hwmon/hwmon3/temp3_label:mem
/sys/class/hwmon/hwmon4/temp1_label:edge
/sys/class/hwmon/hwmon5/temp1_label:Tctl
/sys/class/hwmon/hwmon5/temp3_label:Tccd1
/sys/class/hwmon/hwmon5/temp4_label:Tccd2

Type of Change

  • Bug fix
  • New feature
  • Breaking change
  • Refactoring
  • Build / packaging

Testing

Manual Coverage

  • Tested on Niri
  • Tested on Hyprland
  • Tested on Sway
  • Tested on another compositor:
  • Tested with different bar positions and density settings
  • Tested at different interface scaling values
  • Tested with multiple monitors

Screenshots / Videos

Checklist

  • This PR is ready for review
  • I read and followed the relevant guidance in CONTRIBUTING.md.
  • this PR has no code changes.
  • I ran the relevant build or test commands, or explained why they were not run.
  • I self-reviewed the changes.
  • I checked for new warnings or errors.
  • this PR does not change user-facing configuration or behavior.
  • or this PR adds no new user-facing strings.
  • I did not edit non-English translation files unless this PR is explicitly for translation tooling, an import/export sync, or a maintainer-requested locale change.
  • I used the existing canonical names for config keys, IPC names, paths, and identifiers.

Additional Notes

@ItsLemmy

Copy link
Copy Markdown
Collaborator

Thanks for the PR and the report. As discussed on discord, I went another direction after gathering all the data.

@ItsLemmy ItsLemmy closed this Jun 14, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants