Skip to content

fix(serial): deadlock on tx failure (EPROT-66) - #170

Open
QSmally wants to merge 1 commit into
espressif:mainfrom
QSmally:tx-failure-deadlock
Open

fix(serial): deadlock on tx failure (EPROT-66)#170
QSmally wants to merge 1 commit into
espressif:mainfrom
QSmally:tx-failure-deadlock

Conversation

@QSmally

@QSmally QSmally commented Apr 20, 2026

Copy link
Copy Markdown

This patches a deadlock on bus_sema_handle when a tx failure happens, which returns early before releasing the lock.

Tested using the master example on a development board, using injected failures. ESP-IDF v5.4.

Checklist

  • 🚨 This PR does not introduce breaking changes.
  • All CI checks (GH Actions) pass.
  • Documentation is updated as needed.
  • Tests are updated or added as necessary.
  • Code is well-commented, especially in complex areas.
  • Git history is clean — commits are squashed to the minimum necessary.

@CLAassistant

CLAassistant commented Apr 20, 2026

Copy link
Copy Markdown

CLA assistant check
All committers have signed the CLA.

@github-actions github-actions Bot changed the title fix(serial): deadlock on tx failure fix(serial): deadlock on tx failure (EPROT-66) Apr 20, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants