On Sat, 24 May 2025 10:15:44 GMT, Thomas Stuefe <stu...@openjdk.org> wrote:

>> See bug description. We now unblock all signals in both FORK and POSIX_SPAWN 
>> launch modes. This fixes the problem and makes the behavior consistent 
>> across these two modes.
>> 
>> I left the VFORK mode alone since I believe the signal state is still shared 
>> with the parent. One more reason to get rid of this mode soon (it's 
>> deprecated with 25 and will be removed with 26)
>> 
>> The fix just moves the unblocking code from the jspawnhelper over to the 
>> child process function that is called for all three modes.
>> 
>> -----
>> 
>> Testing: I did various manual tests. I also tested that 
>> test/jdk/java/lang/ProcessBuilder/UnblockSignals.java succeeds with both 
>> POSIX_SPAWN and FORK mode now and that the behavior is unchanged (still 
>> broken) for VFORK mode.
>> 
>> ping @RogerRiggs I would like to get this still into JDK 25; the fix is 
>> small and clean enough to not worry me.
>
> Thomas Stuefe has updated the pull request with a new target base due to a 
> merge or a rebase. The incremental webrev excludes the unrelated changes 
> brought in by the merge/rebase. The pull request contains two additional 
> commits since the last revision:
> 
>  - Merge branch 'openjdk:master' into 
> JDK-8357683-process-SIGQUIT-still-blocked-after-JDK-8234262-with-jdk-lang-Process-launchMechanism=FORK-or-VFORK
>  - unblock signals in fork mode

Looks fine; will run some tests in the CI before approving.

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

PR Comment: https://git.openjdk.org/jdk/pull/25428#issuecomment-2919556500

Reply via email to