Changeset 6910 for branches/HeuristicLab.Hive-3.4/sources/HeuristicLab.Clients.Hive.Slave/3.3/Manager

Timestamp:

10/12/11 15:48:53 (13 years ago)

Author:

ascheibe

Message:

#1233 fixed a bug where the slave didn't report the cpu utilization correctly

Location:

branches/HeuristicLab.Hive-3.4/sources/HeuristicLab.Clients.Hive.Slave/3.3/Manager

Files:

• ConfigManager.cs (modified) (5 diffs)
• HeartbeatManager.cs (modified) (1 diff)

branches/HeuristicLab.Hive-3.4/sources/HeuristicLab.Clients.Hive.Slave/3.3/Manager/ConfigManager.cs

@@ -46 +46 @@
     private Slave slave;
     private PerformanceCounter cpuCounter;
+    private PerformanceCounter memCounter;
 
     /// <summary>
@@ -52 +53 @@
     public ConfigManager(TaskManager jobManager) {
       this.jobManager = jobManager;
-      UpdateSlaveInfo();
-    }
-
-    private void UpdateSlaveInfo() {
       cpuCounter = new PerformanceCounter();
       cpuCounter.CategoryName = "Processor";
       cpuCounter.CounterName = "% Processor Time";
       cpuCounter.InstanceName = "_Total";
+      memCounter = new PerformanceCounter("Memory", "Available Bytes", true);
 
       Asleep = false;
@@ -70 +68 @@
       slave.OperatingSystem = Environment.OSVersion.VersionString;
       slave.CpuSpeed = GetCpuSpeed();
-      slave.FreeMemory = GetFreeMemory();
-      slave.HbInterval = (int)Settings.Default.HeartbeatInterval.TotalSeconds;
+
+      UpdateSlaveInfo();
+    }
+
+    private void UpdateSlaveInfo() {
+      if (slave != null) {
+        slave.FreeMemory = GetFreeMemory();
+        slave.HbInterval = (int)Settings.Default.HeartbeatInterval.TotalSeconds;
+      }
     }
 
@@ -240 +245 @@
     /// returns free memory of machine in MB
     /// </summary>    
-    public static int GetFreeMemory() {
+    public int GetFreeMemory() {
       int mb = 0;
 
       try {
-        PerformanceCounter counter = new PerformanceCounter("Memory", "Available Bytes", true);
-        mb = (int)(counter.NextValue() / 1024 / 1024);
+        mb = (int)(memCounter.NextValue() / 1024 / 1024);
       }
       catch { }
@@ -252 +256 @@
 
     public float GetCpuUtilization() {
-      return cpuCounter.NextValue();
+      float cpuVal = 0.0F;
+
+      try {
+        return cpuCounter.NextValue();
+      }
+      catch { }
+      return cpuVal;
     }
   }

branches/HeuristicLab.Hive-3.4/sources/HeuristicLab.Clients.Hive.Slave/3.3/Manager/HeartbeatManager.cs

@@ -97 +97 @@
                 SlaveId = info.Id,
                 FreeCores = info.Cores.HasValue ? info.Cores.Value - SlaveStatusInfo.UsedCores : 0,
-                FreeMemory = ConfigManager.GetFreeMemory(),
+                FreeMemory = ConfigManager.Instance.GetFreeMemory(),
                 CpuUtilization = ConfigManager.Instance.GetCpuUtilization(),
                 JobProgress = ConfigManager.Instance.GetExecutionTimeOfAllJobs(),