Accepting request 1281139 from science:HPC

OBS-URL: https://build.opensuse.org/request/show/1281139
OBS-URL: https://build.opensuse.org/package/show/openSUSE:Factory/openmpi5?expand=0&rev=14

Fix-type-mismatch-error.patch

@@ -0,0 +1,30 @@
+commit fc4fb3a80615992277717e3e29ce585ebe1f2256
+Author: Tomislav Janjusic <tomislavj@nvidia.com>
+Date:   Tue Feb 18 12:25:27 2025 -0600
+
+    Fix type mismatch error
+    
+    Signed-off-by: Tomislav Janjusic <tomislavj@nvidia.com>
+
+diff --git oshmem/mca/sshmem/base/sshmem_base_open.c oshmem/mca/sshmem/base/sshmem_base_open.c
+index 1f0d1eb761e2..2694120e1cfe 100644
+--- oshmem/mca/sshmem/base/sshmem_base_open.c
++++ oshmem/mca/sshmem/base/sshmem_base_open.c
+@@ -31,7 +31,7 @@
+  * globals
+  */
+ 
+-void *mca_sshmem_base_start_address = UINTPTR_MAX;
++void *mca_sshmem_base_start_address = (void *)UINTPTR_MAX;
+ 
+ char * mca_sshmem_base_backing_file_dir = NULL;
+ 
+@@ -49,7 +49,7 @@ mca_sshmem_base_register (mca_base_register_flag_t flags)
+                                  "base",
+                                  "start_address",
+                                  "Specify base address for shared memory region",
+-                                 MCA_BASE_VAR_TYPE_UNSIGNED_LONG_LONG,
++                                 MCA_BASE_VAR_TYPE_UNSIGNED_LONG,
+                                  NULL,
+                                  0,
+                                  MCA_BASE_VAR_FLAG_SETTABLE,

Force-alignment-of-opal_atomic_int128_t-to-be-16B.patch

@@ -0,0 +1,28 @@
+commit d9a5f9d9db092cc5b4d8fd563a834183b2e9b301
+Author: Joseph Schuchart <joseph.schuchart@stonybrook.edu>
+Date:   Wed May 28 10:11:58 2025 -0400
+
+    Force alignment of opal_atomic_int128_t to be 16B
+    
+    Some architectures will align 128bit integer on 8B but require 16B alignment
+    for 128bit CAS instructions and otherwise fall back to a lock-based
+    atomicity model. By forcing 16bit alignment we can ensure that the variables
+    are properly aligned and the fall-back is not triggered.
+    
+    Thanks to Ulrich Weigand for the analysis and proposed fix in https://github.com/open-mpi/ompi/issues/10988#issuecomment-2913388506.
+    
+    Signed-off-by: Joseph Schuchart <joseph.schuchart@stonybrook.edu>
+
+diff --git opal/include/opal_stdatomic.h opal/include/opal_stdatomic.h
+index 354375512088..f7dd8353d3b6 100644
+--- opal/include/opal_stdatomic.h
++++ opal/include/opal_stdatomic.h
+@@ -72,7 +72,7 @@ typedef _Atomic opal_int128_t opal_atomic_int128_t;
+ 
+ #        else
+ 
+-typedef volatile opal_int128_t opal_atomic_int128_t;
++typedef volatile opal_int128_t opal_atomic_int128_t __opal_attribute_aligned__(16);
+ 
+ #        endif
+ 

README.md

