dangernoodle-io / breadboard / 28488985568

100%

push

github

fix(bb_pub): free store-forward ring after idle to reclaim heap (#621)

- add Kconfig knob BB_PUB_BUFFER_IDLE_FREE_TICKS (default 30, range 0..3600;
  depends on BB_PUB_BUFFER_ENABLE) to control the idle-free threshold
- add s_buffer_idle_ticks counter (inside CONFIG_BB_PUB_BUFFER_ENABLE block)
  incremented in Phase 3 per publish tick when ring is empty; reset to 0
  when ring gains entries; destroy+null under s_tick_lock when threshold
  reached — entire path gated on !buffer_always_on() so BUFFER_ALWAYS=y
  is untouched
- add BB_PUB_TESTING seam bb_pub_test_set_idle_free_ticks(n) so tests can
  trigger the reclaim with n=2 without running 30 ticks
- add #include <inttypes.h> to bb_pub.c for PRIu32 in the log message
- add 5 Unity tests: (a) lazy ring created on first failure, (b) freed
  after N idle ticks post-drain, (c) re-created on subsequent failure,
  (d) BUFFER_ALWAYS=y path not freed, (e) threshold=0 disables reclaim
- 2932/2932 host tests pass; make check exits 0; all 4 smoke envs build

Co-authored-by: Claude Sonnet 4.6 <noreply@anthropic.com>

1850 of 1850 branches covered (100.0%)

Branch coverage included in aggregate %.

3190 of 3190 relevant lines covered (100.0%)

1229.17 hits per line