EIVE System Reboot Logic #762

New Issue

meggert · 2023-07-31T10:45:43+02:00

meggert commented

2023-07-31 10:45:43 +02:00

The error handling within EIVE System leading to a reboot calls the sendFullRebootCommand() function. This function will not only reboot, but boot onto the image 0 0. I don't think this is actually what we want here. A reboot self should be sufficient, especially given that our image 0 0 will lag behind for the foreseeable future.

Maybe we should also add a delay between triggering the reboot event and the actual reboot, to ensure that the event can be triggered and stored before actaully rebooting.

The error handling within [EIVE System](https://egit.irs.uni-stuttgart.de/eive/eive-obsw/src/branch/main/mission/system/EiveSystem.cpp) leading to a reboot calls the `sendFullRebootCommand()` function. This function will not only reboot, but boot onto the image 0 0. I don't think this is actually what we want here. A reboot self should be sufficient, especially given that our image 0 0 will lag behind for the foreseeable future. Maybe we should also add a delay between triggering the reboot event and the actual reboot, to ensure that the event can be triggered and stored before actaully rebooting.

muellerr was assigned by meggert

2023-07-31 10:45:43 +02:00

muellerr commented

2023-08-01 09:06:11 +02:00

I think reboot self is better here as well. I would keep the reboot until the PDEC issue is understood bette. A delay sounds reasonable.