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 #3658

Merged

Conversation

yosefe
Copy link
Contributor

@yosefe yosefe commented Jun 3, 2019

No description provided.

 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.
@swx-jenkins1
Copy link

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

@mellanox-github
Copy link
Contributor

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

}

/* print all config table, to force going over the global list in ucs */
void (*print_all_opts)(FILE*,int) = dlsym(ucs_handle,
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

space after ,

@mellanox-github
Copy link
Contributor

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

@swx-jenkins1
Copy link

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

@mellanox-github
Copy link
Contributor

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

@yosefe yosefe merged commit b7c78ab into openucx:master Jun 5, 2019
@yosefe yosefe deleted the topic/ucs-remove-from-global-config-on-unload branch June 5, 2019 09:15
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