-
Notifications
You must be signed in to change notification settings - Fork 423
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
Multiple fixes for UCX v1.5.2 #3662
Conversation
Test FAILed. |
Test FAILed. |
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.
- there was incorrect processing of aliases of configs
20a2f9a
to
0cb7b64
Compare
Test PASSed. |
Test PASSed. |
bot:pipe:retest |
@brminich can you pls take a look? |
} | ||
|
||
/* print all config table, to force going over the global list in ucs */ | ||
void (*print_all_opts)(FILE*,int) = dlsym(ucs_handle, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
minor: space after ,
Fixing multiple issues for redhat per https://bugzilla.redhat.com/show_bug.cgi?id=1715129