ComputeCanada / software-stack-config

8 stars 3 forks source link

Add minimum driver version for CUDA 12.2 #59

Closed bartoldeman closed 11 months ago

mboisson commented 11 months ago

Probably also need to adjust the minimum driver for compatibility version ?

Also, isn't our code wrong ?

        -- Older compat versions need driver 418.40.04+, 11.7 needs 450.36.06+, see
        -- https://docs.nvidia.com/deploy/cuda-compatibility/index.html#use-the-right-compat-package
        if convertToCanonical(cuda_version_two_digits) >= convertToCanonical("11.7") then
            if convertToCanonical(driver_version) >= convertToCanonical("450.36.06") then
            return "compat"
            end
        elseif convertToCanonical(driver_version) >= convertToCanonical("418.40.04") then
            return "compat"
        end

This page: https://docs.nvidia.com/cuda/cuda-toolkit-release-notes/index.html lists 450.80.02 as minimum for all cuda 11.x (other than 11.0) and 525.60.13 as minimum for all cuda 12.x

We probably want to have a second table, for minimum driver version for compatibility mode vs native mode

bartoldeman commented 11 months ago

It's confusing I know but that is "minor version compatibility", i.e. without a compat package. For compat it's still correct using 450.36.06 I believe: https://docs.nvidia.com/deploy/cuda-compatibility/index.html#use-the-right-compat-package

mboisson commented 11 months ago

It is confusing indeed. I trust you.