Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Update CUDA to version 10.1.105, on x86_64
* CUDA version 10.1.105 Support for gcc 8 and clang 7. Include new Matrix Multiply library cublasLt. Various fxes to CUDA libraries and NVCC compiler. * NVIDIA drivers version 418.39 See https://docs.nvidia.com/cuda/archive/10.1/cuda-toolkit-release-notes/index.html .
- Loading branch information
Showing
1 changed file
with
49 additions
and
51 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
ec2ad19
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.
This commit started to fail builds on IB/CMSSW_10_6_X/gcc9.
ec2ad19
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.
CUDA does not support GCC 9
ec2ad19
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 can confirm that by reverting cuda.spec to version 3e9f7f9 I can build cuda on -a slc7_amd64_gcc900.
@smuzaffar, should we revert back? I will make I PR to test it.
ec2ad19
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.
no.
ec2ad19
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 was only talking about 'IB/CMSSW_10_6_X/gcc9' branch where it fails the IB, not in general.
My mistake about GitHub - I made a comment on this commit on "IB/CMSSW_10_6_X/gcc9" branch and I thought it is shown only there, but now I see it is shown everywhere where the commit is included, in the original PR as well.
ec2ad19
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 would still say that we should not revert the update, but rather fix the gcc9 branch to work with the current version of CUDA. At the very least this will make future updates to the spec files easier across the different architectures.
About the update itself: can you point me to a log for the failure, and to a corresponding log for a working gcc9 IB ?
ec2ad19
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.
@gudrutis , please do not revert. If you look at the log files then it says
and in
/tmp/cuda-installer.log
saysIf no objections from @fwyzard then I would suggest to use
--override
option here https://github.com/cms-sw/cmsdist/blob/IB/CMSSW_10_6_X/gcc700/cuda.spec#L39ec2ad19
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.
sure !
ec2ad19
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 have an other fix queued up for CUDA, I will make a PR with also this shortly.
ec2ad19
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.
Here it is: #4906 .
ec2ad19
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.
Ok I see. I cherry-picked fwyzard changes and created PR #4907 to test if changes fix the issue.