Bug 445917 - CPU-Tctl and Tccd missing from filtered sensors
Summary: CPU-Tctl and Tccd missing from filtered sensors
Status: CONFIRMED
Alias: None
Product: ksystemstats
Classification: Frameworks and Libraries
Component: General (show other bugs)
Version: unspecified
Platform: Arch Linux Linux
: NOR normal
Target Milestone: ---
Assignee: KSysGuard Developers
URL:
Keywords:
: 454867 (view as bug list)
Depends on:
Blocks:
 
Reported: 2021-11-22 12:30 UTC by Matthias Mueller
Modified: 2024-02-08 21:03 UTC (History)
15 users (show)

See Also:
Latest Commit:
Version Fixed In:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Matthias Mueller 2021-11-22 12:30:25 UTC
Follow-Up to https://bugs.kde.org/show_bug.cgi?id=438318 as that apparently is fixed. (see https://bugs.kde.org/show_bug.cgi?id=438318#c31)

Unfortunately, GPU Power and CPU-Temperature for the whole CPU and all sensors for the mainboard are still missing/being filtered out while being displayed by 'sensors'
At least on my hardware (Ryzen 3800x and 6900xt on Asrock x470 Taichi)

I still strongly believe that filtering out sensors is the wrong approach, at least without the option to expose the "raw" data or a config file to change which sensors are filtered. Every time new sensors are added by new or obscure hardware, those filters have to be changed in the codebase - and hardcoded filterlists imho are pretty bad practice...
Comment 1 David Redondo 2021-11-22 13:36:24 UTC
Thanks,  as I said there is not intentional filtering going on on the gpu sensors. There is though for CPU sensors, let's have this one for cpu sensors I will create a new issue for gpu sensors (done: 445920)
Comment 2 Matthias Mueller 2021-11-22 13:54:52 UTC
(In reply to David Redondo from comment #1)
> Thanks,  as I said there is not intentional filtering going on on the gpu
> sensors. There is though for CPU sensors, let's have this one for cpu
> sensors I will create a new issue for gpu sensors (done: 445920)

Alright... i personally don't use the cpu sensors currently with systemmonitor as there is the thermal monitor plasmoid, but here's the sensors -u output for k10temp, just in case ;)

k10temp-pci-00c3
Adapter: PCI adapter
Tctl:
  temp1_input: 53.625
Tccd1:
  temp3_input: 49.000

I just noticed that i don't have my mainboard-sensors in lm-sensors on this install (and don't want to search for sensors due to https://gitlab.freedesktop.org/drm/amd/-/issues/1470 for the time being), so the part about the mainboard sensors might be obsolete. I think i recall them missing, too, but i'm not 100% sure and it was still 5.22 i think.
Comment 3 Bill Sun 2022-03-24 05:10:26 UTC
Can we not remove the filter and just add labels to show that these redundant CPU sensors are from another plugin?

k10temp doesn't display per core temps, so technically it's not even redundant.

I have a Ryzen 9 5900X and it's really annoying that I cannot get the overall CPU temp by looking at CCD1 and CCD2. I prefer to put my CPU temp monitor as a widget in my panel as I do overclocking and want to quickly get a feel of the temp behavior with different settings. Without CCD1 and CCD2 temps, I am forced to pick one particular core out of 24(!) and I know that's not a good representation especially when I run single threaded applications.
Comment 4 Arjen Hiemstra 2023-01-25 10:53:04 UTC
*** Bug 454867 has been marked as a duplicate of this bug. ***
Comment 5 Roman Lebedev 2023-10-20 14:15:51 UTC
Due to this "filtering", it is now simply impossible to monitor CPU temperature
on the AMD FX series or processors. Can that filtering please be ripped out?