Skip to content

Commit 7dc770a

Browse files
cchambreauhppritcha
cchambreau
authored and
hppritcha
committed
Remove non-conformant event_get_info extent argument and internal tracking.
Signed-off-by: Chris Chambreau <chambreau1@llnl.gov>
1 parent 7a25e08 commit 7dc770a

File tree

6 files changed

+39
-51
lines changed

6 files changed

+39
-51
lines changed

‎ompi/include/mpi.h.in

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -2871,7 +2871,7 @@ OMPI_DECLSPEC int PMPI_T_event_get_num (int *num_events);
28712871
OMPI_DECLSPEC int PMPI_T_event_get_info (int event_index, char *name, int *name_len,
28722872
int *verbosity, MPI_Datatype *array_of_datatypes,
28732873
MPI_Aint *array_of_displacements, int *num_elements,
2874-
MPI_Aint *extent, MPI_T_enum *enumtype, MPI_Info *info,
2874+
MPI_T_enum *enumtype, MPI_Info *info,
28752875
char *desc, int *desc_len, int *bind);
28762876
OMPI_DECLSPEC int PMPI_T_event_get_index (const char *name, int *event_index);
28772877
OMPI_DECLSPEC int PMPI_T_event_handle_alloc (int event_index, void *obj_handle,
@@ -2971,7 +2971,7 @@ OMPI_DECLSPEC int MPI_T_event_get_num (int *num_events);
29712971
OMPI_DECLSPEC int MPI_T_event_get_info (int event_index, char *name, int *name_len,
29722972
int *verbosity, MPI_Datatype *array_of_datatypes,
29732973
MPI_Aint *array_of_displacements, int *num_elements,
2974-
MPI_Aint *extent, MPI_T_enum *enumtype, MPI_Info *info,
2974+
MPI_T_enum *enumtype, MPI_Info *info,
29752975
char *desc, int *desc_len, int *bind);
29762976
OMPI_DECLSPEC int MPI_T_event_get_index (const char *name, int *event_index);
29772977
OMPI_DECLSPEC int MPI_T_event_handle_alloc (int event_index, void *obj_handle,

‎ompi/mca/osc/rdma/osc_rdma_component.c

Lines changed: 12 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -107,62 +107,62 @@ mca_base_event_list_item_t mca_osc_rdma_events[] = {
107107
[OMPI_OSC_RDMA_EVENT_LOCK_ACQUIRED] = {.name = "lock_acquired", .desc = "Passive-target lock aquired",
108108
.verbosity = OPAL_INFO_LVL_5, .datatypes = &(opal_datatype_t *) {&ompi_mpi_int.dt.super},
109109
.offsets = &(unsigned long) {0}, .num_datatypes = 1, .elements = mca_osc_rdma_target_element,
110-
.extent = sizeof (int), .bind = MCA_BASE_VAR_BIND_MPI_WIN},
110+
.bind = MCA_BASE_VAR_BIND_MPI_WIN},
111111

112112
[OMPI_OSC_RDMA_EVENT_LOCK_RELEASED] = {.name = "lock_released", .desc = "Passive-target lock required",
113113
.verbosity = OPAL_INFO_LVL_5, .datatypes = &(opal_datatype_t *) {&ompi_mpi_int.dt.super},
114114
.offsets = &(unsigned long) {0}, .num_datatypes = 1, .elements = mca_osc_rdma_target_element,
115-
.extent = sizeof (int), .bind = MCA_BASE_VAR_BIND_MPI_WIN},
115+
.bind = MCA_BASE_VAR_BIND_MPI_WIN},
116116

117117
[OMPI_OSC_RDMA_EVENT_PUT_STARTED] = {.name = "put_started", .desc = "Put started to target. Complete event may not exist.",
118118
.verbosity = OPAL_INFO_LVL_5, .datatypes = mca_osc_rdma_rdma_event_types,
119119
.offsets = mca_osc_rdma_rdma_event_offsets, .num_datatypes = 3,
120-
.elements = mca_osc_rdma_rdma_event_elements, .extent = 24, .bind = MCA_BASE_VAR_BIND_MPI_WIN},
120+
.elements = mca_osc_rdma_rdma_event_elements, .bind = MCA_BASE_VAR_BIND_MPI_WIN},
121121

