You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@mesos.apache.org by qi...@apache.org on 2020/10/13 02:32:29 UTC
[mesos] branch master updated: Ignored the directoy
`/dev/nvidia-caps` when globing Nvidia GPU devices.
This is an automated email from the ASF dual-hosted git repository.
qianzhang pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/mesos.git
The following commit(s) were added to refs/heads/master by this push:
new 301902b Ignored the directoy `/dev/nvidia-caps` when globing Nvidia GPU devices.
301902b is described below
commit 301902be4f1332799cf3b3242cd29b4907c21c09
Author: Qian Zhang <zh...@gmail.com>
AuthorDate: Sat Oct 10 15:04:57 2020 +0800
Ignored the directoy `/dev/nvidia-caps` when globing Nvidia GPU devices.
The directory `/dev/nvidia-caps` was introduced in CUDA 11.0, just
ignore it since we only care about the Nvidia GPU device files.
Review: https://reviews.apache.org/r/72945
---
src/slave/containerizer/mesos/isolators/gpu/isolator.cpp | 9 +++++++++
1 file changed, 9 insertions(+)
diff --git a/src/slave/containerizer/mesos/isolators/gpu/isolator.cpp b/src/slave/containerizer/mesos/isolators/gpu/isolator.cpp
index a0be102..99119f9 100644
--- a/src/slave/containerizer/mesos/isolators/gpu/isolator.cpp
+++ b/src/slave/containerizer/mesos/isolators/gpu/isolator.cpp
@@ -443,6 +443,15 @@ Future<Option<ContainerLaunchInfo>> NvidiaGpuIsolatorProcess::_prepare(
}
foreach (const string& device, nvidia.get()) {
+ // The directory `/dev/nvidia-caps` was introduced in CUDA 11.0, just
+ // ignore it since we only care about the Nvidia GPU device files.
+ //
+ // TODO(qianzhang): Figure out how to handle the directory
+ // `/dev/nvidia-caps` more properly.
+ if (device == "/dev/nvidia-caps") {
+ continue;
+ }
+
const string devicePath = path::join(
devicesDir, strings::remove(device, "/dev/", strings::PREFIX), device);