Skip to content

Fix TCP BTL THREAD_MULTIPLE deadlock #1406

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

Closed
jsquyres opened this issue Feb 25, 2016 · 7 comments
Closed

Fix TCP BTL THREAD_MULTIPLE deadlock #1406

jsquyres opened this issue Feb 25, 2016 · 7 comments
Assignees
Milestone

Comments

@jsquyres
Copy link
Member

Discussed at Feb 2016 Dallas meeting. @bosilca says he knows what the problem is and will fix it.

@jsquyres
Copy link
Member Author

jsquyres commented Mar 7, 2016

@hjelmn @bosilca Did this get fixed yet, perchance?

@jsquyres
Copy link
Member Author

jsquyres commented Mar 8, 2016

@bosilca Says that he has large modifications to the TCP BTL (it's a giant change). Not clear yet on how we want to handle this...

@hppritcha
Copy link
Member

@bosilca is there a chance we can have a narrowly focused fix to the existing TCP BTL?

@jsquyres
Copy link
Member Author

@bosilca If we can't get a focused fix, how about this:

  • Have the old TCP BTL code selected on non-THREAD_MULTIPLE
  • Have your new TCP BTL code selected on THREAD_MULTIPLE

This would allow us to have somewhat newer/less-tested code in the THREAD_MULTIPLE path (and therefore make solid advances in the THREAD_MULTIPLE category, modulo bug fixes), but have solid/reliable/time-tested code in the non-THREAD_MULTIPLE case (i.e., the majority of the world).

As the new code gets more testing and we get higher confidence in it, we can remove this division / remove the old code, and just have the new code.

What do you think?

@jsquyres
Copy link
Member Author

@bosilca Any comment?

@bosilca
Copy link
Member

bosilca commented Mar 23, 2016

#1491

@jsquyres
Copy link
Member Author

Fixed via #1491.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants