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