Skip to content

Commit 2f9d87c

Browse files
William Williamsjeffhammond
authored andcommitted
Fix two MPI_T enums
Per discussion in Tools WG session 18MAR, these should both logically be bitfields: CB_REQUIRE should be 0/1/3/7 as each level inherits the requirements of the previous at present but that's not a guarantee for the future, and verbosity should allow future expansion via bitmasking of both audience and level of detail.
1 parent e6b3698 commit 2f9d87c

File tree

1 file changed

+12
-12
lines changed

1 file changed

+12
-12
lines changed

mpi.h

Lines changed: 12 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -511,10 +511,10 @@ typedef struct MPI_T_event_instance_t* MPI_T_event_instance;
511511
#define MPI_T_PVAR_ALL_HANDLES ((MPI_T_pvar_handle)1)
512512

513513
typedef enum MPI_T_cb_safety {
514-
MPI_T_CB_REQUIRE_NONE = 1,
515-
MPI_T_CB_REQUIRE_MPI_RESTRICTED = 2,
514+
MPI_T_CB_REQUIRE_NONE = 0,
515+
MPI_T_CB_REQUIRE_MPI_RESTRICTED = 1,
516516
MPI_T_CB_REQUIRE_THREAD_SAFE = 3,
517-
MPI_T_CB_REQUIRE_ASYNC_SIGNAL_SAFE = 4
517+
MPI_T_CB_REQUIRE_ASYNC_SIGNAL_SAFE = 7
518518
} MPI_T_cb_safety;
519519

520520
typedef enum MPI_T_source_order {
@@ -523,15 +523,15 @@ typedef enum MPI_T_source_order {
523523
} MPI_T_source_order;
524524

525525
enum {
526-
MPI_T_VERBOSITY_USER_BASIC = 1,
527-
MPI_T_VERBOSITY_USER_DETAIL = 2,
528-
MPI_T_VERBOSITY_USER_ALL = 3,
529-
MPI_T_VERBOSITY_TUNER_BASIC = 4,
530-
MPI_T_VERBOSITY_TUNER_DETAIL = 5,
531-
MPI_T_VERBOSITY_TUNER_ALL = 6,
532-
MPI_T_VERBOSITY_MPIDEV_BASIC = 7,
533-
MPI_T_VERBOSITY_MPIDEV_DETAIL = 8,
534-
MPI_T_VERBOSITY_MPIDEV_ALL = 9
526+
MPI_T_VERBOSITY_USER_BASIC = 0x09,
527+
MPI_T_VERBOSITY_USER_DETAIL = 0x0a,
528+
MPI_T_VERBOSITY_USER_ALL = 0x0c,
529+
MPI_T_VERBOSITY_TUNER_BASIC = 0x11,
530+
MPI_T_VERBOSITY_TUNER_DETAIL = 0x12,
531+
MPI_T_VERBOSITY_TUNER_ALL = 0x14,
532+
MPI_T_VERBOSITY_MPIDEV_BASIC = 0x21,
533+
MPI_T_VERBOSITY_MPIDEV_DETAIL = 0x22,
534+
MPI_T_VERBOSITY_MPIDEV_ALL = 0x24
535535
};
536536

537537
enum {

0 commit comments

Comments
 (0)