From ce3662b455ddd52a4f1e04772e6bd5e173916832 Mon Sep 17 00:00:00 2001
From: Alex Huang <alex.huang@citrix.com>
Date: Mon, 6 Feb 2012 17:02:22 -0800
Subject: [PATCH] bug 13743: fixed the concurrent access to vms hashmap status
 13743: resolved fixed reviewed-by: Will Chan

---
 .../com/cloud/hypervisor/xen/resource/XenServerPoolVms.java | 6 ++++--
 1 file changed, 4 insertions(+), 2 deletions(-)

diff --git a/core/src/com/cloud/hypervisor/xen/resource/XenServerPoolVms.java b/core/src/com/cloud/hypervisor/xen/resource/XenServerPoolVms.java
index 3cb6a9e93ea..0970f6d631d 100644
--- a/core/src/com/cloud/hypervisor/xen/resource/XenServerPoolVms.java
+++ b/core/src/com/cloud/hypervisor/xen/resource/XenServerPoolVms.java
@@ -17,6 +17,8 @@
 package com.cloud.hypervisor.xen.resource;
 
 import java.util.HashMap;
+import java.util.Map;
+import java.util.concurrent.ConcurrentHashMap;
 
 import org.apache.log4j.Logger;
 
@@ -26,8 +28,8 @@ import com.cloud.vm.VirtualMachine.State;
  
 public class XenServerPoolVms {
     private static final Logger s_logger = Logger.getLogger(XenServerPoolVms.class);
-    private HashMap<String/* clusterId */, HashMap<String/* vm name */, Pair<String/* host uuid */, State/* vm state */>>> _cluster_vms = 
-        new HashMap<String, HashMap<String, Pair<String, State>>>();
+    private final Map<String/* clusterId */, HashMap<String/* vm name */, Pair<String/* host uuid */, State/* vm state */>>> _cluster_vms = 
+        new ConcurrentHashMap<String, HashMap<String, Pair<String, State>>>();
 
     public HashMap<String, Pair<String, State>> getClusterVmState(String clusterId){
         HashMap<String, Pair<String, State>> _vms= _cluster_vms.get(clusterId);