-
-
Notifications
You must be signed in to change notification settings - Fork 249
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Configure OpenJ9 xlinux builds for jdk8 jdk11 with --enable-jitserver #1464
Comments
does that include xLinuxXL ? |
Requires "protoc" |
At the OpenJ9 project we are using protobuf 3.7.1 to build it. |
Indeed we standardized protobuf 3.7.1 across all linux platforms. |
@mpirvu can you please answer about whether --enable-jitserver should be done for xLinuxXL builds? (Note it's enabled at OpenJ9). |
For uniformity we should probably do xLinuxXL as well, though JITServer typically helps with small containers whereas xlinuxxl is normally used for applications with very large heaps. |
Adopt slaves have protoc 3.5.1 |
That's a new requirement on or systems then as we explicitly set it to 3.5.1 which I believe is the one that your configure check looks for. @jdekonin based on @pshipton's comment above have you put a different version on the OpenJ9 machines than what's in our playbooks? |
Spot checking a few x/p/z internal and external it seems x has 3.5.1 and p/z have 3.7.1 |
For the purposes of this x is the only important one, so you have the same as us ... |
x can work well with protobuf 3.5.1. It was suggested to move xlinux up to 3.7.1 so that all platforms have the same version, but this is not strictly required |
seem to be linking issues: eclipse-openj9/openj9#8267 |
Fixed - now building at https://ci.adoptopenjdk.net/job/build-scripts/job/jobs/job/jdk8u/job/jdk8u-linux-x64-openj9-linuxXL/455/console Bear in mind that only the first build box is configured appropriately for this at present, so I've removed the |
@sxa555 I was wondering about the status of this issue. Are the nightly xlinux builds done at adoptopenjdk supposed to have JITServer code compiled in now? |
not until we implement this issue |
@mpirvu The quarterly releases will have that option enabled - nightlies will follow shortly after. This support was added at the last minute and therefore we had an effective code freeze on adjustments to the build scripts during the release cycle |
Thanks @sxa555 |
Pls configure OpenJ9 xlinux builds for jdk8 and jdk11 with --enable-jitserver
This includes both the head stream and the January (OpenJ9 0.18.0) release.
eclipse-openj9/openj9#8147
eclipse-openj9/openj9#7841
@mpirvu fyi
The text was updated successfully, but these errors were encountered: