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

Cleanup (shared_mutex, named_thread); fix cellVdecClose #5173

Merged
merged 8 commits into from Sep 27, 2018

Conversation

Nekotekina
Copy link
Member

No description provided.

Rewrite to use unsigned arithmetic, shrink to u32
Ensure zero default unlocked state (will need it later)
Inline all public methods, rewrite lock_upgrade()
Remove try_lock_degrade(), lock_degrade() methods
Implement lock_downgrade() (now trivial)
Remove is_reading(), added is_free()
Added reader_lock::try_upgrade()
lf_queue<>: unbound FIFO queue with dynamic linked-list
lf_value<>: concurrently-assignable value readable without locking at the cost of memory (using dynamic linked list)

Add atomic_t<>::compare_exchange
Writable sections ARE supported
Don't clear force_boot on shortcut Load()
Thanks Micaelis#6971
Old class named_thread renamed to old_thread
It's too hard to move in a single commit
@OmarAglan
Copy link

The only user that marge the pr with (WIP) Tag.

@AniLeo
Copy link
Member

AniLeo commented Sep 27, 2018

So?

@Nekotekina
Copy link
Member Author

What's the problem? Regression?

@Nekotekina Nekotekina changed the title [WIP] Cleanup Cleanup (shared_mutex, named_thread); fix cellVdecClose Sep 27, 2018
@OmarAglan
Copy link

@Nekotekina @AniLeo it just for fun :)

@HenningJW
Copy link

It sounded hostile, that's why so many people reacted with thumbs down. But yeah, this happens all the time, communication through text, language barrier etc. Good to see that all was in good spirit.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants