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);