itsscb/tokio
1
0
Fork 0
mirror of https://github.com/tokio-rs/tokio.git synced 2025-10-01 12:20:39 +00:00
Code Issues Packages Projects Releases Wiki Activity

Merge pull request #253 from alexcrichton/perf

Browse Source 
Conditionally call `consume_queue` on messages
This commit is contained in:
Alex Crichton 2017-09-13 16:33:26 -05:00 committed by GitHub
commit fb05eb34af
1 changed files with 16 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

19
src/reactor/mod.rs
View File

@ -580,9 +580,22 @@ impl Remote {
self.with_loop(|lp| { self.with_loop(|lp| {
match lp { match lp {
Some(lp) => { Some(lp) => {
// Need to execute all existing requests first, to ensure // We want to make sure that all messages are received in
// that our message is processed "in order" // order, so we need to consume pending messages before
lp.consume_queue(); // delivering this message to the core. The actually
// `consume_queue` function, however, can be somewhat slow
// right now where receiving on a channel will acquire a
// lock and block the current task.
//
// To speed this up check the message queue's readiness as a
// sort of preflight check to see if we've actually got any
// messages. This should just involve some atomics and if it
// comes back false then we know for sure there are no
// pending messages, so we can immediately deliver our
// message.
if lp.rx_readiness.0.readiness().is_readable() {
lp.consume_queue();
}
lp.notify(msg); lp.notify(msg);
} }
None => { None => {