RFR (XS) JDK-8015272: Make @Contended within the same group to use the same oop map
Aleksey Shipilev
aleksey.shipilev at oracle.com
Thu Nov 6 13:00:38 UTC 2014
Hi,
The Halloween is over, but here is a creepy undead patch from the past.
http://cr.openjdk.java.net/~shade/8015272/webrev.01/
https://bugs.openjdk.java.net/browse/JDK-8015272
8u does not need the patch, but it would be nice to have it in 9.
I have checked:
* Still the same chunk of code as for non- at Contended cases
* Still builds fine on Linux x86_64 fastdebug/release
* Still passes all hotspot/test/runtime jtreg tests
* Still passes the JPRT
Thanks,
-Aleksey.
On 10/14/2013 01:01 AM, Aleksey Shipilev wrote:
> Hi Christian,
>
> Your call. I'm merely announcing the patch is ready. :)
>
> -Aleksey.
>
> On 10/13/2013 10:02 PM, Christian Tornqvist wrote:
>> Hi Aleksey
>>
>> We're well past Feature Complete (May 23) and Zero Bug Bounce is around the
>> corner, this is not the time to push enhancements. In my opinion this should
>> be done in the next 8u or in jdk9.
>>
>> Thanks,
>> Christian
>>
>> -----Original Message-----
>> From: hotspot-runtime-dev-bounces at openjdk.java.net
>> [mailto:hotspot-runtime-dev-bounces at openjdk.java.net] On Behalf Of Aleksey
>> Shipilev
>> Sent: Sunday, October 13, 2013 12:11 PM
>> To: hotspot-runtime-dev at openjdk.java.net
>> Subject: RFR (XS) JDK-8015272: Make @Contended within the same group to use
>> the same oop map
>>
>> Hi,
>>
>> Please review the simple improvement:
>> http://cr.openjdk.java.net/~shade/8015272/webrev.00/
>>
>> I have copy-pasted the same block from the non- at Contended handling, because
>> it is generic for both cases. The change is also on the path which is
>> excercized only with @Contended with the same tag. Both @Contended
>> regression tests cover this case, as well as j.l.Thread containing
>> @Contended over the TLR state implicitly tests it in every VM run.
>>
>> Testing:
>> - Linux x86_64 fastdebug/release: all hotspot/test/runtime jtreg
>> - JPRT full cycle against hotspot-rt
>> - vm.quick (running)
>>
>> -Aleksey.
>>
>
More information about the hotspot-runtime-dev
mailing list