Skip to content

Commit aab3fc5

Browse files
author
David Holmes
committed
8370207: Test sun/misc/SunMiscSignalTest.java crashes after JDK-8369631
Reviewed-by: kbarrett, coleenp
1 parent 43e036b commit aab3fc5

File tree

2 files changed

+4
-6
lines changed

2 files changed

+4
-6
lines changed

src/hotspot/os/posix/signals_posix.cpp

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -357,12 +357,12 @@ static void jdk_misc_signal_init() {
357357
}
358358

359359
void os::signal_notify(int sig) {
360-
// Signal thread is not created with ReduceSignalUsage and jdk_misc_signal_init
361-
// initialization isn't called. This code is also never called.
362-
assert(!ReduceSignalUsage, "Should not reach here if ReduceSignalUsage is set");
363-
360+
// Signal thread is not created with ReduceSignalUsage and jdk_misc_signal_init
361+
// initialization isn't called.
362+
if (!ReduceSignalUsage) {
364363
AtomicAccess::inc(&pending_signals[sig]);
365364
sig_semaphore->signal();
365+
}
366366
}
367367

368368
static int check_pending_signals() {

test/jdk/ProblemList.txt

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -743,8 +743,6 @@ sun/tools/jstatd/TestJstatdRmiPort.java 8251259,8293577
743743
jdk/incubator/vector/ShortMaxVectorTests.java 8306592 generic-i586
744744
jdk/incubator/vector/LoadJsvmlTest.java 8305390 windows-x64
745745

746-
sun/misc/SunMiscSignalTest.java 8370207 generic-all
747-
748746
############################################################################
749747

750748
# jdk_jfr

0 commit comments

Comments
 (0)