Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Solved Problem
Lack of
reboot
function on SITLThere exists a
shutdown
that does a rather dirty exit()Current state of the change
px4_sitl_default
)tracer.hpp
that was helpful for debugger resource management issuesSolution
A no-hack solution where everything is properly deinited()
It means an unfortunately tentacular in order to achieve the graal: every resource used is then freed()
Non exhaustive list of changes that were necessary:
Pxh
ModuleManager
LockstepScheduler
driver when the SITL module is stoppedWorkItem
removal mechanism. I strongly suspect the existing implementation have race conditions that can be dramatic (to put things in perspective, race happens on module stop())camera_capture.cpp
. Others?) - It could probably be moved to theWorkItem
framework