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
Executor work #53
Executor work #53
Conversation
0469fe7
to
e41926a
Compare
df7f9fb
to
566c408
Compare
bastion-executor/src/lib.rs
Outdated
#![feature(extern_types)] | ||
#![feature(core_intrinsics)] | ||
#![feature(libstd_sys_internals)] | ||
#![feature(thread_local)] | ||
#![feature(const_fn)] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can be removed ;)
#[cfg(target_os = "windows")] | ||
extern crate kernel32; | ||
#[cfg(target_os = "windows")] | ||
extern crate winapi; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can probably be removed
bastion-executor/src/run.rs
Outdated
let future = async move { | ||
future.await; | ||
}; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can be removed ;)
bastion-executor/src/run.rs
Outdated
// Wrap the future into one that drops task-local variables on exit. | ||
// let future = task_local::add_finalizer(future); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can this be removed? :)
@@ -0,0 +1 @@ | |||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This file's empty ;)
let mut stealers = Vec::<Stealer<LightProc>>::new(); | ||
|
||
for core in self.cores { | ||
self.round = core.id; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Distributor.round
doesn't seem to be used anywhere else
We are approximately ready to glue bastion + bastion-executor + lightproc. |
Bastion's Executor Development
https://paper.dropbox.com/doc/Bastion-Executor-Design-Document--AnZMiua4Egf6vE_pXCH_2RGQAg-ged3kQPsVFNOIwuhgkeIY
Fixes #48