string pool synch JEP-349-branch
authormgronlun
Sun, 15 Sep 2019 22:54:35 +0200
branchJEP-349-branch
changeset 58160 c17888eb0d03
parent 58159 892527a70da9
child 58161 c4ee448c9806
string pool synch
src/hotspot/share/jfr/recorder/stringpool/jfrStringPool.cpp
src/hotspot/share/jfr/utilities/jfrTypes.hpp
--- a/src/hotspot/share/jfr/recorder/stringpool/jfrStringPool.cpp	Sun Sep 15 20:23:40 2019 +0200
+++ b/src/hotspot/share/jfr/recorder/stringpool/jfrStringPool.cpp	Sun Sep 15 22:54:35 2019 +0200
@@ -44,22 +44,32 @@
 static uint64_t store_generation = 0;
 static uint64_t serialized_generation = 0;
 
-inline void set_value(uint64_t value, uint64_t* const dest) {
+inline void set_generation(uint64_t value, uint64_t* const dest) {
   assert(dest != NULL, "invariant");
-  const uint64_t current = OrderAccess::load_acquire(dest);
-  if (value != current) {
-    OrderAccess::release_store(dest, value);
+  OrderAccess::release_store(dest, value);
+}
+static void increment_store_generation() {
+  const uint64_t current_serialized = OrderAccess::load_acquire(&serialized_generation);
+  const uint64_t current_stored = OrderAccess::load_acquire(&store_generation);
+  if (current_serialized == current_stored) {
+    set_generation(current_serialized + 1, &store_generation);
   }
 }
-static void inc_store_generation() {
-  set_value(OrderAccess::load_acquire(&serialized_generation) + 1, &store_generation);
+
+static bool increment_serialized_generation() {
+  const uint64_t current_stored = OrderAccess::load_acquire(&store_generation);
+  const uint64_t current_serialized = OrderAccess::load_acquire(&serialized_generation);
+  if (current_stored != current_serialized) {
+    set_generation(current_stored, &serialized_generation);
+    return true;
+  }
+  return false;
 }
-static void set_serialized_generation() {
-  set_value(OrderAccess::load_acquire(&store_generation), &serialized_generation);
+
+bool JfrStringPool::is_modified() {
+  return increment_serialized_generation();
 }
-bool JfrStringPool::is_modified() {
-  return serialized_generation != OrderAccess::load_acquire(&store_generation);
-}
+
 JfrStringPool& JfrStringPool::instance() {
   return *_instance;
 }
@@ -150,13 +160,16 @@
 bool JfrStringPool::add(bool epoch, jlong id, jstring string, JavaThread* jt) {
   assert(jt != NULL, "invariant");
   const bool current_epoch = JfrTraceIdEpoch::epoch();
-  if (current_epoch == epoch) {
+  if (current_epoch != epoch) {
+    return current_epoch;
+  }
+  {
     JfrStringPoolWriter writer(jt);
     writer.write(id);
     writer.write(string);
     writer.inc_nof_strings();
-    inc_store_generation();
   }
+  increment_store_generation();
   return current_epoch;
 }
 
@@ -201,7 +214,6 @@
 typedef CompositeOperation<ExclusiveDiscardOperation, StringPoolReleaseOperation> StringPoolDiscardOperation;
 
 size_t JfrStringPool::write() {
-  set_serialized_generation();
   Thread* const thread = Thread::current();
   WriteOperation wo(_chunkwriter, thread);
   ExclusiveWriteOperation ewo(wo);
@@ -218,7 +230,7 @@
 }
 
 size_t JfrStringPool::clear() {
-  set_serialized_generation();
+  increment_serialized_generation();
   DiscardOperation discard_operation;
   ExclusiveDiscardOperation edo(discard_operation);
   StringPoolReleaseOperation spro(_free_list_mspace, Thread::current(), false);
--- a/src/hotspot/share/jfr/utilities/jfrTypes.hpp	Sun Sep 15 20:23:40 2019 +0200
+++ b/src/hotspot/share/jfr/utilities/jfrTypes.hpp	Sun Sep 15 22:54:35 2019 +0200
@@ -26,6 +26,7 @@
 #define SHARE_JFR_UTILITIES_JFRTYPES_HPP
 
 #include "jfrfiles/jfrEventIds.hpp"
+#include "utilities/globalDefinitions.hpp"
 
 typedef u8 traceid;
 typedef int fio_fd;