Add a new xtune suboption, xtune:footprint by KavinSatheeskumar · Pull Request #23543 · eclipse-openj9/openj9

KavinSatheeskumar · 2026-03-18T13:52:47Z

Add a new xtune sub option, xtune:footprint which tweaks compiler settings to optimize for footprint reduction.

Addresses this issue

mpirvu

Over-arching comment: I don't think we should mix footprint with aggressiveness levels. The aggressiveness here refers to how aggressive the JIT compiler is in improving throughput.

mpirvu

Implemented changes look good to me.
Have you thought how to connect J9_RUNTIME_TUNE_FOOTPRINT to the heuristics? Are heuristics going to access vm->runtimeFlags directly? What if we need omr to know about J9_RUNTIME_TUNE_FOOTPRINT? What happens during CRIU restore?

KavinSatheeskumar · 2026-04-23T20:02:41Z

I was looking into adding CRIU support for this option. However, it seems like none of the options -Xtune: have CRIU support. I was wondering if we should add it in this PR, or have a separate PR which adds all of them?

mpirvu

LGTM

mpirvu · 2026-05-02T13:19:27Z

jenkins compile all jdk26

mpirvu · 2026-05-04T20:12:16Z

jenkins test sanity all jdk26

KavinSatheeskumar · 2026-05-12T13:55:26Z

Test_openjdk26_j9_sanity.openjdk_x86-64_windows_Personal

Failure appears to be caused by an I/O Exception

java.nio.file.AccessDeniedException: C:\Users\jenkins\workspace\Test_openjdk26_j9_sanity.openjdk_x86-64_windows_Personal@tmp\durable-76972f2b\jenkins-log.txt

https://openj9-jenkins.osuosl.org/job/Test_openjdk26_j9_sanity.openjdk_x86-64_linux_Personal/3/

Test failure is an instance of jdk23+ ClassNotFoundException #19779

Test_openjdk26_j9_sanity.openjdk_ppc64_aix_Personal

Appears to be a timeout error

[2026-05-05T07:44:03.147Z] Cancelling nested steps due to timeout
[2026-05-05T07:44:03.166Z] Sending interrupt signal to process

Test_openjdk26_j9_sanity.functional_ppc64_aix_Personal

appears to be an instance of Exception in thread "main" java.awt.AWTError: Can't connect to X11 window server using 'unix:0' #17396

Exception in thread "main" java.awt.AWTError: Can't connect to X11 window server using 'unix:0' as the value of the DISPLAY variable.

All of these appear to be unrelated to the change in this pull request

mpirvu

LGTM

mpirvu · 2026-05-12T14:19:04Z

jenkins test sanity.functional all jdk26

KavinSatheeskumar · 2026-05-12T20:23:26Z

Test_openjdk26_j9_sanity.functional_x86-64_linux_Personal

failed due to the following known infra issues

test list 0, 2, 3, 5

13:21:07  Exception: groovy.lang.MissingMethodException: No signature of method: org.jvnet.hudson.plugins.groovypostbuild.GroovyPostbuildRecorder$BadgeManager.addSummary() is applicable for argument types: (java.lang.String) values: [accept.svg]

1,4

[2026-05-12T16:50:53.910Z] Exception: hudson.AbortException: Failed to run ssh-agent -k

Aside from the infra failure, tests 0, 2, 3 and 5 all passed and was incorrectly marked as failed. Tests 1 and 4 did not run.

KavinSatheeskumar · 2026-05-12T20:28:45Z

Test_openjdk26_j9_sanity.functional_x86-64_windows_Personal

failed due to the following known infra issue

0, 2, 3, 5

[2026-05-12T17:21:07.147Z] Exception: groovy.lang.MissingMethodException: No signature of method: org.jvnet.hudson.plugins.groovypostbuild.GroovyPostbuildRecorder$BadgeManager.addSummary() is applicable for argument types: (java.lang.String) values: [accept.svg]
[2026-05-12T17:21:07.147Z] Possible solutions: createSummary(java.lang.String)

1, 4

[2026-05-12T16:50:53.910Z] Exception: hudson.AbortException: Failed to run ssh-agent -k

Aside from the infra failure, tests 0, 2, 3 and 5 all passed and was incorrectly marked as failed. Tests 1 and 4 did not run.

KavinSatheeskumar · 2026-05-12T20:32:06Z

Test_openjdk26_j9_sanity.functional_s390x_linux_Personal

failed due to the following known infra issue

0, 1, 2

