Skip to content

cpuinfo: PPC64 support #2

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
Open
leitao opened this issue Apr 11, 2018 · 4 comments
Open

cpuinfo: PPC64 support #2

leitao opened this issue Apr 11, 2018 · 4 comments
Labels
enhancement

Comments

@leitao
Copy link

leitao commented Apr 11, 2018

Hello @Maratyszcza and team,

I would like to have PPC64 support on cpuinfo project. I was able to fork it[1], and together with some friends (Rogério and Alfredo), got it partially ported to PPC64. There are some missing things, but, at this moment, all the tests are passing.

That said, I have some questions regarding the PPC64 port:

  1. Are you interested in adding PPC64 support into cpuinfo project?
  2. If you kindly accept our contribution, what is the best method to contribute it?
  3. Do you want to have the PPC64 support fully implemented before us trying to contribute, or, can we start the review process with what the current code, while we get the port complete.

Thank you,
Breno

[1] https://github.com/leitao/cpuinfo/
@leitao leitao changed the title PPC64 support cpuinfo: PPC64 support Apr 11, 2018
@Maratyszcza
Copy link
Contributor

Maratyszcza commented Apr 13, 2018 •
edited
Loading

Hi @leitao and friends,

Thank you for working on PPC64 port!
Here're the answers to your questions:

  1. I'm interested in accepting PPC64 port into cpuinfo, but don't have resources to work on it myself.
  2. The preferred way of contribution is through a pull request to this repo. Your contributions must be licensed under the same license as cpuinfo (2-clause "Simplified" BSD).
  3. As I expect PPC64 port to have quite a lot of code, I suggest you create PRs for a non-master branch of cpuinfo (e.g. ppc64 branch), so we can start review early. When the port is complete, we'll merge it into master.

@cdluminate
Copy link

It looks that ppc64el support is missing.

@Adesoj
Copy link

Adesoj commented Dec 18, 2018

That means the ppc64 is missing

@jaesharp jaesharp mentioned this issue Dec 7, 2020
Open
@kiplingw
Copy link
Contributor

I think this is a great idea. @madscientist159 of Raptor Computing or IBM should be able to provide a test bed to experiment with.

@antoinecarme antoinecarme mentioned this issue Apr 17, 2022
Open
@kartheekgottipati kartheekgottipati mentioned this issue Apr 11, 2023
Closed
BODAPATIMAHESH added a commit to BODAPATIMAHESH/cpuinfo that referenced this issue May 5, 2025
@BODAPATIMAHESH
Remove redundant 'disabled' declaration for PowerPC64 Linux
33cc865 
The 'disabled' field was declared for configurations other than SMT8
mode, but it is already handled during PowerPC64 Linux system
initialization. This change removes the redundant code.
make test:
    Start 1: init-test
1/3 Test pytorch#1: init-test ........................   Passed    0.02 sec
    Start 2: get-current-test
2/3 Test pytorch#2: get-current-test .................   Passed    0.02 sec
    Start 3: power-features-test
3/3 Test pytorch#3: power-features-test ..............   Passed    0.02 sec

100% tests passed, 0 tests failed out of 3
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
6 participants
@leitao @Maratyszcza @kiplingw @cdluminate @prashanthswami @Adesoj