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

magma: fix __ieee128 is undefined on ppc64le with glibc 2.26 #11930

Merged

Conversation

branfosj
Copy link
Member

@branfosj branfosj commented Dec 23, 2020

(created using eb --new-pr)

This is the magma part of #11913. The 2019a version also includes the CUDA SM patch from #11861 (which is added to avoid needing different patches for the 2019a and 2019b versions of magma).

CUDA 11 does not require this fix, so it is not applied to magma in 2020a/b toolchains.

@branfosj
Copy link
Member Author

Test report by @branfosj
SUCCESS
Build succeeded for 2 out of 2 (2 easyconfigs in total)
bear-pg0306u11a.bear.cluster - Linux RHEL 8.2, POWER, 8335-GTX (power9le), Python 3.6.8
See https://gist.github.com/d7ec006c3e43313e7744fad7ef356228 for a full test report.

@branfosj
Copy link
Member Author

Test report by @branfosj
SUCCESS
Build succeeded for 2 out of 2 (2 easyconfigs in total)
bear-pg0212u17a.bear.cluster - Linux centos linux 8.2.2004, x86_64, Intel(R) Xeon(R) CPU E5-2640 v4 @ 2.40GHz (broadwell), Python 3.6.8
See https://gist.github.com/be76970e74957d67bd7fd30bc14bca44 for a full test report.

@boegel boegel added the bug fix label Dec 24, 2020
@boegel boegel added this to the next release (4.3.3?) milestone Dec 24, 2020
@boegel
Copy link
Member

boegel commented Dec 27, 2020

@boegelbot please test @ generoso
CORE_CNT=16

@boegelbot
Copy link
Collaborator

@boegel: Request for testing this PR well received on generoso

PR test command 'EB_PR=11930 EB_ARGS= /apps/slurm/default/bin/sbatch --job-name test_PR_11930 --ntasks="16" ~/boegelbot/eb_from_pr_upload_generoso.sh' executed!

  • exit code: 0
  • output:
Submitted batch job 12345

Test results coming soon (I hope)...

- notification for comment with ID 751444526 processed

Message to humans: this is just bookkeeping information for me,
it is of no use to you (unless you think I have a bug, which I don't).

Copy link
Member

@boegel boegel left a comment

Choose a reason for hiding this comment

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

lgtm

@boegel
Copy link
Member

boegel commented Dec 27, 2020

Test report by @boegel
SUCCESS
Build succeeded for 2 out of 2 (2 easyconfigs in total)
node3307.joltik.os - Linux centos linux 7.9.2009, x86_64, Intel(R) Xeon(R) Gold 6242 CPU @ 2.80GHz (cascadelake), Python 3.6.8
See https://gist.github.com/93d5eea09477f962de237a95a78d114a for a full test report.

@boegel
Copy link
Member

boegel commented Dec 27, 2020

Test report by @boegel
SUCCESS
Build succeeded for 2 out of 2 (2 easyconfigs in total)
node3563.doduo.os - Linux RHEL 8.2, x86_64, AMD EPYC 7552 48-Core Processor (zen2), Python 3.6.8
See https://gist.github.com/e5a7c26cb945fb001de2192d5dceb8f9 for a full test report.

@boegel
Copy link
Member

boegel commented Dec 27, 2020

Going in, thanks @branfosj!

@boegel boegel merged commit aa74459 into easybuilders:develop Dec 27, 2020
@branfosj branfosj deleted the 20201223190623_new_pr_magma254 branch December 27, 2020 16:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants