RFR (S): 8067868: Add GCOld as a JTreg test
Dmitry Fazunenko
dmitry.fazunenko at oracle.com
Fri Dec 19 13:36:18 UTC 2014
Hi Bengt,
I didn't try to understand the logic in detail, I think this is not new
test.
What I noticed:
- the code doesn't meet java coding style (indent should be 4)
- checkTrees()method is declared, but all invocations are commented out.
If we don't need the method we can remove it
- this test never fails (only if OOM or crash). Is it expected?
Thanks,
Dima
On 18.12.2014 15:45, Bengt Rutisson wrote:
>
> Hi everyone,
>
> Could I have a couple of reviews for this change to make GCOld a JTreg
> test?
>
> Root repo changes:
> http://cr.openjdk.java.net/~brutisso/8067868/root-webrev.00/
>
> HotSpot repo changes:
> http://cr.openjdk.java.net/~brutisso/8067868/hotspot-webrev.00/
>
> Bug report:
> https://bugs.openjdk.java.net/browse/JDK-8067868
>
> Thanks,
> Bengt
More information about the hotspot-gc-dev
mailing list