Golang#101
Draft
MatthewKhouzam wants to merge 7 commits intoeclipse-tracecompass-incubator:masterfrom
Draft
Conversation
This allows to audit transactions and ensure that traces artifacts are cleaned. Signed-off-by: Matthew Khouzam <matthew.khouzam@ericsson.com> Change-Id: Iebc84073250ef568bf36b7bf5ba29701343f755d
Signed-off-by: Matthew Khouzam <matthew.khouzam@ericsson.com> Change-Id: I05dbd2b12a779efb98b0ae976254abd46c2d4911
TODO: * clean up debt * test more * Add heap analysis * Add callsite analysis * Add callstack analysis * Add Object Life Analysis Signed-off-by: Matthew Khouzam <matthew.khouzam@ericsson.com> Change-Id: I0dcd991c9cef862de990901486a4ccbe6d3fc814
Signed-off-by: Matthew Khouzam <matthew.khouzam@ericsson.com> Change-Id: I00af9097710c675a532653a785c73ef2d64b2c70
Signed-off-by: Matthew Khouzam <matthew.khouzam@ericsson.com> Change-Id: Iabb664c94c8b7f5991031f6ef64d733f6eb9af58
Has memory usage... Signed-off-by: Matthew Khouzam <matthew.khouzam@ericsson.com> Change-Id: I9b6161b1db8c3142571611a0f28351afc384e74a
Signed-off-by: Matthew Khouzam <matthew.khouzam@ericsson.com> Change-Id: Iae1533332f04ba289f0a9c7246363ac687331760
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This patchset adds golang trace support to Trace Compass
https://go.dev/blog/execution-traces-2024 <- The trace format and its events were designed around relatively efficient emission, but required tooling to parse and keep around the state of the entirety of a trace. A few hundred MiB trace could require several GiB of RAM to analyze!
This is not the case with the trace compass golang parser. It seems to scale linearly with the trace size. But this is not ready for primetime yet.
In GolangTrace there's a datastructure called "fAwfulBackup" that stores ecery event in ram. It needs to go away before we merge the code.