Skip to content

Commit 697f47d

Browse files
committed
---
yaml --- r: 105274 b: refs/heads/master c: 91bed14 h: refs/heads/master v: v3
1 parent e5ec662 commit 697f47d

File tree

2 files changed

+15
-1
lines changed

2 files changed

+15
-1
lines changed

[refs]

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
---
2-
refs/heads/master: 80f92f5c5fedadd131842977c0b9b21806f3902f
2+
refs/heads/master: 91bed14ca8085887a26d029d785d853ad2587718
33
refs/heads/snap-stage1: e33de59e47c5076a89eadeb38f4934f58a3618a6
44
refs/heads/snap-stage3: b8601a3d8b91ad3b653d143307611f2f5c75617e
55
refs/heads/try: db814977d07bd798feb24f6b74c00800ef458a13

trunk/src/libgreen/sched.rs

Lines changed: 14 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -832,6 +832,20 @@ impl Scheduler {
832832
}
833833

834834
pub fn maybe_yield(mut ~self, cur: ~GreenTask) {
835+
// It's possible for sched tasks to possibly call this function, and it
836+
// just means that they're likely sending on channels (which
837+
// occasionally call this function). Sched tasks follow different paths
838+
// when executing yield_now(), which may possibly trip the assertion
839+
// below. For this reason, we just have sched tasks bail out soon.
840+
//
841+
// Sched tasks have no need to yield anyway because as soon as they
842+
// return they'll yield to other threads by falling back to the event
843+
// loop. Additionally, we completely control sched tasks, so we can make
844+
// sure that they never execute more than enough code.
845+
if cur.is_sched() {
846+
return cur.put_with_sched(self)
847+
}
848+
835849
// The number of times to do the yield check before yielding, chosen
836850
// arbitrarily.
837851
rtassert!(self.yield_check_count > 0);

0 commit comments

Comments
 (0)