Skip to content

add elfutils as build dependency for Clang 11.0.1 built with gcccuda toolchain#13008

Merged
boegel merged 1 commit intoeasybuilders:developfrom
nordmoen:clang_offload_fix
Jun 3, 2021
Merged

add elfutils as build dependency for Clang 11.0.1 built with gcccuda toolchain#13008
boegel merged 1 commit intoeasybuilders:developfrom
nordmoen:clang_offload_fix

Conversation

@nordmoen
Copy link
Copy Markdown
Contributor

@nordmoen nordmoen commented Jun 2, 2021

If elfutils is missing during build, CUDA offloading support is not enabled as one would expect with the gcccuda toolchain.

This fixes #13006

@boegel boegel changed the title {compiler}[Clang/gcccuda-2020b] Add elfutils as build dependency add elfutils as build dependency for Clang 11.0.1 built with gcccuda toolchain Jun 2, 2021
@boegel boegel added the bug fix label Jun 2, 2021
@boegel boegel added this to the release after 4.4.0 milestone Jun 2, 2021
@boegel
Copy link
Copy Markdown
Member

boegel commented Jun 2, 2021

@nordmoen Can we come up with a sanity check command that verifies whether Clang was built with offload support, to catch this problem going forward?

@boegel
Copy link
Copy Markdown
Member

boegel commented Jun 2, 2021

@nordmoen Can we come up with a sanity check command that verifies whether Clang was built with offload support, to catch this problem going forward?

Ah, that's taken care of in easybuilders/easybuild-easyblocks#2454, excellent.

One small thing: if you use fixes in the PR description when referring to an open issue, the issue will be closed as soon as the PR is merged, that's a bit less bookkeeping for us (I've tweaked solves to fixes in the description of this PR for that reason).

@boegel
Copy link
Copy Markdown
Member

boegel commented Jun 2, 2021

@boegelbot please test @ generoso
EB_ARGS="--include-easyblocks-from-pr 2454"

@boegel
Copy link
Copy Markdown
Member

boegel commented Jun 2, 2021

Test report by @boegel
Using easyblocks from PR(s) easybuilders/easybuild-easyblocks#2454
FAILED
Build succeeded for 0 out of 1 (1 easyconfigs in total)
node3505.doduo.os - Linux RHEL 8.2, x86_64, AMD EPYC 7552 48-Core Processor (zen2), Python 3.6.8
See https://gist.github.com/4323d6178064b72e5ae04a29fafbfa79 for a full test report.

@boegelbot
Copy link
Copy Markdown
Collaborator

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

PR test command 'EB_PR=13008 EB_ARGS="--include-easyblocks-from-pr 2454" /apps/slurm/default/bin/sbatch --job-name test_PR_13008 --ntasks=4 ~/boegelbot/eb_from_pr_upload_generoso.sh' executed!

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

Test results coming soon (I hope)...

Details

- notification for comment with ID 852945577 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).

@boegelbot
Copy link
Copy Markdown
Collaborator

Test report by @boegelbot
Using easyblocks from PR(s) easybuilders/easybuild-easyblocks#2454
FAILED
Build succeeded for 0 out of 1 (1 easyconfigs in total)
generoso-c1-s-1 - Linux centos linux 8.2.2004, x86_64, Intel(R) Xeon(R) CPU E5-2667 v3 @ 3.20GHz (haswell), Python 3.6.8
See https://gist.github.com/25535d070e043ed0dce4ed9187707e9b for a full test report.

@nordmoen
Copy link
Copy Markdown
Contributor Author

nordmoen commented Jun 2, 2021

The failures are due to missing --cuda-compute-capabilities flag to EB

@boegel
Copy link
Copy Markdown
Member

boegel commented Jun 2, 2021

@boegelbot please test @ generoso
EB_ARGS="--include-easyblocks-from-pr 2454 --cuda-compute-capabilities 7.0"

@boegelbot
Copy link
Copy Markdown
Collaborator

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

PR test command 'EB_PR=13008 EB_ARGS="--include-easyblocks-from-pr 2454 --cuda-compute-capabilities 7.0" /apps/slurm/default/bin/sbatch --job-name test_PR_13008 --ntasks=4 ~/boegelbot/eb_from_pr_upload_generoso.sh' executed!

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

Test results coming soon (I hope)...

Details

- notification for comment with ID 852948483 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).

@boegel
Copy link
Copy Markdown
Member

boegel commented Jun 2, 2021

Test report by @boegel
Using easyblocks from PR(s) easybuilders/easybuild-easyblocks#2454
SUCCESS
Build succeeded for 1 out of 1 (1 easyconfigs in total)
node3505.doduo.os - Linux RHEL 8.2, x86_64, AMD EPYC 7552 48-Core Processor (zen2), Python 3.6.8
See https://gist.github.com/b6a9e10073ee9c3fd17867d8a5dfd457 for a full test report.

@boegel
Copy link
Copy Markdown
Member

boegel commented Jun 2, 2021

Test report by @boegel
Using easyblocks from PR(s) easybuilders/easybuild-easyblocks#2454
SUCCESS
Build succeeded for 1 out of 1 (1 easyconfigs in total)
node2617.swalot.os - Linux centos linux 7.9.2009, x86_64, Intel(R) Xeon(R) CPU E5-2660 v3 @ 2.60GHz (haswell), Python 3.6.8
See https://gist.github.com/c60cdcddc7263655af7dfeb43a423934 for a full test report.

@boegelbot
Copy link
Copy Markdown
Collaborator

Test report by @boegelbot
Using easyblocks from PR(s) easybuilders/easybuild-easyblocks#2454
SUCCESS
Build succeeded for 1 out of 1 (1 easyconfigs in total)
generoso-c1-s-2 - Linux centos linux 8.2.2004, x86_64, Intel(R) Xeon(R) CPU E5-2667 v3 @ 3.20GHz (haswell), Python 3.6.8
See https://gist.github.com/bba591d1f592e965df6dbb331aa1cd9c for a full test report.

@boegel
Copy link
Copy Markdown
Member

boegel commented Jun 2, 2021

Test report by @boegel
Using easyblocks from PR(s) easybuilders/easybuild-easyblocks#2454
FAILED
Build succeeded for 4 out of 5 (1 easyconfigs in total)
easybuild2.novalocal - Linux centos linux 8.3.2011, POWER, IBM pSeries (emulated by qemu) (power9le), Python 3.6.8
See https://gist.github.com/7c3bd92eb7c82c898f3e4b9cc6c3b80a for a full test report.

edit: this is an RPATH-related problem, it has little to do with the fix being done here...

@boegel
Copy link
Copy Markdown
Member

boegel commented Jun 2, 2021

Test report by @boegel
Using easyblocks from PR(s) easybuilders/easybuild-easyblocks#2454
FAILED
Build succeeded for 0 out of 1 (1 easyconfigs in total)
select-pika-c6gd-2xlarge-0001 - Linux centos linux 8.3.2011, AArch64, ARM UNKNOWN (graviton2), Python 3.6.8
See https://gist.github.com/9ac7843de7038cb15e4450b3e85e7917 for a full test report.

edit: another tweak is required in the easyblock to support aarch64, see easybuilders/easybuild-easyblocks#2454 (review)

@boegel
Copy link
Copy Markdown
Member

boegel commented Jun 3, 2021

Test report by @boegel
Using easyblocks from PR(s) easybuilders/easybuild-easyblocks#2454
SUCCESS
Build succeeded for 1 out of 1 (1 easyconfigs in total)
select-pika-c6gd-2xlarge-0001 - Linux centos linux 8.3.2011, AArch64, ARM UNKNOWN (graviton2), Python 3.6.8
See https://gist.github.com/98129881d291b18ad480d310dd75be74 for a full test report.

@boegel boegel merged commit 785f2cd into easybuilders:develop Jun 3, 2021
@boegel
Copy link
Copy Markdown
Member

boegel commented Jun 3, 2021

Test report by @boegel
Using easyblocks from PR(s) easybuilders/easybuild-easyblocks#2454
SUCCESS
Build succeeded for 1 out of 1 (1 easyconfigs in total)
node3518.doduo.os - Linux RHEL 8.2, x86_64, AMD EPYC 7552 48-Core Processor (zen2), Python 3.6.8
See https://gist.github.com/3c2a889fa0e87ecd3bccc1ede654c994 for a full test report.

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.

Building Clang with gcccuda requires elfutils

3 participants