CarryOn icon indicating copy to clipboard operation
CarryOn copied to clipboard

[Fabric 1.21.4] Memory leak

Open ColdingNewb opened this issue 1 year ago • 1 comments

On version carryon-fabric-1.21.4-2.3.0.22 I've experienced a memory leak, I was able to root it down to this mod after a whole night Basically the way to replicate it is by just simply in my case Carry a villager, and just simply run around/fly around, and eventually the ram usage will slowly increase more and more without dumping back down, eventually going to 100% and just requiring a restart of the game.

ColdingNewb avatar Feb 14 '25 15:02 ColdingNewb

I noticed this would only be an issue client side, because I run my own server and had no issues with memory playing on it server sided. I can try my best to give more information as needed.

ColdingNewb avatar Feb 14 '25 15:02 ColdingNewb

Hi, I profiled the game for an hour while holding a villager (both in first and third person) and it all looks very normal:

Image So i don't know if this is a carry on issue

Tschipp avatar Apr 21 '25 09:04 Tschipp

Holding it is indeed normal for me and no memory leaks occur, it is only when I start going further and further from the initial pickup location that the memory leak starts occurring where the GC would not clear all of it and eventually it'll go up from 20% then 30% minimum memory usage until it reaches 99%+ and causes it to stutter and then freeze.

ColdingNewb avatar Apr 27 '25 23:04 ColdingNewb

My follow up is that I no longer can recreate the issue anymore, I'm not sure what changed, if anything on my end changed, because when I had this issue it was just me and one other person who had this issue and only occurred when we picked up an entity (specifically the villager) and moved further and further from the initial pickup location. If I do somehow run into the issue again would you be able to tell me how I could show you these memory issues?

ColdingNewb avatar Apr 27 '25 23:04 ColdingNewb

I used the Java VisualVM profiler for this, it can attach to any java process and is (somewhat) easy to use. I will close this issue for now, feel free to reopen a new one if it comes up again.

Tschipp avatar Jul 22 '25 21:07 Tschipp