Skip to content

Conversation

@joe-explr
Copy link

No description provided.

hppritcha and others added 30 commits July 31, 2025 14:32
related to open-mpi#12193

Signed-off-by: Howard Pritchard <howardp@lanl.gov>
Signed-off-by: Howard Pritchard <howardp@lanl.gov>
Signed-off-by: Howard Pritchard <howardp@lanl.gov>
CID 1659063 - fix potential memory leaks
Signed-off-by: George Bosilca <gbosilca@nvidia.com>
CID 1660795 - fix potential memory leak
Signed-off-by: Howard Pritchard <howardp@lanl.gov>
Signed-off-by: Tim Jammer <tim.jammer@tu-darmstadt.de>
Fix build error for build configured with --without-ft
Signed-off-by: Howard Pritchard <howardp@lanl.gov>
Signed-off-by: Howard Pritchard <howardp@lanl.gov>
without this patch --with-knem=no doesn't work

this is useful when knem software is installed but the knem device is not enabled.
So this message:

WARNING: Open MPI failed to open the /dev/knem device due to a
permissions problem.  Please check with your system administrator to
get the permissions fixed, or set the smsc MCA variable to "^knem" to
silence this warning and run without knem support.

is avoided without needing the user to set an MCA parameter, etc.

Signed-off-by: Howard Pritchard <howardp@lanl.gov>
Signed-off-by: hasegawa.kento <hasegawa.kento@fujitsu.com>
Signed-off-by: hasegawa.kento <hasegawa.kento@fujitsu.com>
Signed-off-by: hasegawa.kento <hasegawa.kento@fujitsu.com>
…o_disable

configury: allow knem to be disabled
mpi-f08.F90 always includes sizeof_f08.h, so compilation fails if the dependency isnt there. Script will print empty file if the flag isnt set, so should not be a big deal

Signed-off-by: Yussur Mustafa Oraji <yussur.oraji@tu-darmstadt.de>
- add the man pages for the newly implemented MPI_Request_get_status_all/any/some functions.
- c-bindings: add support for const request args

Three new functions were added to the MPI API as part of the 4.1 standard.
These used an array of const MPI_Request s which the bindings code
didn't support.

This commit also fixes back the mpi.h prototypes to include the constants
and required changes to the new template files.

- request_get_status variants: add f08 interfaces

also switch MPI_Request_get_status to use new method for generating
f08 bindings.

- Update fortran bindings interfaces generation code.

- f90/f77 interfaces will be added as another commit.
- use-mpi-ignore: start using binding infrastructure
- request_get_status_mult: add f77 functions

Signed-off-by: Edgar Gabriel <edgar.gabriel@amd.com>
Signed-off-by: Howard Pritchard <howardp@lanl.gov>
Turns out that in commit 6bd36a7 we had a function that is not part of the MPI standard.
This showed while working on ABI support - which requires us to pay attention to the truth
rather than make stuff up.

This commit removes our made up MPI_Session_set_info method.
Turns out who ever was doing the fortran bindings knew this wasn't a method in the standard
so there's no need to change the fortran bindings.  Same thing applies to the man pages.

Related to open-mpi#13280

Signed-off-by: Howard Pritchard <howardp@lanl.gov>
…tus-mult

add support for request_get_status_any/all/some
Thanks to Ben Menadue for pointing out that ompi_fortran_string_c2f()
missed a case to properly terminate the resulting Fortran string when
copying from a longer C source string.

Signed-off-by: Jeff Squyres <jeff@squyres.com>
…ring-copy

fortran: fix ompi string c2f where len(fstr) < len(cstr)
use-mpi-f08: Always call generation script for sizeof_f08
We only support named types with this first pass.

related to open-mpi#12076

Fortran interfaces will be added once Fortran infrastructure
additions in PR open-mpi#13279 are merged in to main.

Signed-off-by: Howard Pritchard <howardp@lanl.gov>
Signed-off-by: Howard Pritchard <howardp@lanl.gov>
…o_not_in_mpi

sessions:  oops, we made up a function
hppritcha and others added 15 commits October 20, 2025 13:10
…shas2

move shas for 3rd-party/openpmix and prrte
…tused

avoid data race on set LASTUSED attribute
for comm and instance (aka session) handles

Signed-off-by: Howard Pritchard <howardp@lanl.gov>
buffer_attach: make sure to set to null in ctor
exposed while testing ABI PR open-mpi#13280

Signed-off-by: Howard Pritchard <howardp@lanl.gov>
This commit has the following updates:
- converts the xpmem-based collectives to SMSC-based for collective operations.
- fixes coverity issues
- misc bug-fixes

Signed-off-by: Manu Shantharam <manu.shantharam@amd.com>
Updates to the acoll component
    - converts the xpmem-based collectives to SMSC-based for collective operations.
    - fixes coverity issues
    - misc bug-fixes
ofi: Share the domain among MTL and BTL
Signed-off-by: Howard Pritchard <howardp@lanl.gov>
…8d9c03932b

ompi-prte: advance to sha 8d9c03932b
Signed-off-by: Joseph Schuchart <joseph.schuchart@stonybrook.edu>
@joe-explr joe-explr marked this pull request as draft November 3, 2025 08:36
@github-actions
Copy link

