Integrated: 8347830: [premain] UseCompatibleCompressedOops is broken after merging with mainline
Vladimir Kozlov
kvn at openjdk.org
Mon May 19 15:45:09 UTC 2025
On Fri, 16 May 2025 02:33:45 GMT, Vladimir Kozlov <kvn at openjdk.org> wrote:
> The issue was that we did not allocate space "noaccess prefix" before heap for read-only page when we setup type of encoding with `UseCompatibleCompressedOops` flag. I update code which reserves heap for compressed oops to take into account this flag.
>
> An other issue was that `UseCompatibleCompressedOops` flag was set during CDS arguments consistency checkup. But `UseCompressedOops` flag is set by GC ergonomics. I moved code to CDS ergonomics setting method.
>
> During testing I hit assert that AOT code address table is missing some code blob address. But it was actually card table base address which pointed inside CodeCache. I added special case to check for it early.
>
> Tested premain-tier1.
This pull request has now been integrated.
Changeset: fc222b7a
Author: Vladimir Kozlov <kvn at openjdk.org>
URL: https://git.openjdk.org/leyden/commit/fc222b7a841f4f828fad867dc13a02817b82899e
Stats: 63 lines in 3 files changed: 39 ins; 16 del; 8 mod
8347830: [premain] UseCompatibleCompressedOops is broken after merging with mainline
Reviewed-by: asmehra, adinn
-------------
PR: https://git.openjdk.org/leyden/pull/67
More information about the leyden-dev
mailing list