-
Notifications
You must be signed in to change notification settings - Fork 229
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
Some updates for easier Embench runs (+ some coremark/matmul_32b_float improvements) #2372
Some updates for easier Embench runs (+ some coremark/matmul_32b_float improvements) #2372
Conversation
Signed-off-by: Pascal Gouedo <pascal.gouedo@dolphin.fr>
…rmances. Signed-off-by: Pascal Gouedo <pascal.gouedo@dolphin.fr>
Replacement of TB timer by internal cycle counter to be platform independent. Signed-off-by: Pascal Gouedo <pascal.gouedo@dolphin.fr>
Signed-off-by: Pascal Gouedo <pascal.gouedo@dolphin.fr>
Signed-off-by: Pascal Gouedo <pascal.gouedo@dolphin.fr>
Additional updates to run multiple Embench runs in parallel (different configs, sequential or parallel mode). Signed-off-by: Pascal Gouedo <pascal.gouedo@dolphin.fr>
…mances. Signed-off-by: Pascal Gouedo <pascal.gouedo@dolphin.fr>
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.
LGTM. Only two files are non-embench-test specific (mk/Common.mk
and mk/uvmt/uvmt.mk
) and these changes look good.
I will defer to others who are more familiar with Embench to give the final approval and merge. Unfortunately, that is probably Marton Teilgard who is no longer at SiLabs. @silabs-hfegran, can you have a look?
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.
Looks good to me, but I have one question wrt. random bus stalls that I would like answered before approving this PR
@silabs-hfegran said:
Did @pascalgouedo answer your question Henrik? |
Hi @MikeOpenHWGroup |
No description provided.