On Mon, Apr 25, 2022 at 11:31:22AM -0400, Bryan Steele wrote: > On Mon, Apr 25, 2022 at 05:20:46PM +0200, Claudio Jeker wrote: > > On Sun, Apr 24, 2022 at 07:06:19PM +0200, Claudio Jeker wrote: > > > On Ryzen CPUs each CCD has a temp sensor. If the CPU has CCDs (which > > > excludes Zen APU CPUs) this should show additional temp info. This is > > > based on info from the Linux k10temp driver. > > > > > > Additionally use the MSRs defined in "Open-Source Register Reference For > > > AMD Family 17h Processors" to measure the CPU core frequency. > > > That should be the actuall speed of the CPU core during the measuring > > > interval. > > > > > > On my T14g2 the output is now for example: > > > ksmn0.temp0 63.88 degC Tctl > > > ksmn0.frequency0 3553141515.00 Hz CPU0 > > > ksmn0.frequency1 3549080315.00 Hz CPU2 > > > ksmn0.frequency2 3552369937.00 Hz CPU4 > > > ksmn0.frequency3 3546055048.00 Hz CPU6 > > > ksmn0.frequency4 3546854449.00 Hz CPU8 > > > ksmn0.frequency5 3543869698.00 Hz CPU10 > > > ksmn0.frequency6 3542551127.00 Hz CPU12 > > > ksmn0.frequency7 4441623647.00 Hz CPU14 > > > > > > It is intresting to watch turbo kick in and how temp causes the CPU to > > > throttle. > > > > > > I only tested this on systems with APUs so I could not thest the Tccd temp > > > reporting. > > > > With the frequence sensor moved to cpu(4) this just adds the Tccd > > additional temparature sensors. It does not fix the duplication of the > > ksmn(4) sensors. That needs to be fixed by no attaching on the duplicate > > root complexes. How that is done I still need to figure out. > > > > -- > > :wq Claudio > > I think this looks good now, I'm not too worried about the tCTL offsets. > That can be figured out later. ok brynet@ > > As for the attaching issue, would only attaching one device in the > kernel device be a nasty hack? If I understand, this should make the > duplicate complex attach as ppb(4) again. > > ksmn0 at pci? >
But that would break systems with multiple CPU sockets. Like the two socket EPYC server. -- :wq Claudio