122122
[OMPI_OSC_RDMA_EVENT_PUT_COMPLETE] = {.name = "put_complete", .desc = "Put completed on target",
123123
.verbosity = OPAL_INFO_LVL_5, .datatypes = mca_osc_rdma_rdma_event_types,
124124
.offsets = mca_osc_rdma_rdma_event_offsets, .num_datatypes = 3,
125-
.elements = mca_osc_rdma_rdma_event_elements, .extent = 24, .bind = MCA_BASE_VAR_BIND_MPI_WIN},
125+
.elements = mca_osc_rdma_rdma_event_elements, .bind = MCA_BASE_VAR_BIND_MPI_WIN},
126126

127127
[OMPI_OSC_RDMA_EVENT_GET_STARTED] = {.name = "get_started", .desc = "Put started to target. Complete event may not exist.",
128128
.verbosity = OPAL_INFO_LVL_5, .datatypes = mca_osc_rdma_rdma_event_types,
129129
.offsets = mca_osc_rdma_rdma_event_offsets, .num_datatypes = 3,
130-
.elements = mca_osc_rdma_rdma_event_elements, .extent = 24, .bind = MCA_BASE_VAR_BIND_MPI_WIN},
130+
.elements = mca_osc_rdma_rdma_event_elements, .bind = MCA_BASE_VAR_BIND_MPI_WIN},
131131

132132
[OMPI_OSC_RDMA_EVENT_GET_COMPLETE] = {.name = "get_complete", .desc = "Put completed on target",
133133
.verbosity = OPAL_INFO_LVL_5, .datatypes = mca_osc_rdma_rdma_event_types,
134134
.offsets = mca_osc_rdma_rdma_event_offsets, .num_datatypes = 3,
135-
.elements = mca_osc_rdma_rdma_event_elements, .extent = 24, .bind = MCA_BASE_VAR_BIND_MPI_WIN},
135+
.elements = mca_osc_rdma_rdma_event_elements, .bind = MCA_BASE_VAR_BIND_MPI_WIN},
136136

137137
[OMPI_OSC_RDMA_EVENT_FLUSH_STARTED] = {.name = "flush_started", .desc = "Flush started on target",
138138
.verbosity = OPAL_INFO_LVL_5, .datatypes = &(opal_datatype_t *) {&ompi_mpi_int.dt.super},
139139
.offsets = &(unsigned long) {0}, .num_datatypes = 1, .elements = mca_osc_rdma_target_element,
140-
.extent = sizeof (int), .bind = MCA_BASE_VAR_BIND_MPI_WIN},
140+
.bind = MCA_BASE_VAR_BIND_MPI_WIN},
141141

142142
[OMPI_OSC_RDMA_EVENT_FLUSH_COMPLETE] = {.name = "flush_complete", .desc = "Flush complete on target",
143143
.verbosity = OPAL_INFO_LVL_5, .datatypes = &(opal_datatype_t *) {&ompi_mpi_int.dt.super},
144144
.offsets = &(unsigned long) {0}, .num_datatypes = 1, .elements = mca_osc_rdma_target_element,
145-
.extent = sizeof (int), .bind = MCA_BASE_VAR_BIND_MPI_WIN},
145+
.bind = MCA_BASE_VAR_BIND_MPI_WIN},
146146

147147
[OMPI_OSC_RDMA_EVENT_PSCW_EXPOSE_START] = {.name = "pscw_expose_start", .desc = "PSWW exposure started",
148148
.verbosity = OPAL_INFO_LVL_5, .datatypes = &(opal_datatype_t *) {&ompi_mpi_int.dt.super},
149149
.offsets = &(unsigned long) {0}, .num_datatypes = 0, .elements = mca_osc_rdma_target_element,
150-
.extent = sizeof (int), .bind = MCA_BASE_VAR_BIND_MPI_WIN},
150+
.bind = MCA_BASE_VAR_BIND_MPI_WIN},
151151

152152
[OMPI_OSC_RDMA_EVENT_PSCW_EXPOSE_COMPLETE] = {.name = "pscw_expose_complete", .desc = "PSWW exposure complete",
153153
.verbosity = OPAL_INFO_LVL_5, .datatypes = &(opal_datatype_t *) {&ompi_mpi_int.dt.super},
154154
.offsets = &(unsigned long) {0}, .num_datatypes = 0, .elements = mca_osc_rdma_target_element,
155-
.extent = sizeof (int), .bind = MCA_BASE_VAR_BIND_MPI_WIN},
155+
.bind = MCA_BASE_VAR_BIND_MPI_WIN},
156156

