Ownasaurus / TAStm32

Console replay device utilizing a STM32F4
GNU General Public License v3.0
66 stars 14 forks source link

Add --cpu-temp-fix flag to fix high CPU usage #60

Closed ghost closed 3 years ago

ghost commented 3 years ago

Add a --cpu-temp-fix flag that is used as a temporary fix to deal with a tight loop in the main_loop function until a better solution can be found. The --cpu-temp-fix flag adds a 10ms wait time at the end of the while loop to slow the loop down enough to use a small fraction of the CPU usage it was using before. As the --cpu-temp-fix flag is only a temporary workaround to the high CPU usage issue, it is not known at this moment if having the flag set by default will ruin any payloads that rely on very high data rates, so it is not set by default. It is recommended for most runs to set the --cpu-temp-fix flag.

Ownasaurus commented 3 years ago

Thanks for addressing this concern @ViGrey. This PR inspired me to come up with a permanent solution that works in all test cases. Therefore there is no need for this flag or hardcoded wait anymore. Resolved in 4482dc3. Enjoy!