RFR: 8347830: [premain] UseCompatibleCompressedOops is broken after merging with mainline [v2]

Vladimir Kozlov kvn at openjdk.org
Fri May 16 18:09:41 UTC 2025


> 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.

Vladimir Kozlov has updated the pull request incrementally with one additional commit since the last revision:

  Added missing checks. Address comment.

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

Changes:
  - all: https://git.openjdk.org/leyden/pull/67/files
  - new: https://git.openjdk.org/leyden/pull/67/files/b4ed6324..4563786d

Webrevs:
 - full: https://webrevs.openjdk.org/?repo=leyden&pr=67&range=01
 - incr: https://webrevs.openjdk.org/?repo=leyden&pr=67&range=00-01

  Stats: 7 lines in 1 file changed: 1 ins; 2 del; 4 mod
  Patch: https://git.openjdk.org/leyden/pull/67.diff
  Fetch: git fetch https://git.openjdk.org/leyden.git pull/67/head:pull/67

PR: https://git.openjdk.org/leyden/pull/67


More information about the leyden-dev mailing list