Re: do I need "volatile" for HashMap? when I apply ReentrantReadWriteLock on it.



easy wrote:
fomr javaoc of ReentrantReadWriteLock
an example:
class CachedData {
Object data;
volatile boolean cacheValid;
ReentrantReadWriteLock rwl = new ReentrantReadWriteLock();

void processCachedData() {
rwl.readLock().lock();
if (!cacheValid) {
.....
use(data);
rwl.readLock().unlock();
}
}
Using volatile here makes sense to me.

booleans used for loop control and things like that need to be volatile only if accessed from more than one thread. In your example above that isn't clear. In either case if your boolean were used in a synchronized block or as in this case a locked section of code it would not need it's own synchronization to make changes in other threads visible.

but if in my class looks like this,
class CachedData {
HashMap<Key, Obj> data; // <---- here
ReentrantReadWriteLock rwl = new ReentrantReadWriteLock();

void processCachedData(Key k) {
rwl.readLock().lock();
if (!data.containsKey(k)) { // <--- here
.....
use(data);
rwl.readLock().unlock();
}
}

should I declare as
"volatile HashMap"

No.

or for which "variable" type should I apply volatile in such
situation?

thanks.


volatile is used mainly to ensure multi-thread visibility. If you are going to change the value of the variable in another thread and want to see those changes then make the variable volatile. But as I said above if the variable is only accessed inside a synchronized block or some other synchronization code (eg ReentrantReadWriteLock) then all variables are made visible to this thread. It is a freebie side effect.

You might want to invest a few bucks in the book "JAVA Concurrency in Practice", by Brian Goetz. Very valuable when writing multi-threaded Java code.

--

Knute Johnson
email s/nospam/knute/
.



Relevant Pages

  • Re: Boolean access in multithreaded environment
    ... > I have one thread assigning true/false to boolean and another thread ... > accessing the value of the boolean. ... > need not make it volatile. ... > volatile/restrict its access by lock? ...
    (microsoft.public.pocketpc.developer)
  • do I need "volatile" for HashMap? when I apply ReentrantReadWriteLock on it.
    ... class CachedData { ... ReentrantReadWriteLock rwl = new ReentrantReadWriteLock; ... void processCachedData() { ... Using volatile here makes sense to me. ...
    (comp.lang.java.programmer)
  • Re: Boolean access in multithreaded environment
    ... > accessing the value of the boolean. ... > Someone told me that boolean assignment is atomic operation and thats why ... > need not make it volatile. ... > volatile/restrict its access by lock? ...
    (microsoft.public.pocketpc.developer)
  • Boolean access in multithreaded environment
    ... I have one thread assigning true/false to boolean and another thread ... Someone told me that boolean assignment is atomic operation and thats why I ... need not make it volatile. ...
    (microsoft.public.pocketpc.developer)