157157
[OMPI_OSC_RDMA_EVENT_PSCW_ACCESS_START] = {.name = "pscw_access_start", .desc = "PSWW access epoch started",
158158
.verbosity = OPAL_INFO_LVL_5, .datatypes = &(opal_datatype_t *) {&ompi_mpi_int.dt.super},
159159
.offsets = &(unsigned long) {0}, .num_datatypes = 1, .elements = mca_osc_rdma_target_element,
160-
.extent = sizeof (int), .bind = MCA_BASE_VAR_BIND_MPI_WIN},
160+
.bind = MCA_BASE_VAR_BIND_MPI_WIN},
161161

162162
[OMPI_OSC_RDMA_EVENT_PSCW_ACCESS_COMPLETE] = {.name = "pscw_access_complete", .desc = "PSWW access epoch complete",
163163
.verbosity = OPAL_INFO_LVL_5, .datatypes = &(opal_datatype_t *) {&ompi_mpi_int.dt.super},
164164
.offsets = &(unsigned long) {0}, .num_datatypes = 1, .elements = mca_osc_rdma_target_element,
165-
.extent = sizeof (int), .bind = MCA_BASE_VAR_BIND_MPI_WIN},
165+
.bind = MCA_BASE_VAR_BIND_MPI_WIN},
166166

167167
[OMPI_OSC_RDMA_EVENT_FENCE] = {.name = "fence", .desc = "Fence called", .verbosity = OPAL_INFO_LVL_5, .bind = MCA_BASE_VAR_BIND_MPI_WIN},
168168
};

‎ompi/mca/pml/ob1/pml_ob1_component.c

Lines changed: 16 additions & 17 deletions
Original file line numberDiff line numberDiff line change
@@ -97,74 +97,73 @@ mca_base_event_list_item_t mca_pml_ob1_events[] = {
9797
[MCA_PML_OB1_EVENT_MESSAGE_ARRIVED] = {.name = "message_arrived", .desc = "Message arrived for match",
9898
.verbosity = OPAL_INFO_LVL_5, .datatypes = mca_pml_ob1_match_hdr_types,
9999
.offsets = mca_pml_ob1_match_hdr_offsets, .num_datatypes = 4,
100-
.elements = mca_pml_ob1_match_hdr_names, .extent = 12, .bind = MCA_BASE_VAR_BIND_MPI_COMM},
100+
.elements = mca_pml_ob1_match_hdr_names, .bind = MCA_BASE_VAR_BIND_MPI_COMM},
101101

102102
[MCA_PML_OB1_EVENT_SEARCH_POSTED_BEGIN] = {.name = "search_posted_begin", .desc = "Begin searching posted message queue",
103103
.verbosity = OPAL_INFO_LVL_5, .datatypes = &mca_pml_ob1_match_hdr_types[1],
104104
.offsets = &mca_pml_ob1_match_hdr_offsets[1], .num_datatypes = 3,
105-
.elements = &mca_pml_ob1_match_hdr_names[1], .extent = 8, .bind = MCA_BASE_VAR_BIND_MPI_COMM},
105+
.elements = &mca_pml_ob1_match_hdr_names[1], .bind = MCA_BASE_VAR_BIND_MPI_COMM},
106106

107107
[MCA_PML_OB1_EVENT_SEARCH_POSTED_END] = {.name = "search_posted_end", .desc = "Finished searching posted message queue",
108108
.verbosity = OPAL_INFO_LVL_5, .datatypes = &mca_pml_ob1_match_hdr_types[1],
109109
.offsets = &mca_pml_ob1_match_hdr_offsets[1], .num_datatypes = 3,
110-
.elements = &mca_pml_ob1_match_hdr_names[1], .extent = 8, .bind = MCA_BASE_VAR_BIND_MPI_COMM},
110+
.elements = &mca_pml_ob1_match_hdr_names[1], .bind = MCA_BASE_VAR_BIND_MPI_COMM},
111111

112112
[MCA_PML_OB1_EVENT_SEARCH_UNEX_BEGIN] = {.name = "search_unexpected_begin", .desc = "Begin searching unexpected message queue",
113113
.verbosity = OPAL_INFO_LVL_5, .datatypes = &(opal_datatype_t *) {&ompi_mpi_aint.dt.super},
114114
.offsets = &(unsigned long) {0}, .num_datatypes = 1, .elements = mca_pml_ob1_request_element,
115-
.extent = sizeof (MPI_Aint), .bind = MCA_BASE_VAR_BIND_MPI_COMM},
115+
.bind = MCA_BASE_VAR_BIND_MPI_COMM},
116116

