From 2ba918ccfee1e10fa91548bbc6d5f795d291a884 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Fr=C3=A9d=C3=A9ric=20Desbiens?= Date: Thu, 10 Sep 2026 08:15:50 -0400 Subject: [PATCH] Pointed the SMP coverage comment at the pull request that made its measurements The paragraph explaining why the SMP floor sits at 99 rather than 100 ended with a bare reference that resolves to nothing a reader of this repository can open. It named a source outside the tree in place of one inside it. The real source is #677: it wrote the four tests that closed 53 of the 64 lines, took the measurements the paragraph quotes -- the 180,003 windows with zero handovers, and the three remaining lines in tx_thread_smp_utilities.c -- and raised the floor from 98 to 99. Citing it gives the next reader somewhere to go. This is the same correction #718 made across five other files; this line was missed because it sits in regression_test.yml rather than the template. Comment only; no behaviour change. Assisted-by: Claude Code (Opus 5) --- .github/workflows/regression_test.yml | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/.github/workflows/regression_test.yml b/.github/workflows/regression_test.yml index 014ceec9a..0de8aa822 100644 --- a/.github/workflows/regression_test.yml +++ b/.github/workflows/regression_test.yml @@ -77,7 +77,8 @@ jobs: # three are in tx_thread_smp_utilities.c: one is a range guard placed # after the shift it is meant to guard, so reaching it needs undefined # behaviour, and two are a priority-inheritance branch that needs the - # mutex owner genuinely executing on another core. See T17. + # mutex owner genuinely executing on another core. See #677, which + # made these measurements and raised the floor. coverage_thresholds: '99 100' freertos: permissions: