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

Builds should archive debugimage archives #1596

Closed
keithc-ca opened this issue Mar 9, 2020 · 11 comments · Fixed by #1597
Closed

Builds should archive debugimage archives #1596

keithc-ca opened this issue Mar 9, 2020 · 11 comments · Fixed by #1597
Assignees
Labels
enhancement Issues that enhance the code or documentation of the repo in any way
Milestone

Comments

@keithc-ca
Copy link
Contributor

OpenJ9 builds also produce build/.../images/debug-image containing debug information relating to native code. That folder contains platform-specific (.debuginfo, .pdb, .dSYM or .diz) files in locations that relate to the corresponding executable or shared library in a jdk.

Copying the debug-image folder over top of the jdk makes symbols available to greatly ease the task of debugging an issue with native code.

Archives of this information are already made available in nightly and pull-request builds at ci.eclipse.org/openj9 (e.g. [1]); it would be helpful to make them available via adoptopenjdk.net as well.

[1] https://ci.eclipse.org/openj9/view/Build_Nightly/job/Build_JDK11_x86-64_linux_Nightly/lastSuccessfulBuild/

@sxa sxa added the enhancement Issues that enhance the code or documentation of the repo in any way label Mar 9, 2020
@sxa sxa added this to TODO in temurin-build via automation Mar 9, 2020
@sxa sxa added this to the March 2020 milestone Mar 9, 2020
@pshipton
Copy link
Contributor

pshipton commented Mar 9, 2020

JDK8 should also be configured with the --disable-zip-debug-info option so the debug image contains .debuginfo and .pdb files instead of .diz files which the user (developer) would have to unzip before they can be used with a debugger.
eclipse-openj9/openj9#8808

@pshipton
Copy link
Contributor

pshipton commented Mar 16, 2020

FYI eclipse-openj9/openj9#8874. The OpenJ9 project is going to include the debug-image into the testimage archive, to reduce the number of different files that need to be specified and expanded when running the tests. We are looking at modifying the test scripting to recognize the presence of the debug-image and copy/move the files to the correct place in the JVM build so they are used if a crash occurs when running the tests.
https://github.com/AdoptOpenJDK/openjdk-tests/blob/6d4670a158607cb0e0fd30e76b5c6a0f03a7a74d/get.sh#L273

@pshipton
Copy link
Contributor

@keithc-ca what would it take to have a Hotspot build create a debug-image as well?

@keithc-ca
Copy link
Contributor Author

@keithc-ca what would it take to have a Hotspot build create a debug-image as well?

I expect the additions for OpenJ9 would also work for hotspot. We introduced a new main target ([1]) which invokes a new makefile ([2]). The changes are similar, but not identical, in other versions.

[1] https://github.com/ibmruntimes/openj9-openjdk-jdk11/blob/openj9/closed/custom/Main.gmk#L72-L75
[2] https://github.com/ibmruntimes/openj9-openjdk-jdk11/blob/openj9/closed/DebugImage.gmk

@sxa
Copy link
Member

sxa commented Mar 18, 2020

@johnoliver Can you take a look at this please - the OpenJ9 team want it available for the April release

@sxa sxa added this to To do in Top Priorities Mar 18, 2020
@karianna
Copy link
Contributor

@keithc-ca - is #1597 the PR that resolves this (in part?)

@keithc-ca
Copy link
Contributor Author

@keithc-ca - is #1597 the PR that resolves this (in part?)

Yes, that's the expectation.

@karianna karianna linked a pull request Mar 18, 2020 that will close this issue
@sxa
Copy link
Member

sxa commented Mar 18, 2020

@karianna We'll also need API/website updates to fulfil the other parts of this

@karianna karianna moved this from To do to In progress in Top Priorities Mar 23, 2020
temurin-build automation moved this from TODO to Done Mar 24, 2020
Top Priorities automation moved this from In progress to Done Mar 24, 2020
@pshipton
Copy link
Contributor

I'm not sure why #1597 closed this. I think there are still parts remaining as Stuart mentioned in the previous comment.

@karianna
Copy link
Contributor

Linked PRs auto close issues. The API and website issues should be added to their respective repos and the top priority project added to them.

@pshipton
Copy link
Contributor

I can't add the top priority project, but I've created
https://github.com/AdoptOpenJDK/openjdk-api/issues/195
AdoptOpenJDK/openjdk-website#696

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement Issues that enhance the code or documentation of the repo in any way
Projects
No open projects
temurin-build
  
Done
Development

Successfully merging a pull request may close this issue.

5 participants