adam/Alepha
1
0
Fork 0
forked from Alepha/Alepha
Code Pull Requests Activity

Fix stalls due to lacking custom terminal notifications.

Browse Source 
`Dropbox` and `Mailbox` should be roughly the same interface, but
with different stall semantics.  However, terminal operations don't
wait.  `finish( custom )` should be a handoff without wait.

`Dropbox`'s primary thread-coordination works in terms of an
acquire/release semantic whereby `release` doesn't block, so this
naturally works.  However, `Mailbox` works in terms of the `Interlock`
primitive which allows for a terminal-case `checkpoint`.  Therefore
`finish( custom )` has to call the `checkpoint` api in `Mailbox`
This commit is contained in:
ADAM David Alan Martin
2024-01-05 23:57:47 -05:00
parent 5a67b8d0ec
commit da34142bf1
2 changed files with 23 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

10
Atomic/Dropbox.h
View File

@ -153,10 +153,18 @@ namespace Alepha::Hydrogen::Atomic ::detail:: Dropbox_m
sendoff(); sendoff();
} }
void
finish( auto newNotification )
{
auto &preparing= drops.producer.acquireBox();
preparing.notification= std::make_exception_ptr( std::move( newNotification ) );
sendoff();
}
void void
finish() finish()
{ {
notify( build_exception< FinishedCondition >( "" ) ); finish( build_exception< FinishedCondition >( "" ) );
} }
}; };
} }

15
Atomic/Mailbox.h
View File

@ -241,6 +241,19 @@ namespace Alepha::Hydrogen::Atomic ::detail:: Mailbox_m
checkpoint( lock ); checkpoint( lock );
} }
/*!
* @brief This function will pump the mailbox, and deliver a custom notification
* event to the other side, without waiting (last step).
*/
void
finish( auto newNotification )
{
auto notification_ptr= std::make_exception_ptr( std::move( newNotification ) );
Alepha::unique_lock lock( access );
notification= std::move( notification_ptr );
checkpoint( lock );
}
/*! /*!
* @brief This function will pump the mailbox, and deliver a custom notification * @brief This function will pump the mailbox, and deliver a custom notification
@ -249,7 +262,7 @@ namespace Alepha::Hydrogen::Atomic ::detail:: Mailbox_m
void void
notify( auto newNotification ) notify( auto newNotification )
{ {
auto notification_ptr= std::make_exception_ptr( std::move( notification ) ); auto notification_ptr= std::make_exception_ptr( std::move( newNotification ) );
Alepha::unique_lock lock( access ); Alepha::unique_lock lock( access );
assertion( not finished ); assertion( not finished );
notification= std::move( notification_ptr ); notification= std::move( notification_ptr );