RFR[JDK11]: 8204691: HeapRegion.apply_to_marked_objects_other_vm_test fails with assert(!hr->is_free() || hr->is_empty()) failed: Free region 0 is not empty for set Free list #

[Thomas Schatzl]

Hi,

On Wed, 2018-07-04 at 20:13 -0400, Kim Barrett wrote:
> Please review this fix of the HeapRegion gtest.
> 
> The test modifies a region's "top" to unexpected values without
> ensuring that no allocation might use the region and no GC might run
> while the region is in that invalid state.  We solve this by
> executing the test code in its very own safepoint, and by saving and
> then restoring the region's top back to its original value before
> completing the test.  And since we are doing all that, there's no
> longer any reason to run the test in a separate VM.

  looks good, but the actual test is still run in a separate VM.
Intentional?

Thanks,
  Thomas