Skip to content
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

M4 – The CPU frequency is wrong #11

Closed
vladkens opened this issue Nov 29, 2024 · 7 comments
Closed

M4 – The CPU frequency is wrong #11

vladkens opened this issue Nov 29, 2024 · 7 comments

Comments

@vladkens
Copy link
Owner

          The CPU frequency is wrong in my case. It says @ 1MHz. I'm on 15.1.1.
image

Originally posted by @plmi in #5 (comment)

@vladkens
Copy link
Owner Author

Hi, @plmi.

I see you are using an M4, which I have yet had a chance to test on.

Can you give me please debug log by running macmon debug? Like it was in this issue #9 (comment)

@hungrydonkey
Copy link

@vladkens Hi, I'm using an M4 and have the same issue. I'd like to provide the debug log.
M4_debug.txt

vladkens added a commit that referenced this issue Nov 30, 2024
@vladkens
Copy link
Owner Author

vladkens commented Nov 30, 2024

Hi, @hungrydonkey. Thank you for logs.

Some strange value in voltage-states1-sram: (f) and voltage-states5-sram: (f). The M1-M3 chips had a value in Hz, and I converted them to MHz. But now macmon shows the values as 0 1 1 1 1 2 2 2 2, which looks like the values are now stored in KHz (This may have sense because there is a value 4 on P-Core, which would result in an integer overflow - (2 ** 32 - 1) / 1000 / 1000 = 4294 MHz max).

Can you check a fix please? Here is latest build: https://github.com/vladkens/macmon/actions/runs/12099700778

Download archive and run macmon binary. If frequencies still showed wrong, can you run ./macmon debug again on this build – I updated debug log to show raw Hz values.

Also because file in quarantine by default, you should remove it first:

/usr/bin/xattr -d com.apple.quarantine ./macmon

@hungrydonkey
Copy link

hungrydonkey commented Dec 1, 2024

@vladkens Ok, here is the new log.
M4_debug_with_the_potential_fix.txt

@hungrydonkey
Copy link

The cpu frequency seems to be right, but the gpu is not.
new

vladkens added a commit that referenced this issue Dec 1, 2024
@vladkens
Copy link
Owner Author

vladkens commented Dec 1, 2024

@hungrydonkey Ah yeah, I missed that the GPU values are still in Hz.

Thank you for debugging information. M4 support released in v0.4.0. To update, run:

brew update && brew upgrade macmon

@hungrydonkey
Copy link

@hungrydonkey Ah yeah, I missed that the GPU values are still in Hz.

Thank you for debugging information. M4 support released in v0.4.0. To update, run:

brew update && brew upgrade macmon

@vladkens Very nice, thank you very much!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants