RFR: 8148397: Create new tests for IHOP (original) (raw)

Michail Chernov michail.chernov at oracle.com
Mon Mar 21 17:45:33 UTC 2016

Dmitry,

Thanks for reviewing, test was renamed as you proposed.

http://cr.openjdk.java.net/~mchernov/8148397/webrev.01/

Thanks Michail

On 21.03.2016 17:12, Dmitry Fazunenko wrote:

Michail,

The fix looks good to me. Could you consider renaming: test/stress/gc/TestIHOPParallel --> test/stress/gc/TestStressIHOPMultiThread Thanks, Dima

On 18.03.2016 18:57, Michail Chernov wrote: Hi,

Could I have a review for this change, please: Bug: https://bugs.openjdk.java.net/browse/JDK-8148397 Webrev: http://cr.openjdk.java.net/~mchernov/8148397/webrev.00/ TestIHOPParallel.java - stress test for Adaptive IHOP with logging. Test allocates and frees memory in several threads. Executes with enabled logging for IHOP: -Xlog:gc+ihop=debug,gc+ihop+ergo=debug,gc+ergo=debug . Expects for no crashes during working. TestIHOPStatic.java - provokes concurrent cycle initiation with static IHOP during allocation to check that occupancy and threshold (IHOP) are valid and occupancy is greater than threshold. TestIHOPErgo.java - provokes concurrent cycle initiation with static and adaptive IHOP. Checks that occupancy and threshold are valid. Checks that adaptive IHOP prediction is activated during some iterations. Executed on several configurations with nightly VM options. Thanks Michail

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

More information about the hotspot-gc-dev mailing list