Re: [OMPI devel] 1.8.7 rc1 out for review

2015-07-10 Thread Paul Hargrove
Gilles, Applying both of your patches to the 1.8.7rc1 tarball appears to give the desired results on both of the systems I tested. -Paul On Thu, Jul 9, 2015 at 9:39 PM, Paul Hargrove wrote: > Gilles, > > I see now that your patches aren't applying because they have DOS line > endings (CRLF vs

Re: [OMPI devel] 1.8.7 rc1 out for review

2015-07-10 Thread Paul Hargrove
Gilles, If I am correctly understanding the purpose of deprecating the old interfaces then the openib btl will *eventually* need to support !OMPI_HAVE_CONNECTX_XRC && OMPI_HAVE_CONNECTX_XRC_DOMAINS However, I am not in a position to decide if that is really required for 1.8.x or not. -Paul

Re: [OMPI devel] 1.8.7 rc1 out for review

2015-07-10 Thread Paul Hargrove
Gilles, I see now that your patches aren't applying because they have DOS line endings (CRLF vs LF). I need to strip those to get the patches applied. I will report results with your patch on both my "old" and "new" systems as soon as possible. -Paul On Thu, Jul 9, 2015 at 9:26 PM, Gilles Gouai

Re: [OMPI devel] 1.8.7 rc1 out for review

2015-07-10 Thread Gilles Gouaillardet
Paul, the openib btl need to be reworked ... currently, we have #define HAVE_XRC (1 == OMPI_HAVE_CONNECTX_XRC) /* and it is impossible to have !OMPI_HAVE_CONNECTX_XRC && OMPI_HAVE_CONNECTX_XRC_DOMAINS */ but with your patch, this becomes possible and HAVE_XRC would be zero, which is incorrec

Re: [OMPI devel] 1.8.7 rc1 out for review

2015-07-10 Thread Gilles Gouaillardet
Paul, i just applied the patch on the tarball, and it worked for me. anyway, the IBV_SRQT_XRC test was misplaced (and i just read you already found out ...) we need if for XRC_DOMAINS and *not* for XRC the newly attached patch will (hopefully) fix this Cheers, Gilles On 7/10/2015 11:06 AM,

Re: [OMPI devel] 1.8.7 rc1 out for review

2015-07-10 Thread Paul Hargrove
Gilles, I've made another observation about what I believe is an error in the XRC configure probe. If I am following the code below correctly, then *both* ConnectX and ConnectIB depend on ibv_create_xrc_rcv_qp being defined. However, that function is marked as deprecated (presumably in favor of i

Re: [OMPI devel] 1.8.7 rc1 out for review

2015-07-09 Thread Paul Hargrove
Gilles, The patch didn't apply to the 1.8.7rc1 tarball. So, I made the change manually and ran autogen.pl. The result is that one fewer configure test runs, but "ConnectX XRC support" is still disabled: Diffing the configure output: checking for ibv_resize_cq... yes checking for struct ibv_dev

Re: [OMPI devel] 1.8.7 rc1 out for review

2015-07-09 Thread Gilles Gouaillardet
Thanks Paul, i just found an other bug ... (and i should be blamed for it) here is attached a patch. basically, xrc was incorrectly disabled on "older" ofed stacks Cheers, Gilles On 7/10/2015 10:06 AM, Paul Hargrove wrote: Gilles, A bzip2-compressed config.log is attached. I am unsure ho

Re: [OMPI devel] 1.8.7 rc1 out for review

2015-07-09 Thread Paul Hargrove
Gilles, A bzip2-compressed config.log is attached. I am unsure how to determine the OFED version, because the admins have prevented normal users from reading the RPM database. Perhaps the following helps: $ nm /usr/lib64/libibverbs.a | grep -i xrc 00e0 T ibv_cmd_close_xrc_domain

Re: [OMPI devel] 1.8.7 rc1 out for review

2015-07-09 Thread Gilles Gouaillardet
Paul, can you please compress and post your config.log ? what is the OFED version you are running ? on master, that fix did the trick on mellanox test cluster (recent OFED version) but did not enable XRC on lanl test clusters (my best bet is an old OFED library) Thanks Gilles On 7/10/2015 9

Re: [OMPI devel] 1.8.7 rc1 out for review

2015-07-09 Thread Paul Hargrove
Preliminary report: 1) I find that "ConnectX XRC support" is still not detected as it was in 1.8.4 and earlier: $ grep 'ConnectX XRC support' openmpi-1.*-icc-14/LOG/configure.log| sort -u openmpi-1.8-linux-x86_64-icc-14/LOG/configure.log:checking if ConnectX XRC support is enabled... yes openmp

[OMPI devel] 1.8.7 rc1 out for review

2015-07-09 Thread Ralph Castain
In the usual place: http://www.open-mpi.org/software/ompi/v1.8/ Please test and let me know of any issues that surface. My intent is to release this next week. Ralph