-
Notifications
You must be signed in to change notification settings - Fork 311
ABI Change Wishlist
Ken Raffenetti edited this page May 2, 2023
·
16 revisions
- Update MPI_Status struct
- Simplify count hi/lo and canceled
- Increase
MPI_MAX_INFO_VAL- Allow for more information in a single info hint, e.g. for RMA optimization
- Remove (or do not provide by default) functions removed from the MPI Standard
- e.g.
MPI_ERRHANDLER_CREATE
- e.g.
- Increase value of
MPI_MAX_PORT_NAME(and other string max values) - Change values of
MPI_STATUS_IGNOREandMPI_STATUSES_IGNORE- These constants are defined as
(MPI_Status *)1, which causes recent compilers to generate a warning:/home/glci/petsc/src/sys/utils/mpits.c:125:14: warning: ‘MPI_Testall’ accessing 20 bytes in a region of size 0 [-Wstringop-overflow=] 125 | ierr = MPI_Testall(nsends,sendreqs,&sent,MPI_STATUSES_IGNORE);CHKERRMPI(ierr);
- Today, if a user supplies
NULLin the status argument, MPICH (correctly) considers it an error. - Option 1: Define to
(MPI_Status *)0orNULL. MPICH no longer throws an error onNULL. - Option 2: Use address of dummy struct.
NULLis still considered an error by MPICH.
- These constants are defined as
- Remove
MPIR_Topo_typename from enum containingMPI_GRAPH,MPI_CART, andMPI_DIST_GRAPH- The named enum in not defined by MPI and is not used in any API signature. It is probably OK to remove it now, but we can be conservative and wait.
- Other named enums that can be unnamed:
MPIR_Win_flavor,MPIR_Win_model,MPIR_Combiner_enum,MPIR_T_verbosity_t,MPIR_T_bind_t,MPIR_T_scope_t,MPIR_T_pvar_class_t