Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Take temperature from thermal zones #341

Closed
Camerooooon opened this issue Jun 18, 2022 · 16 comments · Fixed by #347
Closed

Take temperature from thermal zones #341

Camerooooon opened this issue Jun 18, 2022 · 16 comments · Fixed by #347
Assignees
Labels
enhancement New ideas or features to add
Milestone

Comments

@Camerooooon
Copy link
Collaborator

We should consider thermal zones as an alternative to systems that do not have individual cpu temperature read outs.

@Shuzhengz
Copy link
Collaborator

yep good idea

@Camerooooon
Copy link
Collaborator Author

I would also like to add that thermal zones could also be displayed in addition to regular cpu core temp read outs in acs monit.

@Shuzhengz
Copy link
Collaborator

Shuzhengz commented Jun 18, 2022

im pretty sure most hardware monitoring software have a package temp somewhere, so we could just add it in

@JakeRoggenbuck JakeRoggenbuck added the enhancement New ideas or features to add label Jun 18, 2022
@JakeRoggenbuck JakeRoggenbuck added this to the 0.1.10 milestone Jun 18, 2022
@Camerooooon
Copy link
Collaborator Author

We should take a look at the source for the "sensors" application. That program was able to read out core temps on my server where auto clock speed otherwise wasn't.

@Camerooooon
Copy link
Collaborator Author

We should take a look at the source for the "sensors" application. That program was able to read out core temps on my server where auto clock speed otherwise wasn't.

I suspect that they use thermal zones in order to achieve this

@Shuzhengz
Copy link
Collaborator

that would make sense since hyperthreaded cores are not physical

@Shuzhengz
Copy link
Collaborator

ok i just tested on my other laptop, and turns out that one has 19 thermal zones instead of 2
we need to find the one thats x86_pkg_temp type and use the package temperature instead of a bunch of virtual temps

@Shuzhengz
Copy link
Collaborator

i suspect it's caused by intel vpro

@Camerooooon
Copy link
Collaborator Author

ok i just tested on my other laptop, and turns out that one has 19 thermal zones instead of 2

we need to find the one thats x86_pkg_temp type and use the package temperature instead of a bunch of virtual temps

Are the other thermal zones still useful to us?

1 similar comment
@Camerooooon
Copy link
Collaborator Author

ok i just tested on my other laptop, and turns out that one has 19 thermal zones instead of 2

we need to find the one thats x86_pkg_temp type and use the package temperature instead of a bunch of virtual temps

Are the other thermal zones still useful to us?

@Shuzhengz
Copy link
Collaborator

after running stress test it seems like all the other sensors are virtual and the value of the temperature does not change

@Shuzhengz
Copy link
Collaborator

after running cat /sys/class/thermal/thermal_zone*/type it shows the following
acpitz SEN8 SEN9 SENA SENB SENC SEND pch_cannonlake B0D4 x86_pkg_temp iwlwifi_1 INT3400_Thermal SEN0 SEN1 SEN2 SEN3 SEN4 SEN5 SEN6 SEN7

@Shuzhengz
Copy link
Collaborator

it seems like all the SEN* types have static value
iwlwifi is probably the dedicated wifi card

@Shuzhengz
Copy link
Collaborator

Shuzhengz commented Jun 18, 2022

the MSR address 0x1b1 also outputs the package temperature in hex (you have to convert it into actual readings using the 22:16 output bitfield)

@Camerooooon
Copy link
Collaborator Author

Is it alrighty if I take this issue? @Shuzhengz

@Shuzhengz
Copy link
Collaborator

o yee

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New ideas or features to add
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants