stan/jdk-24
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

6888847: TaskQueue needs release_store() for correctness on RMO machines

Browse Source 
See title.

Reviewed-by: jmasa, ysr, jcoomes, iveresov, tonyp
This commit is contained in:
Bob Vandette 2009-10-07 09:48:42 -04:00
parent 9288a18639
commit 9681f68f38
1 changed files with 2 additions and 2 deletions

4
hotspot/src/share/vm/utilities/taskqueue.hpp

@ -207,7 +207,7 @@ bool GenericTaskQueue<E>::push_slow(E t, uint dirty_n_elems) {
// Actually means 0, so do the push.
uint localBot = _bottom;
_elems[localBot] = t;
_bottom = increment_index(localBot);
OrderAccess::release_store(&_bottom, increment_index(localBot));
return true;
}
return false;
@ -485,7 +485,7 @@ template<class E> inline bool GenericTaskQueue<E>::push(E t) {
assert((dirty_n_elems >= 0) && (dirty_n_elems < N), "n_elems out of range.");
if (dirty_n_elems < max_elems()) {
_elems[localBot] = t;
_bottom = increment_index(localBot);
OrderAccess::release_store(&_bottom, increment_index(localBot));
return true;
} else {
return push_slow(t, dirty_n_elems);