RFR (XS) CR 8015493: runtime/contended/OopMaps.java fails with OutOfMemory

[Aleksey Shipilev]

Good idea to run this in JPRT. Yes please, I will need the tiny
instruction how to do this (offlist is fine).

-Aleksey.

On 05/28/2013 06:16 PM, Christian Tornqvist wrote:
> Hi Aleksey,
> 
> Change looks good, but please run this test in jprt on all platforms (this
> is not a part of a normal jprt run, but you just need to run this specific
> test so it'll be quick). Let me know if you need help doing this.
> 
> Thanks,
> Christian
> 
> -----Original Message-----
> From: hotspot-runtime-dev-bounces at openjdk.java.net
> [mailto:hotspot-runtime-dev-bounces at openjdk.java.net] On Behalf Of Aleksey
> Shipilev
> Sent: den 28 maj 2013 10:10
> To: hotspot-runtime-dev at openjdk.java.net
> Subject: RFR (XS) CR 8015493: runtime/contended/OopMaps.java fails with
> OutOfMemory
> 
> Hi,
> 
> This is the tiny fix for the nightly failure on the regression test:
>   http://cr.openjdk.java.net/~shade/8015493/webrev.01/
> 
> Reasons:
>   Due to the nature of test, R1 instance size is large (1880 bytes on
>   my Linux x86_64), and we allocate 100K of them during the test. We
>   do that in the test because we want to have some of the objects
>   pushed through the garbage collection to catch unusual behavior.
>   The flip side is, we have ~200 Mb heap allocated just for R1 objects.
>   While it works nicely on some platforms, the default heap sizes
>   may fail the test. This makes the issue the test bug.
> 
> Fix:
>   Allocate 10K of R1 objects in the test, require -Xmx128m. The target
>   heap occupancy for the test is then ~20Mb, so we have lots of
>   headroom for all platforms.
> 
> Testing:
>  (this falls into my definition of being trivial, so:)
>  * jtreg: runtime/contended/ on Linux x86_64
>  * manual testing with 8015270 partially reverted, OopMaps fails on Linux
> x86_64, as would anyone expect for a good regression test
>  * also looked through the code for other runtime/contended regression
> tests, and those seem unaffected
> 
> Thanks,
> -Aleksey.
>