github-actions bot commented Nov 3, 2025

Hello! The Git Commit Checker CI bot found a few problems with this PR:

08f38c3: Notify for Rput variants

  • check_signed_off: does not contain a valid Signed-off-by line

acf1108: Put notify and get notify

  • check_signed_off: does not contain a valid Signed-off-by line

1f1438e: put notify and get notify

  • check_signed_off: does not contain a valid Signed-off-by line

Please fix these problems and, if necessary, force-push new commits back up to the PR branch. Thanks!

@github-actions
Copy link

github-actions bot commented Nov 3, 2025

Hello! The Git Commit Checker CI bot found a few problems with this PR:

f37ec8f: Delete .cache/clangd/index directory

  • check_signed_off: does not contain a valid Signed-off-by line

08f38c3: Notify for Rput variants

  • check_signed_off: does not contain a valid Signed-off-by line

acf1108: Put notify and get notify

  • check_signed_off: does not contain a valid Signed-off-by line

1f1438e: put notify and get notify

  • check_signed_off: does not contain a valid Signed-off-by line

Please fix these problems and, if necessary, force-push new commits back up to the PR branch. Thanks!

@github-actions
Copy link

github-actions bot commented Nov 3, 2025

Hello! The Git Commit Checker CI bot found a few problems with this PR:

8fc7ada: Git Ignore update

  • check_signed_off: does not contain a valid Signed-off-by line

f37ec8f: Delete .cache/clangd/index directory

  • check_signed_off: does not contain a valid Signed-off-by line

08f38c3: Notify for Rput variants

  • check_signed_off: does not contain a valid Signed-off-by line

acf1108: Put notify and get notify

  • check_signed_off: does not contain a valid Signed-off-by line

1f1438e: put notify and get notify

  • check_signed_off: does not contain a valid Signed-off-by line

Please fix these problems and, if necessary, force-push new commits back up to the PR branch. Thanks!

@github-actions
Copy link

github-actions bot commented Nov 3, 2025

Hello! The Git Commit Checker CI bot found a few problems with this PR:

d17854f: Remove unecessary changes pushed:

  • check_signed_off: does not contain a valid Signed-off-by line

8fc7ada: Git Ignore update

  • check_signed_off: does not contain a valid Signed-off-by line

f37ec8f: Delete .cache/clangd/index directory

  • check_signed_off: does not contain a valid Signed-off-by line

08f38c3: Notify for Rput variants

  • check_signed_off: does not contain a valid Signed-off-by line

acf1108: Put notify and get notify

  • check_signed_off: does not contain a valid Signed-off-by line

1f1438e: put notify and get notify

  • check_signed_off: does not contain a valid Signed-off-by line

Please fix these problems and, if necessary, force-push new commits back up to the PR branch. Thanks!

@github-actions
Copy link

github-actions bot commented Nov 3, 2025

Hello! The Git Commit Checker CI bot found a few problems with this PR:

8fc7ada: Git Ignore update

  • check_signed_off: does not contain a valid Signed-off-by line

f37ec8f: Delete .cache/clangd/index directory

  • check_signed_off: does not contain a valid Signed-off-by line

08f38c3: Notify for Rput variants

  • check_signed_off: does not contain a valid Signed-off-by line

acf1108: Put notify and get notify

  • check_signed_off: does not contain a valid Signed-off-by line

1f1438e: put notify and get notify

  • check_signed_off: does not contain a valid Signed-off-by line

Please fix these problems and, if necessary, force-push new commits back up to the PR branch. Thanks!

@github-actions
Copy link

github-actions bot commented Nov 3, 2025

Hello! The Git Commit Checker CI bot found a few problems with this PR:

8fc7ada: Git Ignore update

  • check_signed_off: does not contain a valid Signed-off-by line

f37ec8f: Delete .cache/clangd/index directory

  • check_signed_off: does not contain a valid Signed-off-by line

08f38c3: Notify for Rput variants

  • check_signed_off: does not contain a valid Signed-off-by line

acf1108: Put notify and get notify

  • check_signed_off: does not contain a valid Signed-off-by line

1f1438e: put notify and get notify

  • check_signed_off: does not contain a valid Signed-off-by line

Please fix these problems and, if necessary, force-push new commits back up to the PR branch. Thanks!

@github-actions
Copy link

github-actions bot commented Nov 3, 2025

Hello! The Git Commit Checker CI bot found a few problems with this PR:

d9f5d2b: Cleaning up the PR

  • check_signed_off: does not contain a valid Signed-off-by line

8fc7ada: Git Ignore update

  • check_signed_off: does not contain a valid Signed-off-by line

f37ec8f: Delete .cache/clangd/index directory

  • check_signed_off: does not contain a valid Signed-off-by line

08f38c3: Notify for Rput variants

  • check_signed_off: does not contain a valid Signed-off-by line

acf1108: Put notify and get notify

  • check_signed_off: does not contain a valid Signed-off-by line

1f1438e: put notify and get notify

  • check_signed_off: does not contain a valid Signed-off-by line

Please fix these problems and, if necessary, force-push new commits back up to the PR branch. Thanks!

@devreal
Copy link
Owner

devreal commented Nov 3, 2025

Closing this, please open a new PR from a fresh branch.

@devreal devreal closed this Nov 3, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.