-
Notifications
You must be signed in to change notification settings - Fork 41
RMA WG 01 17 2019
James Dinan edited this page Jan 17, 2019
·
1 revision
- Wait/test changes to put-with-signal proposal (Naveen)
- Not recorded
- None
- None
- Latest update on the pt-to-pt synchronization issue under the memory model umbrella : 1) introduce shmem_signal which is atomic wrt itself and shmem_wait_until but not wrt other shmem_atomic_ operations; 2) specify shmem_wait_until to be atomic wrt both shmem_signal and shmem_atomic_ operations; 3) shmem_signal and shmem_atomic_ can be used for pt-to-pt synchronization
- There is concern that shmem_wait_until performance will be badly affected because it has to go through network atomics for some platforms
- Khaled thinks that there is no OpenSHMEM implementation that does network atomics in shmem_wait_until but applications still work so adding this restriction on shmem_wait_until is not needed
- The note to implementors in shmem_wait_until prohibits partial update so current implementations not using network atomics suggests that the implementors note is ignored : should the implementors note be removed?
- Khaled suggested that making the signal 1 byte wide will ensure that partial updates are not seen because no known implementation splits a message at less than 1 byte : the problem there is 1 byte signal is restrictive for something like continuously increasing counter used as signal
- Khaled and Manju thinks there is value in allowing partial updates to be seen by shmem_wait_until for an usecase where the signal is just used to communicate arrival of the initiator PE : proposal for two variants of shmem_wait came up; 1) shmem_wait can observe partial updates 2) shmem_wait_signal cannot observe partial updates
- The change in shmem_wait_until to make it atomic wrt to shmem_signal and shmem_atomic_ affects the shmem_waitsome/testsome proposal by Dave : what happens to the scheduled reading in next specification meeting?
- Naveen and Bob are fine with the proposed changes in shmem_wait_until and introduction of shmem_signal : shmem_put_with_signal to rely on shmem_signal in that case
- Manju brought up the similarity that the modified shmem_wait_until has to shmem_atomic_fetch
- Khaled mentioned that user feedback is needed before changing shmem_wait_until
-
Working Groups
-
Errata