Skip to content

Commit

Permalink
Explicitly state the signal update is AMO
Browse files Browse the repository at this point in the history
Based on recent review comments, it looks like it would be more
clear if we state that the signal update is an atomic operation.
We have added this as part of the Notes to Implementers section.
  • Loading branch information
naveen-rn committed Jan 8, 2019
1 parent 5376cdf commit de10f79
Showing 1 changed file with 7 additions and 0 deletions.
7 changes: 7 additions & 0 deletions content/shmem_put_signal.tex
Original file line number Diff line number Diff line change
Expand Up @@ -77,6 +77,13 @@
routine does not imply delivery of the put routine's data.
}

\apiimpnotes{
Implementations must ensure that put-with-signal routines are compatible
with all point-to-point synchronization interfaces. The delivery of
\signal{} flag on the remote \ac{PE} must not cause partial updates. This
requires the update on \signal{} flag to be an atomic memory operation.
}

\begin{apiexamples}

\apicexample
Expand Down

0 comments on commit de10f79

Please sign in to comment.