Skip to content

Refactor TR::DefaultCompilationStrategy::processJittedSample - #15680

Merged
ymanton merged 12 commits into
eclipse-openj9:masterfrom
dsouzai:refactorPJM
Aug 9, 2022
Merged

Refactor TR::DefaultCompilationStrategy::processJittedSample#15680
ymanton merged 12 commits into
eclipse-openj9:masterfrom
dsouzai:refactorPJM

Conversation

@dsouzai

@dsouzai dsouzai commented Aug 8, 2022

Copy link
Copy Markdown
Contributor

This PR is a retry of #15137; the main difference is that I removed the fatal assert. I've verified there's no issues by running extended.system tests internally.

dsouzai added 12 commits August 2, 2022 15:59
Signed-off-by: Irwin D'Souza <dsouzai.gh@gmail.com>
Signed-off-by: Irwin D'Souza <dsouzai.gh@gmail.com>
Signed-off-by: Irwin D'Souza <dsouzai.gh@gmail.com>
Signed-off-by: Irwin D'Souza <dsouzai.gh@gmail.com>
Signed-off-by: Irwin D'Souza <dsouzai.gh@gmail.com>
Signed-off-by: Irwin D'Souza <dsouzai.gh@gmail.com>
Signed-off-by: Irwin D'Souza <dsouzai.gh@gmail.com>
Signed-off-by: Irwin D'Souza <dsouzai.gh@gmail.com>
Signed-off-by: Irwin D'Souza <dsouzai.gh@gmail.com>
…JittedSample

Signed-off-by: Irwin D'Souza <dsouzai.gh@gmail.com>
Signed-off-by: Irwin D'Souza <dsouzai.gh@gmail.com>
@dsouzai

dsouzai commented Aug 8, 2022

Copy link
Copy Markdown
Contributor Author

@ymanton do you mind reviewing again? Also as per #15137 (comment) I suppose we should run the sanity.system and extended.system suites.

@ymanton ymanton left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Difference to #15137 is minor. LGTM.

@ymanton

ymanton commented Aug 8, 2022

Copy link
Copy Markdown
Member

Jenkins test sanity.system,extended.system all jdk11

@dsouzai

dsouzai commented Aug 9, 2022

Copy link
Copy Markdown
Contributor Author

Test_openjdk11_j9_sanity.system_x86-64_linux_Personal failure is due to

[2022-08-08T21:04:01.154Z] CL2 j> 2022/08/08 18:03:59.386 Writing report data ...
[2022-08-08T21:04:03.736Z] CL2 stderr Exception in thread "main" java.lang.BootstrapMethodError: java.lang.invoke.StringConcatException: Generator failed
[2022-08-08T21:04:03.737Z] CL2 stderr 	at net.adoptopenjdk.test.jlm.resources.ExpectedMBeanInfo.writeOutCategoryReport(ExpectedMBeanInfo.java:238)
[2022-08-08T21:04:03.737Z] CL2 stderr 	at net.adoptopenjdk.test.jlm.resources.ExpectedMBeanInfo.writeOutChecklistReport(ExpectedMBeanInfo.java:284)
[2022-08-08T21:04:03.737Z] CL2 stderr 	at net.adoptopenjdk.test.jlm.resources.ThreadData.writeMBeanInfo(ThreadData.java:900)
[2022-08-08T21:04:03.737Z] CL2 stderr 	at net.adoptopenjdk.test.jlm.resources.ThreadData.writeData(ThreadData.java:427)
[2022-08-08T21:04:03.737Z] CL2 stderr 	at net.adoptopenjdk.test.jlm.remote.ThreadProfiler.getStatsViaServer(ThreadProfiler.java:199)
[2022-08-08T21:04:03.737Z] CL2 stderr 	at net.adoptopenjdk.test.jlm.remote.ThreadProfiler.main(ThreadProfiler.java:99)
[2022-08-08T21:04:03.737Z] CL2 stderr Caused by: java.lang.invoke.StringConcatException: Generator failed
[2022-08-08T21:04:03.737Z] CL2 stderr 	at java.base/java.lang.invoke.StringConcatFactory.generate(StringConcatFactory.java:764)
[2022-08-08T21:04:03.737Z] CL2 stderr 	at java.base/java.lang.invoke.StringConcatFactory.doStringConcat(StringConcatFactory.java:665)
[2022-08-08T21:04:03.737Z] CL2 stderr 	at java.base/java.lang.invoke.StringConcatFactory.makeConcatWithConstants(StringConcatFactory.java:581)
[2022-08-08T21:04:03.737Z] CL2 stderr 	at java.base/java.lang.invoke.MethodHandleResolver.invokeBsm(MethodHandleResolver.java:176)
[2022-08-08T21:04:03.737Z] CL2 stderr 	at java.base/java.lang.invoke.MethodHandleResolver.resolveInvokeDynamic(MethodHandleResolver.java:238)
[2022-08-08T21:04:03.737Z] CL2 stderr 	... 6 more
[2022-08-08T21:04:03.737Z] CL2 stderr Caused by: java.lang.NullPointerException
[2022-08-08T21:04:03.737Z] CL2 stderr 	at java.base/java.lang.invoke.BruteArgumentMoverHandle.permuteArguments(BruteArgumentMoverHandle.java:286)
[2022-08-08T21:04:03.737Z] CL2 stderr 	at java.base/java.lang.invoke.MethodHandles.dropArgumentsUnsafe(MethodHandles.java:2961)
[2022-08-08T21:04:03.737Z] CL2 stderr 	at java.base/java.lang.invoke.MethodHandles.dropArguments(MethodHandles.java:2978)
[2022-08-08T21:04:03.737Z] CL2 stderr 	at java.base/java.lang.invoke.StringConcatFactory$MethodHandleInlineCopyStrategy.generate(StringConcatFactory.java:1536)
[2022-08-08T21:04:03.737Z] CL2 stderr 	at java.base/java.lang.invoke.StringConcatFactory.generate(StringConcatFactory.java:756)
[2022-08-08T21:04:03.737Z] CL2 stderr 	... 10 more

which is seen before in #8972 (comment)

@ymanton
ymanton merged commit 7da0d0b into eclipse-openj9:master Aug 9, 2022
@dsouzai
dsouzai deleted the refactorPJM branch February 3, 2023 16:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants