RFR (M): 8059575: JEP-JDK-8043304: Test task: Tiered Compilation level transition tests
Igor Veresov
igor.veresov at oracle.com
Wed Dec 10 22:26:51 UTC 2014
Looks ok.
However, one general comment... I skimmed through the WB API and noticed that you guys get nmethod* and operate on it without any precausions, it’s not really safe. nmethods can go away at anytime (starting from concurrent nmethod::_state modification and ending with the codeblob deallocation). So you have to either disable the sweeper, and/or make sure you cooperate with nmethod::make_not_entrant_or_zombie().
igor
> On Dec 10, 2014, at 9:59 AM, Pavel Punegov <pavel.punegov at oracle.com> wrote:
>
> Hi Tobias and Igor,
>
> could you please review this: http://cr.openjdk.java.net/~ppunegov/8059575/webrev.05/
>
> I replaced the check for OSR-methods to skip transitions after deoptimization as they may make intermittent failures.
> See 67-70 lines at LevelTransitionTest.java
>
> I will file a separate RFE to improve this test to be able to check transitions after deopt too.
>
> Testing: jprt, locally
>
> On 08.12.2014 10:30, Tobias Hartmann wrote:
>> Hi Pavel,
>>
>> looks good (not a reviewer).
>>
>> Best,
>> Tobias
>>
>> On 05.12.2014 21:17, Pavel Punegov wrote:
>>> Hi Tobias,
>>>
>>> see new webrev: http://cr.openjdk.java.net/~ppunegov/8059575/webrev.04/
>>>
>>> I added TransitionsTestExecutor.java class that checks if TieredCompilation is
>>> available in the VM it was started with. Then it starts test for
>>> CompilatonPolicyChoice 2 and 3.
>>>
>>> On 01.12.2014 11:27, Tobias Hartmann wrote:
>>>> Hi Pavel,
>>>>
>>>> the tests fail with a client VM:
>>>>
>>>>> Error occurred during initialization of VM
>>>>> Incompatible compilation policy selected
>>>> Please execute them on JPRT to make sure they work on all platforms.
>>>>
>>>> Best,
>>>> Tobias
>>>>
>>>> On 28.11.2014 13:27, Pavel Punegov wrote:
>>>>> Hi all,
>>>>>
>>>>> please review new tests developed as a part of the test task:
>>>>> JBS: https://bugs.openjdk.java.net/browse/JDK-8059575
>>>>> webrev: http://cr.openjdk.java.net/~ppunegov/8059575/webrev.00/
>>>>>
>>>>> Description:
>>>>> There are two tests to verify level transitions in Tiered Compilation:
>>>>> 1. LevelTransitionTest verifies that in case of no load on compiler queues test
>>>>> methods go through common transition patterns:
>>>>> 0 -> 3 (2) -> 4 for most of methods, 0 ->3(2) -> 1 and 0 -> 1 for different
>>>>> trivial methods, and 0 -> 4 after deoptimization.
>>>>> Test provokes compilations of the method and checks that all transitions are
>>>>> correct for CompilationPolicyChoice=2 and 3.
>>>>>
>>>>> 2. ConstantGettersTransitions is a test adapted from the test Tobias used for
>>>>> 8056071 (see [*] for details). It checks that all constant getters are always
>>>>> compiled with C1.
>>>>>
>>>>> [*] https://bugs.openjdk.java.net/browse/JDK-8028590
>>>>>
>>>>> Testing: done locally
>>>>>
>
> --
> Thanks,
> Pavel Punegov
>
More information about the hotspot-compiler-dev
mailing list