On 2018-06-18 16:23, Peter Levart wrote:
Hi Claes,
On 06/18/2018 03:54 PM, Claes Redestad wrote:
I'd suggest something simple like this to ensure correctness, while
keeping the number of volatile reads to a minimum:
http://cr.openjdk.java.net/~redestad/8199435.00/
Then file a follow-up RFE to investigate if we can make these fields
non-volatile, e.g. using careful fencing as suggested
by Peter.
OK, but the constructor will still need a releaseFence at the end to
prevent a potential write that unsafely publishes Properties instance
to float before the volatile writes of 'map' and 'defaults'...
You might want to use Unsafe directly for that since VarHandle could
cause bootstrap issues.
I don't follow.. which constructor needs a fence in the suggested patch?
/Claes
