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