RFR: 8282475: SafeFetch should not rely on existence of Thread::current
    Johannes Bechberger 
    duke at openjdk.java.net
       
    Mon Mar  7 11:36:21 UTC 2022
    
    
  
The WXMode for the current thread (on MacOS aarch64) is currently stored in the thread class which is unnecessary as the WXMode is bound to the current OS thread, not the current instance of the thread class.
This pull request moves the storage of the current WXMode into a thread local global variable in `os` and changes all related code. SafeFetch depended on the existence of a thread object only because of the WXMode. This pull request therefore removes the dependency, making SafeFetch usable in more contexts.
-------------
Commit messages:
 - Fix include for threadWXSetters.inline.hpp
 - Remove thread parameter from ThreadWXEnable
 - Remove thread parameter from os methods
 - Remove wx_init and current thread assert in safefetch
 - Use os::current_thread_change_wx instead of thread methods
Changes: https://git.openjdk.java.net/jdk/pull/7727/files
 Webrev: https://webrevs.openjdk.java.net/?repo=jdk&pr=7727&range=00
  Issue: https://bugs.openjdk.java.net/browse/JDK-8282475
  Stats: 150 lines in 31 files changed: 26 ins; 47 del; 77 mod
  Patch: https://git.openjdk.java.net/jdk/pull/7727.diff
  Fetch: git fetch https://git.openjdk.java.net/jdk pull/7727/head:pull/7727
PR: https://git.openjdk.java.net/jdk/pull/7727
    
    
More information about the hotspot-dev
mailing list