Skip to content

Conversation

@jmtd
Copy link
Member

@jmtd jmtd commented Sep 23, 2025

Enabled by default

https://issues.redhat.com/browse/OPENJDK-4106

Depends upon: #616

@jmtd jmtd added the ubi10 RHEL UBI10 label Sep 23, 2025
@jmtd
Copy link
Member Author

jmtd commented Sep 25, 2025

I've just had some news which implies I might need to design this so that I don't break older JDKs after all…

@jmtd jmtd force-pushed the 4106-ubi10-UseCompactObjectHeaders branch from c3f8229 to d257a52 Compare September 30, 2025 13:19
jmtd added a commit to jmtd/redhat-openjdk-containers that referenced this pull request Sep 30, 2025
jmtd added 9 commits November 11, 2025 13:29
Signed-off-by: Jonathan Dowland <[email protected]>
Not available yet. Co-install JDK21 and forcibly remove.

Refresh singleton-jdk cekit module: The alternatives workaround is
no longer needed.

Signed-off-by: Jonathan Dowland <[email protected]>
Source jvm-options (if it exists) from the Cekit "JRE" module
path, rather than JDK.

Move the evaluation of 'jvm_specific_options' out of the GC
options function (gc_config) and into the general command line
assembly.

At present, none of the Cekit modules (JDK or JRE) provide the
jvm-options file.

Signed-off-by: Jonathan Dowland <[email protected]>
In the JRE module only (so far); define jvm_specific_options
containing -XX:+UseCompactObjectHeaders unless
JAVA_COMPACT_OBJECT_HEADERS is defined to "false".

Signed-off-by: Jonathan Dowland <[email protected]>
To avoid repeating ourselves, recognising that the JRE module
is a subset of the configuration required for the JDK modules,
include the former from the latter.

Signed-off-by: Jonathan Dowland <[email protected]>
Signed-off-by: Jonathan Dowland <[email protected]>
@jmtd jmtd force-pushed the 4106-ubi10-UseCompactObjectHeaders branch from ba0fca6 to e662465 Compare November 17, 2025 17:01
@jmtd jmtd marked this pull request as ready for review November 17, 2025 17:02
Copy link
Contributor

@jerboaa jerboaa left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks OK.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ubi10 RHEL UBI10

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants