Hi Mandy,
that's actually the opposite, the 2nd subtest is run only in modes other than
Xcomp, as w/ Xcomp the test creates lots of adapters and used to lead to JVM
failure as described in 7049122. I tried to reproduce this failure, but in
vain,.. after a bit more historical digging, I realized that the underlying
problem was 7009641, which has been fixed in hs25/jdk8. so I've changed the fix
for 8249697 to simply return run w/ '-DRicochetTest.MAX_ARITY=255':
http://cr.openjdk.java.net/~iignatyev//8249697/webrev.02
I've verified that the test passes w/ Xcomp and
- -XX:+TieredCompilation (c1 + c2);
- -XX:-TieredCompilation (c2-only);
- -XX:+NeverActAsServerClassMachine (emulated-client, c1-only)
the test was run 100 times on {linux,windows,macos}-x64 w/ 0 failures.
Thanks,
-- Igor
> On Jul 18, 2020, at 9:32 PM, Mandy Chung <mandy.ch...@oracle.com> wrote:
>
>
>
> On 7/17/20 8:54 PM, Igor Ignatyev wrote:
>> http://cr.openjdk.java.net/~iignatyev/8249697/webrev.00/
>> <http://cr.openjdk.java.net/~iignatyev/8249697/webrev.00/>
>>
>
> I suggest to change this:
> 32 * @comment The following test creates an unreasonable number of
> adapters in -Xcomp mode (7049122)
>
> To:
>
> @bug 8249697
> @summary verify very high number of adapters in -Xcomp mode
>
> Otherwise, looks fine.
>
> Mandy
>> Hi all,
>>
>> could you please review this small and trivial patch for
>> java/lang/invoke/RicochetTest.java test?
>> from JBS:
>>> a run of java/lang/invoke/RicochetTest.java w/ MAX_ARITY=255 was removed
>>> from all configurations by JDK-7049122, yet the problem manifests itself
>>> only w/ Xcomp. as now we have @requires to filter out tests from certain
>>> configurations, the test can be updated to run MAX_ARITY=255 in all configs
>>> but Xcomp.
>> the patch splits the test into two subtests, each one w/ one @run, and use
>> @requires to exclude one w/ MAX_ARITY=255 from execution if Xcomp flag is
>> used.
>>
>> JBS: https://bugs.openjdk.java.net/browse/JDK-8249697
>> <https://bugs.openjdk.java.net/browse/JDK-8249697>
>> webrev: http://cr.openjdk.java.net/~iignatyev/8249697/webrev.00/
>> <http://cr.openjdk.java.net/~iignatyev/8249697/webrev.00/>
>> testing: java/lang/invoke/RicochetTest.java on {linux,windows,macos}-x64 w/
>> and w/o -Xcomp; Xcomp runs, as expected, had only 1 test run
>>
>> Thanks,
>> -- Igor
>>
>> JDK-7049122 : https://bugs.openjdk.java.net/browse/JDK-7049122
>> <https://bugs.openjdk.java.net/browse/JDK-7049122>
