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.

Sign up for GitHub

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

Jump to bottom

Bug fix: CPU frequency unit in header was MHz, now fixed to GHz #155

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Bug fix: CPU frequency unit in header was MHz, now fixed to GHz #155

wants to merge 1 commit into from

Conversation

ajitjadhav28
Copy link

Fixed: #153

Other Changes:

  • typescript correct return type string | null instead of string | undefined
  • commented and added reference in updateCpuCoresFrequencyProc()

@ajitjadhav28
Bug fix: CPU frequency unit in header was MHz, now fixed to GHz
3e5b3e6 
Other Changes:
 - typescript correct return type `string | null` instead of `string | undefined`
 - commented and added reference in `updateCpuCoresFrequencyProc()`
@ajitjadhav28
Copy link
Author

ajitjadhav28 commented Oct 20, 2024 •
edited
Loading

Fix screenshot:
image

The sysfs returns frequency in KHz, previous code divides input frequency twice by 1000. So unit should be GHz.

Source: https://www.kernel.org/doc/Documentation/cpu-freq/user-guide.txt#:~:text=scaling_cur_freq%20%3A%20Current%20frequency%20of%20the,and%20cpufreq%20core%2C%20in%20KHz

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

CPU Frequency header shows MHz instead of GHz
1 participant
@ajitjadhav28