RFR (xxs) JDK-8012525 gc/metaspace/G1AddMetaspaceDependency.java Test fails a safepoint timeout assertion or hangs.

[Thomas Schatzl]

On Tue, 2013-10-01 at 11:44 +0200, Mikael Gerdin wrote:
> Hi
> 
> Please review this small test fix to work around a problem where the 
> test accidentally provokes a very slow execution path in the VM.
> This slow path can cause very long GC pauses or delayed safepoints.
> 
> Summary of the bug:
> The byte array allocation causes a big MemRegion to be stored as the 
> thread's deferred_card_mark. This can cause two separate slow code paths:
>[...]
> I've ran the test with the large allocation removed and it still 
> successfully reproduces the bug it was written to provoke on an unfixed 
> build, so my suggestion is to simply remove the byte array allocation.
> 
> The bug is unfortunately not externally visible.
> 
> Webrev: http://cr.openjdk.java.net/~mgerdin/8012525/webrev

Looks good.

Thomas