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

docs: update debug.md to match the new trace output #220

Merged
1 commit merged into from
Dec 18, 2017

Conversation

jbech-linaro
Copy link
Contributor

This is the updates needed here due to the trace changes in OP-TEE/optee_os#2011.

Copy link
Contributor

@etienne-lms etienne-lms left a comment

Choose a reason for hiding this comment

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

no issue/typo found. looks nice.

docs/debug.md Outdated
also copied to the TA development kit. Whenever you are confronted with an error
message reporting a serious error and showing a "Call stack:" line, you may use
the symbolize script.
read or write permission fault).
Copy link
Contributor

Choose a reason for hiding this comment

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

Alignment issues also?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yes, your are probably right.

ending up in a `TEE_Panic(0)` call.

OP-TEE provides a helper script called `symbolize.py` to facilitate the analysis
of such issues. It is located in the OP-TEE OS source tree, and is also copied
Copy link
Contributor

Choose a reason for hiding this comment

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

remove coma.

s/copied to/installed with/ ?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

It's Jerome's wording and I think copied to is fine here, it's not that we are running something like "make install" (so I'll keep the current wording).

@etienne-lms
Copy link
Contributor

Reviewed-by: Etienne Carriere <etienne.carriere@linaro.org>

Signed-off-by: Joakim Bech <joakim.bech@linaro.org>
Reviewed-by: Etienne Carriere <etienne.carriere@linaro.org>
@ghost ghost merged commit bdfce4a into OP-TEE:master Dec 18, 2017
@jbech-linaro jbech-linaro deleted the trace_improvments branch December 18, 2017 16:46
This pull request was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants