From de10f796008b7318d7ca42534198f9650087432e Mon Sep 17 00:00:00 2001 From: Naveen Namashivayam Ravichandrasekaran Date: Tue, 8 Jan 2019 12:18:58 -0600 Subject: [PATCH] Explicitly state the signal update is AMO 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. --- content/shmem_put_signal.tex | 7 +++++++ 1 file changed, 7 insertions(+) diff --git a/content/shmem_put_signal.tex b/content/shmem_put_signal.tex index e657a7a00..15f9f0e4a 100644 --- a/content/shmem_put_signal.tex +++ b/content/shmem_put_signal.tex @@ -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