Conversation
| index 17239ce8ef..ee5c913551 100644 | ||
| --- a/opal/mca/btl/vader/btl_vader_fbox.h | ||
| +++ b/opal/mca/btl/vader/btl_vader_fbox.h | ||
| @@ -138,9 +138,6 @@ static inline bool mca_btl_vader_fbox_sendi (mca_btl_base_endpoint_t *ep, unsign |
There was a problem hiding this comment.
ok, but the original patch seems to contain a bit more? Can this be ignored?
https://github.com/open-mpi/ompi/pull/5705/files
There was a problem hiding this comment.
The original patch combines two suggested patches from the original issue 1, 2. The comments mention the first one did not solve the issue, so I tried the second and it worked for the versions tested (2.0.5, 3.0.2, 3.1.2). I briefly tried building with the entire patch, but found that vader's source had diverged enough that it would require a separate patch for each minor release.
|
I'm fairly certain a segfault I see with 3.1.2 is the vader issue so it would be great to merge this. |
|
Thanks for the clarification @sknigh and the effort you put into this! |
4 participants
Vader bug causes some MPI codes to sporadically segfault.