@@ -1,12 +0,0 @@
-
-## Build Results
-
-Current state of openmpi5 in openSUSE:Factory is
-
-![Factory build results](https://br.opensuse.org/status/openSUSE:Factory/openmpi5/standard)
-
-The current state of openmpi5 in the devel project build (science:HPC)
-
-![Devel project build results](https://br.opensuse.org/status/science:HPC/openmpi5)
-
-

mtl-ofi-fix-missing-definition-of-container_of.patch

@@ -0,0 +1,24 @@
+commit 75b6dad5959ba75fcba8c64cc0a364d820e03ebc
+Author: Nicolas Morey <nmorey@suse.com>
+Date:   Fri Dec 20 12:44:38 2024 +0100
+
+    mtl: ofi: fix missing definition of container_of
+    
+    Signed-off-by: Nicolas Morey <nmorey@suse.com>
+
+diff --git ompi/mca/mtl/ofi/mtl_ofi_request.h ompi/mca/mtl/ofi/mtl_ofi_request.h
+index f68c2a91baa0..6d748fd59bdc 100644
+--- ompi/mca/mtl/ofi/mtl_ofi_request.h
++++ ompi/mca/mtl/ofi/mtl_ofi_request.h
+@@ -15,6 +15,11 @@
+ 
+ #include "mtl_ofi.h"
+ 
++#ifndef container_of
++#define container_of(ptr, type, field) \
++        ((type *) ((char *)ptr - offsetof(type, field)))
++#endif
++
+ #define TO_OFI_REQ(_ptr_ctx) \
+     container_of((_ptr_ctx), struct ompi_mtl_ofi_request_t, ctx)
+ 

openmpi5.changes

@@ -1,62 +1,3 @@
--------------------------------------------------------------------
-Fri Oct 31 07:56:03 UTC 2025 - Nicolas Morey <nicolas.morey@suse.com>
-
-- Update to 5.0.9
-  - Functionality Enhancements
-    - GPFS: Added support for GPFS 5.2.3-0 and newer versions
-    - OFI: Enhanced accelerator memory support with proper rcache flag handling
-    - OFI: Added memory monitor export for better memory management
-    - ROCm: Added missing header for memcpy operations in accelerator component
-    - UCC: Set node local id for better collective operation performance
-    - UCX: Set node local id in PML component for improved communication
-  - Bug Fixes and Minor Enhancements
-    - PML OB1: Fixed critical bug in MCA_PML_OB1_ADD_ACK_TO_PENDING that could cause memory overruns or allocation failures
-    - CUDA: Fixed CUDA detection when multiple libcuda.so files exist in CUDA install directory
-    - Fortran: Fixed off-by-one string copy error in C2F string conversion
-    - Fortran: Fixed ompi string c2f conversion when Fortran string length is less than C string length
-    - Threading: Fixed OMPI_MPI_THREAD_LEVEL environment variable handling to allow useful overrides in threaded library use cases
-    - Threading: Enhanced OMPI_MPI_THREAD_LEVEL to accept both numeric (0-3) and string ('multiple', 'MPI_THREAD_MULTIPLE', etc.) values
-    - OSC: Fixed rdma component when not using ob1 PML
-    - S390x: Fixed alignment of opal_atomic_int128_t to be 16-byte aligned
-    - Configury: Improved Fortran complex(real16) testing and module file cleanup
-    - Documentation: Fixed MCA environment variable prefix documentation for PRRTE
-    - Documentation: Updated MPI_Init*/MPI_Finalize*/MPI_Session_* man pages with numerous improvements
-    - Build system: Removed whitespace from conftestval-style tests and cleaned up configuration
-- Drop patches merged upstream:
-  - Force-alignment-of-opal_atomic_int128_t-to-be-16B.patch
-  - mtl-ofi-fix-missing-definition-of-container_of.patch
-
--------------------------------------------------------------------
-Tue Aug  5 18:48:52 UTC 2025 - Nicolas Morey <nicolas.morey@suse.com>
-
-- Enable Grid Cluster Scheduler support (jsc#PED-13373)
-
--------------------------------------------------------------------
-Thu Jun 12 08:35:43 UTC 2025 - Nicolas Morey <nicolas.morey@suse.com>
-
-- Update to 5.0.8
-  - Functionality Enhancements
-    - UCC collective operations:
-      Improved handling of MPI_IN_PLACE in allgather, allgatherv, alltoall,
-      alltoallv, gather, gatherv, scatter, and scatterv operations
-    - UCX OSC: Added support for "no_locks" info key to disable lock table usage
-    - OFI MTL: Enhanced CXI provider support for better compatibility
-    - AARCH64: Added SVE detection alongside NEON in aarch64 op component
-    - Fortran: Fixed common symbol sizes and alignments for better compatibility
-  - Bug Fixes and Minor Enhancements
-    - Hugepage Mpool: Fixed sizing of hugepages for better memory management
-    - Configure: Fixed --with-prrte=internal option handling
-    - Documentation: Updated shared memory and networking documentation
-    - Build system: Fixed support for flang on OSX
-    - Removed unused spread contrib sample scripts
-    - Various documentation updates and improvements
-    - CI: Removed Ubuntu 18 and RHEL 7 from test matrix
-    - Fixed type mismatch errors in shared memory components
-    - Updated Java configuration for latest Fedora releases
-    - Fixed memory zone reference counting in UCX dynamic windows
-- Remove Fix-type-mismatch-error.patch to fix a compilation error
-  as it was merged upstream.
-
 -------------------------------------------------------------------
 Mon May 26 15:52:53 UTC 2025 - Nicolas Morey <nicolas.morey@suse.com>
 

openmpi5.spec

@@ -75,7 +75,7 @@ ExclusiveArch:  do_not_build
 #############################################################################
 
 Name:           openmpi5%{?testsuite:-testsuite}
-Version:        5.0.9
+Version:        5.0.7
 Release:        0
 Summary:        An implementation of MPI/SHMEM (Version 5)
 License:        BSD-3-Clause
@@ -85,8 +85,10 @@ Source0:        https://download.open-mpi.org/release/open-mpi/v5.0/openmpi-%{ve
 Source2:        openmpi5-rpmlintrc
 Source4:        mpivars.sh
 Source5:        mpivars.csh
-Source100:      README.md
 Patch1:         romio341-backport-fixes-from-mpich.patch
+Patch2:         mtl-ofi-fix-missing-definition-of-container_of.patch
+Patch3:         Fix-type-mismatch-error.patch
+Patch4:         Force-alignment-of-opal_atomic_int128_t-to-be-16B.patch
 Provides:       mpi
 BuildRoot:      %{_tmppath}/%{name}-%{version}-build
 # Exclude 32b archs
@@ -352,7 +354,6 @@ export HOSTNAME=OBS
            --enable-mpi-thread-multiple \
            --disable-wrapper-rpath \
            --with-slurm \
-           --with-sge \
 %if 0%{?with_ucx}
            --with-ucx \
            --with-ucx-libdir=/usr/%_lib \