Skip to content

nvproxy: keep uvmFDMemmapFile.pfm refcount up to date#12657

Merged
copybara-service[bot] merged 1 commit intomasterfrom
test/cl875885672
Feb 27, 2026
Merged

nvproxy: keep uvmFDMemmapFile.pfm refcount up to date#12657
copybara-service[bot] merged 1 commit intomasterfrom
test/cl875885672

Conversation

@copybara-service
Copy link

@copybara-service copybara-service bot commented Feb 26, 2026

nvproxy: keep uvmFDMemmapFile.pfm refcount up to date

Otherwise, mappings owned by the PreciseHostFileMapper leak after the uvmFD is
released, interfering with future mappings of the same nvidia-uvm addresses.

Without platform/kvm, hitting this requires the application workload to touch
the same cudaMallocManaged memory addresses with syscalls (e.g. read/write)
using two distinct uvmFDs (e.g. in two separate sequentially-executed process
trees). On platform/kvm, hitting this only requires the application to
allocate the same cudaMallocManaged memory addresses using two distinct
uvmFDs, but use of nvidia-uvm on platform/kvm is discouraged due to #11436.

@copybara-service copybara-service bot added the exported Issue was exported automatically label Feb 26, 2026
@copybara-service copybara-service bot changed the title nvproxy: unmap nvidia-uvm from sentry when FD is closed nvproxy: keep uvmFDMemmapFile.pfm refcount up to date Feb 27, 2026
@copybara-service copybara-service bot force-pushed the test/cl875885672 branch 6 times, most recently from a8d969a to d426c0c Compare February 27, 2026 22:18
Otherwise, mappings owned by the PreciseHostFileMapper leak after the uvmFD is
released, interfering with future mappings of the same nvidia-uvm addresses.

Without platform/kvm, hitting this requires the application workload to touch
the same `cudaMallocManaged` memory addresses with syscalls (e.g. read/write)
using two distinct uvmFDs (e.g. in two separate sequentially-executed process
trees). On platform/kvm, hitting this only requires the application to
*allocate* the same `cudaMallocManaged` memory addresses using two distinct
uvmFDs, but use of nvidia-uvm on platform/kvm is discouraged due to #11436.

PiperOrigin-RevId: 876436522
@copybara-service copybara-service bot merged commit a12250c into master Feb 27, 2026
0 of 2 checks passed
@copybara-service copybara-service bot deleted the test/cl875885672 branch February 27, 2026 23:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

exported Issue was exported automatically

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant