-
Notifications
You must be signed in to change notification settings - Fork 17.8k
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
x/build: set up ppc64/linux and ppc64le/linux POWER9 builders #30138
Comments
cc @dmitshur |
@ceseo Who are you asking? From the point of view of the Go project, I think anywhere reliable is fine. |
@ianlancetaylor thanks! I'd like the opinion of any of the maintainers before start setting them up. So, I guess I'll set them up at OSU, since our current builders POWER8 are there too. |
I hereby bless continued use of OSU. |
@bradfitz cool, thanks! What sizing did you use for the current POWER8 builders? (disk/CPUs/memory) |
|
Excellent! I'll get this going then. |
We need to decide what builder host type names we should use for the new builders. I see we already have "host-linux-ppc64-osu" and "host-linux-ppc64le-osu" host types: @ceseo In the original issue, you said:
Can you elaborate on how the new instructions are different? What are the exact GOOS and GOARCH values? Is it still Given that "host-linux-ppc64-osu" and "host-linux-ppc64le-osu" host names are already taken, do you think these would be appropriate?
("-osu" suffix since they'll also be running at OSU.) |
I have a change ready that creates a new env var like GOARM for ppc64/ppc64le. POWER9 has a new ISA, so we want to be able to generate these new instructions in SSA and take advantage of extra performance without breaking POWER8. However, we need to be able to test them. That's why we need a builder.
There's no need for a ppc64 (BE) builder, since there are no Linux distributions supporting POWER9 on big endian at the moment. AIX people might want a ppc64 POWER9 builder in the future, though. |
Change https://golang.org/cl/163757 mentions this issue: |
Excellent! Thanks for sending the CL. I will be able to get to it tomorrow or so, but review from others is welcome too. |
This is deployed now. The new builder is showing up on build.golang.org: |
Reopening this as they're failing with:
Looks like some config work remains. |
Yes, I saw that. Sorry for the delay... I was on vacation. Trying to fix it now. |
@bradfitz it seems to be working now. Please see if you're happy with that and then you can close this issue again. |
In preparation to the work I intend to submit for go1.13, we would certainly need new POWER9 builders to test the new instructions.
Do you have any preferences on where to set these up? Is OSU still fine?
Thanks!
The text was updated successfully, but these errors were encountered: