# HG changeset patch # User mgronlun # Date 1568638631 -7200 # Node ID 38b5442bcab453f7b3f87dbec2840b581264c514 # Parent 105d32b27809cc93cc695cb6d1867f09741224fe some stuff diff -r 105d32b27809 -r 38b5442bcab4 src/hotspot/share/jfr/recorder/service/jfrRecorderService.cpp --- a/src/hotspot/share/jfr/recorder/service/jfrRecorderService.cpp Mon Sep 16 13:50:20 2019 +0200 +++ b/src/hotspot/share/jfr/recorder/service/jfrRecorderService.cpp Mon Sep 16 14:57:11 2019 +0200 @@ -237,14 +237,12 @@ cw.reserve(sizeof(u4)); cw.write(EVENT_CHECKPOINT); cw.write(JfrTicks::now()); - cw.write((int64_t)0); // duration + cw.write(0); // duration cw.write(delta_to_last_checkpoint); - cw.write(false); // flushpoint - cw.write((u4)1); // nof types in this checkpoint + cw.write(GENERIC); // checkpoint type + cw.write(1); // nof types in this checkpoint cw.write(type_id); - const int64_t number_of_elements_offset = cw.current_offset(); - cw.reserve(sizeof(u4)); - return number_of_elements_offset; + return cw.reserve(sizeof(u4)); } template diff -r 105d32b27809 -r 38b5442bcab4 src/hotspot/share/jfr/recorder/storage/jfrStorage.cpp --- a/src/hotspot/share/jfr/recorder/storage/jfrStorage.cpp Mon Sep 16 13:50:20 2019 +0200 +++ b/src/hotspot/share/jfr/recorder/storage/jfrStorage.cpp Mon Sep 16 14:57:11 2019 +0200 @@ -314,7 +314,7 @@ assert(buffer != NULL, "invariant"); assert(buffer->retired(), "invariant"); const size_t unflushed_size = buffer->unflushed_size(); - buffer->reinitialize(); + buffer->concurrent_reinitialization(); log_registration_failure(unflushed_size); } diff -r 105d32b27809 -r 38b5442bcab4 src/hotspot/share/jfr/recorder/storage/jfrStorageUtils.inline.hpp --- a/src/hotspot/share/jfr/recorder/storage/jfrStorageUtils.inline.hpp Mon Sep 16 13:50:20 2019 +0200 +++ b/src/hotspot/share/jfr/recorder/storage/jfrStorageUtils.inline.hpp Mon Sep 16 14:57:11 2019 +0200 @@ -47,7 +47,7 @@ inline bool ConcurrentWriteOp::process(typename Operation::Type* t) { const u1* const current_top = t->concurrent_top(); const size_t unflushed_size = t->pos() - current_top; - if (unflushed_size == 0 || t->excluded()) { + if (unflushed_size == 0) { t->set_concurrent_top(current_top); return true; } @@ -58,8 +58,7 @@ template inline bool ConcurrentWriteOpExcludeRetired::process(typename Operation::Type* t) { - if (t->retired()) { - assert(t->empty(), "invariant"); + if (t->retired() || t->excluded()) { return true; } return ConcurrentWriteOp::process(t); @@ -70,7 +69,7 @@ assert(t != NULL, "invariant"); const u1* const current_top = t->top(); const size_t unflushed_size = t->pos() - current_top; - if (unflushed_size == 0 || t->excluded()) { + if (unflushed_size == 0) { return true; } const bool result = _operation.write(t, current_top, unflushed_size);