mpich icon indicating copy to clipboard operation
mpich copied to clipboard
verified publisher icon pmodels

ch4/ofi: add MPIDI_OFI_ENABLE_DATA back

Open hzhou opened this issue 4 years ago • 2 comments

Pull Request Description

In PR #2985 we merged MPIDI_OFI_ENABLE_DATA and MPIDI_OFI_ENABLE_TAG, thus making support of cq_data a requirement. This unnecessarily restricts our support to some legacy providers, gni being one of the important ones.

This PR tries to revert that change.

[skip warnings]

Author Checklist

  • [x] Provide Description Particularly focus on why, not what. Reference background, issues, test failures, xfail entries, etc.
  • [x] Commits Follow Good Practice Commits are self-contained and do not do two things at once. Commit message is of the form: module: short description Commit message explains what's in the commit.
  • [ ] Passes All Tests Whitespace checker. Warnings test. Additional tests via comments.
  • [x] Contribution Agreement For non-Argonne authors, check contribution agreement. If necessary, request an explicit comment from your companies PR approval manager.

hzhou avatar Jan 24 '22 21:01 hzhou

test:mpich/ch4/ofi

hzhou avatar Jan 24 '22 22:01 hzhou

test:mpich/ch4/ofi

hzhou avatar Jan 25 '22 14:01 hzhou

test:mpich/ch4/ofi

hzhou avatar Jan 12 '23 22:01 hzhou

test:mpich/ch4/ofi

test:mpich/custom netmod: ch4:ofi env: MPIR_CVAR_CH4_OFI_ENABLE_DATA=0

hzhou avatar Jan 23 '23 00:01 hzhou

test:mpich/spellcheck

hzhou avatar Jan 23 '23 16:01 hzhou