Skip to content

Commit

Permalink
Rename async_channel to sync_channel (#1078)
Browse files Browse the repository at this point in the history
The previous `async_channel` was asynchronous in the sense
that it had an internal buffer that allowed a sender and receiver
to not have to synchronously rendezvous in order to exchange
a message/data. However, it was not asynchronous in the sense
of language-level async/await, so the naming would be confusing. 

We rename that channel implementation to `sync_channel`, 
which is more accurate in the sense that Senders and Receivers 
interact with it synchronously (not using async/await), which means
the current task may be blocked when waiting on a send or receive 
operation. 

This is in preparation for an upcoming PR that will introduce a channel
that supports language-level async/await syntax. 

Signed-off-by: Klimenty Tsoutsman <klim@tsoutsman.com>
  • Loading branch information
tsoutsman authored Dec 5, 2023
1 parent f2fe78a commit 92295d7
Show file tree
Hide file tree
Showing 16 changed files with 174 additions and 174 deletions.
40 changes: 20 additions & 20 deletions Cargo.lock

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

4 changes: 2 additions & 2 deletions applications/bm/Cargo.toml
Original file line number Diff line number Diff line change
Expand Up @@ -53,8 +53,8 @@ path = "../../kernel/memory"
[dependencies.rendezvous]
path = "../../kernel/rendezvous"

[dependencies.async_channel]
path = "../../kernel/async_channel"
[dependencies.sync_channel]
path = "../../kernel/sync_channel"

[dependencies.simple_ipc]
path = "../../kernel/simple_ipc"
Expand Down
Loading

0 comments on commit 92295d7

Please sign in to comment.