[jdk22u] Integrated: 8323782: Race: Thread::interrupt vs. AbstractInterruptibleChannel.begin

[Richard Reingruber]

On Fri, 5 Apr 2024 16:34:11 GMT, Richard Reingruber <rrich at openjdk.org> wrote:

> Hi,
> 
> this PR contains a backport of commit https://github.com/openjdk/jdk/commit/4018b2b19629ddb8cd7a56e064dfef371f23e5fa.
> 
> I'd like to do the backport because jdk 22 is affected by the issue. The included test hangs without the change.
> Besides it will reduce the noise in jck tests we're doing regularly (the test api/java_net/ServerSocket/AcceptInterruptibleTests_PlatformThreads fails sometimes because of the issue).
> 
> The original fix does not apply due to renaming of a local variable. Also the context differs because there's no `postInterrupt` method to be called.
> Nevertheless I'd consider the risk low because of the small size of the fix.
> 
> Tested with the included test.
> 
> The fix passed our CI testing: JTReg tests: tier1-4 of hotspot and jdk. All of Langtools and jaxp. JCK, SPECjvm2008, SPECjbb2015, Renaissance Suite, and SAP specific tests.
> Testing was done with fastdebug builds on the main platforms and also on Linux/PPC64le.

This pull request has now been integrated.

Changeset: 1188d404
Author:    Richard Reingruber <rrich at openjdk.org>
URL:       https://git.openjdk.org/jdk22u/commit/1188d404bc4879fc388209cd32e6c4e46178e200
Stats:     103 lines in 2 files changed: 97 ins; 5 del; 1 mod

8323782: Race: Thread::interrupt vs. AbstractInterruptibleChannel.begin

Reviewed-by: phh
Backport-of: 4018b2b19629ddb8cd7a56e064dfef371f23e5fa

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

PR: https://git.openjdk.org/jdk22u/pull/130