RFR(S): 8157984 - [TESTBUG] Several compiler tests fails when are executed with -XX:TieredStopAtLevel=1

Vladimir Ivanov vladimir.x.ivanov at oracle.com
Fri Jul 8 17:05:36 UTC 2016



> I've prepared version which has more compact requires expression but
> doesn't support explicit TieredStopAtLevel=4.
> http://cr.openjdk.java.net/~dpochepk/8157984/webrev.03/

Looks fine.

Best regards,
Vladimir Ivanov

>>>> General question: why do you fix the tests differently?
>>> Because TestRangeCheckSmearing can be modified to run on all tiers and
>>> we'll keep good test coverage.
>>
>> OK.
>>
>>>> test/compiler/tiered/NonTieredLevelsTest.java:
>>>>
>>>> + * @requires vm.opt.TieredStopAtLevel!=1 &
>>>> vm.opt.TieredStopAtLevel!=2 & vm.opt.TieredStopAtLevel!=3
>>>>
>>>> Why isn't it just vm.opt.TieredStopAtLevel=4?
>>> because unfortunately jtreg only parses explicit vm options for
>>> "vm.opt.*" parameter i.e. only those which are present in comandline.
>>> In case vm.opt.TieredStopAtLevel == 4 is specified in requires it won't
>>> work in case of default vm options (so, 4th level is set by default),
>>> so, we have to use such construction.
>>
>> Can you ignore the test if TieredStopAtLevel is overridden on
>> command-line? I don't see much value in supporting
>> -XX:TieredStopAtLevel=4 case.
>>
>> Best regards,
>> Vladimir Ivanov
>>
>>> I've created v02 with testlibrary changes :
>>> http://cr.openjdk.java.net/~dpochepk/8157984/webrev.02/
>>>
>>> Thanks,
>>> Dmitrij
>>>>
>>>> Best regards,
>>>> Vladimir Ivanov
>>>>
>>>> On 7/6/16 7:56 PM, Dmitrij Pochepko wrote:
>>>>> Hi,
>>>>>
>>>>> please review fix for 8157984 - [TESTBUG] Several compiler tests fails
>>>>> when are executed with -XX:TieredStopAtLevel=1
>>>>>
>>>>>
>>>>> 3 tests failed with TieredStopAtLevl=1 option included (these tests
>>>>> were
>>>>> quarantined by
>>>>> http://hg.openjdk.java.net/jdk9/hs/hotspot/rev/1bf96735f3be))
>>>>>
>>>>> 1) compiler/tiered/NonTieredLevelsTest.java modified to have
>>>>> respective
>>>>> "requires" tag
>>>>> 2) compiler/rangechecks/TestRangeCheckSmearing.java modified to have
>>>>> correct checks inside test
>>>>> 3) separate issue for compiler/ciReplay/TestVM_no_comp_level.sh was
>>>>> created so, respective @ignore tag changed.
>>>>>
>>>>> webrev: http://cr.openjdk.java.net/~dpochepk/8157984/webrev.01/
>>>>> CR: https://bugs.openjdk.java.net/browse/JDK-8157984
>>>>>
>>>>> I've tested these locally on linux-arm64 providing various vm flags
>>>>> like
>>>>> TieredCompilation, TieredStopAtLevel.
>>>>>
>>>>>
>>>>> Thanks,
>>>>> Dmitrij
>>>
>


More information about the hotspot-compiler-dev mailing list