[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