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

feat(mpz-common): async sync primitives #152

Merged
merged 4 commits into from
Jun 11, 2024

Conversation

sinui0
Copy link
Collaborator

@sinui0 sinui0 commented Jun 6, 2024

This PR adds async synchronization primitives. I learned that the shared KOS variant needs to be able to hold a lock across an await point.

@sinui0 sinui0 requested review from themighty1 and th4s June 6, 2024 17:47
Copy link
Member

@th4s th4s left a comment

Choose a reason for hiding this comment

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

nice 👍

@sinui0
Copy link
Collaborator Author

sinui0 commented Jun 10, 2024

@themighty1 as per our conversation, i've updated the syncer tests to poll the order more explicitly.

Copy link
Collaborator

@themighty1 themighty1 left a comment

Choose a reason for hiding this comment

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

thanks, nice work!

@sinui0 sinui0 merged commit 5a20d29 into threading-refactor Jun 11, 2024
@sinui0 sinui0 deleted the feature/async-mutex branch June 11, 2024 06:07
sinui0 added a commit that referenced this pull request Jun 11, 2024
commit 63a586d
Author: sinu.eth <[email protected]>
Date:   Mon Jun 10 23:09:01 2024 -0700

    feat(mpz-ot): impl more OT traits on shared KOS (#153)

commit 5a20d29
Author: sinu.eth <[email protected]>
Date:   Mon Jun 10 23:07:17 2024 -0700

    feat(mpz-common): async sync primitives (#152)

    * feat(mpz-common): async sync primitives

    * update syncer test

    * add unsync lock method

    * update async syncer test

commit 86eebbf
Author: sinu.eth <[email protected]>
Date:   Fri Jun 7 09:28:05 2024 -0700

    feat(mpz-common): add type alias for test st executor (#154)
sinui0 added a commit that referenced this pull request Jun 25, 2024
* feat(mpz-common): async sync primitives

* update syncer test

* add unsync lock method

* update async syncer test
sinui0 added a commit that referenced this pull request Jun 25, 2024
* feat(mpz-common): async sync primitives

* update syncer test

* add unsync lock method

* update async syncer test
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.

3 participants