crownstone / bluenet

Bluenet is the in-house firmware on Crownstone hardware. Functions: switching, dimming, energy monitoring, presence detection, indoor localization, switchcraft.
https://crownstone.rocks
91 stars 62 forks source link

Feature/microapp watchdog #163

Closed mrquincle closed 2 years ago

mrquincle commented 2 years ago

This implements the watchdog feature for microapps: https://github.com/crownstone/bluenet/issues/147

mrquincle commented 2 years ago

It must be checked if other failure modes of a microapp might crash bluenet.

mrquincle commented 2 years ago

https://github.com/crownstone/crownstone-microapp/tree/test/faulty-microapp now contains an illegal instruction. This indeed crashes bluenet without being detected. Will fix.

mrquincle commented 2 years ago

Again reverted to draft.

  1. @vliedel sees no point for a RebootManager as a more general class taking care of storing information across reboots.
  2. This means that all code concerning reboots has to be moved to the microapp class.
  3. Also code/config that sets up the event handler must be reverted because @vliedel sees no need for more information when the watchdog triggers.

Will probably then be after holidays.

mrquincle commented 2 years ago

To not have PRs open, let's just close it. Good luck next time with an implementation. :-)