test/hotspot/jtreg/vmTestbase/nsk/jdwp/ReferenceType/Instances/instances001/instances001.java

@@ -155,14 +155,14 @@
 
         String testClassName = nsk.jdwp.ReferenceType.Instances.instances001.TestClass.class.getName();
 
         long typeID = debuggee.getReferenceTypeID(createTypeSignature(testClassName));
 
-        // Note!
-        // Test broken - assumes that no GC is run before heap walk.
-        // G1 fails, just like ZGC, if en explicitly GC is done here.
-        // forceGC();
+        // Note! This test is broken, in the sense that it incorrectly assumes
+        // that no GC can happen before it walks the heap. In practice, it seems
+        // to only affect this test when using ZGC. However, this test will also
+        // fail when using other GCs if an explicit GC is done here.
 
         // create command with maxInstances=1, only 1 instance should be returned
         testClass(typeID, 1, 1, false, 0);
         // create command with maxInstances=0, all instances should be returned
         testClass(typeID, 0, expectedInstances, false, 0);