Accepting request 1301458 from science:HPC

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

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)
+ 

openmpi-5.0.8.tar.bz2

@@ -0,0 +1,3 @@
+version https://git-lfs.github.com/spec/v1
+oid sha256:53131e1a57e7270f645707f8b0b65ba56048f5b5ac3f68faabed3eb0d710e449
+size 30293034

openmpi5.changes

@@ -1,31 +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>
 

openmpi5.spec

@@ -75,7 +75,7 @@ ExclusiveArch:  do_not_build
 #############################################################################
 
 Name:           openmpi5%{?testsuite:-testsuite}
-Version:        5.0.9
+Version:        5.0.8
 Release:        0
 Summary:        An implementation of MPI/SHMEM (Version 5)
 License:        BSD-3-Clause
@@ -85,8 +85,9 @@ 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:         Force-alignment-of-opal_atomic_int128_t-to-be-16B.patch
 Provides:       mpi
 BuildRoot:      %{_tmppath}/%{name}-%{version}-build
 # Exclude 32b archs