Project

General

Profile

Feature #28567

Add CPU temperature graph in Reporting menu on AMD Ryzen

Added by Roland Buczyński over 1 year ago. Updated 7 months ago.

Status:
Done
Priority:
Nice to have
Assignee:
Benno Rice
Category:
OS
Target version:
Estimated time:
Severity:
Low Medium
Reason for Closing:
Reason for Blocked:
Needs QA:
Yes
Needs Doc:
No
Needs Merging:
No
Needs Automation:
No
Support Suite Ticket:
n/a
Hardware Configuration:

AMD Ryzen 1700
FreeNAS 11.1-U1


Description

Add CPU temperature graphs in Reporting menu for AMD Ryzen processors

I don't know if it's possible in current FreeNAS state because when i'm trying to check temps manually via CLI using
sysctl -a | grep temp

I don't get anything useful
device amdtemp
device coretemp
net.inet6.ip6.use_tempaddr: 0
net.inet6.ip6.temppltime: 86400
net.inet6.ip6.tempvltime: 604800
net.inet6.ip6.prefer_tempaddr: 0
hw.sfxge.restart_attempts: 3
hw.usb.template: 0
kstat.zfs.misc.zcompstats.attempts: 25706636

Same for the
sysctl dev.cpu.0

dev.cpu.0.cx_method: C1/hlt
dev.cpu.0.cx_usage_counters: 409506447
dev.cpu.0.cx_usage: 100.00% last 663us
dev.cpu.0.cx_lowest: C2
dev.cpu.0.cx_supported: C1/1/0
dev.cpu.0.freq_levels: 3400/3562 2700/2835 1550/1312
dev.cpu.0.freq: 3400
dev.cpu.0.%parent: acpi0
dev.cpu.0.%pnpinfo: _HID=none _UID=0
dev.cpu.0.%location: handle=\_PR
.P000
dev.cpu.0.%driver: cpu
dev.cpu.0.%desc: ACPI CPU _

no values.jpg (206 KB) no values.jpg Christopher Manns, 12/24/2018 08:59 AM
46347

Related issues

Related to FreeNAS - Bug #35164: Move CPU Temperature from Reporting to the dashboard in the new UIClosed

History

#1 Updated by Dru Lavigne over 1 year ago

  • Category set to Middleware
  • Assignee changed from Release Council to William Grzybowski

William: are you aware of any sysctl values that could be used?

#2 Updated by William Grzybowski over 1 year ago

  • Category changed from Middleware to OS
  • Assignee changed from William Grzybowski to Alexander Motin

Alexander, we have a custom cputemp collectd plugin which uses dev.cpu.X.temperature .

It seems we lack these to be reported on Ryzen? Maybe just a commit we need to backport?

#3 Updated by Roland Buczyński over 1 year ago

#4 Updated by Alexander Motin over 1 year ago

  • Status changed from Not Started to Blocked
  • Reason for Blocked set to Dependent on a related task to be completed

Roland Buczyński wrote:

Maybe this?

https://reviews.freebsd.org/D12217

Yes, that seem to be it. Unfortunately for some reason it is still not merged to FreeBSD stable/11. I've written an email to the commit author.

#5 Updated by Alexander Motin over 1 year ago

  • Status changed from Blocked to Not Started
  • Reason for Blocked deleted (Dependent on a related task to be completed)

Commit author answered that he is not interested in stable/11 branch, but sees no reason why somebody else could not do it. ;)

#6 Updated by Alexander Motin over 1 year ago

  • Assignee changed from Alexander Motin to Benno Rice
  • Reason for Blocked deleted (Dependent on a related task to be completed)

Benno, could you look on backporting of that code to FreeBSD stable/11? I'd expect it to be a pretty mechanical operation.

#7 Updated by Benno Rice over 1 year ago

  • Status changed from Not Started to In Progress

This has already landed in stable/11:

https://svnweb.freebsd.org/base?view=revision&revision=329767

The only thing we'll need to do is add the amdsmn driver to our kernel config if we want to link it in statically. I can confirm that kldloading amdtemp.ko on my Threadripper system automatically loads amdsmn.ko if we want to go that way.

#8 Updated by Benno Rice over 1 year ago

  • Status changed from In Progress to Blocked
  • Reason for Blocked set to Other: make note in comments

We don't yet have the required revisions merged into freenas/os. Once they're in I can add the amdsmn driver to the relevant kernel configs.

#9 Updated by Benno Rice over 1 year ago

  • Status changed from Blocked to In Progress
  • Reason for Blocked deleted (Other: make note in comments)

PR: https://github.com/freenas/build/pull/41

amdtemp driver isn't present in TrueNAS so I won't bother modifying its kernel config.

#10 Updated by Benno Rice over 1 year ago

  • Status changed from In Progress to Done

Build passed, PR merged.

#11 Updated by Dru Lavigne over 1 year ago

  • Target version changed from 11.2-RC2 to 11.2-BETA1
  • Needs Doc changed from Yes to No
  • Needs Merging changed from Yes to No

#12 Updated by Dru Lavigne about 1 year ago

  • Status changed from Done to Ready for Testing

#13 Updated by Bonnie Follweiler about 1 year ago

  • Related to Bug #35164: Move CPU Temperature from Reporting to the dashboard in the new UI added

#14 Updated by Bonnie Follweiler about 1 year ago

I can't test this is the New GUI until bug https://redmine.ixsystems.com/issues/35164 is done

#15 Updated by Bonnie Follweiler about 1 year ago

The cpu temperature reporting works in the old UI but we don't know about the new UI until bug https://redmine.ixsystems.com/issues/35164 is done

#16 Updated by Nick Wolff about 1 year ago

  • Status changed from Ready for Testing to Passed Testing

Marking this as passed as it was an os/services issue and and not connected to new ui

#17 Updated by Dru Lavigne about 1 year ago

  • Status changed from Passed Testing to Done

#18 Updated by Christopher Manns 7 months ago

46347

Nick Wolff wrote:

Marking this as passed as it was an os/services issue and and not connected to new ui

I don't see this in 11.2 stable on the dashboard ( no values or "0") in the new UI (same with CPU usage %). Is there a config change that I need to make somewhere?

#19 Updated by Alexander Motin 7 months ago

Christopher, your problem may be the same or different. Instead of bumping random old ticket please create your own and attach full debug information. Zero CPU usage is likely unrelated to CPU temperature.

Also available in: Atom PDF