refac(melpomene): run TCP serial driver on Tokio #20
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.
This branch rewrites the TCP serial port simulated driver to run in
Melpomene's Tokio runtime, rather than spawning its own thread. This
should make Melpomene a bit more efficient, since the runtime thread can
run other simulated drivers while the serial driver task is waiting for
IO, instead of doing a bunch of little sleeps.
I think this shouldn't result in any real behavioral change, although
I haven't tested it super extensively. If I open
crowtty
and then killit after a while, Melpomene still sees the TCP stream
disconnection...but I wasn't sure how I was supposed to actually start
the simulated serial port after launching
crowtty
? Any suggestions onfurther testing would be appreciated! :)