Skip to content

Commit 2060462

Browse files
committed
Add 'compute' capability to list of defaults.
For most practical purposes, it should be fine to set NVIDIA_DRIVER_CAPABILITIES=all nowadays. Historically, these different capabilities exist because they were added incrementally, with varying degrees of stability. It's fairly common to run with GPUs in containers today, but a few years ago the driver didn't support them very well, and it was important to make sure the libraries being injected into the container actually worked in a containerized environment. When they didn't, it was common to get information leaks, crashes, or even silent failures. In the past, whenever a new set of libraries was being vetted for injected, a new capability was added to make sure that users had control to explicitly include only those libraries they were comfortable having injected into their containers. The idea being that whoever puts together a container image for use with GPUs should have the knowledge of what capabilities the software in that container image requires, and can set the NVIDIA_DRIVER_CAPABILITIES envvar in that image appropriately. After some back and forth, we've decided it doesn't quite make sense to set it to "all" just yet, but we should set it to "utility, compute" instead of just "utility", so that at least the core CUDA libraries work by default (once installed in the container). Signed-off-by: Kevin Klues <[email protected]>
1 parent 8cfb3c2 commit 2060462

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

pkg/container_config.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -28,7 +28,7 @@ const (
2828

2929
const (
3030
allDriverCapabilities = "compute,compat32,graphics,utility,video,display,ngx"
31-
defaultDriverCapabilities = "utility"
31+
defaultDriverCapabilities = "utility,compute"
3232
)
3333

3434
const (

0 commit comments

Comments
 (0)