xtest: add command to get system time #694
Conversation
host/xtest/stats.c
Outdated
| op.paramTypes = TEEC_PARAM_TYPES(TEEC_VALUE_OUTPUT, | ||
| TEEC_VALUE_OUTPUT, | ||
| TEEC_NONE, TEEC_NONE); | ||
| res = TEEC_InvokeCommand(&sess, STATS_CMD_SYSTEM_TIME, &op, &eo); |
There was a problem hiding this comment.
This command could be implemented in the "OS test" TA (ta/os_test/os_test.c).
There was a problem hiding this comment.
This command could be implemented in the "OS test" TA (
ta/os_test/os_test.c).
Thanks for your comment. It seems "OS test" TA hasn't been included into our real product. Can we keep it in stats PTA?
There was a problem hiding this comment.
This command could be implemented in the "OS test" TA (ta/os_test/os_test.c).
It is worth creating a new xtest --os-test [ARGS] interface rather than using existing --stats entries and related PTA_STATS, to print these kind of core state info?
There was a problem hiding this comment.
Please ignore my suggestion, we'll keep the thing in the stats PTA.
weizhaojiang
force-pushed
the
main_0922
branch
from
72264ee
to
c654f8d
Compare
host/xtest/stats.c
Outdated
|
|
||
| printf("REE time: %"PRId32" Seconds, %"PRId32" Milliseconds\n", | ||
| op.params[0].value.a, op.params[0].value.b); | ||
| printf("TEE time: %"PRId32" Seconds, %"PRId32" Milliseconds\n", |
There was a problem hiding this comment.
No particular reason for capitalizing Seconds and Milliseconds.
Command format: xtest --stats --time Acked-by: Etienne Carriere <etienne.carriere@foss.st.com> Reviewed-by: Jerome Forissier <jerome.forissier@linaro.org> Tested-by: Weizhao Jiang <weizhaoj@amazon.com> Signed-off-by: Weizhao Jiang <weizhaoj@amazon.com>
weizhaojiang
force-pushed
the
main_0922
branch
from
c654f8d
to
fd2c790
Compare
Command format: xtest --stats --time
Tested-by: Weizhao Jiang weizhaoj@amazon.com