RFR: 8162852: Mark stress compiler and gc tests with stress keyword

Leonid Mesnik Leonid.Mesnik at oracle.com
Thu Aug 4 08:58:35 UTC 2016


Thanks for review.

Leonid

On 09.06.2016 18:04, Vladimir Kozlov wrote:
> Thank you, Leonid, for offlist explanation.
> Tests will be moved to different testing tier since their long 
> execution time.
> These tests run fixed (and long) time instead of until completion. And 
> "stress" keyword is already used for similar tests.
> With this information I agree with changes.
>
> Thanks,
> Vladimir
>
> On 8/2/16 11:46 AM, Vladimir Kozlov wrote:
>> "stress" is somewhat confusing word. I would expect "serial" based on
>> the problem description:
>>
>> "These tests doesn't allow to gain performance benefits executing all
>> hotspot jtreg tests concurrently."
>>
>> Why these test so special? Please, explain how it affects these tests
>> execution? Are they excluded?
>>
>> Thanks,
>> Vladimir
>>
>> On 8/1/16 4:09 AM, Leonid Mesnik wrote:
>>> Please review following patch which marks 'stress' tests with
>>> corresponding keyword.
>>>
>>> Issue: https://bugs.openjdk.java.net/browse/JDK-8162852
>>>
>>> lmesnik at lmesnik-desktop:~/ws/hs-fast/hotspot$ hg diff
>>> diff -r 2ea13db80d5c
>>> test/compiler/codecache/stress/RandomAllocationTest.java
>>> --- a/test/compiler/codecache/stress/RandomAllocationTest.java Thu Jul
>>> 21 16:29:21 2016 +0200
>>> +++ b/test/compiler/codecache/stress/RandomAllocationTest.java Mon Aug
>>> 01 13:55:00 2016 +0300
>>> @@ -28,6 +28,7 @@
>>>
>>>   /*
>>>    * @test RandomAllocationTest
>>> + * @key stress
>>>    * @library /testlibrary /test/lib
>>>    * @modules java.base/jdk.internal.misc
>>>    *          java.management
>>> diff -r 2ea13db80d5c
>>> test/compiler/codecache/stress/UnexpectedDeoptimizationTest.java
>>> --- a/test/compiler/codecache/stress/UnexpectedDeoptimizationTest.java
>>> Thu Jul 21 16:29:21 2016 +0200
>>> +++ b/test/compiler/codecache/stress/UnexpectedDeoptimizationTest.java
>>> Mon Aug 01 13:55:00 2016 +0300
>>> @@ -24,6 +24,7 @@
>>>
>>>   /*
>>>    * @test UnexpectedDeoptimizationTest
>>> + * @key stress
>>>    * @library /testlibrary /test/lib
>>>    * @modules java.base/jdk.internal.misc
>>>    *          java.management
>>> diff -r 2ea13db80d5c test/compiler/codegen/7184394/TestAESMain.java
>>> --- a/test/compiler/codegen/7184394/TestAESMain.java    Thu Jul 21
>>> 16:29:21 2016 +0200
>>> +++ b/test/compiler/codegen/7184394/TestAESMain.java    Mon Aug 01
>>> 13:55:00 2016 +0300
>>> @@ -25,6 +25,7 @@
>>>   /**
>>>    * @test
>>>    * @bug 7184394
>>> + * @key stress
>>>    * @summary add intrinsics to use AES instructions
>>>    * @library /testlibrary
>>>    *
>>> diff -r 2ea13db80d5c test/gc/g1/ihop/TestIHOPErgo.java
>>> --- a/test/gc/g1/ihop/TestIHOPErgo.java Thu Jul 21 16:29:21 2016 +0200
>>> +++ b/test/gc/g1/ihop/TestIHOPErgo.java Mon Aug 01 13:55:00 2016 +0300
>>> @@ -24,6 +24,7 @@
>>>    /*
>>>    * @test TestIHOPErgo
>>>    * @bug 8148397
>>> + * @key stress
>>>    * @summary Test checks that behavior of Adaptive and Static IHOP at
>>> concurrent cycle initiation
>>>    * @requires vm.gc.G1
>>>    * @requires !vm.flightRecorder
>>>



More information about the hotspot-compiler-dev mailing list