[RFR] 8061715: gc/g1/TestShrinkAuxiliaryData15.java fails with java.lang.RuntimeException: heap decommit failed - after > before

Jesper Wilhelmsson jesper.wilhelmsson at oracle.com
Fri Feb 27 16:16:55 UTC 2015


Looks good!
/Jesper

Andrey Zakharov skrev den 26/2/15 18:55:
> Updated test with
>   - updated 2 spaces for c++ code and code style
>   - updated copyrights
>   - renamed variable hotCardTableSize
>   - added output for current page size
>
> webrev:
> http://cr.openjdk.java.net/~azakharov/8061715/webrev.05/
>
> testing: yes (sthjprt 2015-02-26-172927.gtee.auxdata - I will restart solaris on
> complete)
>
> Thanks.
>
>>>>
>>>> - in TestShrinkAuxiliaryData.java, line 107 seems to be debugging code.
>>>> Afaik jtreg already automatically puts all output into the log files.
> This is why this code here: OutputAnalyzer with ProcessBuilder doesn't
> implicitly output anything.
>
>>>>
>>>> - line 174, I do not understand the comment. What does "if auxdata size
>>>> will be more than page size it would not decommit auxiliary data size is
>>>> about ~3.6% of heap size" seem to be at least two sentences.
> Fixed
>>>>
>>>> - what do the two reasons given in the comment of
>>>> checkEnvApplicability() actually mean? I.e. why do you not run the test
>>>> if large pages are enabled?
> I dont want to run this test on box where page size is larger than auxiliary
> data size for region, where decommitment wouldn't occurs for sure.
>



More information about the hotspot-gc-dev mailing list