Integrated: 8307824: Clean up Finalizable.java and finalize terminology in vmTestbase/nsk/share

Leonid Mesnik lmesnik at openjdk.org
Tue Jun 4 19:21:06 UTC 2024


On Fri, 31 May 2024 18:22:47 GMT, Leonid Mesnik <lmesnik at openjdk.org> wrote:

> The fix removes finalization cleanup from vmTestbase.
> The last to classes that use it are: DebugeeBinder and SocketIOPipe.
> The DebugeeBinder is used in jdi and jdwp tests and is always linked with debuggee process. So the DebugeeProcess.waitFor() is the good place to close binder and free all it's resources.
> The SocketIOPipe is used directly in AOD tests where it should be closed after test execution.
> 
> The OPipe (child of SocketIOPipe) also used in jdi and jdwp tests where it is connected directly in tests. However is also connected with debuggee and could be closed in  DebugeeProcess.waitFor().
> 
> The VMOutOfMemoryException001 test is fixed to release some memory after throwing OOME so Sytem.exit() could complete successfully. Previously some memory freed during VM shutdown hook. 
> 
> I verified that cleanup printed that corresponding 'close' method has been already called before VM shutdown phase for debugger process. 
> Additionally, run all vmTestbase tests to verify there are no failures,

This pull request has now been integrated.

Changeset: 244f6ac2
Author:    Leonid Mesnik <lmesnik at openjdk.org>
URL:       https://git.openjdk.org/jdk/commit/244f6ac222fa98fba4fb99bf5bccd36e3e6c5de1
Stats:     314 lines in 10 files changed: 24 ins; 271 del; 19 mod

8307824: Clean up Finalizable.java and finalize terminology in vmTestbase/nsk/share

Reviewed-by: sspitsyn, cjplummer

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

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


More information about the serviceability-dev mailing list