pr feedback - added locking for setting dropped event handler

cchambreau · jsquyres · commit c3da3686b85c · 2022-06-28T16:42:52.000-04:00
Signed-off-by: Chris Chambreau &lt;chambreau1@llnl.gov&gt;
diff --git a/ompi/mpi/tool/event_set_dropped_handler.c b/ompi/mpi/tool/event_set_dropped_handler.c
@@ -34,7 +34,11 @@ int MPI_T_event_set_dropped_handler (MPI_T_event_registration handle, MPI_T_even
         return MPI_T_ERR_INVALID_HANDLE;
     }
 
+    ompi_mpit_lock ();
+
     mca_base_event_registration_set_dropped_handler (handle, (mca_base_event_dropped_cb_fn_t) dropped_cb_function);
 
+    ompi_mpit_unlock ();
+
     return MPI_SUCCESS;
 }

Original file line number	Diff line number	Diff line change
`@@ -34,7 +34,11 @@ int MPI_T_event_set_dropped_handler (MPI_T_event_registration handle, MPI_T_even`
`34`	`34`	`return MPI_T_ERR_INVALID_HANDLE;`
`35`	`35`	`}`
`36`	`36`
	`37`	`+ ompi_mpit_lock ();`
	`38`	`+`
`37`	`39`	`mca_base_event_registration_set_dropped_handler (handle, (mca_base_event_dropped_cb_fn_t) dropped_cb_function);`
`38`	`40`
	`41`	`+ ompi_mpit_unlock ();`
	`42`	`+`
`39`	`43`	`return MPI_SUCCESS;`
`40`	`44`	`}`