Skip to content

nvidia-peer-memory: prefer in-tree nvidia_peermem over legacy nv_peer_mem - #1390

Open
100-JM wants to merge 1 commit into
NVIDIA:masterfrom
100-JM:fix/nvidia-peer-memory-in-tree
Open

nvidia-peer-memory: prefer in-tree nvidia_peermem over legacy nv_peer_mem#1390
100-JM wants to merge 1 commit into
NVIDIA:masterfrom
100-JM:fix/nvidia-peer-memory-in-tree

Conversation

@100-JM

@100-JM 100-JM commented Sep 3, 2026

Copy link
Copy Markdown

Problem

roles/nvidia-peer-memory assumes the out-of-tree nv_peer_mem DKMS module on DGX systems. Since driver R470.42.01 the equivalent nvidia_peermem module ships in-tree with the driver and nv_peer_mem is not available, so the role fails on current DGX OS:

modprobe: FATAL: Module nv_peer_mem not found in directory /lib/modules/6.8.0-106-generic

There is no variable to disable the role from slurm-cluster.yml; only the nvidia-peer-memory tag.

Reproduced on DGX OS 7.5.0, driver 580.126.20.

Fix

Probe for nvidia_peermem with modinfo. When present, load it and skip the legacy DKMS/nv_peer_mem/service path. When absent, behaviour is unchanged.

Verification

Failure reproduced on DGX OS 7.5.0 / driver 580.126.20. On the same system lsmod shows the in-tree module already loaded (nvidia_peermem 16384 0, used by ib_uverbs), so the new probe takes the in-tree branch and the legacy DKMS/nv_peer_mem path is skipped.

…_mem

Driver R470.42.01 and later ship nvidia_peermem as part of the driver;
the out-of-tree nv_peer_mem DKMS module does not exist there. The role
unconditionally ran dkms autoinstall + modprobe nv_peer_mem on DGX
systems and failed on current DGX OS.

Probe for the in-tree module with modinfo and load it when present;
fall back to the legacy nv_peer_mem path only when it is absent.

Observed on DGX OS 7.5.0 / driver 580.126.20:
  modprobe: FATAL: Module nv_peer_mem not found in directory /lib/modules/6.8.0-106-generic

Signed-off-by: 백지명 <wlaud9805@gmail.com>
@100-JM
100-JM marked this pull request as ready for review September 3, 2026 13:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant