Skip to content

Commit 1db8540

Browse files
committed
increase timing slack for sync tests
1 parent 3d5a516 commit 1db8540

File tree

1 file changed

+6
-4
lines changed
  • src/tools/miri/tests/pass/concurrency

1 file changed

+6
-4
lines changed

src/tools/miri/tests/pass/concurrency/sync.rs

Lines changed: 6 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -201,8 +201,9 @@ fn park_timeout() {
201201
thread::park_timeout(Duration::from_millis(200));
202202
// Normally, waiting in park/park_timeout may spuriously wake up early, but we
203203
// know Miri's timed synchronization primitives do not do that.
204-
205-
assert!((200..1000).contains(&start.elapsed().as_millis()));
204+
// We allow much longer sleeps as well since the macOS GHA runners seem very oversubscribed
205+
// and sometimes just pause for 1 second or more.
206+
assert!((200..2000).contains(&start.elapsed().as_millis()));
206207
}
207208

208209
fn park_unpark() {
@@ -219,8 +220,9 @@ fn park_unpark() {
219220
thread::park();
220221
// Normally, waiting in park/park_timeout may spuriously wake up early, but we
221222
// know Miri's timed synchronization primitives do not do that.
222-
223-
assert!((200..1000).contains(&start.elapsed().as_millis()));
223+
// We allow much longer sleeps as well since the macOS GHA runners seem very oversubscribed
224+
// and sometimes just pause for 1 second or more.
225+
assert!((200..2000).contains(&start.elapsed().as_millis()));
224226

225227
t2.join().unwrap();
226228
}

0 commit comments

Comments
 (0)