diff options
| author | Frédéric Desbiens <[email protected]> | 2026-09-10 09:06:13 -0400 |
|---|---|---|
| committer | GitHub <[email protected]> | 2026-09-10 09:06:13 -0400 |
| commit | 42f1d2bcf0d5946b008bacdda09ce0f8ce115a3d (patch) | |
| tree | d90b3304b6ed3a2bca5765d54f4cc83c4bb163d3 | |
| parent | 39277cf026e3a235375529fefcddf279f0c8b6fe (diff) | |
Pointed the SMP coverage comment at the pull request that made its measurements (#719)
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) <[email protected]>
| -rw-r--r-- | .github/workflows/regression_test.yml | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/.github/workflows/regression_test.yml b/.github/workflows/regression_test.yml index 014ceec9..0de8aa82 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: |