RFR: 8328665: serviceability/jvmti/vthread/PopFrameTest failed with a timeout [v4]

[Serguei Spitsyn]

On Wed, 27 Mar 2024 20:27:42 GMT, Daniel D. Daugherty <dcubed at openjdk.org> wrote:

>> test/hotspot/jtreg/serviceability/jvmti/vthread/PopFrameTest/libPopFrameTest.cpp line 163:
>> 
>>> 161:     LOG("Main: ensureAtBreakpoint: waitig 5 millis\n");
>>> 162:     if (++attempts > 100) {
>>> 163:       fatal(jni, "Main: ensureAtBreakpoint: waited 1 sec");
>> 
>> 1 second isn't very long when you are talking about something that is relying on debugger/debuggee communication. Maybe wait 100ms at a time for a total of 10 seconds.
>
> Caught this comment in passing. Delays like this should be scaled with
> defaultTimeoutFactor so that test tasks that invoke tests with options
> that can slow the test down, e.g., `-Xcomp`, can be accommodated.
> 
> I believe the defaultTimeoutFact for `-Xcomp` test tasks gets bumped
> from 4 to 10.

Thanks for the comments, Chris and Dan. Updated as Chris suggested. I've added this with `-Xcomp` consideration as the worst case scenario in mind. Now, I think it is more save to make it 10 seconds instead of one. Is it going to be good enough? In fact, I've added this for manual testing to save time in waiting for test completion when it is deadlocked.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/18419#discussion_r1543839536