ch4/ofi: skip ofi_initialized if world size is 1

hzhou · hzhou · commit 42f20cbd6c0b · 2025-07-02T10:29:25.000-05:00
The address exchange will be skipped if it is a single process,
including inserting self address. This results in flush_send in finalize
fail. Simply avoid setting ofi_initialized in the case of single process
make it work.
diff --git a/src/mpid/ch4/netmod/ofi/ofi_init.c b/src/mpid/ch4/netmod/ofi/ofi_init.c
@@ -811,12 +811,11 @@ int MPIDI_OFI_init_world(void)
 {
     int mpi_errno = MPI_SUCCESS;
 
-    if (!MPIDI_OFI_global.got_named_av) {
+    if (!MPIDI_OFI_global.got_named_av && MPIR_Process.size > 1) {
         MPIR_Assert(MPIR_Process.comm_world);
         mpi_errno = MPIDI_OFI_comm_addr_exchange(MPIR_Process.comm_world);
         MPIR_ERR_CHECK(mpi_errno);
     }
-
     ofi_initialized = true;
 
   fn_exit:

Original file line number	Diff line number	Diff line change
`@@ -811,12 +811,11 @@ int MPIDI_OFI_init_world(void)`
`811`	`811`	`{`
`812`	`812`	`int mpi_errno = MPI_SUCCESS;`
`813`	`813`
`814`		`- if (!MPIDI_OFI_global.got_named_av) {`
	`814`	`+ if (!MPIDI_OFI_global.got_named_av && MPIR_Process.size > 1) {`
`815`	`815`	`MPIR_Assert(MPIR_Process.comm_world);`
`816`	`816`	`mpi_errno = MPIDI_OFI_comm_addr_exchange(MPIR_Process.comm_world);`
`817`	`817`	`MPIR_ERR_CHECK(mpi_errno);`
`818`	`818`	`}`
`819`		`-`
`820`	`819`	`ofi_initialized = true;`
`821`	`820`
`822`	`821`	`fn_exit:`