From a4d11307f56bfb8d5f7559315844f5cbc64b6ae4 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 dccd00127..b4dba7247 100644 --- a/src/main/java/com/destroystokyo/paper/PaperConfig.java +++ b/src/main/java/com/destroystokyo/paper/PaperConfig.java @@ -228,4 +228,9 @@ public class PaperConfig { private static void loadPermsBeforePlugins() { loadPermsBeforePlugins = getBoolean("settings.load-permissions-yml-before-plugins", true); } + + 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 2217adf99..c0ab543b9 100644 --- a/src/main/java/net/minecraft/server/RegionFileCache.java +++ b/src/main/java/net/minecraft/server/RegionFileCache.java @@ -9,10 +9,12 @@ import java.io.IOException; import java.util.Iterator; import java.util.Map; import javax.annotation.Nullable; +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 public static synchronized RegionFile a(File file, int i, int j) { File file1 = new File(file, "region"); @@ -26,8 +28,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); @@ -60,6 +62,22 @@ public class RegionFileCache { } // CraftBukkit end + // 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(); -- 2.19.1