[7u-dev] Request for approval: 8038440: backport few C2 fixes

Poonam Bajaj poonam.bajaj at oracle.com
Wed Apr 9 10:18:53 UTC 2014


Thanks Sean. I added the keywords to the bug report.

regards,
Poonam

On 4/9/2014 3:34 PM, Seán Coffey wrote:
> Poonam,
>
> Looks like 8038440 is non-applicable for JDK 8u and JDK 9 then. Can 
> you add the 8-na label (and perhaps a noreg-* label) to bug report ?
>
> Approved for jdk7u-dev.
>
> regards,
> Sean.
>
> On 09/04/14 02:20, Poonam Bajaj wrote:
>> Hi,
>> This is a request to integrate fix for 8038440  to JDK7u-dev.:
>>
>> Bug: JDK-8038440 <https://bugs.openjdk.java.net/browse/JDK-8038440>: 
>> backport few C2 fixes
>> Webrev: http://cr.openjdk.java.net/~poonam/8038440/webrev.01/
>>
>> These changes backport few C2 fixes to 7u. Changes backported:
>> 1. Move disabling UseOptoBiasInlining after the last change to 
>> UseBiasedLocking in src/share/vm/runtime/arguments.cp.
>> 2. In OptoRuntime::new_named_counter() in 
>> src/share/vm/opto/runtime.cpp file, cmpxchg may fail first time when 
>> new named
>> counter is added. It triggers assert in set_next() because _next is 
>> set already. Fix it by resetting _next for each attempt.
>>
>> Code Reviewed by:
>> Vladimir Kozlov , Kevin Walls
>>
>> Testing:
>> JPRT
>>
>> Thanks,
>> Poonam
>>
>


More information about the jdk7u-dev mailing list