Skip to content
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

Closed
pshipton opened this issue Jan 9, 2020 · 18 comments
Closed
Assignees
Labels
openj9 Issues that are enhancements or bugs raised against the OpenJ9 group
Milestone

Comments

@pshipton
Copy link
Contributor

pshipton commented Jan 9, 2020

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

@andrew-m-leonard
Copy link
Contributor

does that include xLinuxXL ?

@andrew-m-leonard
Copy link
Contributor

Requires "protoc"
Current Ubtuntu 16.04 distribution is not late enough either, 2.6.1, needs 3.5.1. Not sure what slaves have?

@pshipton
Copy link
Contributor Author

pshipton commented Jan 9, 2020

At the OpenJ9 project we are using protobuf 3.7.1 to build it.

@mpirvu
Copy link

mpirvu commented Jan 9, 2020

Indeed we standardized protobuf 3.7.1 across all linux platforms.
Moreover, if build machines have an older version of protobuf, it needs to be uninstalled because we've seen that it can cause linking problems.

@pshipton
Copy link
Contributor Author

pshipton commented Jan 9, 2020

@mpirvu can you please answer about whether --enable-jitserver should be done for xLinuxXL builds? (Note it's enabled at OpenJ9).

@mpirvu
Copy link

mpirvu commented Jan 9, 2020

does that include xLinuxXL

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.

@andrew-m-leonard
Copy link
Contributor

Adopt slaves have protoc 3.5.1

@sxa
Copy link
Member

sxa commented Jan 9, 2020

Indeed we standardized protobuf 3.7.1 across all linux platforms.

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?

@AdamBrousseau
Copy link
Contributor

Spot checking a few x/p/z internal and external it seems x has 3.5.1 and p/z have 3.7.1

@sxa
Copy link
Member

sxa commented Jan 9, 2020

For the purposes of this x is the only important one, so you have the same as us ...

@mpirvu
Copy link

mpirvu commented Jan 9, 2020

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

@andrew-m-leonard
Copy link
Contributor

seem to be linking issues: eclipse-openj9/openj9#8267

@sxa
Copy link
Member

sxa commented Jan 10, 2020

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 gcc73 tag from the other one which will stop OpenJ9 builds running on it.

@sxa sxa added the openj9 Issues that are enhancements or bugs raised against the OpenJ9 group label Jan 10, 2020
@sxa sxa added this to the January 2020 milestone Jan 10, 2020
@mpirvu
Copy link

mpirvu commented Jan 15, 2020

@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?

@andrew-m-leonard
Copy link
Contributor

not until we implement this issue

@sxa
Copy link
Member

sxa commented Jan 17, 2020

@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

@mpirvu
Copy link

mpirvu commented Jan 17, 2020

Thanks @sxa555

@andrew-m-leonard
Copy link
Contributor

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
openj9 Issues that are enhancements or bugs raised against the OpenJ9 group
Projects
None yet
Development

No branches or pull requests

5 participants