On 11/4/2013 6:29 PM, David Holmes wrote:
2. In VM.java. booted need not be volatile now that it is only
accessed within a locked region.

Oops my bad! The accessor wasn't synchronized but now is. Your call whether to leave as is or revert to previous.

I'm fine with making isBooted() to a synchronized method since I expect the contention on this method is very low and that makes access of the booted field using the consistent synchronization.

Mandy

Reply via email to