117117
[MCA_PML_OB1_EVENT_SEARCH_UNEX_END] = {.name = "search_unexpected_end", .desc = "Finished searching unexpected message queue",
118118
.verbosity = OPAL_INFO_LVL_5, .datatypes = &(opal_datatype_t *) {&ompi_mpi_aint.dt.super},
119119
.offsets = &(unsigned long) {0}, .num_datatypes = 1, .elements = mca_pml_ob1_request_element,
120-
.extent = sizeof (MPI_Aint), .bind = MCA_BASE_VAR_BIND_MPI_COMM},
120+
.bind = MCA_BASE_VAR_BIND_MPI_COMM},
121121

122122
[MCA_PML_OB1_EVENT_POSTED_INSERT] = {.name = "posted_insert", .desc = "Added request to the posted message queue",
123123
.verbosity = OPAL_INFO_LVL_5, .datatypes = &(opal_datatype_t *) {&ompi_mpi_aint.dt.super},
124124
.offsets = &(unsigned long) {0}, .num_datatypes = 1, .elements = mca_pml_ob1_request_element,
125-
.extent = sizeof (MPI_Aint), .bind = MCA_BASE_VAR_BIND_MPI_COMM},
125+
.bind = MCA_BASE_VAR_BIND_MPI_COMM},
126126

127127
[MCA_PML_OB1_EVENT_POSTED_REMOVE] = {.name = "posted_remove", .desc = "Remove request from the posted message queue",
128128
.verbosity = OPAL_INFO_LVL_5, .datatypes = &(opal_datatype_t *) {&ompi_mpi_aint.dt.super},
129129
.offsets = &(unsigned long) {0}, .num_datatypes = 1, .elements = mca_pml_ob1_request_element,
130-
.extent = sizeof (MPI_Aint), .bind = MCA_BASE_VAR_BIND_MPI_COMM},
130+
.bind = MCA_BASE_VAR_BIND_MPI_COMM},
131131

132132
[MCA_PML_OB1_EVENT_UNEX_INSERT] = {.name = "unex_insert", .desc = "Unexpected message inserted in queue", .verbosity = OPAL_INFO_LVL_5,
133133
.datatypes = &mca_pml_ob1_match_hdr_types[1], .offsets = &mca_pml_ob1_match_hdr_offsets[1],
134-
.num_datatypes = 2, .elements = &mca_pml_ob1_match_hdr_names[1], .extent = 8, .bind = MCA_BASE_VAR_BIND_MPI_COMM},
134+
.num_datatypes = 2, .elements = &mca_pml_ob1_match_hdr_names[1], .bind = MCA_BASE_VAR_BIND_MPI_COMM},
135135

136136
[MCA_PML_OB1_EVENT_UNEX_REMOVE] = {.name = "unex_remove", .desc = "Unexpected message removed from queue", .verbosity = OPAL_INFO_LVL_5,
137137
.datatypes = &mca_pml_ob1_match_hdr_types[1], .offsets = &mca_pml_ob1_match_hdr_offsets[1],
138-
.num_datatypes = 2, .elements = &mca_pml_ob1_match_hdr_names[1], .extent = 8, .bind = MCA_BASE_VAR_BIND_MPI_COMM},
138+
.num_datatypes = 2, .elements = &mca_pml_ob1_match_hdr_names[1], .bind = MCA_BASE_VAR_BIND_MPI_COMM},
139139

140140
[MCA_PML_OB1_EVENT_TRANSFER_BEGIN] = {.name = "transfer_begin", .desc = "Transfer has begun", .verbosity = OPAL_INFO_LVL_5,
141141
.datatypes = &(opal_datatype_t *) {&ompi_mpi_aint.dt.super}, .offsets = &(unsigned long) {0}, .num_datatypes = 1,
142-
.elements = mca_pml_ob1_request_element, .extent = sizeof (MPI_Aint), .bind = MCA_BASE_VAR_BIND_MPI_COMM},
142+
.elements = mca_pml_ob1_request_element, .bind = MCA_BASE_VAR_BIND_MPI_COMM},
143143

144144
[MCA_PML_OB1_EVENT_TRANSFER] = {.name = "transfer", .desc = "Transfer event", .verbosity = OPAL_INFO_LVL_5,
145145
.datatypes = mca_pml_ob1_request_size_types, .offsets = mca_pml_ob1_request_size_offsets, .num_datatypes = 2,
146-
.elements = mca_pml_ob1_request_size_elements, .extent = sizeof (mca_pml_ob1_transfer_event_t),
147-
.bind = MCA_BASE_VAR_BIND_MPI_COMM},
146+
.elements = mca_pml_ob1_request_size_elements, .bind = MCA_BASE_VAR_BIND_MPI_COMM},
148147

