RFR(S): 8035328: closed/compiler/6595044/Main.java failed with timeout
tobias.hartmann at oracle.com
Tue Jun 24 05:53:33 UTC 2014
Thank you, Igor.
On 21.06.2014 05:50, Igor Veresov wrote:
> Looks good.
> On Jun 19, 2014, at 12:15 AM, Tobias Hartmann
> <tobias.hartmann at oracle.com <mailto:tobias.hartmann at oracle.com>> wrote:
>> please review the following patch for 8035328.
>> Bug: https://bugs.openjdk.java.net/browse/JDK-8035328
>> Webrev: http://cr.openjdk.java.net/~thartmann/8035328/webrev.00
>> *This bug is not a test bug. The test executes hundreds of small
>> methods consisting of only a return statement multiple times. This
>> was intended to test for an old (closed) bug in the C2 compiler. The
>> timeout arises when the tiered compilation policy decides to
>> recompile the caller method and therefore marks it as non-entrant.
>> The method is still in use but call sites are not patched. See line
>> 1179 of sharedRuntime.cpp:
>> // We do not patch the call site if the caller nmethod has been made
>> As a result, all call sites have to be resolved on every call which
>> is costly and leads to the timeout (4mins vs. 17s).
>> *The call site patching code in sharedRuntime is changed to patch not
>> entrant call sites as well.
>> *- JPRT
>> - SPECjbb2005, SPECjbb2013
>> - SPECjvm2008-Compiler*
>> *- Nashorn + Octane*
>> *- UTE full run with Kitchensink
-------------- next part --------------
An HTML attachment was scrubbed...
More information about the hotspot-compiler-dev