Merge pull request #189 from intel-innersource/rdementi/fix-race-condition-resctrl

fix element init race condition when resctrl is empty

Author: rdementi

src/resctrl.cpp

@@ -105,11 +105,15 @@ namespace pcm
             }
         }
     }
-    size_t Resctrl::getMetric(Resctrl::FileMapType & fileMap, int core)
+    size_t Resctrl::getMetric(const Resctrl::FileMapType & fileMap, int core)
     {
-        auto files = fileMap[core];
+        auto files = fileMap.find(core);
+        if (files == fileMap.end())
+        {
+            return 0ULL;
+        }
         size_t result = 0;
-        for (auto& f : files)
+        for (auto& f : files->second)
         {
             const auto data = readSysFS(f.c_str(), false);
             if (data.empty() == false)
@@ -143,4 +147,4 @@ namespace pcm
     }
 };
 
- #endif // __linux__
+ #endif // __linux__

src/resctrl.h

@@ -37,7 +37,7 @@ namespace pcm
         typedef std::unordered_map<int, std::vector<std::string> > FileMapType;
         FileMapType L3OCC, MBL, MBT;
         Resctrl() = delete;
-        size_t getMetric(FileMapType & fileMap, int core);
+        size_t getMetric(const FileMapType & fileMap, int core);
         static constexpr auto PCMPath = "/sys/fs/resctrl/mon_groups/pcm";
     public:
         Resctrl(PCM & m) : pcm(m) {}
@@ -50,4 +50,4 @@ namespace pcm
     };
 };
 
- #endif // __linux__
+ #endif // __linux__