New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
8275950: Use only _thread_in_vm in ~ThreadBlockInVMPreprocess() #6120
Changes from 2 commits
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -706,7 +706,6 @@ void SafepointSynchronize::block(JavaThread *thread) { | |
} | ||
|
||
JavaThreadState state = thread->thread_state(); | ||
assert(is_a_block_safe_state(state), "Illegal threadstate encountered: %d", state); | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Getting rid of this assert makes me a bit nervous, but I have to There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. I thought about keeping it but not only we have a single caller(SafepointMechanism::process()) which already has that assert as a guarantee, but also there is no way to add new callers without basically copying all the logic from SafepointMechanism::process() since we cannot just call SS::block(). |
||
thread->frame_anchor()->make_walkable(thread); | ||
|
||
uint64_t safepoint_id = SafepointSynchronize::safepoint_counter(); | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The comment on L231 is now wrong since you're no longer using
a "transition" state. Perhaps, "Change to an unsafe state..."
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fixed.