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

Use OpenBLAS directly on POWER for foss 2021/* toolchains #13836

Closed

Conversation

branfosj
Copy link
Member

@branfosj branfosj commented Aug 26, 2021

(created using eb --new-pr)

I've created this for discussion and marked it as draft while we decide if this is a suitable approach.

For foss/2021a and foss/2021.07 (candidate for foss/2021b) we see a seg fault when running the numpy tests on top of FlexiBLAS with OpenBLAS as the backend (#12968). However, we do not see this issue if we remove FlexiBLAS and use OpenBLAS directly.

This PR keeps using FlexiBLAS / OpenBLAS for other architectures, but uses the architecture specific dependencies to allow us to use OpenBLAS directly on POWER.

With this I can build #13789 with the numpy and scipy test suites passing.

@branfosj branfosj marked this pull request as draft August 26, 2021 10:12
@branfosj branfosj changed the title Use OpenBLAS directly on POWER for foss toolchain Use OpenBLAS directly on POWER for foss 2021/* toolchains Aug 26, 2021
@branfosj branfosj closed this Aug 28, 2021
@branfosj branfosj deleted the 20210826111058_new_pr_foss202107 branch August 28, 2021 13:38
@branfosj branfosj restored the 20210826111058_new_pr_foss202107 branch August 28, 2021 13:38
@branfosj branfosj reopened this Aug 28, 2021
@boegel boegel added this to the next release (4.4.2?) milestone Sep 1, 2021
@branfosj branfosj marked this pull request as ready for review September 1, 2021 08:20
@branfosj
Copy link
Member Author

I've put this back to draft, for two reasons:

@boegel boegel modified the milestones: 4.5.1, release after 4.5.1 Dec 8, 2021
@boegel boegel modified the milestones: 4.5.2, release after 4.5.2 Jan 18, 2022
@boegel boegel modified the milestones: 4.5.3, release after 4.5.3 Feb 9, 2022
@branfosj
Copy link
Member Author

There is information in easybuilders/easybuild-framework#3809 (comment) on using an alternative toolchain on POWER9 for those wanting 2021a and newer. I am no longer working on this, so I am closing this.

@boegel
Copy link
Member

boegel commented Aug 6, 2022

Proper workaround for allowing the use of FlexiBLAS on POWER is implemented in #15885, so there's no need anymore for special workarounds for POWER by mangling the definition of the foss toolchain...

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants