Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

core.thread.osthread: Don't subtract suspend counter if thread has detached itself #14763

Merged
merged 1 commit into from
Dec 31, 2022

Conversation

ibuclaw
Copy link
Member

@ibuclaw ibuclaw commented Dec 30, 2022

Fix FreeBSD failing test suite in master 2 (Electric Boogaloo)

More debugging done in #14754.

@dlang-bot
Copy link
Contributor

dlang-bot commented Dec 30, 2022

Thanks for your pull request, @ibuclaw!

Bugzilla references

Auto-close Bugzilla Severity Description
23593 normal core.thread: suspendAll doesn't wait for all if current thread has detached itself

Testing this PR locally

If you don't have a local development environment setup, you can use Digger to test this PR:

dub run digger -- build "master + dmd#14763"

@ibuclaw ibuclaw requested a review from schveiguy December 30, 2022 21:35
Copy link
Member

@schveiguy schveiguy left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good, let's see if it has any failures. Probably want to rerun the tests a few times.

druntime/src/core/thread/osthread.d Show resolved Hide resolved
@schveiguy
Copy link
Member

If this works, it would be good to have a bug report so it goes in the changelog. I can put one in if you want.

@ibuclaw
Copy link
Member Author

ibuclaw commented Dec 30, 2022

If this works, it would be good to have a bug report so it goes in the changelog. I can put one in if you want.

Go for it!

@ibuclaw ibuclaw changed the title core.thread.osthread: Don't substract suspend counter if thread has detached itself core.thread.osthread: Don't subtract suspend counter if thread has detached itself Dec 30, 2022
@ibuclaw ibuclaw force-pushed the fix-freebsd2 branch 2 times, most recently from 5a1648d to 42e4d4d Compare December 30, 2022 23:54
…rent thread has detached itself.

Don't subtract suspend counter if thread has detached itself.
@ibuclaw
Copy link
Member Author

ibuclaw commented Dec 31, 2022

I'll run the testsuite one more time before hitting auto-merge.

@dlang-bot dlang-bot merged commit 7560d12 into dlang:master Dec 31, 2022
@ibuclaw ibuclaw deleted the fix-freebsd2 branch December 31, 2022 01:13
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