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

UCS/TEST: Remove entry from configuration list when library is unloaded - v1.5.x #3656

Conversation

yosefe
Copy link
Contributor

@yosefe yosefe commented Jun 2, 2019

Fixes #3558
@opoplawski FYI

@swx-jenkins1
Copy link

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

 When UCT is loaded, the static initializer adds it's configuration
tables to the global list in UCS. When UCT is unloaded, need to remove
them from that list, otherwise the list will point to invalid memory.
@yosefe yosefe force-pushed the topic/ucs-remove-from-global-config-on-unload-v1.5.x branch from 0a5b668 to e6d1d5c Compare June 2, 2019 15:54
@mellanox-github
Copy link
Contributor

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

@swx-jenkins1
Copy link

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

@mellanox-github
Copy link
Contributor

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

@yosefe yosefe changed the title UCS/TEST: Remove entry from configuration list when library is unloaded UCS/TEST: Remove entry from configuration list when library is unloaded - v1.5.x Jun 3, 2019
@shamisp
Copy link
Contributor

shamisp commented Jun 3, 2019

@yosefe - are we planning another v1.5.x ?

@yosefe
Copy link
Contributor Author

yosefe commented Jun 3, 2019

replaced by #3662

@yosefe yosefe closed this Jun 3, 2019
@yosefe yosefe deleted the topic/ucs-remove-from-global-config-on-unload-v1.5.x branch July 20, 2019 12:39
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.

4 participants