> This PR uses a shared per session family depth limit + per session depth > value to enforce invalidation. This will remove the container's > (ConcurrentLinkedQueue) strong references to the child sessions, which allows > child sessions to be garbage collected when they are no longer reachable. The > drawback is that the invalidation depth limit will be per family, so once a > session is invalidated at the specified depth, all deeper sessions will be > invalidated, not just its descendants. > > Ran related tests on linux-x64, linux-aarch64, macos-aarch64 and windows-x64: > > make test TEST=test/jdk/sun/security/ssl > make test TEST=test/jdk/javax/net/ssl > > Results: > > [windows-x64-specific-test.log](https://github.com/user-attachments/files/33689537/windows-x64-specific-test.log) > [windows-x64-specific-2-test.log](https://github.com/user-attachments/files/33689538/windows-x64-specific-2-test.log) > [macos-aarch64-specific-test.log](https://github.com/user-attachments/files/33689539/macos-aarch64-specific-test.log) > [macos-aarch64-specific-2-test.log](https://github.com/user-attachments/files/33689540/macos-aarch64-specific-2-test.log) > [linux-x64-specific-test.log](https://github.com/user-attachments/files/33689541/linux-x64-specific-test.log) > [linux-x64-specific-2-test.log](https://github.com/user-attachments/files/33689542/linux-x64-specific-2-test.log) > [linux-aarch64-specific-test.log](https://github.com/user-attachments/files/33689543/linux-aarch64-specific-test.log) > [linux-aarch64-specific-2-test.log](https://github.com/user-attachments/files/33689544/linux-aarch64-specific-2-test.log) > > Low risk - the change is limited to SSLSessionImpl, its cache lookup and the > ticket code. The worst case is an occasional extra full handshake. > > --------- > - [x] I confirm that I make this contribution in accordance with the [OpenJDK > Interim AI Policy](https://openjdk.org/legal/ai).
Roland Mesde has updated the pull request incrementally with one additional commit since the last revision: Add Comment on Constructor + Family Behavior and Renamed isInvalid() to isInvalidated() ------------- Changes: - all: https://git.openjdk.org/jdk/pull/33447/files - new: https://git.openjdk.org/jdk/pull/33447/files/d70bb669..6be0bce0 Webrevs: - full: https://webrevs.openjdk.org/?repo=jdk&pr=33447&range=04 - incr: https://webrevs.openjdk.org/?repo=jdk&pr=33447&range=03-04 Stats: 7 lines in 2 files changed: 2 ins; 0 del; 5 mod Patch: https://git.openjdk.org/jdk/pull/33447.diff Fetch: git fetch https://git.openjdk.org/jdk.git pull/33447/head:pull/33447 PR: https://git.openjdk.org/jdk/pull/33447
