[jdk17u-dev] RFR: 8294399: (ch) Refactor some methods out of sun.nio.ch.UnixFileDispatcherImpl [v2]

[Shruthi Acharya]

On Sun, 20 Jul 2025 07:44:21 GMT, Goetz Lindenmaier <goetz at openjdk.org> wrote:

>> Shruthi Acharya 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 backport-d667895
>>  - Backport d6678952a6de4e5435dab65e7029021832454857
>>    
>>    Signed-off-by: Shruthi <Shruthi.Shruthi1 at ibm.com>
>
> Also, if you want to backport https://bugs.openjdk.org/browse/JDK-8317801, please first backport it to 25, then 21.
> Please read https://wiki.openjdk.org/display/JDKUpdates/How+to+contribute+or+backport+a+fix , it e.g. advises you to first backport to higher releases. Thanks.

@GoeLin Can you please let me know if we can port this backport PR with the below justification?
>I’m backporting 8294399 because it introduces the UnixDispatcher class, which is a necessary foundation for other functional fixes, including the one in https://github.com/openjdk/jdk/pull/25817. Although it’s a refactoring change, it helps separate platform-specific logic into a dedicated class, making the code cleaner and easier to maintain. More importantly, the fix that address real issues (async close handling and race conditions) are built on top of this refactoring. Without backporting 8294399, those fixes cannot be cleanly or safely applied to JDK 17.

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

PR Comment: https://git.openjdk.org/jdk17u-dev/pull/3631#issuecomment-3189579351