<div dir="ltr"><div>Hi, Runtime and GC team</div><div><br></div><div>(This email has been sent out to hotspot-dev alias, but I did not receive the notice. Since i already registered to it and did not receive it so I decided send the copy again to hotspot-rumtime-dev an hotspot-gc-dev).</div><div><br></div><div>  This is Yumin now  have a question to ask for help --- I am not sure if it is a bug.</div><div>   As stated in the subject, found some weakly referenced object not GC'ed. </div><div><br></div><div><br></div><div>  Code here is the test case we used to check for this problem.</div><div><br></div><div>  There are two classes AAA and AAB which are all loaded by the TestClassLoader but different instances which are independent. TestClassLoader extends from URLClassLoader.</div><div><br></div><div>  AAA has a field type of AAB which will be set to null in the test.</div><div><br></div><div>public class AAA {</div><div>     private AAB aab;</div><div>     public AAA() {</div><div>         aab = new AAB();</div><div>     }</div><div>     public void clear() {</div><div>         aab = null;</div><div>     }</div><div>}</div><div><br></div><div>public class AAB {</div><div> }</div><div><br></div><div>The two class have to be in different jars(here AAA.jar contains AAA.class, and AAB.jar contains AAB.jar), which are not in the class path so are being loaded from different locations for the test.</div><div><br></div><div>import java.net.URL;</div><div>import java.net.URLClassLoader;</div><div>import java.util.WeakHashMap;</div><div><br></div><div><br></div><div>public class TestLoader extends URLClassLoader {</div><div>        public static WeakHashMap<TestLoader,Object> map=new WeakHashMap<TestLoader,Object>();</div><div>        private static int count=0;</div><div>        public TestLoader(URL[] urls){</div><div>                super(urls);</div><div>                map.put(this, new Object());</div><div>        }</div><div>        @SuppressWarnings("resource")</div><div>        public Class<?> loadClass(String name) throws ClassNotFoundException {</div><div>                if (name.equals("AAB") && count==0) {</div><div>                        try {</div><div>                            count=1;</div><div>                            URL[] urls = new URL[1];</div><div>                            urls[0] = new URL("file:///home/nijiaben/tmp/AAB.jar"); // You need to use your own location for AAB.jar here!!!</div><div>                            return new TestLoader(urls).loadClass("AAB");</div><div>                        } catch (Exception e){</div><div>                            e.printStackTrace();</div><div>                        }</div><div>                } else {</div><div>                        return super.loadClass(name);</div><div>                }</div><div>                return null;</div><div>        }</div><div>}</div><div><br></div><div>TestLoader puts itself in the WeakHashMap --- upon "AAB" loading, it uses new instance of TestLoader. the new Object is just a object as value which has nothing to strongly or weakly to the key.</div><div><br></div><div>  TTest.java is the test program to start with, basically self explained.</div><div><br></div><div>import java.lang.reflect.Method;</div><div>import java.net.URL;</div><div><br></div><div>// Author: Jiapeng Li</div><div><br></div><div>public class TTest {</div><div>    private Object aaa;</div><div>    public static void main(String args[]){</div><div>        try {</div><div>            TTest tt = new TTest();</div><div>            //Move tt to old gen and clear aab in aaa</div><div>            test(tt);</div><div>            // do a final full GC, the TestLoader for aab should be purged.</div><div>            System.gc();</div><div>            System.out.println("finished");  // stop here in debugger and dump heap</div><div>        }catch (Exception e){</div><div>            e.printStackTrace();</div><div>        }</div><div>    }</div><div><br></div><div>    @SuppressWarnings("resource")</div><div>        public static void test(TTest tt){</div><div>        try {</div><div>            // New instance of TestLoader which will load AAA from AAA.jar</div><div>            URL[] urls = new URL[1];</div><div>            urls[0] = new URL("file:///home/nijiaben/tmp/AAA.jar");  // You have to use your own location for AAA.jar here!!!</div><div>            tt.aaa=new TestLoader(urls).loadClass("AAA").newInstance();</div><div>            // young GC will not purge the class loader, after 10 times of full GC, it should move it to old gen</div><div>            for (int i = 0; i < 10; i++) {</div><div>                System.gc();</div><div>                Thread.sleep(1000);</div><div>            }</div><div>            //  set aaa.aab= null，so next full gc will collect  it</div><div>            Method[] methods=tt.aaa.getClass().getDeclaredMethods();</div><div>            for (Method m : methods){</div><div>                if (m.getName().equals("clear")) {</div><div>                        m.invoke(tt.aaa);</div><div>                        break;</div><div>                }</div><div>            }</div><div>        } catch (Exception e) {</div><div>            e.printStackTrace();</div><div>        }</div><div>    }</div><div>}</div><div><br></div><div>  After final  full GC, there should be no instance of AAB exists, but the instance of AAB's class loader (TestLoader) still not cleaned by GC. See the stop point above, we dumped heap after final full GC. Following is the graph for reference which still holds for the WeakHashMap.</div><div><br></div><div><img src="cid:ii_1545a285dc627168" alt="Inline image 1" width="513" height="188"><br></div><div><br></div><div>  Notice that the dependency records the two instances of TestLoaders in order that they are related,   When AAA is loaded, it loads AAB and after AAB loaded, dependency records the relationship in _dependencies. </div><div><br></div><div>but we don't have 'remove' method to disengage them for GC. </div><div>When GC, _dependencies.oops_do, the is_alive Closure still marks the TestLoader for AAB alive even though it has nothing to refer to due to the dependency.</div><div><br></div><div>void ClassLoaderData::oops_do(OopClosure* f, KlassClosure* klass_closure, bool must_claim) {</div><div>  if (must_claim && !claim()) {</div><div>    return;</div><div>  }</div><div><br></div><div>  f->do_oop(&_class_loader);</div><div>  _dependencies.oops_do(f);</div><div>  _handles->oops_do(f);</div><div>  if (klass_closure != NULL) {</div><div>    classes_do(klass_closure);</div><div>  }</div><div>}</div><div><br></div><div>Is this a bug or a design consideration? </div><div><br></div><div>Any comments are appreciated!</div><div><br></div><div>Thanks</div><div>Yumin</div></div>