[2026-05-12T16:08:36.791Z] Exception: groovy.lang.MissingMethodException: No signature of method: org.jvnet.hudson.plugins.groovypostbuild.GroovyPostbuildRecorder$BadgeManager.addSummary() is applicable for argument types: (java.lang.String) values: [accept.svg]
[2026-05-12T16:08:36.791Z] Possible solutions: createSummary(java.lang.String)

All three tests ran successfully, and were incorrectly marked as failing

KavinSatheeskumar · 2026-05-12T20:33:32Z

Test_openjdk26_j9_sanity.functional_ppc64le_linux_Personal

failed due to the following known infra issue

groovy.lang.MissingPropertyException: No such property: parallelTestParams for class: groovy.lang.Binding

These tests did not run

KavinSatheeskumar · 2026-05-12T20:35:42Z

Test_openjdk26_j9_sanity.functional_ppc64_aix_Personal

these tests failed due to known infra issues

0, 1, 2

[2026-05-12T19:23:00.970Z] Exception: groovy.lang.MissingMethodException: No signature of method: org.jvnet.hudson.plugins.groovypostbuild.GroovyPostbuildRecorder$BadgeManager.addSummary() is applicable for argument types: (java.lang.String) values: [accept.svg]
[2026-05-12T19:23:00.970Z] Possible solutions: createSummary(java.lang.String)

All these tests passed, but were incorrectly marked as failing

KavinSatheeskumar · 2026-05-12T20:39:23Z

Test_openjdk26_j9_sanity.functional_aarch64_mac_Personal

failed due to known infra issues

0, 1

[2026-05-12T15:35:09.616Z] Exception: groovy.lang.MissingMethodException: No signature of method: org.jvnet.hudson.plugins.groovypostbuild.GroovyPostbuildRecorder$BadgeManager.addSummary() is applicable for argument types: (java.lang.String) values: [accept.svg]
[2026-05-12T15:35:09.617Z] Possible solutions: createSummary(java.lang.String)

All tests ran successfully, and were incorrectly marked as failing

KavinSatheeskumar · 2026-05-12T20:40:46Z

Test_openjdk26_j9_sanity.functional_aarch64_linux_Personal

failed due to known infra issue

groovy.lang.MissingPropertyException: No such property: parallelTestParams for class: groovy.lang.Binding

none of these tests ran

KavinSatheeskumar · 2026-05-12T20:42:55Z

Build_JDK26_x86-64_mac_Personal

failed due to known infra issue

java.io.FileNotFoundException: /var/lib/jenkins/jobs/Build_JDK26_x86-64_mac_Personal/builds/11/program.dat (No such file or directory)

build failed due to an issue on the build machine, and not the code

mpirvu · 2026-05-13T13:28:08Z

jenkins test sanity.functional xlinux jdk26

mpirvu · 2026-05-13T18:33:05Z

The test on xlinux passed. The other failed due to infra issues. Given this and the fact that the new is protected by the newly created option, this PR is ready to be merged.

keithc-ca · 2026-05-13T20:28:36Z

