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

Roadmap #3

Open
1 of 3 tasks
reconbot opened this issue Jan 5, 2021 · 0 comments
Open
1 of 3 tasks

Roadmap #3

reconbot opened this issue Jan 5, 2021 · 0 comments

Comments

@reconbot
Copy link
Member

reconbot commented Jan 5, 2021

Very subject to change, I'm just playing around right now but this feels very possible in ways it hasn't before.

A rust based bining following the abstract binding api that be be used instead of the bindings package today. I'm considering the new bindings api because I think it will be easier to make, but the usage of serialport/stream is so high that this has to work with it eventually.

Must haves

  • N-API for electron and new node versions
  • as performant as the current implementation for reads and writes
  • prebuilt binaries

Nice to haves

  • similar if not improved behavior from serialport/bindings
  • rpi support

Design exploration

A napi.rs based rust bindings that provides a binding class and utility functions. This gives us prebuilt binaries for many platforms and N-API support for electron and worker threads.

The async patterns we need to use here are not very clear to me. I think a mix of serialport-rs and custom async code will get us to the performance metrics. To start we can stick all the serialport-rs code behind tokio's task::spawn_blocking() but this spawns temporary threads for every call which isn't performant for any operation.

The current bindings put all c++ operations on an event loop provided by v8. And put the unix fs read/write operations onto an IO thread pool. For windows the operations are "async" and don't need the thread pool as I understand it. In any case this isn't consistent.

The bindings design has a read and write queue. The current inflight read, inflight write and any administrative function (eg, not a read or write) can all be inflight at the same time being worked by the event loop or by the thread pool. I wonder if drain blocks the event loop in our current setup?

What is blocking?

  • Reads
  • Writes
  • drain
  • open (maybe)

Opening, setting baud rates and other com settings don't appear to be blocking. This isn't a comprehensive list. Tokio claims to be able to do async io on unix platforms unclear about windows and unclear about serialports.

Possible designs

Dedicating threads for reads, writes and drain, and having a single tokio thread for administrative functions. Pros, everything has dedicated resources, cons we'll need to provide our own communication channels for reads and writes and wont be able to use tokio for that, possible lower performance without a poller telling us when we can resume reading/writing.

Assuming we can do unix async io for serialports with tokio, we use spawn_blocking for windows serialport reading and reading and writing through serialport-rs. Everything else goes on the default tokio thread pool except drain which we use spawn_blocking as well. Pros, easy setup, easy reasoning about execution, cons possible poor performance for windows.

Assuming we can do unix async io for serialports with tokio, we write an async windows serialport struct for reading and writing. Everything goes on the default tokio thread pool except drain which we use spawn_blocking for the blocking thread_pool. Pros, easy setup, easy reasoning about execution, cons, a lot of work possibly for windows serialport writing/reading.

todo

  • research thread communication methods, maybe I don't need tokio to make this easy ryon is interesting in this space
  • confirm the read and write queue behavior in the bindings spec
  • confirm that spawn_blocking spawns a new thread each time. I see references to that and to a blocking pool and how you can turn a normal thread into a blocking thread. Its just very confusing. it does use a thread pool for blocking operations
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

No branches or pull requests

1 participant