RFR (S) 8016752: Test case for 8005857/8016740 (assert in GC_locker from PSOldGen::expand with -XX:+PrintGCDetails and Verbose)
dmitry.fazunenko at oracle.com
Fri Nov 20 11:16:12 UTC 2015
Thanks a lot for looking at the fix. I totally agree with you: CR
should be well descriptive. Fixed.
A question: This test is for Parallel only, because it's a regression
test for a problem in Parallel.
It could be easily made applicable for all collectors to check that
PrintGCDetails + Verbose doesn't crash when OOM happens.
What is your opinion, does it make any sense or that case is very unlikely?
On 20.11.2015 13:46, Thomas Schatzl wrote:
> On Mon, 2015-11-16 at 20:46 +0300, Dmitry Fazunenko wrote:
>> may I have a couple of reviews for a fix made Andrey Zakharov, please:
>> Comments: during the fix TEST.ROOT is modified: java.version is
>> registered as a property that can be used in the @requires tag.
>> It allows to declare a test as fastdebug specific:
>> @requires java.version ~= ".*fastdebug"
> looks okay. Is it possible to update the CR to include a description
> of the problem (+ change the summary), not just referencing other bugs.
More information about the hotspot-gc-dev