diff --git a/Spigot-Server-Patches/0122-Sanitise-RegionFileCache-and-make-configurable.patch b/Spigot-Server-Patches/0122-Sanitise-RegionFileCache-and-make-configurable.patch new file mode 100644 index 0000000000..6031d78a80 --- /dev/null +++ b/Spigot-Server-Patches/0122-Sanitise-RegionFileCache-and-make-configurable.patch @@ -0,0 +1,81 @@ +From 1c73ddcbe1da095e26689b846f058d03523e4d7d Mon Sep 17 00:00:00 2001 +From: Antony Riley +Date: Tue, 29 Mar 2016 08:22:55 +0300 +Subject: [PATCH] Sanitise RegionFileCache and make configurable. + +RegionFileCache prior to this patch would close every single open region +file upon reaching a size of 256. +This patch modifies that behaviour so it closes the the least recently +used RegionFile. +The implementation uses a LinkedHashMap as an LRU cache (modified from HashMap). +The maximum size of the RegionFileCache is also made configurable. + +diff --git a/src/main/java/com/destroystokyo/paper/PaperConfig.java b/src/main/java/com/destroystokyo/paper/PaperConfig.java +index 5a9e032..cd25fb8 100644 +--- a/src/main/java/com/destroystokyo/paper/PaperConfig.java ++++ b/src/main/java/com/destroystokyo/paper/PaperConfig.java +@@ -185,4 +185,9 @@ public class PaperConfig { + public static void healthDeath() { + setHealthBeforeDeathEvent = getBoolean("settings.set-health-before-death-event", false); + } ++ ++ public static int regionFileCacheSize = 256; ++ private static void regionFileCacheSize() { ++ regionFileCacheSize = getInt("settings.region-file-cache-size", 256); ++ } + } +diff --git a/src/main/java/net/minecraft/server/RegionFileCache.java b/src/main/java/net/minecraft/server/RegionFileCache.java +index 5eb01c3..9744e72 100644 +--- a/src/main/java/net/minecraft/server/RegionFileCache.java ++++ b/src/main/java/net/minecraft/server/RegionFileCache.java +@@ -8,10 +8,12 @@ import java.io.File; + import java.io.IOException; + import java.util.Iterator; + import java.util.Map; ++import com.destroystokyo.paper.PaperConfig; // Paper ++import java.util.LinkedHashMap; // Paper + + public class RegionFileCache { + +- public static final Map a = Maps.newHashMap(); // Spigot - private -> public ++ public static final Map a = new LinkedHashMap(PaperConfig.regionFileCacheSize, 0.75f, true); // Spigot - private -> public, Paper - HashMap -> LinkedHashMap + + // Paper start + public static synchronized RegionFile a(File file, int i, int j) { +@@ -32,8 +34,8 @@ public class RegionFileCache { + file1.mkdirs(); + } + +- if (RegionFileCache.a.size() >= 256) { +- a(); ++ if (RegionFileCache.a.size() >= PaperConfig.regionFileCacheSize) { // Paper ++ trimCache(); // Paper + } + + RegionFile regionfile1 = new RegionFile(file2); +@@ -43,6 +45,22 @@ public class RegionFileCache { + } + } + ++ // Paper Start ++ private static synchronized void trimCache() { ++ Iterator> itr = RegionFileCache.a.entrySet().iterator(); ++ int count = RegionFileCache.a.size() - PaperConfig.regionFileCacheSize; ++ while (count-- >= 0 && itr.hasNext()) { ++ try { ++ itr.next().getValue().c(); ++ } catch (IOException ioexception) { ++ ioexception.printStackTrace(); ++ ServerInternalException.reportInternalException(ioexception); ++ } ++ itr.remove(); ++ } ++ } ++ // Paper End ++ + public static synchronized void a() { + Iterator iterator = RegionFileCache.a.values().iterator(); + +-- +1.9.1 +