RFR: 8339725: Concurrent GC crashed due to GetMethodDeclaringClass [v13]

[Leonid Mesnik]

On Fri, 13 Sep 2024 02:33:11 GMT, Liang Mao <lmao at openjdk.org> wrote:

>> test/hotspot/jtreg/serviceability/jvmti/GetMethodDeclaringClass/TestUnloadedClass.java line 30:
>> 
>>> 28:  * @summary Stress test GetMethodDeclaringClass
>>> 29:  * @requires vm.jvmti
>>> 30:  * @requires (os.family == "linux") & (vm.debug != true)
>> 
>> Sorry, that missed initially. Why test is linux and non-debug only?
>> `* @requires (os.family == "linux") & (vm.debug != true)`
>
> The test depends on pthread and it's a stress test that fastdebug ran out of time. We don't want to trade off the crash reproduction probability to make fastdebug pass.

The fastdebug testing is very important for hotspot testing. Is it possible just to increase timeout so test pass with fastdebug? You can make 2 tests - 1 for product and 1 for fastdebug with different timeouts, if you want, but I would recommend just to increase the timeout. 
Also, please exclude test from tier1_serviceability in TEST.groups because it takes too many time for tier1.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/20907#discussion_r1758091768