Uploaded image for project: 'Minecraft: Java Edition'
  1. Minecraft: Java Edition
  2. MC-105406

OpenGL Leak in View Frustum


    • Icon: Bug Bug
    • Resolution: Fixed
    • Minecraft 16w41a
    • Minecraft 1.10.2
    • None
    • NVIDIA + Linux
    • Unconfirmed

      There is an OpenGL leak in the View Frustum and subsequently all individual Chunk Renderers + Vertex Buffers.

      Without digging too deeply I believe this leak is either 1) a regression or 2) only now obvious due to the use of VBOs by default, which (on NVIDIA + Linux at least) have an allocation time that grows exponentially as more and more buffers are already allocated.

      To attempt to reproduce this issue, first enable VBOs. Then join a world and fly around causing lots of chunks to be loaded. Disconnect from the world and repeat this process. After perhaps half a dozen repetitions, the world load time will be very slow (up to 2000ms above normal time has been observed). Note that at this point disabling use of VBOs will restore normal world load time (and toggling them back on will make the issue apparent once more).

      The cause of this is that when resetting the Global Renderer on world join, the View Frustum is nulled out before the cleanup method (guarded by a null check) can be run. To fix this leak the cleanup method must be called the line before the View Frustum is nulled out.

      Once this additional cleanup is called world load time remains constant even with the use of VBOs.

      I have based my terminology on the MCP namings, A patch with hopefully sufficient context is below:

      --- a/src/minecraft/net/minecraft/client/renderer/RenderGlobal.java
      +++ b/src/minecraft/net/minecraft/client/renderer/RenderGlobal.java
      @@ -456,61 +456,66 @@ public class RenderGlobal implements IWorldEventListener, IResourceManagerReload
            * set null to clear
           public void setWorldAndLoadRenderers(@Nullable WorldClient worldClientIn)
               if (this.theWorld != null)
               this.frustumUpdatePosX = Double.MIN_VALUE;
               this.frustumUpdatePosY = Double.MIN_VALUE;
               this.frustumUpdatePosZ = Double.MIN_VALUE;
               this.frustumUpdatePosChunkX = Integer.MIN_VALUE;
               this.frustumUpdatePosChunkY = Integer.MIN_VALUE;
               this.frustumUpdatePosChunkZ = Integer.MIN_VALUE;
               this.theWorld = worldClientIn;
               if (worldClientIn != null)
      -            this.viewFrustum = null;
      +            if (this.viewFrustum != null)
      +            {
      +                this.viewFrustum.deleteGlResources();
      +                this.viewFrustum = null;
      +            }
                   if (this.renderDispatcher != null)
                   this.renderDispatcher = null;
            * Loads all the renderers and sets up the basic settings usage
           public void loadRenderers()
               if (this.theWorld != null)
                   if (this.renderDispatcher == null)
                       this.renderDispatcher = new ChunkRenderDispatcher();
                   this.displayListEntitiesDirty = true;
                   this.renderDistanceChunks = this.mc.gameSettings.renderDistanceChunks;
                   boolean flag = this.vboEnabled;
                   this.vboEnabled = OpenGlHelper.useVbo();
                   if (flag && !this.vboEnabled)

            grum [Mojang] Grum (Erik Broes)
            md_5 [Mod] md_5
            15 Vote for this issue
            5 Start watching this issue