I will try it.

Jeff Squyres wrote:
This commit does what we previously discussed: it only compiles the XOOB openib CPC if XRC support is actually present (vs. having a stub XOOB when XRC is not present). This is on the /tmp-public/openib-cpc branch.

I have some hermon hca's, but due to dumb issues, I don't have XRC- capable OFED on those nodes yet. It'll probably take me a few more days before I have that ready.

Could someone try the openib-cpc tmp branch and ensure I didn't break the case where XRC support is available? It is easy to tell if the XOOB CPC compiled in -- run this command:

ompi_info --param btl openib --parsable | grep xoob

If the output is empty, then XOOB was not compiled in. If you see output, then XOOB was compiled in.

Thanks!



Begin forwarded message:

From: jsquy...@osl.iu.edu
Date: December 14, 2007 12:10:24 PM EST
To: svn-f...@open-mpi.org
Subject: [OMPI svn-full] svn:open-mpi r16959
Reply-To: de...@open-mpi.org

Author: jsquyres
Date: 2007-12-14 12:10:23 EST (Fri, 14 Dec 2007)
New Revision: 16959
URL: https://svn.open-mpi.org/trac/ompi/changeset/16959

Log:
Only compile in the XOOB CPC if a) configure found that we have XRC
support available and b) the user didn't disable connectx support.

Text files modified:
tmp-public/openib-cpc/config/ ompi_check_openib.m4 | 3 ++- tmp-public/openib-cpc/ompi/mca/btl/openib/ Makefile.am | 8 ++++++-- tmp-public/openib-cpc/ompi/mca/btl/openib/ configure.m4 | 8 ++++++++ tmp-public/openib-cpc/ompi/mca/btl/openib/connect/ btl_openib_connect_base.c | 2 ++ tmp-public/openib-cpc/ompi/mca/btl/openib/connect/ btl_openib_connect_xoob.c | 23 -----------------------
  5 files changed, 18 insertions(+), 26 deletions(-)

Modified: tmp-public/openib-cpc/config/ompi_check_openib.m4
= = = = = = = = ======================================================================
--- tmp-public/openib-cpc/config/ompi_check_openib.m4   (original)
+++ tmp-public/openib-cpc/config/ompi_check_openib.m4 2007-12-14 12:10:23 EST (Fri, 14 Dec 2007)
@@ -102,7 +102,8 @@
    AS_IF([test "$ompi_check_openib_happy" = "yes"],
          [AC_CHECK_DECLS([IBV_EVENT_CLIENT_REREGISTER], [], [],
                          [#include <infiniband/verbs.h>])
- AC_CHECK_FUNCS([ibv_get_device_list ibv_resize_cq ibv_open_xrc_domain])])
+           AC_CHECK_FUNCS([ibv_get_device_list ibv_resize_cq])
+           AC_CHECK_FUNCS([ibv_open_xrc_domain], [$1_have_xrc=1])])

    CPPFLAGS="$ompi_check_openib_$1_save_CPPFLAGS"
    LDFLAGS="$ompi_check_openib_$1_save_LDFLAGS"

Modified: tmp-public/openib-cpc/ompi/mca/btl/openib/Makefile.am
= = = = = = = = ======================================================================
--- tmp-public/openib-cpc/ompi/mca/btl/openib/Makefile.am       (original)
+++ tmp-public/openib-cpc/ompi/mca/btl/openib/Makefile.am 2007-12-14 12:10:23 EST (Fri, 14 Dec 2007)
@@ -55,14 +55,18 @@
    connect/btl_openib_connect_base.c \
    connect/btl_openib_connect_oob.c \
    connect/btl_openib_connect_oob.h \
-    connect/btl_openib_connect_xoob.c \
-    connect/btl_openib_connect_xoob.h \
    connect/btl_openib_connect_rdma_cm.c \
    connect/btl_openib_connect_rdma_cm.h \
    connect/btl_openib_connect_ibcm.c \
    connect/btl_openib_connect_ibcm.h \
    connect/connect.h

