-
Notifications
You must be signed in to change notification settings - Fork 396
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 new core dump tool for OSX #6014
Conversation
FYI @manqingl |
@klangman @JamesKingdon : FYI |
On one of the Open9 DDR tests, the utility creates a vastly inflated core file (100GB+) running the following command:
Looking into the issue now. |
8fae4d1
to
df293fe
Compare
Fixed the issue with core file size inflation. This PR is ready for review. |
Updated the PR, can you take another look please @keithc-ca |
c8376ec
to
e257dad
Compare
@keithc-ca I've updated the PR, can you take another look please? |
e257dad
to
2f9ea8d
Compare
@keithc-ca Updated the PR again. |
2f9ea8d
to
e7d0cd3
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think this should be squashed: one commit is sufficient for one new file.
Please create an issue to address removal of memory allocations (that will fail in low memory scenarios).
e7d0cd3
to
c88ec5b
Compare
Squashed commits into one. |
Dumping core using the signal requires an the task-get-allow entitlement, which prevents notarization. Thus we dump the core with a user space tool which does not require an entitlement. Signed-off-by: Mike Zhang <[email protected]>
c88ec5b
to
cfed42b
Compare
Jenkins build all |
Dumping core using the signal requires an the task-get-allow
entitlement, which prevents notarization. Thus we dump the core
with a user space tool which does not require an entitlement.
Signed-off-by: Mike Zhang [email protected]
Resolves #5813
fyi @keithc-ca