Looks good!

Thanks,
David

On 31/07/2015 8:56 AM, Kim Barrett wrote:
On Jul 30, 2015, at 5:33 PM, David Holmes <david.hol...@oracle.com> wrote:

So I think I can either:

1. Go ahead with my change + Peter's change.

2. Give this back to core-libs while I step carefully away :-)

I *think* option (1) is at least an improvement.  But I completely
missed Peter's race, despite having specifically looked for problems
there, so take my opinion with an appropriate quantity of salt.

I vote for 1 as well. I think we have now given good coverage to the two 
sources of problems (the two lock-free regions):
- reference queue state can be seen while queue state is in transition
- queue empty state can be seen while reference state is in transition

New webrev, with both changes:

http://cr.openjdk.java.net/~kbarrett/8132306/webrev.01/

Reply via email to