From 20604621e49b8e47770644cb9b103bd3a94f8102 Mon Sep 17 00:00:00 2001 From: Kevin Klues Date: Mon, 7 Dec 2020 12:10:23 +0000 Subject: [PATCH] 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 --- pkg/container_config.go | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/pkg/container_config.go b/pkg/container_config.go index 04171b33..d91fccd1 100644 --- a/pkg/container_config.go +++ b/pkg/container_config.go @@ -28,7 +28,7 @@ const ( const ( allDriverCapabilities = "compute,compat32,graphics,utility,video,display,ngx" - defaultDriverCapabilities = "utility" + defaultDriverCapabilities = "utility,compute" ) const (