Request for review (S): 8006398: Add regression tests for deprectated GCs

[Jesper Wilhelmsson]

Bengt,

The tests looks fine, ship'em!
/Jesper

On 12/2/13 1:56 PM, Bengt Rutisson wrote:
>
> Hi again,
>
> Christian has pushed his test library change and it has propagated to the
> hotspot-gc repository. So, this change is ready to be pushed. Just need some
> reviews. Any takers? :)
>
> Thanks,
> Bengt
>
> On 1/16/13 1:58 PM, Bengt Rutisson wrote:
>>
>> Hi all,
>>
>> Could I have a couple of reviews for this change?
>> http://cr.openjdk.java.net/~brutisso/8006398/webrev.00/
>>
>> Recently we deprecated some GC combinations. Those should now print a warning
>> at startup. Other GC combinations should not print any warnings.
>>
>> With the new process handling support that Christian Törnqvist is adding to
>> the JTREG tests for hotspot it is very easy to write test that start a VM and
>> checks the output.
>>
>> This changes makes use of Christian's testlibrary to verify that warnings are
>> printed as expected.
>>
>> I'm also adding the "gc" keyword to JTREG to make it possible to filter out GC
>> tests. We should probably use this for all test in the the /gc folder, but I
>> think that should be done as a separate change.
>>
>> The webrev above is based on Christian's webrev to add the testlibrary:
>> http://cr.openjdk.java.net/~brutisso/8006413/webrev.00/
>>
>> Thanks,
>> Bengt
>>
>