RFR (M) #5 CR 8003985: Support @Contended annotation

Jesper Wilhelmsson jesper.wilhelmsson at oracle.com
Thu Jan 10 15:15:22 PST 2013


Aleksey,

I have looked through the change and I don't see anything obviously 
wrong with it, but this change is in a part of HotSpot that I don't know 
at all so I wouldn't count this as a review.

You are adding a TODO in classFileParser.cpp , is that intentional?

I have followed the discussions around @contended over the last year and 
I feel comfortable sponsoring this change once it has been approved by 
proper reviewers.
/Jesper

On 10/1/13 4:16 PM, Aleksey Shipilev wrote:
> Hi,
>
> This is the final call for reviews for @Contended:
>    http://cr.openjdk.java.net/~shade/8003985/webrev.01/
>
> Changes:
>   - VM: none since latest revision.
>   - JDK: pruned some Javadocs as requested by CCC reviewers
>
> Testing:
>   - (respin to make sure everything still intact)
>   - Test8003985 passes on Linux x86_64
>   - JPRT builds passed
>   - JPRT testing passed
>
> Status:
>   - This change is ready to be pushed
>   - JEP-142 is in Funded state (http://openjdk.java.net/jeps/142)
>   - CCC is in place, and Approved
>
> I would need a sponsor to push this.
>
> Thanks,
> Aleksey.



More information about the hotspot-dev mailing list