-
Notifications
You must be signed in to change notification settings - Fork 720
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
CRIU BytecodeInterpreter enables DO_HOOKS #17245
Conversation
jenkins test sanity xlinux jdk11 |
jenkins test sanity plinux,zlinux,win,alinux64 jdk11 |
jenkins test sanity amac jdk11 |
jenkins test sanity xlinux jdk17 |
Looks like the OpenJ9-JDK11-x86-64_linux-20230421-180956 build fixes the regression (and possibly even improves startup by ~1.5%). |
No issues on JDK17 either. |
Should also check the footprint (RSS) when compiling an extra bytecode interpreter. |
@dsouzai mentioned in the openj9 meeting that the Footprint perf results above are RSS. |
3e6101e
to
9faa6ee
Compare
criuBytecodeLoopCompressed/criuBytecodeLoopFull are used when CRIU is enabled. Signed-off-by: Jason Feng <fengj@ca.ibm.com>
9faa6ee
to
f8f3d73
Compare
@keithc-ca anymore concerns |
@pshipton we will go ahead with this change |
jenkins compile win jdk11 |
@JasonFengJ9 please make a change for 0.38 |
@tajila please review 0.38 PR |
CRIU
BytecodeInterpreter
enablesDO_HOOKS
criuBytecodeLoopCompressed
/criuBytecodeLoopFull
are used whenCRIU
is enabled.Signed-off-by: Jason Feng fengj@ca.ibm.com