Integrated: 8248609: [Graal] vmTestbase/nsk/jdi/VoidValue/toString/tostring001/TestDescription.java failed with Unexpected com.sun.jdi.ObjectCollectedException

[Chris Plummer]

On Thu, 18 Jul 2024 18:50:48 GMT, Chris Plummer <cjplummer at openjdk.org> wrote:

> The test is failing with an ObjectCollectedException.  The test hits a SUSPEND_ALL breakpoint. It then uses JDI to allocate an Object on the debuggee side:
> 
>             testedObject = testedClass.newInstance(thread, ctor, params, 0);
> 
> Since we are under a SUSPEND_ALL, the object is not initially at risk of getting GC'd. However, the test then calls invokeMethod() in a loop:
> 
>                 if (method.isStatic()) {
>                     voidValue = (VoidValue) testedClass.invokeMethod(thread, method, params, 0);
>                 } else {
>                     voidValue = (VoidValue) testedObject.invokeMethod(thread, method, params, 0);
>                 }
> 
> On the first iteration of the loop, invokeMethod() will do a resumeAll() so it can execute the method on the specified thread. During this time a GC can happen, and that GC is likely to collect the object that testedObject is mirroring since it is only weakly kept alive. Then on a subsequent iteration of the loop, testedObject.invokeMethod() is called again, but this time you get the ObjectCollectedException because the object that testedObject is mirroring has been collected. The test needs to add a call to testedObject.disableCollection() to prevent it from being collected.
> 
> I'm not able to reproduce this failure, but the bug is pretty clear. Testing is in progress. I'll run tier1 CI and also tier2 and tier5 test tasks that run this test.

This pull request has now been integrated.

Changeset: e7e48a78
Author:    Chris Plummer <cjplummer at openjdk.org>
URL:       https://git.openjdk.org/jdk/commit/e7e48a780a34007994f830869fdb74ba1cb5b3fe
Stats:     5 lines in 1 file changed: 5 ins; 0 del; 0 mod

8248609: [Graal] vmTestbase/nsk/jdi/VoidValue/toString/tostring001/TestDescription.java failed with Unexpected com.sun.jdi.ObjectCollectedException

Reviewed-by: amenkov, lmesnik

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

PR: https://git.openjdk.org/jdk/pull/20242