Errata: fix a couple of spots in server upcall functions

Server upcall functions were required to return prior to executing
the callback function. This was a limitation of the implementation
and not a requirement of the Standard - the statements should
never have been included here.

FWIW: the limitation has subsequently been removed from the
reference implementation.

In the pmix_server_iof_fn_t, clarify that an error status can be
returned either directly by the function or in the callback function.
It does not require that the host execute the callback to return
an error, as was implied by the current language.

Signed-off-by: Ralph Castain <[email protected]>

Author: rhc54

Chap_API_Server.tex

@@ -955,7 +955,7 @@ \subsection{\code{PMIx_server_deregister_resources}}
 
 A \code{NULL} \refarg{cbfunc} reference indicates that the function is to be executed as a blocking operation.
 % TODO: If this is one of those functions that can be blocking or non-blocking, then
-% it needs a full return code explanation, not the returnsimple 
+% it needs a full return code explanation, not the returnsimple
 
 \advicermend
 
@@ -3318,8 +3318,7 @@ \subsection{\code{pmix_server_tool_connection_fn_t}}
 Approval or rejection of the connection request shall be returned in the
 \refarg{status} parameter of the \refapi{pmix_tool_connection_cbfunc_t}. If
 the connection is refused, the \ac{PMIx} server library must terminate the
-connection attempt. The host must not execute the callback function prior to
-returning from the \ac{API}.
+connection attempt.
 
 %%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%
 \subsubsection{Tool connection attributes}
@@ -3435,7 +3434,7 @@ \subsection{\code{pmix_server_log_fn_t}}
 %%%%
 \descr
 
-Log data on behalf of a client. This function is not intended for output of computational results, but rather for reporting status and error messages. The host must not execute the callback function prior to returning from the \ac{API}.
+Log data on behalf of a client. This function is not intended for output of computational results, but rather for reporting status and error messages.
 
 
 %%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%
@@ -3974,7 +3973,7 @@ \subsection{\code{pmix_server_iof_fn_t}}
 %%%%
 \descr
 
-Request the specified IO channels be forwarded from the given array of processes. An error shall be returned in the callback function if the requested service from any of the requested processes cannot be provided.
+Request the specified IO channels be forwarded from the given array of processes. An error shall be returned (either directly or in the callback function) if the requested service from any of the requested processes cannot be provided.
 
 \adviceimplstart
 The forwarding of stdin is a \textit{push} process - processes cannot request that it be \textit{pulled} from some other source. Requests including the \refconst{PMIX_FWD_STDIN_CHANNEL} channel will return a \refconst{PMIX_ERR_NOT_SUPPORTED} error.