Skip to content
This repository was archived by the owner on Sep 30, 2022. It is now read-only.

Shmem fixes (Don't merge until #1041 is merged) #1329

Closed
wants to merge 10 commits into from
Closed

Shmem fixes (Don't merge until #1041 is merged) #1329

wants to merge 10 commits into from

Conversation

@karasevb
Copy link
Member

@karasevb karasevb commented Aug 20, 2016 •
edited
Loading

Cherry-picked from open-mpi/ompi#1867 and #1041
Don't merge until #1041 is merged.

igor-ivanov and others added 10 commits August 20, 2016 13:20
@igor-ivanov @karasevb
oshmem: Align OSHMEM API with spec v1.3 (Add spml/get_nb interface)
af07e80
@igor-ivanov @karasevb
oshmem: Align OSHMEM API with spec v1.3 (Add nonblocking put/get C fu…
8621023 
…nctions)
@igor-ivanov @karasevb
oshmem: Align OSHMEM API with spec v1.3 (Add nonblocking put/get Fort…
dcbd0fd 
…ran functions)
@igor-ivanov @karasevb
oshmem: Align OSHMEM API with spec v1.3 (Add mans for nonblocking put…
f181d9d 
…/get Fortran functions)
@igor-ivanov @karasevb
oshmem: Align OSHMEM API with spec v1.3 (update shmem.h)
c4657de
@igor-ivanov @karasevb
oshmem: Align OSHMEM API with spec v1.3 (update spml/ucx)
f6c355d
@igor-ivanov @karasevb
oshmem: Align OSHMEM API with spec v1.3 (update spml/ikrit)
92e0973
@igor-ivanov @karasevb
oshmem: Align OSHMEM API with spec v1.3 (update spml/yoda)
f8fcf51
@hpcraink @karasevb
Correct "configure --help" output and amend the default setting if us…
5443fdb 
…er provides a wrong input value, like "runtime" (which works for MPI, but not for OSHMEM)
@hpcraink @karasevb
Fix missing include and missing MCA_SPML_CALL.
8a09acc
@karasevb karasevb mentioned this pull request Aug 20, 2016
Closed
@mellanox-github
Copy link

Test FAILed.
See http://bgate.mellanox.com/jenkins/job/gh-ompi-release-pr/2104/ for details.

@jladd-mlnx
Copy link
Member

@artpol84 Please have a look.

@artpol84
Copy link
Contributor

bot:milestone:v2.1.0

@ompiteam-bot ompiteam-bot added this to the v2.1.0 milestone Aug 22, 2016
@artpol84
Copy link
Contributor

bot:label:WIP-DNM

@artpol84 artpol84 mentioned this pull request Aug 24, 2016
Closed
@jsquyres
Copy link
Member

bot:mellanox:retest

@mellanox-github
Copy link

Test FAILed.
See http://bgate.mellanox.com/jenkins/job/gh-ompi-release-pr/2129/ for details.

@artpol84
Copy link
Contributor

@jsquyres this PR is ok, but the base PR is failing.

@jsquyres
Copy link
Member

jsquyres commented Aug 26, 2016 •
edited
Loading

@artpol84 Ah, you mean this one won't pass CI until #1041 is merged?

@artpol84
Copy link
Contributor

@jsquyres right!

@artpol84
Copy link
Contributor

At least I hope so - we'll see once #1041 will be merged and this commit will get a chance to be verified.

@jsquyres
Copy link
Member

bot:mellanox:retest

@mellanox-github
Copy link

Test FAILed.
See http://bgate.mellanox.com/jenkins/job/gh-ompi-release-pr/2169/ for details.

@jsquyres
Copy link
Member

@karasevb @artpol84 Getting a compile failure in in SPML UCX.

@artpol84
Copy link
Contributor

@jsquyres this is expected:

we depend on #1041 which in turn depends on #913. Won't get compiled untill those 2 are in.

@jsquyres
Copy link
Member

@artpol84 Ok

@jsquyres
Copy link
Member

The ompi-release repo is now closed; all release branches have been moved to the ompi repo (https://github.com/open-mpi/ompi). Please re-open this PR on the corresponding branch on the ompi repo.

@jsquyres jsquyres mentioned this pull request Sep 22, 2016
Closed
@artpol84 artpol84 mentioned this pull request Sep 27, 2016
Closed
@artpol84
Copy link
Contributor

ported to ompi (open-mpi/ompi#2127).
Please close this one to avoid confusion.

@jsquyres jsquyres closed this Sep 27, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

No reviews

Assignees

No one assigned

Labels

pushed-back RM-Approved WIP-DNM

Projects

None yet

Milestone

v2.1.0

Development

Successfully merging this pull request may close these issues.

9 participants

@karasevb @mellanox-github @jladd-mlnx @artpol84 @jsquyres @hppritcha @ompiteam-bot @igor-ivanov @hpcraink