Skip to content

Conversation

@raoatifshad
Copy link

Added CUDA 13.1.0 support since CUDA 13.x introduces compatibility
with GCC 15 and Clang 20, which is required for newer toolchains.

Additionally, address a known CUDA installer issue related to the
default log file path (/tmp/cuda-installer.log). When the file is
created by another user, subsequent installations may fail due to
permission errors.

To avoid this, explicitly create the required directory before the
installation and clean it up afterward. This ensures the installer
can write logs safely in multi-user build environments.

References:

…CC 15 and Clang 20 & added a check for tmp dir
@cmsbuild
Copy link
Contributor

A new Pull Request was created by @raoatifshad for branch IB/CMSSW_16_0_X/g14.

@akritkbehera, @cmsbuild, @iarspider, @raoatifshad, @smuzaffar can you please review it and eventually sign? Thanks.
@ftenchini, @mandrenguyen, @sextonkennedy you are the release manager for this.
cms-bot commands are listed here

@cmsbuild
Copy link
Contributor

cmsbuild commented Dec 18, 2025

cms-bot internal usage

@fwyzard
Copy link
Contributor

fwyzard commented Dec 18, 2025

Why do we use GCC 15 in the g14 branch ?

@smuzaffar
Copy link
Contributor

smuzaffar commented Dec 18, 2025

@fwyzard , title os PR is misleading, we still use GCC 14 in this branch.
As we do not have GCC 15 IBs, so we can not test it. This PR I would like to test with c++23 IBs ( which are using this branch). If you want, we can try testing it for default gcc13 IBs too

@cmsbuild
Copy link
Contributor

Pull request #10257 was updated.

@fwyzard
Copy link
Contributor

fwyzard commented Dec 18, 2025

OK, but then I'm confused - CUDA 13.x does not support C++23 ...

@fwyzard
Copy link
Contributor

fwyzard commented Dec 18, 2025

As for testing in the main branch, I have no objections to testing it, but we cannot move to CUDA 13.x until we agree to drop support for Pascal and Volta GPUs :-(

@smuzaffar
Copy link
Contributor

OK, but then I'm confused - CUDA 13.x does not support C++23 ...

that is what I wanted to test, there is good chance that it will fail :-)

@smuzaffar smuzaffar changed the base branch from IB/CMSSW_16_0_X/g14 to IB/CMSSW_16_1_X/g14 December 18, 2025 13:45
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.

5 participants