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