JDK-6983747 : Remove obsolete dl_mutex lock
  • Type: Enhancement
  • Component: hotspot
  • Sub-Component: runtime
  • Affected Version: 7,8,9
  • Priority: P3
  • Status: Resolved
  • Resolution: Fixed
  • OS: generic
  • CPU: generic
  • Submitted: 2010-09-10
  • Updated: 2015-06-03
  • Resolved: 2015-04-02
The Version table provides details related to the release that this issue/RFE will be addressed.

Unresolved : Release in which this issue/RFE will be addressed.
Resolved: Release in which this issue/RFE has been resolved.
Fixed : Release in which this issue/RFE has been fixed. The release containing this fix may be available for download as an Early Access Release or a General Availability Release.

To download the current JDK release, click here.
JDK 9
9 b64Fixed
Related Reports
Relates :  
JDK-6721093 - -XX:AppendRatio=N not supported
Relates :  
JDK-6626677 - Error: Unimplemented()/HPI sysMonitorExit is broken on linux
Description
The fix for 6721093 copied functionality from the old HPI interface back into the VM. A few builds before this was implemented (b30) we finally closed out the long-lingering 6626677 (and related bugs) by deleting the obsolete definitions that caused locking to try to be used around the dl_* functions. 6721093 took the now obsolete commentary from HPI that locking is needed and introduced a new dl_mutex lock. This is completely unnecessary as per the eval for 6626677.
Comments
Ah, that's why we didn't find it! Thanks :)
11-03-2014
Fixed word-swap in summary and description. Should refer to "dl_mutex lock" not "dl_lock mutex". Hence a grep would indeed have not found dl_lock. Apologies for that.
11-03-2014
This has not gone: bussund0416 /scratch/dh198349/jdk9-hs-rt/hotspot > grepsrc.sh dl_mutex ./src/os/aix/vm/os_aix.cpp:static pthread_mutex_t dl_mutex; // Used to protect dlsym() calls */ ./src/os/aix/vm/os_aix.cpp: pthread_mutex_lock(&dl_mutex); ./src/os/aix/vm/os_aix.cpp: pthread_mutex_unlock(&dl_mutex); ./src/os/aix/vm/os_aix.cpp: pthread_mutex_init(&dl_mutex, NULL); ./src/os/linux/vm/os_linux.cpp:static pthread_mutex_t dl_mutex; ./src/os/linux/vm/os_linux.cpp: pthread_mutex_lock(&dl_mutex); ./src/os/linux/vm/os_linux.cpp: pthread_mutex_unlock(&dl_mutex); ./src/os/linux/vm/os_linux.cpp: pthread_mutex_init(&dl_mutex, NULL);
11-03-2014
This is already gone, closing as not an issue
10-03-2014
EVALUATION See description
10-09-2010