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

*** 155,168 ****
  
          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();
  
          // 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);
--- 155,168 ----
  
          String testClassName = nsk.jdwp.ReferenceType.Instances.instances001.TestClass.class.getName();
  
          long typeID = debuggee.getReferenceTypeID(createTypeSignature(testClassName));
  
!         // 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);