From c4d93f5c31db13719b327533c1a31dd81eeaf20b Mon Sep 17 00:00:00 2001 From: Serge Barral Date: Wed, 7 Aug 2024 10:29:13 +0200 Subject: [PATCH] Disable Loom tests for broadcaster due to Loom bug --- asynchronix/src/ports/output/broadcaster.rs | 16 ++++++++++++++++ 1 file changed, 16 insertions(+) diff --git a/asynchronix/src/ports/output/broadcaster.rs b/asynchronix/src/ports/output/broadcaster.rs index 4300392..f3cc423 100644 --- a/asynchronix/src/ports/output/broadcaster.rs +++ b/asynchronix/src/ports/output/broadcaster.rs @@ -770,6 +770,14 @@ mod tests { ) } + // This tests fails with "Concurrent load and mut accesses" even though the + // `task_list` implementation which triggers it does not use any unsafe. + // This is most certainly related to this Loom bug: + // + // https://github.com/tokio-rs/loom/issues/260 + // + // Disabling until the bug is fixed. + #[ignore] #[test] fn loom_broadcast_basic() { const DEFAULT_PREEMPTION_BOUND: usize = 3; @@ -843,6 +851,14 @@ mod tests { }); } + // This tests fails with "Concurrent load and mut accesses" even though the + // `task_list` implementation which triggers it does not use any unsafe. + // This is most certainly related to this Loom bug: + // + // https://github.com/tokio-rs/loom/issues/260 + // + // Disabling until the bug is fixed. + #[ignore] #[test] fn loom_broadcast_spurious() { const DEFAULT_PREEMPTION_BOUND: usize = 3;