Skip to content

Ensure that the CSV files define the required devices when only a iGPU is present#1556

Merged
elezar merged 2 commits intoNVIDIA:mainfrom
elezar:fix-purecsv
Jan 23, 2026
Merged

Ensure that the CSV files define the required devices when only a iGPU is present#1556
elezar merged 2 commits intoNVIDIA:mainfrom
elezar:fix-purecsv

Conversation

@elezar
Copy link
Member

@elezar elezar commented Jan 6, 2026

In oder to allow the dGPU to be filtered, one needs to ensure that the /dev/nvidia1 device node is NOT injected into a container when only the iGPU is requested. This was added in #1461.

The change in this PR update that implementation to:

  1. Fix a bug in the fallback CSV spec generator where the /dev/nvidia1 device node was always being filtered out.
  2. Update the logic to use the fallback CSV spec generator when only a single NVML device is present.

Note this does call out the fact that some applications that expect a filtered device node may not work as expected in containers when both iGPUs and dGPUs are present.

elezar added 2 commits January 6, 2026 14:07
…nabled

This change allows the "multiple-csv-devices" feature to be disabled
using the feature flag. Without this change the /dev/nvidia1 device node
is filtered out of the required set of devices which breaks certain
applications.

Signed-off-by: Evan Lezar <elezar@nvidia.com>
This change ensure that the pure CSV mode (with no device
node filtering) is used when a single device is present on
the system. This enables use cases such as VPI applications where
the /dev/nvidia1 device node is used even though an iGPU is present.

Signed-off-by: Evan Lezar <elezar@nvidia.com>
@coveralls
Copy link

Pull Request Test Coverage Report for Build 20749603432

Details

  • 0 of 34 (0.0%) changed or added relevant lines in 1 file are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage decreased (-0.06%) to 36.872%

Changes Missing Coverage Covered Lines Changed/Added Lines %
pkg/nvcdi/lib-csv.go 0 34 0.0%
Totals Coverage Status
Change from base Build 20280018211: -0.06%
Covered Lines: 5203
Relevant Lines: 14111

💛 - Coveralls

@elezar elezar merged commit ea32bf1 into NVIDIA:main Jan 23, 2026
16 checks passed
@elezar elezar deleted the fix-purecsv branch January 23, 2026 14:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants