Java - is volatile required with synchronized?

Question

In the following simple scenario:

class A {
  int x;
  Object lock;

  ...

  public void method(){
    synchronized(lock){
      // modify/read x and act up         


        

Answer 1

No it does not, synchronised already has a memory barrier inserted after it, so all Threads will see the update that the current thread performs, taking into account that the other threads will synchronise on the same lock.

Volatile, just like synchronised has memory barriers that are attached to it - depending on the cpu it is store/load/full barrier that ensures that an update from one thread is visible to the other(s). I assume this is performed with cpu cache invalidation.

EDIT From what I've just read, the store buffers are flushed to the CPU cache, and this is how the visibility is achieved.

Answer 2

Simplified answer: If thread A updates a field and then releases a lock, then thread B will be guaranteed to see the update after thread B has acquired the same lock.

Note, "release a lock" means exit a synchronized block, and "acquire the same lock" means synchronize on the same object.