Skip to content

Commit 3224fa4

Browse files
committed
---
yaml --- r: 105892 b: refs/heads/auto c: 91bed14 h: refs/heads/master v: v3
1 parent b894a76 commit 3224fa4

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
@@ -13,7 +13,7 @@ refs/heads/try3: 9387340aab40a73e8424c48fd42f0c521a4875c0
1313
refs/tags/release-0.3.1: 495bae036dfe5ec6ceafd3312b4dca48741e845b
1414
refs/tags/release-0.4: e828ea2080499553b97dfe33b3f4d472b4562ad7
1515
refs/tags/release-0.5: 7e3bcfbf21278251ee936ad53e92e9b719702d73
16-
refs/heads/auto: 80f92f5c5fedadd131842977c0b9b21806f3902f
16+
refs/heads/auto: 91bed14ca8085887a26d029d785d853ad2587718
1717
refs/heads/servo: af82457af293e2a842ba6b7759b70288da276167
1818
refs/tags/release-0.6: b4ebcfa1812664df5e142f0134a5faea3918544c
1919
refs/tags/0.1: b19db808c2793fe2976759b85a355c3ad8c8b336

branches/auto/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)