Request for Review (s) - 8060463: ParNew: SurvivorAlignmentInBytes greater then YoungPLABSize cause assert(obj != NULL | (original) (raw)

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 michail.chernov at oracle.com
Thu Mar 24 12:33:45 UTC 2016


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/



More information about the hotspot-gc-dev mailing list