Skip to content

Enable k10temp kernel module#2094

Merged
agners merged 1 commit intohome-assistant:devfrom
agners:enable-k10temp-sensor
Aug 31, 2022
Merged

Enable k10temp kernel module#2094
agners merged 1 commit intohome-assistant:devfrom
agners:enable-k10temp-sensor

Conversation

@agners
Copy link
Member

@agners agners commented Aug 30, 2022

The k10temp module allows to monitor CPU temperature on AMD systems.

The k10temp module allows to monitor CPU temperature on AMD systems.
@agners agners added the board/generic-x86-64 Generic x86-64 Boards (like Intel NUC) label Aug 30, 2022
@agners agners merged commit 4edc71b into home-assistant:dev Aug 31, 2022
Copy link

@bearhntr bearhntr left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

When and How will I see this change? Will it come out in a 2022.x.x release?

@agners
Copy link
Member Author

agners commented Sep 1, 2022

OS releases are separate. This will be part of OS 9.0. Currently a development build is on its way. If you enable the development channel it should be part of 9.0.dev20220831.

@agners agners deleted the enable-k10temp-sensor branch September 1, 2022 07:37
@bearhntr
Copy link

bearhntr commented Sep 1, 2022 via email

@agners
Copy link
Member Author

agners commented Sep 1, 2022

If you want, you can verify if CPU temp indeed works with this change. You can install the latest development build from https://os-builds.home-assistant.io/9.0.dev20220831/ or use the built-in update mechanism.

ha su options --channel dev
ha su reload
ha os update

Switch back to stable channel afterwards

ha su options --channel beta
ha su reload

@bearhntr
Copy link

bearhntr commented Sep 1, 2022 via email

@agners
Copy link
Member Author

agners commented Sep 2, 2022

Tried the commands you gave - and I got this:

"This" did not make it to GitHub 😅

@bearhntr
Copy link

bearhntr commented Sep 2, 2022 via email

@agners
Copy link
Member Author

agners commented Sep 2, 2022

@bearhntr yeah I don't see anything here, you can't use email response for this it seems.

@bearhntr
Copy link

bearhntr commented Sep 2, 2022 via email

@agners
Copy link
Member Author

agners commented Sep 3, 2022

Check ha os info, and use ha os update (not core).

@bearhntr
Copy link

bearhntr commented Sep 3, 2022 via email

@agners
Copy link
Member Author

agners commented Sep 5, 2022

Hm, weird that the systemmonitor does not pick it up. On my AMD system it seems to generate the following labels like Tctl and CPU, which are in the list of labels classified as CPU temperature (see https://github.com/home-assistant/core/blob/dev/homeassistant/components/systemmonitor/sensor.py#L291).

Would be interesting to see what your system produces exactly, from the OS shell:

docker exec -it homeassistant /bin/bash
python3
>>> import psutil
>>> psutil.sensors_temperatures()

@agners
Copy link
Member Author

agners commented Sep 14, 2022

FYI, this should fix the temperature reading with systemmonitor integration: home-assistant/core#78472

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

board/generic-x86-64 Generic x86-64 Boards (like Intel NUC)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants