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

Add z/TPF specific configuration diagnostic information to Javacore #19219

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

jjohnst-us
Copy link
Contributor

@jjohnst-us jjohnst-us commented Mar 26, 2024

To assist with gathering context information for problem determination on z/TPF several fields dealing with system memory configuration and other settings are added to the Javacore for z/TPF only. The added sub-section is especially useful for determining if system limits were reached, and it can also be used for sizing system resources to run Java applications on z/TPF.

@jjohnst-us jjohnst-us force-pushed the master branch 3 times, most recently from aad72f5 to 63c71c2 Compare May 8, 2024 01:37
@jjohnst-us jjohnst-us requested a review from pshipton May 8, 2024 01:39
@pshipton pshipton requested a review from keithc-ca May 8, 2024 16:23
runtime/rasdump/javadump.cpp Outdated Show resolved Hide resolved
runtime/rasdump/javadump.cpp Outdated Show resolved Hide resolved
runtime/rasdump/javadump.cpp Outdated Show resolved Hide resolved
runtime/rasdump/javadump.cpp Show resolved Hide resolved
runtime/rasdump/javadump.cpp Outdated Show resolved Hide resolved
runtime/rasdump/javadump.cpp Outdated Show resolved Hide resolved
runtime/rasdump/javadump.cpp Outdated Show resolved Hide resolved
runtime/rasdump/javadump.cpp Outdated Show resolved Hide resolved
runtime/rasdump/javadump.cpp Outdated Show resolved Hide resolved
runtime/rasdump/javadump.cpp Outdated Show resolved Hide resolved
@keithc-ca
Copy link
Contributor

The summary in the commit message is too long (should be no more than 70 characters); suggest removing the second "z/TPF".

@jjohnst-us jjohnst-us requested a review from keithc-ca May 9, 2024 14:42
@jjohnst-us jjohnst-us changed the title Add z/TPF specific configuration diagnostic information to z/TPF Javacore Add z/TPF specific configuration diagnostic information to Javacore May 9, 2024
runtime/rasdump/javadump.cpp Outdated Show resolved Hide resolved
runtime/rasdump/javadump.cpp Outdated Show resolved Hide resolved
runtime/rasdump/javadump.cpp Outdated Show resolved Hide resolved
runtime/rasdump/javadump.cpp Outdated Show resolved Hide resolved
runtime/rasdump/javadump.cpp Outdated Show resolved Hide resolved
runtime/rasdump/javadump.cpp Outdated Show resolved Hide resolved
runtime/rasdump/javadump.cpp Outdated Show resolved Hide resolved
runtime/rasdump/javadump.cpp Outdated Show resolved Hide resolved
runtime/rasdump/javadump.cpp Outdated Show resolved Hide resolved
@jjohnst-us
Copy link
Contributor Author

All recommended updates have been applied.

@jjohnst-us jjohnst-us requested a review from keithc-ca May 11, 2024 00:52
Comment on lines +1785 to +1786
#if defined(J9ZTPF)

Copy link
Contributor

Choose a reason for hiding this comment

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

Please put the blank line before the new block.

Comment on lines +1789 to +1791
int rc;

rc = mprivatestats(&ztpfNativeStats);
Copy link
Contributor

Choose a reason for hiding this comment

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

Please join these so rc has an initial value.

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.

None yet

3 participants