RFR(S): 8157984 - [TESTBUG] Several compiler tests fails when are executed with -XX:TieredStopAtLevel=1
Dmitrij Pochepko
dmitrij.pochepko at oracle.com
Fri Jul 8 17:02:28 UTC 2016
Hi,
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/
Thanks,
Dmitrij
>
>>> 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