+if MCA_btl_openib_have_xrc
+sources += \
+    connect/btl_openib_connect_xoob.c \
+    connect/btl_openib_connect_xoob.h
+endif
+
# Make the output library in this directory, and name it either
# mca_<type>_<name>.la (for DSO builds) or libmca_<type>_<name>.la
# (for static builds).

Modified: tmp-public/openib-cpc/ompi/mca/btl/openib/configure.m4
= = = = = = = = ======================================================================
--- tmp-public/openib-cpc/ompi/mca/btl/openib/configure.m4      (original)
+++ tmp-public/openib-cpc/ompi/mca/btl/openib/configure.m4 2007-12-14 12:10:23 EST (Fri, 14 Dec 2007)
@@ -18,6 +18,14 @@
# $HEADER$
#

+# MCA_btl_openib_POST_CONFIG([should_build])
+# ------------------------------------------
+AC_DEFUN([MCA_btl_openib_POST_CONFIG], [
+    AS_IF([test $1 -eq 0 -a "$enable_dist" = "yes"],
+ [AC_MSG_ERROR([BTL openib is disabled but --enable-dist specifed. This will result in a bad tarball. Aborting configure.])]) + AM_CONDITIONAL([MCA_btl_openib_have_xrc], [test $1 -eq 1 -a "x $btl_openib_have_xrc" = "x1" -a "x$ompi_want_connectx_xrc" = "x1"])
+])
+

# MCA_btl_openib_CONFIG([action-if-can-compile],
#                      [action-if-cant-compile])

Modified: tmp-public/openib-cpc/ompi/mca/btl/openib/connect/ btl_openib_connect_base.c = = = = = = = = ====================================================================== --- tmp-public/openib-cpc/ompi/mca/btl/openib/connect/ btl_openib_connect_base.c (original) +++ tmp-public/openib-cpc/ompi/mca/btl/openib/connect/ btl_openib_connect_base.c 2007-12-14 12:10:23 EST (Fri, 14 Dec 2007)
@@ -34,7 +34,9 @@
 */
static ompi_btl_openib_connect_base_funcs_t *all[] = {
    &ompi_btl_openib_connect_oob,
+#if HAVE_XRC
    &ompi_btl_openib_connect_xoob,
+#endif
    &ompi_btl_openib_connect_rdma_cm,
    &ompi_btl_openib_connect_ibcm,
    NULL

Modified: tmp-public/openib-cpc/ompi/mca/btl/openib/connect/ btl_openib_connect_xoob.c = = = = = = = = ====================================================================== --- tmp-public/openib-cpc/ompi/mca/btl/openib/connect/ btl_openib_connect_xoob.c (original) +++ tmp-public/openib-cpc/ompi/mca/btl/openib/connect/ btl_openib_connect_xoob.c 2007-12-14 12:10:23 EST (Fri, 14 Dec 2007)
@@ -42,8 +42,6 @@
    xoob_finalize,
};

-#if HAVE_XRC
-
typedef enum {
    SEND,
    RECV
@@ -831,24 +829,3 @@
        free(rem_info->rem_srqs);
    }
}
-
-#else
-/* In case if the XRC was disabled during compilation we will print message and return error */
-static int xoob_init(void)
-{
-    printf("xoob init\n");
-    return OMPI_ERR_NOT_IMPLEMENTED;
-}
-
-static int xoob_start_connect(mca_btl_base_endpoint_t *e)
-{
-    printf("xoob start connect\n");
-    return OMPI_ERR_NOT_IMPLEMENTED;
-}
-
-static int xoob_finalize(void)
-{
-    printf("xoob finalize\n");
-    return OMPI_ERR_NOT_IMPLEMENTED;
-}
-#endif
_______________________________________________
svn-full mailing list
svn-f...@open-mpi.org
http://www.open-mpi.org/mailman/listinfo.cgi/svn-full



Reply via email to