aboutsummaryrefslogtreecommitdiffhomepage
path: root/patches/server/0717-Fix-swamp-hut-cat-generation-deadlock.patch
diff options
context:
space:
mode:
authormaxcom1 <[email protected]>2024-03-23 22:26:17 +0100
committerGitHub <[email protected]>2024-03-23 17:26:17 -0400
commitb6001403e9703cadaa6e8c8558e732b91c3c6d6e (patch)
treea8c57bbc334a8ad48d4ad2b43db335667b142bee /patches/server/0717-Fix-swamp-hut-cat-generation-deadlock.patch
parent9ec7dfcbc41c6e625de0050b6997160a75df9f44 (diff)
downloadPaper-b6001403e9703cadaa6e8c8558e732b91c3c6d6e.tar.gz
Paper-b6001403e9703cadaa6e8c8558e732b91c3c6d6e.zip
Add methods to change entity physics (#10334)
Diffstat (limited to 'patches/server/0717-Fix-swamp-hut-cat-generation-deadlock.patch')
-rw-r--r--patches/server/0717-Fix-swamp-hut-cat-generation-deadlock.patch62
1 files changed, 62 insertions, 0 deletions
diff --git a/patches/server/0717-Fix-swamp-hut-cat-generation-deadlock.patch b/patches/server/0717-Fix-swamp-hut-cat-generation-deadlock.patch
new file mode 100644
index 0000000000..bac78b77ff
--- /dev/null
+++ b/patches/server/0717-Fix-swamp-hut-cat-generation-deadlock.patch
@@ -0,0 +1,62 @@
+From 0000000000000000000000000000000000000000 Mon Sep 17 00:00:00 2001
+From: Spottedleaf <[email protected]>
+Date: Sat, 12 Mar 2022 06:31:13 -0800
+Subject: [PATCH] Fix swamp hut cat generation deadlock
+
+The worldgen thread will attempt to get structure references
+via the world's getChunkAt method, which is fine if the gen is
+not cancelled - but if the chunk was unloaded, the call will block
+indefinitely. Instead of using the world state, we use the already
+supplied ServerLevelAccessor which will always have the chunk available.
+
+diff --git a/src/main/java/net/minecraft/world/entity/animal/Cat.java b/src/main/java/net/minecraft/world/entity/animal/Cat.java
+index 534630b0161c8d869e49e7a59572193550be0671..7dfd2b17e82a80683af28779d0bd8f64a909c3b6 100644
+--- a/src/main/java/net/minecraft/world/entity/animal/Cat.java
++++ b/src/main/java/net/minecraft/world/entity/animal/Cat.java
+@@ -364,7 +364,7 @@ public class Cat extends TamableAnimal implements VariantHolder<CatVariant> {
+ });
+ ServerLevel worldserver = world.getLevel();
+
+- if (worldserver.structureManager().getStructureWithPieceAt(this.blockPosition(), StructureTags.CATS_SPAWN_AS_BLACK).isValid()) {
++ if (worldserver.structureManager().getStructureWithPieceAt(this.blockPosition(), StructureTags.CATS_SPAWN_AS_BLACK, world).isValid()) { // Paper - Fix swamp hut cat generation deadlock
+ this.setVariant((CatVariant) BuiltInRegistries.CAT_VARIANT.getOrThrow(CatVariant.ALL_BLACK));
+ this.setPersistenceRequired();
+ }
+diff --git a/src/main/java/net/minecraft/world/level/StructureManager.java b/src/main/java/net/minecraft/world/level/StructureManager.java
+index b33a015b834873f279bf33a64974ef440a37df79..b59581b92aed0bc1b09008c695b0b112c3e65743 100644
+--- a/src/main/java/net/minecraft/world/level/StructureManager.java
++++ b/src/main/java/net/minecraft/world/level/StructureManager.java
+@@ -44,7 +44,12 @@ public class StructureManager {
+ }
+
+ public List<StructureStart> startsForStructure(ChunkPos pos, Predicate<Structure> predicate) {
+- Map<Structure, LongSet> map = this.level.getChunk(pos.x, pos.z, ChunkStatus.STRUCTURE_REFERENCES).getAllReferences();
++ // Paper start - Fix swamp hut cat generation deadlock
++ return this.startsForStructure(pos, predicate, null);
++ }
++ public List<StructureStart> startsForStructure(ChunkPos pos, Predicate<Structure> predicate, @Nullable ServerLevelAccessor levelAccessor) {
++ Map<Structure, LongSet> map = (levelAccessor == null ? this.level : levelAccessor).getChunk(pos.x, pos.z, ChunkStatus.STRUCTURE_REFERENCES).getAllReferences();
++ // Paper end - Fix swamp hut cat generation deadlock
+ ImmutableList.Builder<StructureStart> builder = ImmutableList.builder();
+
+ for(Map.Entry<Structure, LongSet> entry : map.entrySet()) {
+@@ -108,13 +113,18 @@ public class StructureManager {
+ }
+
+ public StructureStart getStructureWithPieceAt(BlockPos pos, TagKey<Structure> structureTag) {
++ // Paper start - Fix swamp hut cat generation deadlock
++ return this.getStructureWithPieceAt(pos, structureTag, null);
++ }
++ public StructureStart getStructureWithPieceAt(BlockPos pos, TagKey<Structure> structureTag, @Nullable ServerLevelAccessor levelAccessor) {
++ // Paper end - Fix swamp hut cat generation deadlock
+ Registry<Structure> registry = this.registryAccess().registryOrThrow(Registries.STRUCTURE);
+
+ for(StructureStart structureStart : this.startsForStructure(new ChunkPos(pos), (structure) -> {
+ return registry.getHolder(registry.getId(structure)).map((reference) -> {
+ return reference.is(structureTag);
+ }).orElse(false);
+- })) {
++ }, levelAccessor)) { // Paper - Fix swamp hut cat generation deadlock
+ if (this.structureHasPieceAt(pos, structureStart)) {
+ return structureStart;
+ }