149148
[MCA_PML_OB1_EVENT_TRANSFER_END] = {.name = "transfer_end", .desc = "Transfer has completed", .verbosity = OPAL_INFO_LVL_5,
150149
.datatypes = &(opal_datatype_t *) {&ompi_mpi_aint.dt.super}, .offsets = &(unsigned long) {0}, .num_datatypes = 1,
151-
.elements = mca_pml_ob1_request_element, .extent = sizeof (MPI_Aint), .bind = MCA_BASE_VAR_BIND_MPI_COMM},
150+
.elements = mca_pml_ob1_request_element, .bind = MCA_BASE_VAR_BIND_MPI_COMM},
152151

153152
[MCA_PML_OB1_EVENT_RECEIVE_CANCELED] = {.name = "cancel", .desc = "Receive request canceled", .verbosity = OPAL_INFO_LVL_5,
154153
.datatypes = &(opal_datatype_t *) {&ompi_mpi_aint.dt.super}, .offsets = &(unsigned long) {0}, .num_datatypes = 1,
155-
.elements = mca_pml_ob1_request_element, .extent = sizeof (MPI_Aint), .bind = MCA_BASE_VAR_BIND_MPI_COMM},
154+
.elements = mca_pml_ob1_request_element, .bind = MCA_BASE_VAR_BIND_MPI_COMM},
156155

157156
[MCA_PML_OB1_EVENT_REQUEST_FREE] = {.name = "free", .desc = "Request object freed", .verbosity = OPAL_INFO_LVL_5,
158157
.datatypes = &(opal_datatype_t *) {&ompi_mpi_aint.dt.super}, .offsets = &(unsigned long) {0}, .num_datatypes = 1,
159-
.elements = mca_pml_ob1_request_element, .extent = sizeof (MPI_Aint), .bind = MCA_BASE_VAR_BIND_MPI_COMM},
158+
.elements = mca_pml_ob1_request_element, .bind = MCA_BASE_VAR_BIND_MPI_COMM},
160159

161160
[MCA_PML_OB1_EVENT_REQUEST_ACTIVATE] = {.name = "request_activate", .desc = "Request activated", .verbosity = OPAL_INFO_LVL_5,
162161
.datatypes = &(opal_datatype_t *) {&ompi_mpi_aint.dt.super}, .offsets = &(unsigned long) {0}, .num_datatypes = 1,
163-
.elements = mca_pml_ob1_request_element, .extent = sizeof (MPI_Aint), .bind = MCA_BASE_VAR_BIND_MPI_COMM},
162+
.elements = mca_pml_ob1_request_element, .bind = MCA_BASE_VAR_BIND_MPI_COMM},
164163

165164
[MCA_PML_OB1_EVENT_REQUEST_COMPLETE] = {.name = "request_complete", .desc = "Request completed", .verbosity = OPAL_INFO_LVL_5,
166165
.datatypes = &(opal_datatype_t *) {&ompi_mpi_aint.dt.super}, .offsets = &(unsigned long) {0}, .num_datatypes = 1,
167-
.elements = mca_pml_ob1_request_element, .extent = sizeof (MPI_Aint), .bind = MCA_BASE_VAR_BIND_MPI_COMM},
166+
.elements = mca_pml_ob1_request_element, .bind = MCA_BASE_VAR_BIND_MPI_COMM},
168167
};
169168

170169
mca_pml_base_component_2_1_0_t mca_pml_ob1_component = {

‎ompi/mpi/tool/event_get_info.c

Lines changed: 1 addition & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -30,7 +30,7 @@
3030
int MPI_T_event_get_info (int event_index, char *name, int *name_len,
3131
int *verbosity, MPI_Datatype *array_of_datatypes,
3232
MPI_Aint *array_of_displacements, int *num_elements,
33-
MPI_Aint *extent, MPI_T_enum *enumtype, MPI_Info *info,
33+
MPI_T_enum *enumtype, MPI_Info *info,
3434
char *desc, int *desc_len, int *bind)
3535
{
3636
mca_base_event_t * const event;
@@ -125,10 +125,6 @@ int MPI_T_event_get_info (int event_index, char *name, int *name_len,
125125
*bind = event->event_bind;
126126
}
127127

128-
if (NULL != extent) {
129-
*extent = event->event_extent;
130-
}
131-
132128
if (NULL != info) {
133129
*info = OBJ_NEW(ompi_info_t);
134130
}

0 commit comments

Comments
 (0)