Skip to content
This repository has been archived by the owner. It is now read-only.
Permalink
Browse files
8268350: Remove assert that ensures thread identifier remains the same
Reviewed-by: tschatzl, kbarrett
  • Loading branch information
lkorinth committed Jun 30, 2021
1 parent 90eb118 commit 6b64a7956ceb0c10d3868c67a923a8de12433aee
@@ -87,7 +87,7 @@ void ParallelArguments::initialize() {
}

if (FLAG_IS_DEFAULT(ParallelRefProcEnabled) && ParallelGCThreads > 1) {
//FLAG_SET_DEFAULT(ParallelRefProcEnabled, true);
FLAG_SET_DEFAULT(ParallelRefProcEnabled, true);
}
}

@@ -71,12 +71,7 @@ class PromotionFailedInfo : public CopyFailedInfo {

void register_copy_failure(size_t size) {
CopyFailedInfo::register_copy_failure(size);
if (_thread_trace_id == 0) {
_thread_trace_id = JFR_THREAD_ID(Thread::current());
} else {
assert(JFR_THREAD_ID(Thread::current()) == _thread_trace_id,
"The PromotionFailedInfo should be thread local.");
}
_thread_trace_id = JFR_THREAD_ID(Thread::current());
}

void reset() {
@@ -51,7 +51,7 @@ public static void main(String args[]) throws Exception {
}
if (GC.Parallel.isSupported()) {
noneGCSupported = false;
testFlag(new String[] { "-XX:+UseParallelGC" }, false);
testFlag(new String[] { "-XX:+UseParallelGC" }, true);
}
if (GC.G1.isSupported()) {
noneGCSupported = false;

1 comment on commit 6b64a79

@openjdk-notifier

This comment has been minimized.

Copy link

@openjdk-notifier openjdk-notifier bot commented on 6b64a79 Jun 30, 2021

Please sign in to comment.