dangernoodle-io / breadboard / 28341571809

100%

push

github

feat: bb_mdns batch ring via bb_ring drop-new + semaphore wakeup (B1-257) (#590)

replace the hand-rolled single-static-slot dispatch queue (QueueHandle_t
s_evt_queue + tagged-pointer macros + s_batch_item/batch_item_alloc/free)
with:
  - bb_ring_t s_batch_ring — capacity BB_MDNS_BATCH_RING_DEPTH, entry size
    sizeof(bb_mdns_batch_item_t), BB_RING_REJECT_NEW policy
  - SemaphoreHandle_t s_ring_sem — binary semaphore; replaces s_evt_queue
    as the wakeup signal for the dispatch task

batch_do_flush_locked: builds bb_mdns_batch_item_t on stack (txt
pointer-relocation preserved), clears s_batch.count, releases
s_evt_pool_lock, calls bb_ring_push by value. on BB_ERR_NO_SPACE (ring
full) increments drop counter + re-arms flush timer for 50 ms retry.
on BB_OK calls xSemaphoreGive(s_ring_sem).

dispatch task: xSemaphoreTake(s_ring_sem, portMAX_DELAY), then drains all
ring entries (peek_oldest + pop + dispatch_one) inside s_evt_pool_lock,
releasing the lock around each dispatch_one call so the knot
on_peer_discovered callback runs outside the lock (B1-372).

bb_mdns_init: creates s_ring_sem (xSemaphoreCreateBinary) and s_batch_ring
(bb_ring_create_ex), replacing xQueueCreate(s_evt_queue).

add BB_MDNS_BATCH_RING_DEPTH Kconfig (default 4) with heap-cost note
(capacity × (entry meta + sizeof(bb_mdns_batch_item_t)), SPIRAM-preferred
on ESP-IDF; internal heap on no-PSRAM). update BB_MDNS_BATCH_MAX help text.
add bb_ring to bb_mdns PRIV_REQUIRES.

consuming app (TaipanMiner) must add platform/espidf/bb_ring_espidf to
EXTRA_COMPONENT_DIRS for PSRAM routing on bitaxe — no TM change needed here.

no host-side bb_mdns_ring_*_for_test symbols existed at origin/main (all
from discarded 1a701d9); no deletions required. coalesce/dispatch host
tests unchanged (test_bb_mdns_coalesce.c still green, 2787/2787 pass).

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

1786 of 1786 branches covered (100.0%)

Branch coverage included in aggregate %.

3080 of 3080 relevant lines covered (100.0%)

1245.73 hits per line