Request for Review (s) - 8060463: ParNew: SurvivorAlignmentInBytes greater then YoungPLABSize cause assert(obj != NULL || plab->words_remaining() < word_sz) failed: Else should have been able to allocate

[Michail Chernov]

Hi Jon,

Two minor comments:

1. Test should have @requires because you use specified GC and 
ExplicitGCInvokesConcurrent. Our nightly testing uses 
ExplicitGCInvokesConcurrent, so there can be options conflict.
Requires can be next:
   @requires (vm.gc == "ConcMarkSweep" | vm.gc == null)
            & vm.opt.ExplicitGCInvokesConcurrent != true

2. Test does not need any @modules or @library because it does not use 
such modules/libraries.

Thanks
Michail


On 03/21/2016 06:04 AM, Jon Masamitsu wrote:
>
> https://bugs.openjdk.java.net/browse/JDK-8060463
> http://cr.openjdk.java.net/~jmasa/8060463/webrev.00/
>
>