trim cold

Author: Alex Peck

BitFaster.Caching.UnitTests/Lru/ConcurrentLruTests.cs

@@ -953,6 +953,25 @@ public void WhenWarmThenClearedIsWarmIsReset()
             lru.Count.Should().Be(capacity.Hot + capacity.Warm + capacity.Cold);
         }
 
+        [Fact]
+        public void WhenWarmThenTrimIsWarmIsReset()
+        {
+            for (int i = 0; i < 20; i++)
+            {
+                lru.GetOrAdd(i, k => k.ToString());
+            }
+
+            lru.Trim(6);
+            lru.Count.Should().Be(3);
+
+            for (int i = 0; i < 20; i++)
+            {
+                lru.GetOrAdd(i, k => k.ToString());
+            }
+
+            lru.Count.Should().Be(capacity.Hot + capacity.Warm + capacity.Cold);
+        }
+
         [Fact]
         public void WhenItemsAreDisposableClearDisposesItemsOnRemove()
         {

BitFaster.Caching/Lru/ConcurrentLruCore.cs

@@ -423,7 +423,6 @@ public void AddOrUpdate(K key, V value)
         public void Clear()
         {
             this.TrimLiveItems(itemsRemoved: 0, this.Count, ItemRemovedReason.Cleared);
-            Volatile.Write(ref this.isWarm, false);
         }
 
         /// <summary>
@@ -532,6 +531,11 @@ private void TrimLiveItems(int itemsRemoved, int itemCount, ItemRemovedReason re
                     trimWarmAttempts++;
                 }
             }
+
+            if (Volatile.Read(ref this.counter.warm) < this.capacity.Warm)
+            {
+                Volatile.Write(ref this.isWarm, false);
+            }
         }
 
         private void TrimWarmOrHot(ItemRemovedReason reason)