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 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
@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
runtime/rasdump/javadump.cpp Outdated Show resolved Hide resolved
runtime/rasdump/javadump.cpp Outdated Show resolved Hide resolved
@jjohnst-us
Copy link
Contributor Author

@keithc-ca , should be all good.

runtime/rasdump/javadump.cpp Outdated Show resolved Hide resolved
_OutputStream.writeInteger(ztpfNativeStats.total31bitHeap1MBFrames, "%zu\n");
_OutputStream.writeCharacters("1TPFLIMEMPS EMPS keypoint A setting: ");
_OutputStream.writeInteger(ecbp2()->ce2proc->iproc_maxheap_31, "%zu\n");
_OutputStream.writeCharacters("1TPFHWGC Total highwater mark GC heap 1MB frames (limited by -Xmx): ");
Copy link
Contributor

Choose a reason for hiding this comment

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

Please replace the tab within the string literal with a suitable number of spaces.

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