8265919: RunThese30M fails "assert((!(((((JfrTraceIdBits::load(value)) & ((1 << 4) << 8)) != 0))))) failed: invariant"

Markus Grönlund · Markus Grönlund · commit ffa34ed42907 · 2021-06-25T09:41:23.000Z
Reviewed-by: egahlin
diff --git a/src/hotspot/share/jfr/recorder/checkpoint/types/jfrTypeSet.cpp b/src/hotspot/share/jfr/recorder/checkpoint/types/jfrTypeSet.cpp
@@ -428,8 +428,6 @@ static void do_previous_epoch_artifact(JfrArtifactClosure* callback, T* value) {
   assert(value != NULL, "invariant");
   if (USED_PREVIOUS_EPOCH(value)) {
     callback->do_artifact(value);
-    assert(IS_NOT_SERIALIZED(value), "invariant");
-    return;
   }
   if (IS_SERIALIZED(value)) {
     CLEAR_SERIALIZED(value);

Original file line number	Diff line number	Diff line change
`@@ -428,8 +428,6 @@ static void do_previous_epoch_artifact(JfrArtifactClosure* callback, T* value) {`
`428`	`428`	`assert(value != NULL, "invariant");`
`429`	`429`	`if (USED_PREVIOUS_EPOCH(value)) {`
`430`	`430`	`callback->do_artifact(value);`
`431`		`- assert(IS_NOT_SERIALIZED(value), "invariant");`
`432`		`- return;`
`433`	`431`	`}`
`434`	`432`	`if (IS_SERIALIZED(value)) {`
`435`	`433`	`CLEAR_SERIALIZED(value);`