RFR: 8146728: TestPrintGCDetailsVerbose is never run by jtreg

Kim Barrett kim.barrett at oracle.com
Tue Jan 12 18:53:21 UTC 2016


On Jan 12, 2016, at 1:46 PM, Kim Barrett <kim.barrett at oracle.com> wrote:
> 
>> On Jan 12, 2016, at 11:02 AM, Stefan Johansson <stefan.johansson at oracle.com> wrote:
>>>> CR:
>>>> https://bugs.openjdk.java.net/browse/JDK-8146728
>>>> 
>>>> Webrev:
>>>> http://cr.openjdk.java.net/~kbarrett/8146728/webrev.00/
>>> The change looks good if we want to continue running the test with debug-builds only, but I wonder if it really makes sense to do so. I suspect that the reason we had the restriction was that Verbose was only available in debug-builds. But now when using -Xlog:gc*=trace the test should work with product builds as well, or am I missing something?
>> 
>> The test was created to verify that some assertions were no longer triggered.  Not product => no assertions to be triggered.
> 
> That said, there probably is some benefit to beating on (GC) logging in a product build.  So sure, I’ll make that change.

In which case, should I also remove the jdk.debug requires property?  This is the only test presently using this feature.  It can always be added back if some test needs it in the future.

I think this is also the only test using ~= (regex match) in a @requires.




More information about the hotspot-gc-dev mailing list