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

v2.x: oshmem: Align OSHMEM API with spec v1.3 (nonblocking put/get) (this PR depends on #913) #1041

Closed
wants to merge 8 commits into from

Conversation

igor-ivanov
Copy link
Member

@igor-ivanov
Copy link
Member Author

:bot🏷️enhancement

@mellanox-github
Copy link

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

@igor-ivanov
Copy link
Member Author

@miked-mellanox @alex-mikheev it seems that v2.x(spml/ucx) does not have open-mpi/ompi@f627608e4 when trunk and v1.10 does.
Do you see any issue with this commit in v2.x?

@mike-dubman
Copy link
Member

@igor-ivanov - could you please pr it into v2.x
thanks

@igor-ivanov
Copy link
Member Author

@miked-mellanox I found related PR(v2.x) as #913
in waiting state.
@hppritcha @jsquyres can we apply #913

@igor-ivanov
Copy link
Member Author

:bot:milestone:v2.1.0

@ompiteam-bot ompiteam-bot modified the milestones: v2.1.0, v2.0.0 Mar 22, 2016
@hppritcha
Copy link
Member

@igor-ivanov since we don't currently have a separate 2.1.0 branch, we aren't merging in PR's with milestone 2.1.0 till after 2.0.0 is released.

@igor-ivanov
Copy link
Member Author

@hppritcha it is ok, just this one should be on top of #913

@hppritcha
Copy link
Member

I'm confused about what's going on with respect to shmem.h.in with this PR and PR 1042.

In particular, whats going on with
7a53bad
and
939eaad

?
👎

until this is clarified.

@hppritcha
Copy link
Member

bot:retest

@hppritcha
Copy link
Member

Can someone fix UCX so MLNX jenkins passes?

@mellanox-github
Copy link

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

@igor-ivanov
Copy link
Member Author

@hppritcha i see issue with shmem.h.in thanks. (Fixed)
ucx failure relates #913 (this PR depens on one )

@igor-ivanov
Copy link
Member Author

@hppritcha just to double check. Fix for issue in shmem.h.in was provided (see comment above #1041 (comment)). Do you see another issue?

@ggouaillardet
Copy link
Contributor

:bot:retest

@mellanox-github
Copy link

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

@artpol84
Copy link
Contributor

artpol84 commented Aug 7, 2016

bot:mellanox:retest

@mellanox-github
Copy link

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

@artpol84
Copy link
Contributor

artpol84 commented Aug 7, 2016

@igor-ivanov looks like a compiler issue. Can you please check?

@artpol84
Copy link
Contributor

@igor-ivanov did you had a chance to investigate?

@artpol84
Copy link
Contributor

artpol84 commented Sep 27, 2016

Was backed up by open-mpi/ompi#2095, please close it to avoid confusion.

@jsquyres jsquyres closed this Sep 27, 2016
@mellanox-github
Copy link

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

@mellanox-github
Copy link

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

@mellanox-github
Copy link

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

@mellanox-github
Copy link

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

@mellanox-github
Copy link

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

@mellanox-github
Copy link

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

@mellanox-github
Copy link

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

@mellanox-github
Copy link

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

@mellanox-github
Copy link

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

@mellanox-github
Copy link

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

@mellanox-github
Copy link

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

@mellanox-github
Copy link

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

@mellanox-github
Copy link

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

@mellanox-github
Copy link

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

@mellanox-github
Copy link

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

@mellanox-github
Copy link

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

@mellanox-github
Copy link

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

@mellanox-github
Copy link

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

@mellanox-github
Copy link

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

@mellanox-github
Copy link

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

@mellanox-github
Copy link

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

@mellanox-github
Copy link

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

@mellanox-github
Copy link

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

@mellanox-github
Copy link

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

@mellanox-github
Copy link

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

@mellanox-github
Copy link

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

@mellanox-github
Copy link

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

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

8 participants