-
Notifications
You must be signed in to change notification settings - Fork 198
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
Update help of xtest --stats #699
Conversation
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.
Signed-off-by:
tag is missing in the commit message.
I suggest to rephrase the message in describing what's done:
The command 'xtest --stats --time' is
notnow documented in the usage
description.
Acked-by: Etienne Carriere <[email protected]>
once addressed.
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.
With my comment addressed:
Acked-by: Jerome Forissier <[email protected]>
host/xtest/stats.c
Outdated
@@ -58,6 +58,7 @@ static int usage(void) | |||
fprintf(stderr, " --alloc Print allocation statistics\n"); | |||
fprintf(stderr, " --memleak Dump memory leak data on secure console\n"); | |||
fprintf(stderr, " --ta Print loaded TAs context\n"); | |||
fprintf(stderr, " --time Print system time\n"); |
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.
Print REE and TEE time
12b42c6
to
4a39a73
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.
could you format the commit message header line into xtest: ...
? maybe:
xtest: add help for --stats --time
.
Please also add a period in the message:
The command 'xtest --stats --time' is now documented in the usage
- description
+description.
The command 'xtest --stats --time' is now documented in the usage description. Acked-by: Etienne Carriere <[email protected]> Acked-by: Jerome Forissier <[email protected]> Signed-off-by: Julien Jayat <[email protected]>
This pull request has been marked as a stale pull request because it has been open (more than) 30 days with no activity. Remove the stale label or add a comment, otherwise this pull request will automatically be closed in 5 days. Note that you can always re-open a closed pull request at any time. |
The command 'xtest --stats --time' is not documented in the usage
description