RFR(S): 8059846: InstanceKlass should use MutexLockerEx to acquire OsrList_lock
tobias.hartmann at oracle.com
Tue Oct 14 15:05:12 UTC 2014
Thanks, David and Vladimir.
On 14.10.2014 16:58, Vladimir Kozlov wrote:
> Looks good. Should be reviewed by runtime too since changes are in
> On 10/14/14 5:54 AM, David Chase wrote:
>> Hello Tobias,
>> Not a Reviewer here, but it looks correct to me.
>> One question — above I see a NEEDS_CLEANUP notation.
>> Do we know (does someone know?) if it refers to this change?
>> If so, we could remove that :-).
I don't know if this statement is related to my changes but I assume it refers
to the comment '// only one compilation can be active'. Maybe someone from the
runtime team knows.
>> On 2014-10-14, at 3:55 AM, Tobias Hartmann <tobias.hartmann at oracle.com> wrote:
>>> please review this small code cleanup that replaces the explicit locking of
>>> OsrList_lock by a MutexLockerEx instantiation.
>>> Bug: https://bugs.openjdk.java.net/browse/JDK-8059846
>>> Webrev: http://cr.openjdk.java.net/~thartmann/8059846/webrev.00/
More information about the hotspot-compiler-dev