task: size of the key locks array should be at least 32, and larger i…

…f you have a lot of CPUs
alastairtree · Sep 21, 2020 · e0e1fe4 · e0e1fe4 · jnyrup · Sep 26, 2020
1 parent 0140063
commit e0e1fe4
Showing 1 changed file with 6 additions and 1 deletion.
diff --git a/LazyCache/CachingService.cs b/LazyCache/CachingService.cs
@@ -13,7 +13,7 @@ public class CachingService : IAppCache
     {
         private readonly Lazy<ICacheProvider> cacheProvider;
 
-        private readonly int[] keyLocks = new int[32];
+        private readonly int[] keyLocks;
 
         public CachingService() : this(DefaultCacheProvider)
         {
@@ -22,12 +22,17 @@ public CachingService() : this(DefaultCacheProvider)
         public CachingService(Lazy<ICacheProvider> cacheProvider)
         {
             this.cacheProvider = cacheProvider ?? throw new ArgumentNullException(nameof(cacheProvider));
+            var lockCount = Math.Max(Environment.ProcessorCount * 8, 32);
+            keyLocks = new int[lockCount];
         }
 
         public CachingService(Func<ICacheProvider> cacheProviderFactory)
         {
             if (cacheProviderFactory == null) throw new ArgumentNullException(nameof(cacheProviderFactory));
             cacheProvider = new Lazy<ICacheProvider>(cacheProviderFactory);
+            var lockCount = Math.Max(Environment.ProcessorCount * 8, 32);
+            keyLocks = new int[lockCount];
+
         }
 
         public CachingService(ICacheProvider cache) : this(() => cache)