Integrated: 8367721: Test compiler/arguments/TestCompileTaskTimeout.java crashed: SIGSEGV

[Manuel Hässig]

On Wed, 17 Sep 2025 06:57:29 GMT, Manuel Hässig <mhaessig at openjdk.org> wrote:

> The test `TestCompileTaskTimeout.java` runs `java -Xcomp -XX:CompileTaskTimeout=1 --version` to demonstrate that the timeout works. Part of the timeout working involves it printing the method of the compile task. Inspecting the core file of the execution that failed with a `SIGSEGV` in the compile task timeout signal handler, the backtrace looks as follows:
> 
> #n   <called signal handler>
> #n+1 CompilerThreadTimeoutLinux::signal_handler()
> #n+2 <called signal handler>
> #n+3 timer_settime()
> #n+4 CompilerThreadTimeoutLinux::disarm()
> #n+5 CompileTaskWrapper::~CompileTaskWrapper()
> 
> So, the compile task hit the timeout during destruction of the underlying `CompileTaskWrapper`. Since the timeout was disarmed only after setting the task to null in the destructor, the signal handler segfaulted when trying to access the method of the compile task to print it out. This PR addresses this issue by moving up the disarmament of the timeout to the top of the destructor.
> 
> Because this issue can only be triggered with bad --- or good, depending on your view --- luck on timing, I could not devise a regression test. But this is not too big of an issue, since the CI already caught this issue.
> 
> Testing:
>  - [x] Github Actions
>  - [x] tier1,tier2,tier3 plus stress testing on Oracle supported platforms

This pull request has now been integrated.

Changeset: 04dcaa34
Author:    Manuel Hässig <mhaessig at openjdk.org>
URL:       https://git.openjdk.org/jdk/commit/04dcaa3412d07c407aed604874095acaf81d7309
Stats:     5 lines in 1 file changed: 4 ins; 1 del; 0 mod

8367721: Test compiler/arguments/TestCompileTaskTimeout.java crashed: SIGSEGV

Reviewed-by: mchevalier, chagedorn

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

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