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

[Bengt Rutisson]

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
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://mail.openjdk.org/pipermail/hotspot-gc-dev/attachments/20130212/3f51c5a9/attachment.htm>