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

report bogomips with the rest of cpu info #3231

Closed
wants to merge 1 commit into from

Conversation

joewilliams
Copy link

@joewilliams joewilliams commented Jan 22, 2025

Add bogomips to the cpuinfo output.

@joewilliams joewilliams force-pushed the bogo branch 2 times, most recently from f373bf3 to 458ab29 Compare January 22, 2025 21:44
@SuperQ
Copy link
Member

SuperQ commented Jan 22, 2025

This needs a DCO sign-off. You can use git commit -s --amend to add it.

Copy link
Member

@SuperQ SuperQ left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thinking about this, I'm not sure what use this is. Bogomips is not really a useful metric nor useful for inventory.

Any specific reason why this should be included?

@joewilliams
Copy link
Author

@SuperQ we would like to use it as an agnostic "size" of a server. For instance, a server with higher cores * bogomips probably has a bigger compute capacity than a smaller one regardless of CPU vendor, generation, etc.

@SuperQ
Copy link
Member

SuperQ commented Jan 25, 2025

This will not be helpful for your use case. You can not use label values in queries that way, they are treated as opaque string values.

I also don't think you're going to get the value out of bogomips that you think you will. Here's a simple example.

  • My laptop: 13th gen i7-1365U
  • My old VPS host: Xeon E-2236

CPU benchmark comparison shows that the Xeon is about 5-6% slower than my laptop. Although, I'm not sure if that benchmark site results include P and E cores correctly.

But from a bogomips perspective, it says the Xeon is 6799.81, and the laptop is 5376.00. And that still doesn't take into account P and E cores as it shows them to be the same.

I highly suggest you don't use bogomips for any kind of comparisons. They haven't really been useful for 20 years, since the introduction of cpu frequency scaling and hyperthreading.

@joewilliams
Copy link
Author

Thanks for the info!

@joewilliams joewilliams deleted the bogo branch January 25, 2025 21:32
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

Successfully merging this pull request may close these issues.

2 participants