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

[SHMEM 1.6 Sec 9.10] Collectives section committee changes #535

Closed
Closed
Changes from 1 commit
Commits
Show all changes
25 commits
Select commit Hold shift + click to select a range
6072be8
Deprecate blocks for Collect, Broadcast, update def apireturnvalues
kwaters4 Mar 20, 2024
14c87b8
Deprecate active-set language in Collectives, missing Reductions
kwaters4 Mar 20, 2024
8997a4e
Reductions, Programming Model, strided teams active set langauge depr…
kwaters4 Mar 28, 2024
9fa187b
Indent in shmem_alltoall
kwaters4 Mar 28, 2024
111a29c
Update content/collective_intro.tex
kwaters4 Apr 26, 2024
dda26c6
Update content/shmem_broadcast.tex
kwaters4 Apr 26, 2024
45e710b
Update shmem_reductions.tex
kwaters4 Apr 26, 2024
41a0024
Update shmem_team_split_strided API Note, arbirary to any positive in…
kwaters4 Jul 26, 2024
a6532ef
Fix Whitespace in shmem_alltoall
kwaters4 Jul 26, 2024
c3b23e5
Fix whitespace shmem_broadcast
kwaters4 Jul 26, 2024
a3b9ea7
Edit Whitespace in shmem_collect
kwaters4 Jul 26, 2024
a621dd4
Fix Whitespace in collective_intro
kwaters4 Jul 26, 2024
a2d9daa
Fix Typo in shmem_alltoall
kwaters4 Jul 26, 2024
9a6a048
Merge branch 'openshmem-org:master' into master
kwaters4 Aug 29, 2024
fd98952
Update content/shmem_team_split_strided.tex
kwaters4 Aug 29, 2024
a1e23bd
scan: 488 section committee edits (nelems/overlap)
davidozog Aug 29, 2024
129573e
Update content/shmem_broadcast.tex
kwaters4 Aug 30, 2024
421cc8b
Remove active language in reduction api args
kwaters4 Aug 30, 2024
6573d12
Merge pull request #8 from kwaters4/dep_active_lang
kwaters4 Aug 30, 2024
0b47caf
collectives: clarify src buffer entry requirements
davidozog Aug 30, 2024
de1315d
Remove unnecessary new line
maawad Aug 30, 2024
f1216b9
Merge pull request #11 from davidozog/pr/remove-newline-after-comma
davidozog Aug 30, 2024
6a6ce3f
Merge pull request #9 from davidozog/pr/scan_edits
davidozog Aug 30, 2024
8095ea4
collectives: "array" instead of source "buffer"
davidozog Aug 30, 2024
00213da
Merge pull request #10 from davidozog/pr/src_buffer_readiness
davidozog Aug 30, 2024
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Fix whitespace shmem_broadcast
kwaters4 authored Jul 26, 2024

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature.
commit c3b23e5649ae9201c9b68b3fe7e77f726469f7af
34 changes: 17 additions & 17 deletions content/shmem_broadcast.tex
Original file line number Diff line number Diff line change
@@ -96,9 +96,9 @@
Upon return from a team-based broadcast routine, the following are true for the local
\ac{PE}:
\begin{itemize}
\item The \dest{} data object is updated.
\item The \source{} data object may be safely reused.
\end{itemize}
\item The \dest{} data object is updated.
\item The \source{} data object may be safely reused.
\end{itemize}

\begin{DeprecateBlock}
\openshmem active-set broadcast routines are collective routines over an active set.
@@ -111,30 +111,30 @@

For active-set-based broadcasts:
\begin{itemize}
\item The \VAR{dest} object is updated on all PEs other than the root PE.
\item All \acp{PE} in the active set defined by the
\VAR{PE\_start}, \VAR{logPE\_stride}, \VAR{PE\_size} triplet
must participate in the operation.
\item Only \acp{PE} in the active set may call the routine. If a
\ac{PE} not in the active set calls an active-set-based
\item The \VAR{dest} object is updated on all PEs other than the root PE.
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Suggested change
\item The \VAR{dest} object is updated on all PEs other than the root PE.
\item The \VAR{dest} object is updated on all \acp{PE} other than the root PE.

\item All \acp{PE} in the active set defined by the
\VAR{PE\_start}, \VAR{logPE\_stride}, \VAR{PE\_size} triplet
must participate in the operation.
\item Only \acp{PE} in the active set may call the routine. If a
\ac{PE} not in the active set calls an active-set-based
collective routine, the behavior is undefined.
\item The values of arguments \VAR{PE\_root}, \VAR{PE\_start},
\item The values of arguments \VAR{PE\_root}, \VAR{PE\_start},
\VAR{logPE\_stride}, and \VAR{PE\_size} must be the same value
on all \acp{PE} in the active set.
\item The value of \VAR{PE\_root} must be between \CONST{0} and
\item The value of \VAR{PE\_root} must be between \CONST{0} and
\VAR{PE\_size $-$ 1}.
\item The same \VAR{pSync} work array must be passed by all \acp{PE}
\item The same \VAR{pSync} work array must be passed by all \acp{PE}
in the active set.
\end{itemize}

Before any \ac{PE} calls a active-set-based broadcast routine, the following
conditions must be ensured:
\begin{itemize}
\item The \dest{} array on all \acp{PE} participating in the broadcast
is ready to accept the broadcast data.
\item The \VAR{pSync} array on all \acp{PE} in the
active set is not still in use from a prior call to an \openshmem
collective routine.
\item The \dest{} array on all \acp{PE} participating in the broadcast
is ready to accept the broadcast data.
\item The \VAR{pSync} array on all \acp{PE} in the
active set is not still in use from a prior call to an \openshmem
collective routine.
\end{itemize}
Otherwise, the behavior is undefined.