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

Add a JSON file with information on cpuid #86

Merged
merged 9 commits into from
Feb 17, 2024
Merged

Conversation

alalazo
Copy link
Member

@alalazo alalazo commented Feb 15, 2024

We'll use this information to detect features on Windows, since the OS doesn't expose them in any other way.

Modifications:

  • Add vendor detection parameters in cpuid.json
  • Add flags in cpuid.json
  • Add cpuid_schema.json
  • Add tests for various architectures using cpuid (I guess a CSV collection of input and output from the registers)
  • Update CI

@alalazo alalazo added the enhancement New feature or request label Feb 15, 2024
@alalazo alalazo marked this pull request as ready for review February 15, 2024 21:06
@alalazo
Copy link
Member Author

alalazo commented Feb 15, 2024

@isuruf I added a JSON file with the information we need from cpuid. I also think I spotted a couple of bugs in py-cpuinfo, but that needs to be verified. I'll submit companion PRs to the other repos and ping you for test-driving them if you're available.

@isuruf
Copy link
Contributor

isuruf commented Feb 15, 2024

Looks great. Thanks.

tgamblin pushed a commit to archspec/archspec that referenced this pull request Feb 17, 2024
This PR builds on #129 and #149 to add support for Windows. Features detection is done using `cpuid`. The `cpuid` calls that we need to detect features are in archspec/archspec-json#86. We also forked [archspec/cpuid](https://github.com/archspec/cpuid) and use it here as a submodule.

Modifications:
- [x] Add a new method to detect CPU features and vendor using `cpuid`
- [x] Add unit tests for the new method, including Windows
- [x] update archspec-json

---------

Co-authored-by: Isuru Fernando <isuruf@gmail.com>
@alalazo alalazo merged commit ad11750 into master Feb 17, 2024
1 check passed
@alalazo alalazo deleted the features/windows branch February 17, 2024 06:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants