Skip to content

Commit e9f3ee4

Browse files
committed
---
yaml --- r: 142643 b: refs/heads/try2 c: ddbccec h: refs/heads/master i: 142641: a46db5b 142639: f159364 v: v3
1 parent 1b9f3be commit e9f3ee4

File tree

3 files changed

+19
-21
lines changed

3 files changed

+19
-21
lines changed

[refs]

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,7 @@ refs/heads/snap-stage3: 78a7676898d9f80ab540c6df5d4c9ce35bb50463
55
refs/heads/try: 519addf6277dbafccbb4159db4b710c37eaa2ec5
66
refs/tags/release-0.1: 1f5c5126e96c79d22cb7862f75304136e204f105
77
refs/heads/ndm: f3868061cd7988080c30d6d5bf352a5a5fe2460b
8-
refs/heads/try2: e65d0cbabebc73f2c9733a7ed158576c9702e71e
8+
refs/heads/try2: ddbccecc27e7a6be144c9c55d25a5b56cac0a179
99
refs/heads/dist-snap: ba4081a5a8573875fed17545846f6f6902c8ba8d
1010
refs/tags/release-0.2: c870d2dffb391e14efb05aa27898f1f6333a9596
1111
refs/tags/release-0.3: b5f0d0f648d9a6153664837026ba1be43d3e2503

branches/try2/src/libstd/rt/mod.rs

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -267,11 +267,10 @@ pub fn run(main: ~fn()) -> int {
267267
};
268268

269269
// Create and enqueue the main task.
270-
let main_cell = Cell::new(main);
271270
let mut new_task = ~Task::new_root();
272271
new_task.on_exit = Some(on_exit);
273272
let main_task = ~Coroutine::with_task(&mut scheds[0].stack_pool,
274-
new_task, main_cell.take());
273+
new_task, main);
275274
scheds[0].enqueue_task(main_task);
276275

277276
// Run each scheduler in a thread.

branches/try2/src/libstd/rt/sched.rs

Lines changed: 17 additions & 18 deletions
Original file line numberDiff line numberDiff line change
@@ -177,7 +177,7 @@ impl Scheduler {
177177

178178
rtdebug!("run taking sched");
179179
let sched = Local::take::<Scheduler>();
180-
// XXX: Reenable this once we're using a per-task queue. With a shared
180+
// XXX: Reenable this once we're using a per-scheduler queue. With a shared
181181
// queue this is not true
182182
//assert!(sched.work_queue.is_empty());
183183
rtdebug!("scheduler metrics: %s\n", {
@@ -213,29 +213,29 @@ impl Scheduler {
213213
if sched.resume_task_from_queue() {
214214
// We performed a scheduling action. There may be other work
215215
// to do yet, so let's try again later.
216-
let mut sched = Local::take::<Scheduler>();
217-
sched.metrics.tasks_resumed_from_queue += 1;
218-
sched.event_loop.callback(Scheduler::run_sched_once);
219-
Local::put(sched);
216+
do Local::borrow::<Scheduler, ()> |sched| {
217+
sched.metrics.tasks_resumed_from_queue += 1;
218+
sched.event_loop.callback(Scheduler::run_sched_once);
219+
}
220220
return;
221221
}
222222

223223
// If we got here then there was no work to do.
224224
// Generate a SchedHandle and push it to the sleeper list so
225225
// somebody can wake us up later.
226226
rtdebug!("no work to do");
227-
let mut sched = Local::take::<Scheduler>();
228-
sched.metrics.wasted_turns += 1;
229-
if !sched.sleepy && !sched.no_sleep {
230-
rtdebug!("sleeping");
231-
sched.metrics.sleepy_times += 1;
232-
sched.sleepy = true;
233-
let handle = sched.make_handle();
234-
sched.sleeper_list.push(handle);
235-
} else {
236-
rtdebug!("not sleeping");
227+
do Local::borrow::<Scheduler, ()> |sched| {
228+
sched.metrics.wasted_turns += 1;
229+
if !sched.sleepy && !sched.no_sleep {
230+
rtdebug!("sleeping");
231+
sched.metrics.sleepy_times += 1;
232+
sched.sleepy = true;
233+
let handle = sched.make_handle();
234+
sched.sleeper_list.push(handle);
235+
} else {
236+
rtdebug!("not sleeping");
237+
}
237238
}
238-
Local::put(sched);
239239
}
240240

241241
pub fn make_handle(&mut self) -> SchedHandle {
@@ -462,8 +462,7 @@ impl Scheduler {
462462
// here we know we are home, execute now OR we know we
463463
// aren't homed, and that this sched doesn't care
464464
do this.switch_running_tasks_and_then(task) |sched, last_task| {
465-
let last_task = Cell::new(last_task);
466-
sched.enqueue_task(last_task.take());
465+
sched.enqueue_task(last_task);
467466
}
468467
} else if !homed && !this.run_anything {
469468
// the task isn't homed, but it can't be run here

0 commit comments

Comments
 (0)