Skip to content
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

Updating watchdog service rate #114

Merged
merged 2 commits into from
Jan 6, 2021
Merged

Conversation

ryan-summers
Copy link
Member

@ryan-summers ryan-summers commented Jan 4, 2021

This PR increases the watchdog interval from 2s -> 4s to account for 1.8s potential in the idle task due to ARP timeouts of the W5500. Additionally, this simplifies the fan self-test to not test low-speed operation of the fans to reduce startup time. Finally, a mutex is shorted in the idle task to make other tasks more responsive.

TODO:

  • Test debug and release binaries on hardware for watchdogs
  • Measure watchdog service interval

This PR fixes #94
This PR fixes #110

@jordens
Copy link
Member

jordens commented Jan 4, 2021

Moving to raw sockets for the w5500 and smoltcp on top of that presumably will also help once we can do that.

@ryan-summers
Copy link
Member Author

ryan-summers commented Jan 6, 2021

I created a test binary that flashed LED3 on the booster mainboard whenever the watchdog was serviced. I confirmed that during MQTT-disconnect, the service interval was slightly below 2s, but very close to it, so it's possible we may have been exceeding it previously. These changes should help alleviate any task stackup-related issues.

Additionally, I ran firmware for ~53 minutes continuously and didn't observe any watchdogs/resets. We'll have to observe stability going forward after these changes.

@ryan-summers ryan-summers merged commit 0a1c340 into develop Jan 6, 2021
@ryan-summers ryan-summers deleted the feature/watchdog-updates branch January 6, 2021 10:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

firmware crash Watchdog triggering
2 participants