Review request (7u) JDK-8038440: backport few C2 fixes

[Vladimir Kozlov]

Looks good.

Thanks,
Vladimir

On 4/1/14 6:21 PM, Poonam Bajaj wrote:
> Hi Vladimir,
>
> Sorry, mistake. I have corrected it. Please check the following webrev:
> http://cr.openjdk.java.net/~poonam/8038440/webrev.01/
>
> Thanks,
> Poonam
>
> On 3/31/2014 11:37 PM, Vladimir Kozlov wrote:
>> Poonam,
>>
>> Why you did not remove original UseOptoBiasInlining lines in arguments.cpp?
>>
>> Thanks,
>> Vladimir
>>
>> On 3/30/14 10:36 PM, Poonam Bajaj wrote:
>>> Bug: JDK-8038440 <https://bugs.openjdk.java.net/browse/JDK-8038440>: backport few C2 fixes
>>> Webrev: http://cr.openjdk.java.net/~poonam/8038440/webrev.00/
>>>
>>> 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.
>>>
>>> Testing:
>>> JPRT
>>>
>>> Thanks,
>>> Poonam
>>>