+        double scaleFactor = 1;
+        bool xtune_footprint = javaVM->runtimeFlags & J9_RUNTIME_TUNE_FOOTPRINT;
+        if (xtune_footprint) {
+            scaleFactor = 4;
+        } else {
+#if defined(J9VM_OPT_CRIU_SUPPORT)
+            if (javaVM->internalVMFunctions->isCRaCorCRIUSupportEnabled(javaVM)) {
+                scaleFactor = 2;
+            }
+#endif
+        }
+        if (!_extensions->dnssExpectedRatioMaximum._wasSpecified &&
+			!_extensions->dnssExpectedRatioMinimum._wasSpecified) {
+			_extensions->dnssExpectedRatioMaximum._valueSpecified *= scaleFactor;
+			_extensions->dnssExpectedRatioMinimum._valueSpecified *= scaleFactor;


Please create a follow-up change to indent consistently (with tabs instead of spaces).

Hello, this PR should do that, #23908

Pull request eclipse-openj9#23543 used spaces where it should have used tabs in the file runtime/gc_glue_java/ConfigurationDelegate.hpp. This pull request fixes it. eclipse-openj9#23543

KavinSatheeskumar force-pushed the xtune-footprint branch from 5ad23a6 to 646969a Compare March 18, 2026 14:03

KavinSatheeskumar mentioned this pull request Mar 18, 2026

Add xtune:footprint suboption to xtune eclipse-omr/omr#8170

Closed

mpirvu requested changes Mar 18, 2026

View reviewed changes

Comment thread runtime/vm/jvminit.c Outdated

Comment thread runtime/vm/jvminit.c Outdated

Comment thread runtime/vm/jvminit.c Outdated

Comment thread runtime/compiler/control/J9Options.cpp Outdated

mpirvu self-assigned this Mar 18, 2026

mpirvu added comp:jit comp:vm depends:omr Pull request is dependent on a corresponding change in OMR labels Mar 18, 2026

KavinSatheeskumar force-pushed the xtune-footprint branch from 646969a to 4b1ff86 Compare March 18, 2026 15:30

mpirvu reviewed Mar 18, 2026

View reviewed changes

Comment thread runtime/vm/jvminit.c

mpirvu removed the depends:omr Pull request is dependent on a corresponding change in OMR label Mar 18, 2026

KavinSatheeskumar force-pushed the xtune-footprint branch 3 times, most recently from a45639b to 4998d34 Compare March 20, 2026 19:49

mpirvu requested changes Mar 20, 2026

View reviewed changes

KavinSatheeskumar force-pushed the xtune-footprint branch 3 times, most recently from 899d366 to 4f1f8a8 Compare March 24, 2026 14:47

mpirvu reviewed Mar 24, 2026

View reviewed changes

Comment thread runtime/oti/j9nonbuilder.h Outdated

KavinSatheeskumar force-pushed the xtune-footprint branch from 4f1f8a8 to 4c49ad1 Compare March 25, 2026 15:27

KavinSatheeskumar force-pushed the xtune-footprint branch from 4c49ad1 to d5bf0ba Compare April 8, 2026 03:00

mpirvu reviewed Apr 8, 2026

View reviewed changes

Comment thread runtime/compiler/control/J9Options.cpp Outdated

mpirvu reviewed Apr 8, 2026

View reviewed changes

Comment thread runtime/compiler/control/J9Options.cpp Outdated

mpirvu reviewed Apr 8, 2026

View reviewed changes

Comment thread runtime/compiler/control/J9Options.cpp Outdated

mpirvu reviewed Apr 8, 2026

View reviewed changes

Comment thread runtime/compiler/control/J9Options.cpp Outdated

KavinSatheeskumar force-pushed the xtune-footprint branch 6 times, most recently from 2b80d7b to 30555af Compare April 14, 2026 03:15

KavinSatheeskumar force-pushed the xtune-footprint branch 4 times, most recently from 25eac81 to d579f42 Compare April 21, 2026 13:59

mpirvu approved these changes May 2, 2026

View reviewed changes

Add a new xtune suboption, xtune:footprint

1512975

KavinSatheeskumar force-pushed the xtune-footprint branch from d579f42 to 1512975 Compare May 12, 2026 14:12

mpirvu approved these changes May 12, 2026

View reviewed changes

mpirvu merged commit d869913 into eclipse-openj9:master May 13, 2026
12 of 19 checks passed

keithc-ca reviewed May 13, 2026

View reviewed changes

KavinSatheeskumar mentioned this pull request May 14, 2026

Fix formatting in gc #23908

Merged

pshipton mentioned this pull request Jul 27, 2026

-Xtune:throughput and -Xtune:footprint eclipse-openj9/openj9-docs#1739

Closed

Uh oh!

Conversation

KavinSatheeskumar commented Mar 18, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

mpirvu left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

mpirvu left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

KavinSatheeskumar commented Apr 23, 2026

Uh oh!

mpirvu left a comment

Choose a reason for hiding this comment

Uh oh!

mpirvu commented May 2, 2026

Uh oh!

mpirvu commented May 4, 2026

Uh oh!

KavinSatheeskumar commented May 12, 2026

Uh oh!

mpirvu left a comment

Choose a reason for hiding this comment

Uh oh!

mpirvu commented May 12, 2026

Uh oh!

KavinSatheeskumar commented May 12, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

KavinSatheeskumar commented May 12, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

KavinSatheeskumar commented May 12, 2026

Uh oh!

KavinSatheeskumar commented May 12, 2026

Uh oh!

KavinSatheeskumar commented May 12, 2026

Uh oh!

KavinSatheeskumar commented May 12, 2026

Uh oh!

KavinSatheeskumar commented May 12, 2026

Uh oh!

KavinSatheeskumar commented May 12, 2026

Uh oh!

mpirvu commented May 13, 2026

Uh oh!

mpirvu commented May 13, 2026

Uh oh!

Uh oh!

keithc-ca May 13, 2026

Choose a reason for hiding this comment

Uh oh!

KavinSatheeskumar May 14, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

KavinSatheeskumar commented Mar 18, 2026 •

edited

Loading

KavinSatheeskumar commented May 12, 2026 •

edited

Loading

KavinSatheeskumar commented May 12, 2026 •

edited

Loading

KavinSatheeskumar May 14, 2026 •

edited

Loading