-
-
Notifications
You must be signed in to change notification settings - Fork 223
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
Performance updates #97
Conversation
helpers/helpers.sh
Outdated
# | ||
# Author(s): Michael Messner, Pascal Eckmann | ||
|
||
# Description: Multiple usefull helpers |
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.
Typo: useful
TEMP_PIDS=() | ||
# check for really running PIDs and re-create the array | ||
for PID in ${WAIT_PIDS[*]}; do | ||
#print_output "[*] max pid protection: ${#WAIT_PIDS[@]}" |
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.
Please delete all the comments lines with printing statements.
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.
lets leave it here for further debugging in the next days
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.
Code looks fine to me.
Multiple speed improvements. Emba now takes all the cores:
Started last week with the runtime:
[!] Test ended on Sa 17. Apr 15:11:05 CEST 2021 and took about 01:24:07
Current runtime:
[!] Test ended on Mi 21. Apr 15:37:30 CEST 2021 and took about 00:29:10
There are multiple areas that we can improve in future. Currently a lot of modules are not finally optimised and are not running on multiple cores.
Additionally I have started a Strg-C catcher. So, that we are able to cleanup a bit if we kill emba. Currently it is always a mess.