Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

UCX: fix #1462 (v1.2) #1505

Merged
merged 1 commit into from
May 14, 2017
Merged

UCX: fix #1462 (v1.2) #1505

merged 1 commit into from
May 14, 2017

Conversation

evgeny-leksikov
Copy link
Contributor

  • fix uninitialized UCT err handler in UCP
  • increased UD timeout, fix variable name

@yosefe i think this should be in release

- fix uninitialized UCT err handler in UCP
- increased UD timeout, fix variable name
@yosefe yosefe added this to the v1.2 - release milestone May 12, 2017
@yosefe yosefe added the Bugfix label May 12, 2017
@mellanox-github
Copy link
Contributor

Test PASSed.
See http://hpc-master.lab.mtl.com:8080/job/hpc-ucx-pr/3565/ for details (Mellanox internal link).

@mellanox-github
Copy link
Contributor

Test FAILed.
See http://bgate.mellanox.com/jenkins/job/gh-ucx-pr/1645/ for details.

@mellanox-github
Copy link
Contributor

Test FAILed.
See http://bgate.mellanox.com/jenkins/job/gh-ucx-pr/1648/ for details.

@shamisp
Copy link
Contributor

shamisp commented May 12, 2017

seems like it causes segfault on ARM

14:36:27 [----------] 1 test from udx/test_ucp_perf
14:36:27 [ RUN      ] udx/test_ucp_perf.envelope/0
14:36:27 [cavium:41271:0] Caught signal 11 (Segmentation fault: address not mapped to object at address 0x5800000009)
15:57:45 Build timed out (after 100 minutes). Marking the build as failed.
15:57:45 Build was aborted
15:57:45 TAP Reports Processing: START
15:57:45 Looking for TAP results report in workspace using pattern: **/*.tap
15:57:48 make: *** [test] Terminated

@yosefe
Copy link
Contributor

yosefe commented May 12, 2017

@shamisp it's an issue already fixed on master branch, will port fix to v1.2 next week

@shamisp
Copy link
Contributor

shamisp commented May 12, 2017

So, I guess we have to wait for the bugfix to be ported before we merge this one, right ?

@yosefe yosefe changed the title UCX: fix #1462 for v1.2 UCX: fix #1462 (v1.2) May 14, 2017
@yosefe
Copy link
Contributor

yosefe commented May 14, 2017

bot:retest

@mellanox-github
Copy link
Contributor

Test PASSed.
See http://bgate.mellanox.com/jenkins/job/gh-ucx-pr/1663/ for details.

@mellanox-github
Copy link
Contributor

Test PASSed.
See http://hpc-master.lab.mtl.com:8080/job/hpc-ucx-pr/3580/ for details (Mellanox internal link).

@yosefe yosefe merged commit 3174c0b into openucx:v1.2 May 14, 2017
@yosefe
Copy link
Contributor

yosefe commented May 21, 2017

fixes #1462

@evgeny-leksikov evgeny-leksikov deleted the ud_assert_12 branch July 4, 2017 08:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants