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

Invalid layout of well-known class: java.lang.Thread with Java 11.0.11+8 #10

Open
ebourg opened this issue Apr 12, 2021 · 13 comments
Open

Comments

@ebourg
Copy link

ebourg commented Apr 12, 2021

I tried using DCVEM 11.0.10+5 with OpenJDK 11.0.11+8 and I got the following error:

$ java -dcevm -version
[0.017s][error][class] Name nativeParkEventPointer should be in the SymbolTable since its class is loaded
Error occurred during initialization of VM
Invalid layout of well-known class: java.lang.Thread
@skybber
Copy link
Contributor

skybber commented Apr 13, 2021

Will you get the same error if you combine std 11.0.5 with 11.0.8? I don't think that it is related to dcevm, since dcevm11 code is pretty separated from the standard JVM code and it is not involved unless you use the advanced hotswap.

@ebourg
Copy link
Author

ebourg commented Apr 13, 2021

Do you mean that I should try rebasing DCEVM 11.0.10+5 on top of OpenJDK 11.0.11+8 ?

@skybber
Copy link
Contributor

skybber commented Apr 13, 2021

I did not read used versions properly. It seems strange for me now since the code of hotspot/src/share/vm/classfile/javaClasses.cpp has not been changed at nativeParkEventPointer definition for a long time. You can try to use the same version for dcevm and OpenJDK and the same compilation flags.

@ebourg
Copy link
Author

ebourg commented Apr 13, 2021

Ok I'll give it a try. Do you think you could pull the jdk-11.0.11*_adopt tags into the DCEVM repository?

@skybber
Copy link
Contributor

skybber commented Apr 13, 2021

I'll do that this evening. Btw it can be done if you merge upstream from https://github.com/AdoptOpenJDK/openjdk-jdk11u to master branch.

@ebourg
Copy link
Author

ebourg commented Apr 13, 2021

Thank you, I wasn't sure about the remote used.

@ebourg
Copy link
Author

ebourg commented Apr 13, 2021

java -dcevm -version works after rebasing on top of jdk-11.0.11+8

@skybber
Copy link
Contributor

skybber commented Apr 13, 2021

https://github.com/AdoptOpenJDK/openjdk-jdk11u is merged in this repo's master now .

@ebourg
Copy link
Author

ebourg commented Apr 27, 2021

OpenJDK 11.0.11-ga has been released last week, would it be possible to rebase the dcevm repository and tag a release please?

@skybber
Copy link
Contributor

skybber commented Apr 27, 2021

I'll do that in next days.

@skybber
Copy link
Contributor

skybber commented Apr 27, 2021

11.0.11 is in https://github.com/HotswapProjects/openjdk-jdk11u-dcevm , but Trava build is broken now. I has not resolved it yet (they are closing trava.org and migrate all repos to trava.com) .

@ebourg
Copy link
Author

ebourg commented Apr 27, 2021

Thank you! It built fine in Debian at least.

@ebourg
Copy link
Author

ebourg commented Jul 30, 2021

OpenJDK 11.0.12 has been released last week, DCEVM needs a rebase again.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants