dangernoodle-io / breadboard / 26134911361

100%

push

github

fix(bb_update_check): add set_task_priority() to escape mining starvation on shared cores (#278)

PR #276 made the bb_update_check worker task default to Core 1 to keep
the mbedTLS handshake off the core carrying lwip/wifi/httpd. On boards
where Core 1 also runs a high-priority CPU-bound task (e.g. TaipanMiner
tdongle-s3 mining at prio 20), the worker at prio 1 is starved — the
kick semaphore wakes it but it never gets CPU, so it never reaches the
pause hook that would suspend the mining task.

Add `bb_update_check_set_task_priority(int)` (default 1) so consumers
can raise the worker priority above the contending task on the same
core. Once kicked, the worker preempts, calls mining_pause(), and the
pause hook cleanly suspends mining for the duration of the fetch.

Mirrors `bb_update_check_set_task_core`'s semantics: must be called
before bb_update_check_register_init, no-op after. Host stub mirrors
set_task_core's no-op pattern, exercised by a new coverage test.

Co-authored-by: Claude Opus 4.7 <noreply@anthropic.com>

1000 of 1000 branches covered (100.0%)

Branch coverage included in aggregate %.

2 of 2 new or added lines in 1 file covered. (100.0%)

1797 of 1797 relevant lines covered (100.0%)

999.63 hits per line