Skip to content

Conversation

@janjust
Copy link
Contributor

@janjust janjust commented Jan 22, 2020

Improves multi-threaded performance by adding the option to create
multiple ucx workers in threaded applications.

Co-authored with:
Artem Y. Polyakov [email protected],
Manjunath Gorentla Venkata [email protected]

Signed-off-by: Tomislav Janjusic [email protected]
(cherry picked from commit 3d6bf9f)

@janjust
Copy link
Contributor Author

janjust commented Jan 22, 2020

FYI @artpol84 @jladd-mlnx @yosefe

The v4.0.x multi-worker PR (need to wait for #7327 to go in before merging this PR)

@gpaulsen
Copy link
Member

@janjust Once we merge in #7327 will we need to rebase this PR, or just re-test?
And is that the only reason the LABEL:WIP_DNM! is on this PR or is there another reason?

@janjust
Copy link
Contributor Author

janjust commented Jan 23, 2020

@gpaulsen We’ll need to rebase, and yes that’s the only reason LABEL:WIP_DNM! is on this PR.

@gpaulsen gpaulsen added this to the v4.0.3 milestone Jan 23, 2020
@janjust janjust force-pushed the v4.0.x-oshmem-perf-multi-worker branch from ac9a1e4 to d1f8735 Compare January 23, 2020 19:40
@janjust
Copy link
Contributor Author

janjust commented Jan 23, 2020

@artpol84, @jladd-mlnx Just fyi, this PR needs the commit from #7332

@janjust janjust force-pushed the v4.0.x-oshmem-perf-multi-worker branch from bb51c28 to 1cef05b Compare January 23, 2020 20:04
@artpol84 artpol84 requested a review from hoopoepg January 23, 2020 20:08
@artpol84
Copy link
Contributor

@hoopoepg we should have someone from UCX originally on #7274,
please have a look at this one.

Tomislav Janjusic and others added 2 commits January 24, 2020 17:29
applications.

Improves multi-threaded performance by adding the option to create
multiple ucx workers in threaded applications.

Co-authored with:
Artem Y. Polyakov <[email protected]>,
Manjunath Gorentla Venkata <[email protected]>

Signed-off-by: Tomislav Janjusic <[email protected]>
(cherry picked from commit 3d6bf9f)
- fixed sizeof(char***) by variable datatype
- fixed resorce leak in proc_add

Signed-off-by: Sergey Oblomov <[email protected]>
(cherry picked from commit 8543860)
@janjust janjust force-pushed the v4.0.x-oshmem-perf-multi-worker branch from 1cef05b to 91ab0e2 Compare January 24, 2020 15:30
@janjust
Copy link
Contributor Author

janjust commented Jan 24, 2020

@jladd-mlnx @gpaulsen @hppritcha
PR rebased, ready to go

@gpaulsen gpaulsen merged commit 2549ba2 into open-mpi:v4.0.x Jan 24, 2020
@janjust janjust deleted the v4.0.x-oshmem-perf-multi-worker branch March 27, 2025 02:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants