-
Notifications
You must be signed in to change notification settings - Fork 772
Added CUDA 11.0.2 and related recipes, incl. gompic/2020a and iccifortcuda/2020a #11295
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
Added CUDA 11.0.2 and related recipes, incl. gompic/2020a and iccifortcuda/2020a #11295
Conversation
Like easybuilders#10935 this introduces a new CUDAcore easyconfig to share CUDA between use of GCC and Intel compilers, but unlike easybuilders#10935 this uses a versionsuffix for UCX+CUDA so it does not need any framework changes or MODULEPATH adjustments.
|
@boegelbot please test @ generoso |
|
@bartoldeman: Request for testing this PR well received on generoso PR test command '
Test results coming soon (I hope)... Details- notification for comment with ID 693637644 processed Message to humans: this is just bookkeeping information for me, |
|
Test report by @boegelbot |
|
First; I agree with this approach compared to the other alternatives presented, though this got me thinking about the possibilities here: A relatively minor thing, but, if we ever want to do something like that, it sure would be nicer if the system-level CUDA got to keep using the While writing this, I realized there might be some issues for those who RPATH the UCX libs into their openmpi, installation, which would require multiple versions of openmpi, forever forcing the use of separate |
|
Test report by @bartoldeman |
|
@Micket it's not clear to me from your comment what I should change to this commit -- you said you were ok but did not approve it. |
nevermind, sticking to UCX 1.8.0 and CUDA 11.0.2 makes sense
|
To avoid the On GPU systems you would then configure EasyBuild with That's a good way forward to collapse |
|
Probably a good idea, but beyond this PR :P |
|
Test report by @boegel |
Yeah, I would like to go for an approach like that in the upcoming |
|
Test report by @boegel |
boegel
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think this is ready to go...
@Micket @lexming @akesandgren Any last words on this before we merge it?
easybuild/easyconfigs/u/UCX/UCX-1.8.0-GCCcore-9.3.0-CUDA-11.0.2.eb
Outdated
Show resolved
Hide resolved
|
I'll test this over the weekend |
use OS_PKG_OPENSSL_DEV constant in UCX 1.8.0 easyconfig on top of CUDA 11.0.2
@lexming Any results yet? |
|
I tested this build of UCX with CUDA in a non-GPU node and it works, but there is a big fat warning on every execution This is absolutely fine for this PR, but we should have this in mind in our quest for the one UCX to rule them all. Easiest solution would be to just disable the warning with the envar |
Add GDRCopy to UCX with CUDA
|
@boegelbot please test @ generoso |
|
@bartoldeman: Request for testing this PR well received on generoso PR test command '
Test results coming soon (I hope)... Details- notification for comment with ID 701567955 processed Message to humans: this is just bookkeeping information for me, |
|
Test report by @boegelbot |
|
Fails on generoso because of I'll check what can be done... |
|
I'll try doing something like prebuildopts="PATH=$PATH:/sbin && " because ldconfig usually sits there and sbin is not always in the path -- but need to be offline for a while. |
|
|
Test report by @boegel |
|
Test report by @lexming |
lexming
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
It's not in the standard PATH on generoso.
|
@boegelbot please test @ generoso |
|
@bartoldeman: Request for testing this PR well received on generoso PR test command '
Test results coming soon (I hope)... Details- notification for comment with ID 701712682 processed Message to humans: this is just bookkeeping information for me, |
|
Test report by @boegelbot |
Addressed via ComputeCanada#14
|
Test report by @bartoldeman |
|
Test report by @lexming |
lexming
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
|
Test report by @boegel |
|
Test report by @boegel |
|
Test report by @boegel |
|
Test report by @zao |
|
Test report by @branfosj |
|
Going in, thanks @bartoldeman! |
Like #10935 this introduces a new CUDAcore easyconfig to share
CUDA between use of GCC and Intel compilers, but
unlike #10935 this uses a versionsuffix for UCX+CUDA so it does
not need any framework changes or MODULEPATH adjustments.