-
Notifications
You must be signed in to change notification settings - Fork 143
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
iMac drive bay temperature shown as 249 º C. #48
Comments
Note that all other temperatures seam reasonable (except perhaps ambient air temperature which is very low 4 º C up to 8 º C for iMacs). |
Clone cjbassi#176 |
@mario-grgic, are you still having this issue, and are you willing to help in debugging this? I don't have access to a Mac (or, as may be necessary with this ticket, many Macs) to track the issue down. |
In the current version 3.5.0 hard drive bay temperature is no longer displayed as a statistic, so it's hard to tell if it is fixed per se. The temperatures that are displayed look plausible, except the ambient air temperature which shows 8 º C. |
@mario-grgic , I've created a little program to dump the temperature values directly from SMC. You can download the tarball from here; the link will be valid through Apr 26. The archive contains the source for the program and a compiled executable; you can also compile it yourself with a simple If you would please run the program and post the output, it'd help me see what the raw values are coming out of your computer. |
Here is the output from 2017 15'' Macbook Pro:
And from 2017 27'' iMac:
|
Crap. I forgot carriage returns in the Printfs. Oh well; I can reformat it, thanks. |
The numbers being returned by the C code still look weird, and that's coming directly from SMC calls. There could be a bug in There's another project that provides binaries to do a similar data dump using different SMC libraries. Do you mind downloading one of their releases and providing the temperature output? If those look more reasonable, that project uses a different library; that uses a C binding with an even older copyright, but if it produces more reasonable data, I can replace |
Here is the output of iSMC on 2017 27'' iMac:
|
Well, there's one thing. On the 27", Ok, I've pulled out the relevant code from |
Here it is
|
Much better, thank you. This was a big help, and I should be able to fix this from here. If you get a chance at some point to post results from your 15", just so we can verify that the values are also sane on that device, that'd be great. |
@mario-grgic here's a gotop using iSMC. It's gzip'ed, so you'll have to
Assuming that you do something like:
|
Is there a branch for this code I can checkout and build? |
There is now. Pull branch |
The temperatures look reasonable now. Not sure which ones should be displayed in the most common case (when you just launch gotop with no options). Right now only the top of the list of all available temp stats (that is displayed with |
Yes, there are a ton of temperatures reported on OSX; maybe on Linux, too, but Caleb may have been filtering them. I'm going to mark this as fixed, but I'm also going to add a configuration option so users can specify which temps they want to see, and I'll default to a couple of the main ones as you suggest. Thanks for the report, the help with tracking down the issue, and testing the solution. |
@mario-grgic If you get a chance, could you pull and build the ismc branch? This turned into a bit of a rabbit hole. I added sensible defaults for the thermal sensors, but this required changing how devices are configured and set up. If you wouldn't mind testing a couple things, if you can verify that they work on OSX I'll include this in the 3.6 release. First, verify that the defaults are sensible:
and you should see 5 sensors: one or two CPUs, GPU, drive, memory. Then try:
and you should see all of the Mac temperature sensors. Finally, test configuring the sensors you want to see. I am not enabling a command line option for this, so it has to be done in the config file.
Edit the config file that is printed from the previous command, and uncomment the last line, which should be Thanks again. |
Latest branch build leads to:
|
Ooof. Thanks. I wish I could find a Darwin VM on which to test these things. Hold on a tic. |
Ok, that was dumb. Try again, please. |
Ok, this is much better. The only problem I can think of is that it is going to be hard to come up with defaults across hardware. The defaults that make sense on iMac say are different than I would want to display on MacBook Pro, and are different than the ones on I would use on 12'' Macbook (no fans at all, or discrete GPU). But at least users have the chance to pick themselves what they want displayed. |
Thanks again. |
gotop v3.0.0 displays hard_drive_bay temperature as 249 º C on two different 27'' iMacs (both have fusion drives), but displays correct information on 12'' Macbook and 15'' Macbook Pro.
Required information:
gotop version (
gotop -V
):3.0.0
The output of
uname -a
:Darwin Jupiter.local 19.0.0 Darwin Kernel Version 19.0.0: Wed Sep 25 20:18:50 PDT 2019; root:xnu-6153.11.26~2/RELEASE_X86_64 x86_64
Terminal emulator (e.g. iTerm or gnome terminal):
Apple Terminal.app or iTerm.
Any releveant hardware info:
iMac (Retina 5K, 27-inch, 2017)
3.4 GHz Quad-Core Intel Core i5
40 GB 2400 MHz DDR4
Radeon Pro 570 4 GB
APPLE HDD ST1000DM003:
Capacity: 1 TB (1,000,204,886,016 bytes)
Model: APPLE HDD ST1000DM003
Revision: AQ04
Native Command Queuing: Yes
Queue Depth: 32
Removable Media: No
Detachable Drive: No
BSD Name: disk1
Medium Type: Rotational
Partition Map Type: GPT (GUID Partition Table)
S.M.A.R.T. status: Verified
Volumes:
EFI:
Capacity: 209.7 MB (209,715,200 bytes)
File System: MS-DOS FAT32
BSD Name: disk1s1
Content: EFI
Volume UUID: BDC1974F-6B8C-3DAE-9DB2-3AA3C17BF506
disk1s2:
Capacity: 1 TB (999,995,129,856 bytes)
BSD Name: disk1s2
Content: Apple_APFS
tmux version if using tmux:
Not using tmux
Also please copy or attach
~/.local/state/gotop/errors.log
if it exists and contains logs:errors.log is empty
The text was updated successfully, but these errors were encountered: