* remove redundant sync locking in AssetServerBase since this is already being done by the lower database layers

4 files changed, 7 insertions, 25 deletions

diff --git a/OpenSim/Framework/Communications/Cache/AssetCache.cs b/OpenSim/Framework/Communications/Cache/AssetCache.cs
index 7099760..d0bcc98 100644
--- a/OpenSim/Framework/Communications/Cache/AssetCache.cs
+++ b/OpenSim/Framework/Communications/Cache/AssetCache.cs
@@ -360,7 +360,7 @@ namespace OpenSim.Framework.Communications.Cache
 
                     if (!asset.Temporary)
                     {
-                        m_assetServer.StoreAndCommitAsset(asset);
+                        m_assetServer.StoreAsset(asset);
                     }
                 }
             }
@@ -376,7 +376,7 @@ namespace OpenSim.Framework.Communications.Cache
 
                     if (!asset.Temporary)
                     {
-                        m_assetServer.StoreAndCommitAsset(asset);
+                        m_assetServer.StoreAsset(asset);
                     }
                 }
             }
diff --git a/OpenSim/Framework/Communications/Cache/AssetServerBase.cs b/OpenSim/Framework/Communications/Cache/AssetServerBase.cs
index f65b88f..f729d78 100644
--- a/OpenSim/Framework/Communications/Cache/AssetServerBase.cs
+++ b/OpenSim/Framework/Communications/Cache/AssetServerBase.cs
@@ -44,12 +44,11 @@ namespace OpenSim.Framework.Communications.Cache
         protected BlockingQueue<AssetRequest> m_assetRequests;
         protected Thread m_localAssetServerThread;
         protected IAssetProvider m_assetProvider;
-        protected object m_syncLock = new object();
 
         // Temporarily hardcoded - should be a plugin
         protected IAssetLoader assetLoader = new AssetLoaderFileSystem();
 
-        protected abstract void StoreAsset(AssetBase asset);
+        public abstract void StoreAsset(AssetBase asset);
 
         /// <summary>
         /// This method must be implemented by a subclass to retrieve the asset named in the
@@ -161,18 +160,7 @@ namespace OpenSim.Framework.Communications.Cache
 
         public virtual void UpdateAsset(AssetBase asset)
         {
-            lock (m_syncLock)
-            {
-                m_assetProvider.UpdateAsset(asset);
-            }
-        }
-
-        public void StoreAndCommitAsset(AssetBase asset)
-        {
-            lock (m_syncLock)
-            {
-                StoreAsset(asset);
-            }
+            m_assetProvider.UpdateAsset(asset);
         }
 
         public virtual void Close()
diff --git a/OpenSim/Framework/Communications/Cache/GridAssetClient.cs b/OpenSim/Framework/Communications/Cache/GridAssetClient.cs
index 6caabcd..4206cae 100644
--- a/OpenSim/Framework/Communications/Cache/GridAssetClient.cs
+++ b/OpenSim/Framework/Communications/Cache/GridAssetClient.cs
@@ -78,7 +78,7 @@ namespace OpenSim.Framework.Communications.Cache
             throw new Exception("The method or operation is not implemented.");
         }
 
-        protected override void StoreAsset(AssetBase asset)
+        public override void StoreAsset(AssetBase asset)
         {
             try
             {
diff --git a/OpenSim/Framework/Communications/Cache/SQLAssetServer.cs b/OpenSim/Framework/Communications/Cache/SQLAssetServer.cs
index 95696ee..94a8509 100644
--- a/OpenSim/Framework/Communications/Cache/SQLAssetServer.cs
+++ b/OpenSim/Framework/Communications/Cache/SQLAssetServer.cs
@@ -78,16 +78,10 @@ namespace OpenSim.Framework.Communications.Cache
 
         protected override AssetBase GetAsset(AssetRequest req)
         {
-            AssetBase asset;
-            lock (m_syncLock)
-            {
-                asset = m_assetProvider.FetchAsset(req.AssetID);
-            }
-
-            return asset;
+            return m_assetProvider.FetchAsset(req.AssetID);;
         }
 
-        protected override void StoreAsset(AssetBase asset)
+        public override void StoreAsset(AssetBase asset)
         {
             m_assetProvider.CreateAsset(asset);
         }