Add j.l.Thread.setCurrentThread() inline & fast jni natives - #15428
Conversation
|
@amicic @dmitripivkine This is the first time we've modified the threadObject field of J9VMThread after initial creation. Are there any barriers required? |
|
Please add a comment describing why the liveness check can be omitted. I also want to hear back from the GC guys before merging - we're changing a thread root field from another thread, so there may be barrier implications. |
Missed the initial request for the comment, added it. |
|
I don't think we need a barrier on (strong) roots mutation, even if they are done by another thread. For some concurrent GCs, based on card dirtying barrier, like Concurrent Mark in Gencon |
|
jenkins compile win jdknext |
|
|
exports.cmake need updating as well. |
Signed-off-by: Jason Feng <fengj@ca.ibm.com>
Fixed, please re-launch the PR build. |
|
jenkins compile win,xlinux jdknext |
|
jenkins compile zlinux jdk8 |
Add
j.l.Thread.setCurrentThread()inline & fast jni natives.related #15176
FYI @tajila
Signed-off-by: Jason Feng fengj@ca.ibm.com