--- old/src/share/vm/gc/parallel/gcTaskThread.cpp 2016-09-19 12:03:42.438289062 +0200 +++ new/src/share/vm/gc/parallel/gcTaskThread.cpp 2016-09-19 12:03:42.322289057 +0200 @@ -56,9 +56,10 @@ GCTaskTimeStamp* GCTaskThread::time_stamp_at(uint index) { guarantee(index < GCTaskTimeStampEntries, "increase GCTaskTimeStampEntries"); - if (_time_stamps == NULL) { + GCTaskTimeStamp* time_stamps = (GCTaskTimeStamp*)OrderAccess::load_ptr_acquire(&_time_stamps); + if (time_stamps == NULL) { // We allocate the _time_stamps array lazily since logging can be enabled dynamically - GCTaskTimeStamp* time_stamps = NEW_C_HEAP_ARRAY(GCTaskTimeStamp, GCTaskTimeStampEntries, mtGC); + time_stamps = NEW_C_HEAP_ARRAY(GCTaskTimeStamp, GCTaskTimeStampEntries, mtGC); void* old = Atomic::cmpxchg_ptr(time_stamps, &_time_stamps, NULL); if (old != NULL) { // Someone already setup the time stamps @@ -66,7 +67,7 @@ } } - return &(_time_stamps[index]); + return &(time_stamps[index]); } void GCTaskThread::print_task_time_stamps() { @@ -74,7 +75,8 @@ // Since _time_stamps is now lazily allocated we need to check that it // has in fact been allocated when calling this function. - if (_time_stamps != NULL) { + GCTaskTimeStamp* time_stamps = (GCTaskTimeStamp*)OrderAccess::load_ptr_acquire(&_time_stamps); + if (time_stamps != NULL) { log_debug(gc, task, time)("GC-Thread %u entries: %d", id(), _time_stamp_index); for(uint i=0; i<_time_stamp_index; i++) { GCTaskTimeStamp* time_stamp = time_stamp_at(i);