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

[Bengt Rutisson]

Thanks for the reveiws Erik and Jesper!

Now I just need a Reviewer to look at this too... ;)

Bengt


On 2/12/13 2:39 PM, Jesper Wilhelmsson wrote:
> 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
>>>
>>