Skip to content
Snippets Groups Projects

RFC: Proper Mutexes

Merged rahix requested to merge rahix/mutex into master
All threads resolved!

This MR is the first step towards tackling the locking issues which are plaguing the firmware right now. Commit 82fe0cd7 introduces a new mutex API and the rest of the commits port a few select mutex-uses to it. Those are:

  • Lifecycle/Core 1 mutex
  • Sensor streams
  • API dispatcher lock
  • FAT filesystem

The rationale behind the new mutex design is detailed in the docs added as part of this MR: mutex.rst. The API can be found in mutex.h.

Cc: @schneider

Edited by schneider

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • schneider
  • schneider
  • schneider
  • rahix added 12 commits

    added 12 commits

    • cb86e6f5...c7e40e27 - 7 commits from branch master
    • 562255b8 - feat(epicardium): Add a proper mutex implementation
    • a6f9606d - chore(lifecycle): Port to new mutex API
    • 1ca26b6c - chore(sensor-streams): Port to new mutex API
    • ff7d8eb2 - chore(api-lock): Port to new mutex API
    • e788238f - chore(fatfs): Port to new mutex API

    Compare with previous version

  • rahix added 5 commits

    added 5 commits

    • 9a5a46cd - feat(epicardium): Add a proper mutex implementation
    • cc5f0e29 - chore(lifecycle): Port to new mutex API
    • 44ea81f2 - chore(sensor-streams): Port to new mutex API
    • 75278836 - chore(api-lock): Port to new mutex API
    • 86c7418a - chore(fatfs): Port to new mutex API

    Compare with previous version

  • schneider resolved all threads

    resolved all threads

  • schneider unmarked as a Work In Progress

    unmarked as a Work In Progress

  • merged

  • schneider mentioned in commit a4fefd7e

    mentioned in commit a4fefd7e

  • Please register or sign in to reply
    Loading