Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

tests: add test for priority inversion using overlapping mutexes #7455

Closed

Conversation

haukepetersen
Copy link
Contributor

rebased on #7445

This PR adds an additional test case for demonstrating priority inversion when using overlapping mutexes. As I am not sure (yet), how theoretical such a use case is in practice, I decided to put it in its own PR.

As pointed out in the comments for #7445, the solution for priority inheritance as proposed there does not work for this test case. So either I/we need to re-think #7445, or we decide that this overlapping case is not of our concern?!

@haukepetersen haukepetersen added Discussion: RFC The issue/PR is used as a discussion starting point about the item of the issue/PR Area: tests Area: tests and testing framework labels Aug 7, 2017
@haukepetersen haukepetersen added this to the Release 2017.10 milestone Aug 7, 2017
@miri64
Copy link
Member

miri64 commented Nov 15, 2017

Don't think I'm the right person to be assigned to this PR and with @bergzand being a maintainer now, I think it doesn't hurt if I revoke my assignment.

@miri64 miri64 removed their assignment Nov 15, 2017
@smlng
Copy link
Member

smlng commented Jan 15, 2018

depending PR needs work too, postpone

@kaspar030 kaspar030 removed this from the Release 2018.04 milestone Apr 16, 2018
@tcschmidt tcschmidt requested a review from danpetry May 30, 2018 19:34
@danpetry danpetry added the State: waiting for other PR State: The PR requires another PR to be merged first label Jun 7, 2018
@danpetry
Copy link
Contributor

Hi @haukepetersen and @bergzand . Thanks for the nice work on exploring this edge case and potential solution. My initial feeling with this is that it's an edge case, and in fact the original priority inversion issue was largely theoretical - in other words, that we weren't having a problem with even the main issue, let alone edge cases. So I suggest that we just merge the main ones and pass on the edge case ones. In other words, close this issue and #7461 and save ourselves the trouble of working on it further. What do you think.
It will always be there in "closed" status in case we start having more of a problem, and people can search for closed issues on the subject, and subsequently re-open.

@bergzand
Copy link
Member

@danpetry Sure, I agree with your reasoning. Feel free to close #7461

@stale
Copy link

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
@haukepetersen haukepetersen added the State: don't stale State: Tell state-bot to ignore this issue label Aug 12, 2019
@stale stale bot removed the State: stale State: The issue / PR has no activity for >185 days label Aug 12, 2019
@MrKevinWeiss MrKevinWeiss added this to the Release 2021.07 milestone Jun 21, 2021
@MrKevinWeiss MrKevinWeiss removed this from the Release 2021.07 milestone Jul 15, 2021
@Teufelchen1
Copy link
Contributor

@maribu given your solution in #17040, do you think this PR can be closed / reworked?

@maribu
Copy link
Member

maribu commented Jan 23, 2024

Yes, the upstream test does basically the same.

@maribu maribu closed this Jan 23, 2024
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
Area: tests Area: tests and testing framework Discussion: RFC The issue/PR is used as a discussion starting point about the item of the issue/PR State: don't stale State: Tell state-bot to ignore this issue State: waiting for other PR State: The PR requires another PR to be merged first
Projects
None yet
Development

Successfully merging this pull request may close these issues.

9 participants