RFR - JDK-8179625 [BACKOUT] of 8179305 Avoid repeated calls to JavaThread::last_frame in InterpreterRuntime

[Ioi Lam]

Hi Dean,

That was it! I fixed this and reran the reproducer. No crash in 8000 
iterations, so I think we're good :-)

I'll do more stress testing before submitting the REDO review.

Thanks
- Ioi

On 5/4/17 9:53 AM, dean.long at oracle.com wrote:
> Sorry for not noticing before, but this looks like it could be the 
> cause of the crash:
>
> -    BasicObjectLock* monitor_begin() const {
> -      return _last_frame.interpreter_frame_monitor_end();
> -    }
> -    BasicObjectLock* monitor_end() const {
> -      return _last_frame.interpreter_frame_monitor_begin();
> -    }
>
> _begin and _end are reversed.
>
> dl
>
>
> On 5/4/17 6:40 AM, Ioi Lam wrote:
>> Hi,
>>
>> It turns out JDK- 8179305 causes intermittent failures in JPRT, so it 
>> needs to be backed out quickly:
>>
>> https://bugs.openjdk.java.net/browse/JDK-8179625
>> http://cr.openjdk.java.net/~iklam/jdk10/8179625-backout-8179305-last-frame.v01/ 
>>
>>
>> Thanks
>> - Ioi
>