Skip to content

rfc13: add PMI 1.2 group barrier - #550

Merged
mergify[bot] merged 4 commits into
flux-framework:masterfrom
garlick:pmi1.2
Sep 2, 2026
Merged

rfc13: add PMI 1.2 group barrier#550
mergify[bot] merged 4 commits into
flux-framework:masterfrom
garlick:pmi1.2

Conversation

@garlick

@garlick garlick commented Sep 2, 2026

Copy link
Copy Markdown
Member

Problem: the recent PMI 1.2 group barrier introduced in MPICH to support MPI sessions is not described in RFC 13.

Add it.

@github-actions

github-actions Bot commented Sep 2, 2026

Copy link
Copy Markdown

⚠️ linkcheck failed with status code 2

@github-actions

github-actions Bot commented Sep 2, 2026

Copy link
Copy Markdown

⚠️ linkcheck failed with status code 2

1 similar comment
@github-actions

github-actions Bot commented Sep 2, 2026

Copy link
Copy Markdown

⚠️ linkcheck failed with status code 2

@grondo grondo left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

Problem: RFC 13 does not document the PMI 1.2 PMI_Barrier_group()
API function or its group constants.

Add the function.

Assisted-by: Claude:claude-opus-4.8
Problem: the PMI 1.2 wire protocol defines additional barrier attributes
to support PMI_barrier_group(), but this is not documented in the
RFC 13 ABNF.

Add it.
Problem: the ABNF is not sufficient to guide an implementation of
PMI 1.2 group barriers.

Add some prose, informed by reviewing MPICH 5's PMI_Barrier_group()
implementation.
Problem: The Back Compatibility section does not capture the
implications of the updated version handshake description.

Update this section.
@mergify

mergify Bot commented Sep 2, 2026

Copy link
Copy Markdown
Contributor

Queued — the merge queue status continues in this comment ↓.

@github-actions

github-actions Bot commented Sep 2, 2026

Copy link
Copy Markdown

⚠️ linkcheck failed with status code 2

@mergify

mergify Bot commented Sep 2, 2026

Copy link
Copy Markdown
Contributor

Merge Queue Status

  • Entered queue2026-09-02 21:35 UTC · Rule: default · triggered by rule rebase and merge when passing all checks
  • Checks skipped · PR is already up-to-date
  • Merged2026-09-02 21:35 UTC · at 99ab6c5547df873a25b0562c76fb91b81f726365 · merge

This pull request spent 17 seconds in the queue, including 3 seconds running CI.

Required conditions to merge
  • github-review-approved [🛡 GitHub branch protection]
  • any of [🛡 GitHub branch protection]:
    • check-success = docs/readthedocs.org:flux-rfc
    • check-neutral = docs/readthedocs.org:flux-rfc
    • check-skipped = docs/readthedocs.org:flux-rfc
  • any of [🛡 GitHub branch protection]:
    • check-success = make check
    • check-neutral = make check
    • check-skipped = make check
  • any of [🛡 GitHub branch protection]:
    • check-success = validate commits
    • check-neutral = validate commits
    • check-skipped = validate commits

@mergify
mergify Bot merged commit 99ab6c5 into flux-framework:master Sep 2, 2026
7 of 8 checks passed
@mergify mergify Bot removed the queued label Sep 2, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants