archive/jdk
1
0
Fork 0
mirror of https://github.com/openjdk/jdk.git synced 2025-08-25 22:04:51 +02:00
Code Issues Projects Releases Packages Wiki Activity Actions

8044339: Update FilterSpuriousWakeups documentation. Review "Solaris only" vm options descriptions

Browse source 
Reviewed-by: kvn, dholmes, iignatyev
This commit is contained in:
Igor Ignatyev 2014-06-04 00:25:21 +04:00
parent e09b9cbc3c
commit f19a510532
4 changed files with 10 additions and 16 deletions
Download patch file Download diff file Expand all files Collapse all files

6
hotspot/src/os/linux/vm/os_linux.cpp
View file

@ -5538,10 +5538,8 @@ int os::PlatformEvent::park(jlong millis) {
//
// Thread.interrupt and object.notify{All} both call Event::set.
// That is, we treat thread.interrupt as a special case of notification.
// The underlying Solaris implementation, cond_timedwait, admits
// spurious/premature wakeups, but the JLS/JVM spec prevents the
// JVM from making those visible to Java code. As such, we must
// filter out spurious wakeups. We assume all ETIME returns are valid.
// We ignore spurious OS wakeups unless FilterSpuriousWakeups is false.
// We assume all ETIME returns are valid.
//
// TODO: properly differentiate simultaneous notify+interrupt.
// In that case, we should propagate the notify to another waiter.