From f92996f16b75d128458e543ddb1f4f5b1790a5f5 Mon Sep 17 00:00:00 2001 From: Julian Oes Date: Wed, 1 Jul 2026 15:16:45 +1200 Subject: [PATCH] test: document why 'sensors status' is disabled in test_imu_filtering Replace the terse "triggers TSAN" note with a TODO explaining that 'sensors status' reads sensor module state from the shell thread without synchronization, so it should be re-enabled once that race is fixed. Signed-off-by: Julian Oes --- posix-configs/SITL/init/test/test_imu_filtering | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/posix-configs/SITL/init/test/test_imu_filtering b/posix-configs/SITL/init/test/test_imu_filtering index 7fe127c6d5e..ff2a5cdbbe0 100644 --- a/posix-configs/SITL/init/test/test_imu_filtering +++ b/posix-configs/SITL/init/test/test_imu_filtering @@ -51,7 +51,9 @@ sleep 10 logger off -#sensors status # triggers TSAN +# TODO: re-enable once the data race hit by 'sensors status' is fixed (it reads +# sensor module state from the shell thread without synchronization, which TSAN flags). +#sensors status listener sensor_gyro listener sensor_gyro_fifo listener sensor_gyro_fft