166 2005-06-22 12:25:22 +0200 Memory leak with ATI Mobility Radeon 9700 2010-03-24 07:47:54 +0100 0 0 3 JogAmp Jogl core 1 All all VERIFIED FIXED P3 normal --- 1 sgothel sgothel DEFECT --- oldest_to_newest 182 0 sgothel 2010-03-24 07:47:54 +0100 ---- Reported by kbr 2005-06-22 00:25:22 ---- A very simple test case reported by a user on the javagaming.org JOGL forum was shown to leak memory when run with JOGL 1.1 b12 on ATI Mobility Radeon 9700 hardware on Windows XP with recent drivers. The test case is attached and can also be found in the following thread on javagaming.org (called "JOGL is slowly increasing memory usage! WHY?": http://192.18.37.44/forums/index.php?topic=9865.0 The same test case when ported to LWJGL does not exhibit the memory leak. The difference in behavior lies in JOGL's and LWJGL's OpenGL context management. When JOGL performs an OpenGL repaint cycle in the GLCanvas (on Windows), it locks the surface using the JAWT, makes the OpenGL context current on the resulting HDC, performs the user's rendering, releases the current OpenGL context, and unlocks the surface. The current implementation of LWJGL behaves similarly, but does not release the current OpenGL context before unlocking the surface, and before making the OpenGL context current on the HDC checks to see whether it is already current. This skips the "makeCurrent/free" pattern of the OpenGL context on all but the first paint cycle, possibly for performance reasons. It can be argued that this OpenGL context management is not completely correct, because if the OpenGL context was desired to be operated upon from another thread it would have to cooperatively be released from the thread upon which it was previously made current. In the general case this is not possible. It turns out there is a real bug in ATI's drivers causing memory to be leaked on this particular mobile chip (and perhaps others) during each makeCurrent/free cycle. This is the root cause of the memory leak with the JOGL program and why the LWJGL port of the same program does not leak memory. ---- Additional Comments From kbr 2005-06-22 00:26:13 ---- Created an attachment Main program of test case ---- Additional Comments From kbr 2005-06-22 00:26:59 ---- Created an attachment GLEventListener for test case ---- Additional Comments From kbr 2005-06-22 01:06:20 ---- Worked around memory leak in ATI's OpenGL drivers by adding system property -Djogl.GLContext.nofree which users can specify on the command line. There is no good general-purpose workaround for this bug which works well on all hardware and in all kinds of applications. Issues may remain if this workaround is used and if the GLCanvas is removed and re-added to its parent container. Use at your own risk. ---- Additional Comments From kbr 2005-06-22 01:11:01 ---- I should also mention that a bug has been filed with ATI's Customer Support Ticket system. --- Bug imported by sgothel@jausoft.com 2010-03-24 07:47 EDT --- This bug was previously known as _bug_ 166 at https://jogl.dev.java.net/bugs/show_bug.cgi?id=166 Imported an attachment (id=59) Imported an attachment (id=60) The original submitter of attachment 59 is unknown. Reassigning to the person who moved it here: sgothel@jausoft.com. The original submitter of attachment 60 is unknown. Reassigning to the person who moved it here: sgothel@jausoft.com. 59 2005-06-22 00:26:00 +0200 2005-06-22 00:26:00 +0200 Main program of test case AnimJOGLApp.java text/plain 896 sgothel DQppbXBvcnQgamF2YS5hd3QuKjsNCmltcG9ydCBqYXZhLmF3dC5ldmVudC4qOw0KDQppbXBvcnQg bmV0LmphdmEuZ2FtZXMuam9nbC4qOw0KDQpwdWJsaWMgY2xhc3MgQW5pbUpPR0xBcHANCnsNCiAg c3RhdGljIEFuaW1hdG9yIGFuaW1hdG9yOw0KDQogIHB1YmxpYyBzdGF0aWMgdm9pZCBtYWluKFN0 cmluZ1tdIGFyZ3MpDQogIHsNCiAgICBGcmFtZSBmcmFtZSA9IG5ldyBGcmFtZSgiQW5pbUpPR0xB cHAiKTsNCiAgICAvL0FuaW1hdG9yIGb8ciBCZXdlZ3VuZw0KDQoNCiAgICAvL0dMQ2FudmFzIGb8 ciBBbnplaWdlDQogICAgR0xDYW52YXMgY2FudmFzID0gR0xEcmF3YWJsZUZhY3RvcnkuZ2V0RmFj dG9yeSgpLmNyZWF0ZUdMQ2FudmFzKG5ldyBHTENhcGFiaWxpdGllcygpKTsNCiAgICBjYW52YXMu YWRkR0xFdmVudExpc3RlbmVyKG5ldyBBbmltR0xFdmVudExpc3RlbmVyKCkpOw0KICAgIGZyYW1l LmFkZChjYW52YXMpOw0KDQogICAgLy9BbmltYXRvciBtaXQgWmVpY2hlbmZs5GNoZSBpbml0aWFs aXNpZXJlbg0KICAgIGFuaW1hdG9yID0gbmV3IEFuaW1hdG9yKGNhbnZhcyk7DQoNCiAgICBmcmFt ZS5zZXRTaXplKCA0MDAsDQogICAgICAgICAgICAgICAgICAgNDAwKTsNCiAgICBmcmFtZS5hZGRX aW5kb3dMaXN0ZW5lcihuZXcgV2luZG93QWRhcHRlcigpDQogICAgICB7DQogICAgICAgIHB1Ymxp YyB2b2lkIHdpbmRvd0Nsb3NpbmcoV2luZG93RXZlbnQgZSkNCiAgICAgICAgew0KICAgICAgICAg IGFuaW1hdG9yLnN0b3AoKTsNCiAgICAgICAgICBTeXN0ZW0uZXhpdCgwKTsNCiAgICAgICAgfQ0K ICAgICAgfSk7DQoNCiAgICBmcmFtZS5zZXRWaXNpYmxlKHRydWUpOw0KICAgIC8vQW5pbWF0aW9u IHN0YXJ0ZW4NCiAgICBhbmltYXRvci5zdGFydCgpOw0KICB9DQp9DQo= 60 2005-06-22 00:26:00 +0200 2005-06-22 00:26:00 +0200 GLEventListener for test case AnimGLEventListener.java text/plain 1456 sgothel aW1wb3J0IG5ldC5qYXZhLmdhbWVzLmpvZ2wuKjsNCg0KcHVibGljIGNsYXNzIEFuaW1HTEV2ZW50 TGlzdGVuZXIgaW1wbGVtZW50cyBHTEV2ZW50TGlzdGVuZXINCnsNCg0KICBmbG9hdCB0cmkgPSAw LjBmOw0KDQoNCiAgcHVibGljIHZvaWQgaW5pdChHTERyYXdhYmxlIGRyYXdhYmxlKQ0KICB7DQog ICAgR0wgZ2wgPSBkcmF3YWJsZS5nZXRHTCgpOw0KICAgIGdsLnNldFN3YXBJbnRlcnZhbCgwKTsN CiAgfQ0KDQogIHB1YmxpYyB2b2lkIGRpc3BsYXkoR0xEcmF3YWJsZSBkcmF3YWJsZSkNCiAgew0K ICAgIEdMIGdsID0gZHJhd2FibGUuZ2V0R0woKTsNCiAgICBnbC5nbE1hdHJpeE1vZGUoR0wuR0xf TU9ERUxWSUVXKTsNCiAgICBnbC5nbENsZWFyKEdMLkdMX0NPTE9SX0JVRkZFUl9CSVQgfCBHTC5H TF9ERVBUSF9CVUZGRVJfQklUKTsNCiAgICBnbC5nbExvYWRJZGVudGl0eSgpOw0KDQogICAgZ2wu Z2xUcmFuc2xhdGVmKC0wLjRmLDAuMGYsMC4wZik7DQogICAgZ2wuZ2xSb3RhdGVmKHRyaSwwLjBm LDEuMGYsMC4wZik7DQoNCiAgICBnbC5nbEJlZ2luKEdMLkdMX1RSSUFOR0xFX0ZBTik7DQogICAg Z2wuZ2xDb2xvcjNmKDEuMGYsMC4wZiwwLjBmKTsNCiAgICBnbC5nbFZlcnRleDNmKC0wLjJmLDAu MGYsMC4wZik7DQogICAgZ2wuZ2xDb2xvcjNmKDAuMGYsMS4wZiwwLjBmKTsNCiAgICBnbC5nbFZl cnRleDNmKDAuMmYsMC4wZiwwLjBmKTsNCiAgICBnbC5nbENvbG9yM2YoMC4wZiwwLjBmLDEuMGYp Ow0KICAgIGdsLmdsVmVydGV4M2YoMC4wZiwwLjZmLDAuMGYpOw0KICAgIGdsLmdsRW5kKCk7DQoN CiAgICBnbC5nbExvYWRJZGVudGl0eSgpOw0KICAgIGdsLmdsVHJhbnNsYXRlZigwLjRmLDAuMGYs MC4wZik7DQogICAgZ2wuZ2xSb3RhdGVmKHRyaSwxLjBmLDAuMGYsMC4wZik7DQogICAgZ2wuZ2xC ZWdpbihHTC5HTF9UUklBTkdMRV9GQU4pOw0KICAgIGdsLmdsQ29sb3IzZigxLjBmLDAuMGYsMC4w Zik7DQogICAgZ2wuZ2xWZXJ0ZXgzZigtMC4yZiwwLjBmLDAuMGYpOw0KICAgIGdsLmdsQ29sb3Iz ZigwLjBmLDEuMGYsMC4wZik7DQogICAgZ2wuZ2xWZXJ0ZXgzZigwLjJmLDAuMGYsMC4wZik7DQog ICAgZ2wuZ2xDb2xvcjNmKDAuMGYsMC4wZiwxLjBmKTsNCiAgICBnbC5nbFZlcnRleDNmKDAuMGYs MC42ZiwwLjBmKTsNCiAgICBnbC5nbEVuZCgpOw0KDQogICAgaWYodHJpIDwgMzYwLmYpDQogICAg ICB0cmkgKz0gMC41ZjsNCiAgICBlbHNlDQogICAgICB0cmkgPSAwLjBmOw0KICB9DQoNCiAgcHVi bGljIHZvaWQgcmVzaGFwZShHTERyYXdhYmxlIGRyYXdhYmxlLCBpbnQgeCwgaW50IHksIGludCB3 aWR0aCwgaW50IGhlaWdodCkNCiAgew0KDQogIH0NCg0KICBwdWJsaWMgdm9pZCBkaXNwbGF5Q2hh bmdlZChHTERyYXdhYmxlIGRyYXdhYmxlLCBib29sZWFuIG1vZGVDaGFuZ2VkLCBib29sZWFuIGRl dmljZUNoYW5nZWQpDQogIHsNCg0KICB9DQoNCn0NCg==