Skip to content

core: priority inheritance protocol with mutex overlap support#7461

Closed
bergzand wants to merge 11 commits intoRIOT-OS:masterfrom
bergzand:mutex_prioinvoverlap
Closed

core: priority inheritance protocol with mutex overlap support#7461
bergzand wants to merge 11 commits intoRIOT-OS:masterfrom
bergzand:mutex_prioinvoverlap

Conversation

@bergzand
Copy link
Copy Markdown
Member

@bergzand bergzand commented Aug 8, 2017

This is mostly an academic exercise to see if it is possible to pass the test from #7455.

Each mutex stores the highest priority among tasks waiting in the mutex queue.
By building a linked list of all locked mutexes, it is possible for a thread, when unlocking a mutex, to restore its priority to the highest priority among the locked mutexes (and its own configured priority).

The test from #7455 somehow still fails, but I don't see immediately why, the output looks correct to me 😃.

@bergzand bergzand changed the title Mutex prioinvoverlap core: priority inheritance protocol with mutex overlap support Aug 8, 2017
@miri64 miri64 requested a review from kaspar030 October 28, 2017 15:10
@miri64 miri64 added Area: core Area: RIOT kernel. Handle PRs marked with this with care! Impact: major The PR changes a significant part of the code base. It should be reviewed carefully Type: new feature The issue requests / The PR implemements a new feature for RIOT labels Oct 28, 2017
@miri64 miri64 added the Process: needs >1 ACK Integration Process: This PR requires more than one ACK label Oct 5, 2018
@stale
Copy link
Copy Markdown

stale bot commented Aug 10, 2019

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. If you want me to ignore this issue, please mark it with the "State: don't stale" label. Thank you for your contributions.

@stale stale bot added the State: stale State: The issue / PR has no activity for >185 days label Aug 10, 2019
@stale stale bot closed this Sep 10, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Area: core Area: RIOT kernel. Handle PRs marked with this with care! Impact: major The PR changes a significant part of the code base. It should be reviewed carefully Process: needs >1 ACK Integration Process: This PR requires more than one ACK State: stale State: The issue / PR has no activity for >185 days Type: new feature The issue requests / The PR implemements a new feature for RIOT

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants