In this case, volatile was both incorrect (volatile does nothing about
thread synchronization) and not required (there is an explicit lock).