[jdk17u-dev] Integrated: 8313638: Add test for dump of resolved references
Goetz Lindenmaier
goetz at openjdk.org
Sat Oct 5 18:06:38 UTC 2024
On Tue, 1 Oct 2024 08:25:20 GMT, Goetz Lindenmaier <goetz at openjdk.org> wrote:
> I had to resolve the Whitebox coding. The method added by the backported change already exists in 17.
>
> Originally, WB_GetResolvedReferences was added by "https://bugs.openjdk.org/browse/JDK-8179302: Pre-resolve constant pool string entries and cache resolved_reference arrays in CDS archive" in 10 which did not directly use it, there were closed tests depending on it. See "https://bugs.openjdk.org/browse/JDK-8188791: Move AppCDS from closed repo to open repo". It was removed in "https://bugs.openjdk.org/browse/JDK-8298048: Combine CDS archive heap into a single block" in 21 and added again by this change.
>
> I resolved the code to use the version of this new change to reduce differences to 21.
>
> The old Whitebox code is used in
> runtime/cds/appcds/cacheObject/CheckCachedResolvedReferencesApp.java
> runtime/cds/appcds/cacheObject/GCStressApp.java
> All runtime/cds/appcds/cacheObject/ are still passing.
This pull request has now been integrated.
Changeset: c1efd753
Author: Goetz Lindenmaier <goetz at openjdk.org>
URL: https://git.openjdk.org/jdk17u-dev/commit/c1efd753bb02f8ffc622f7f0d444940bd82843ef
Stats: 216 lines in 5 files changed: 202 ins; 14 del; 0 mod
8313638: Add test for dump of resolved references
Reviewed-by: mdoerr
Backport-of: 83dca6296e3fc7b9912ef7b82e443ce1415a7bcc
-------------
PR: https://git.openjdk.org/jdk17u-dev/pull/2924
More information about the jdk-updates-dev
mailing list