-
Notifications
You must be signed in to change notification settings - Fork 729
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
OpenJ9 Perf Results with OJDK MHs enabled #12728
Comments
For reference, here are the recent results of jruby and nashorn benchmarks on JDK11 comparing the performance of OpenJ9 with the existing OpenJ9 MethodHandle implementation vs the reference OpenJDK implementation (RI). The diff (performance gap) is calculated using the following: Jrubyred-black (lower=better)
hash (higher = better)
mandelbrot (lower=better)
kwargs (lower=better)
weird_sort (higher=better)
Nashorn (lower=better)
|
These results are collected using
|
@DanHeidinga More data will be reported next week with changes that will lead to better performance. |
It is unlikely we will have results with JIT enabled until late next week at the earliest. Once @jdmpapin's inliner work is enabled we can re-bench, but more analysis will likely be needed to identify further opportuntities. |
Nashorn JDK11Setupnashorn: downloaded from link at #5371 Raw values are The Results
|
|
JRuby JDK11SetupOpenJDK sha: d4a63e5e5c09a2d8fde2e2d8af2735e8cea8d27d
|
Evaluation after OJDK MH backport to JDK 11 and 8SetupJDK8OpenJDK sha: 9a702f6d35a13affd72c1b996ca991ae40dac100 JDK11OpenJDK sha: f229543b686dd32abe2b3f5565a46df0d70414b9 JDK17#OpenJDK sha: 549540814d003296474c332e25c5f109ca218cd2 JRubyBenchmarks: https://github.com/headius/bench2018 with
JRuby: JDK8
JRuby: JDK11
JRuby: JDK17
Nashornnashorn: downloaded from link at #5371 Raw values are The Nashorn: JDK8
Nashorn: JDK11
Nashorn: JDK17
|
DayTrader7SetupJDK8OpenJDK sha: 9a702f6d35a13affd72c1b996ca991ae40dac100 JDK11OpenJDK sha: f229543b686dd32abe2b3f5565a46df0d70414b9
|
I think we will need confirmation from (say) Marius Pirvu on the exact throughput situation here for Daytrader7. For example, I would certainly be reluctant to accept 2.62% throughout loss in Daytrader7 warm SCC runs in Java8 on x86 (it is not easy to get that kind of improvement in these kinds of apps being the reason). It is very possible that there is some fluctuation in the score and in fact, there isn't any substantial regression. So we just need to be more certain and debug to reduce the impact if the regression amount is in fact confirmed. |
This issue contains performance data for OpenJ9 with OJDK MHs enabled.
Target benchmarks
java.lang.Record
equality is much slower than HotSpot #12359)The text was updated successfully, but these errors were encountered: