dangernoodle-io / breadboard / 28531755928

100%

push

github

feat: route per-task TWDT subscribe + feed through bb_task_registry (opts + token) (#638)

- add bb_task_registry_opts_t (hw_wdt_subscribe) + bb_task_registry_token_t
  (index/generation) to bb_task_registry_register(); register() performs
  best-effort esp_task_wdt_add(handle) under BB_TASK_REGISTRY_WDT_SUPPORT
  (Kconfig, default y, depends on ESP_TASK_WDT_EN), never failing registration
- deregister() unsubscribes the hw wdt and bumps the slot's generation before
  freeing it, invalidating any outstanding token for that slot
- add lock-free bb_task_registry_feed(token): generation-checked, self-feed
  only (the hw esp_task_wdt_reset() pass-through always resets the CALLING
  task's entry; feed() guards it on ESP-IDF by comparing the token owner's
  handle against xTaskGetCurrentTaskHandle() and warns on a cross-task call
  instead of silently feeding the wrong task's wdt entry)
- generation is published/observed via a release/acquire seqlock pairing:
  register()/deregister() release-store generation after the entry payload
  (handle, wdt_subscribed, stack_budget_bytes) is fully written; feed()
  acquire-loads generation before reading that payload
- every slot's generation is initialized (and reset) to 1, never 0, so a
  zero-initialized/default token can never alias a live slot 0
- register() warns and skips the hw subscribe (leaving wdt_subscribed=false)
  when hw_wdt_subscribe is requested with a NULL handle, instead of a silent
  no-op
- last-feed bookkeeping moved from a 64-bit atomic to a 32-bit atomic
  (lock-free on Xtensa, unlike 64-bit); it is a relative timestamp consumed
  via wrap-safe delta, not an exposed absolute timestamp
- migrate all 19 register() call sites; 18 pass NULL/NULL (no behavior
  change); bb_timer_disp now registers with hw_wdt_subscribe gated by the
  existing CONFIG_BB_TIMER_DISP_WDT_ENABLE and feeds via its own token,
  replacing its self bb_wdt_task_subscribe()/bb_wdt_task_feed() calls
- extend BB_TASK_REGISTRY_T... (continued)

1848 of 1848 branches covered (100.0%)

Branch coverage included in aggregate %.

3186 of 3186 relevant lines covered (100.0%)

1268.86 hits per line