RFR: 8316397: StackTrace/Suspended/GetStackTraceSuspendedStressTest.java failed with: SingleStep event is NOT expected [v2]

[Serguei Spitsyn]

On Wed, 19 Feb 2025 19:19:25 GMT, Patricio Chilano Mateo <pchilanomate at openjdk.org> wrote:

> For this particular event I don’t think the crash can happen with platform threads because get_jvmti_thread_state() will only block for virtual threads. And I don’t see any other blocking points in InterpreterRuntime::at_safepoint() other than that one.

Thank you for the comment.
I agree, the `get_jvmti_thread_state()` will only block for virtual threads.

> But for other events it can happen since the thread might have been blocked and suspended at some point before (looking at post_class_prepare/post_class_load for example). This leads to another question, should we add this suspend check for all JvmtiExport::post_xxx methods?

Right. I wanted to address this separately as you mentioned before that the same issue should exist for other event types.
However, I'll move this check to the `get_jvmti_thread_state()` function. It should work for all events then.

-------------

PR Comment: https://git.openjdk.org/jdk/pull/23490#issuecomment-2672786201