Skip to content

Commit 566e6f1

Browse files
committed
btl/usnic: remove legacy code
Remove compatibility code for multiple versions of BTL_IN_OPAL, BTL_VERSION, and RCACHE_VERSION. This stuff was really only necessary when we were actively swapping code between multiple release branches that had large variations in core OMPI infrastructure. These large variations have now been around for quite a while, so the need for this "compat" layer is significantly reduced. It hasn't been removed simply because a few of the "compat" names a slightly more friendly than the real names (e.g., the SEND/RECV/PUT names). Signed-off-by: Jeff Squyres <jsquyres@cisco.com>
1 parent 8a24416 commit 566e6f1

14 files changed

+62
-760
lines changed

opal/mca/btl/usnic/Makefile.am

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -25,7 +25,7 @@
2525
# $HEADER$
2626
#
2727

28-
AM_CPPFLAGS = -DBTL_IN_OPAL=1 $(opal_ofi_CPPFLAGS) -DOMPI_LIBMPI_NAME=\"$(OMPI_LIBMPI_NAME)\"
28+
AM_CPPFLAGS = $(opal_ofi_CPPFLAGS) -DOMPI_LIBMPI_NAME=\"$(OMPI_LIBMPI_NAME)\"
2929

3030
EXTRA_DIST = README.txt README.test
3131

opal/mca/btl/usnic/README.txt

Lines changed: 12 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -290,9 +290,18 @@ November 2014 / SC 2014
290290
Update February 2015
291291

292292
The usnic BTL code has been unified across master and the v1.8
293-
branches. That is, you can copy the code from
294-
v1.8:ompi/mca/btl/usnic/* to master:opal/mca/btl/usnic*, and then only
295-
have to make 3 changes in the resulting code in master:
293+
branches.
294+
295+
NOTE: As of May 2018, this is no longer true. This was generally
296+
only necessary back when the BTLs were moved from the OMPI layer to
297+
the OPAL layer. Now that the BTLs have been down in OPAL for
298+
several years, this tomfoolery is no longer necessary. This note
299+
is kept for historical purposes, just in case someone needs to go
300+
back and look at the v1.8 series.
301+
302+
That is, you can copy the code from v1.8:ompi/mca/btl/usnic/* to
303+
master:opal/mca/btl/usnic*, and then only have to make 3 changes in
304+
the resulting code in master:
296305

297306
1. Edit Makefile.am: s/ompi/opal/gi
298307
2. Edit configure.m4: s/ompi/opal/gi

opal/mca/btl/usnic/btl_usnic.h

Lines changed: 1 addition & 17 deletions
Original file line numberDiff line numberDiff line change
@@ -12,7 +12,7 @@
1212
* All rights reserved.
1313
* Copyright (c) 2006 Sandia National Laboratories. All rights
1414
* reserved.
15-
* Copyright (c) 2011-2016 Cisco Systems, Inc. All rights reserved.
15+
* Copyright (c) 2011-2019 Cisco Systems, Inc. All rights reserved
1616
* Copyright (c) 2015-2016 Los Alamos National Security, LLC. All rights
1717
* reserved.
1818
* $COPYRIGHT$
@@ -36,24 +36,13 @@
3636
#include "opal/class/opal_hash_table.h"
3737
#include "opal/mca/event/event.h"
3838

39-
#if BTL_IN_OPAL
4039
#include "opal/mca/btl/btl.h"
4140
#include "opal/mca/btl/base/btl_base_error.h"
4241
#include "opal/mca/btl/base/base.h"
4342
#include "opal/mca/rcache/rcache.h"
4443

4544
#include "btl_usnic_compat.h"
4645

47-
#if RCACHE_VERSION < 30
48-
#include "opal/mca/mpool/grdma/mpool_grdma.h"
49-
#endif
50-
#else
51-
#include "ompi/mca/btl/btl.h"
52-
#include "ompi/mca/btl/base/btl_base_error.h"
53-
#include "ompi/mca/btl/base/base.h"
54-
#include "ompi/mca/mpool/grdma/mpool_grdma.h"
55-
#endif
56-
5746
BEGIN_C_DECLS
5847

5948
/*
@@ -164,16 +153,11 @@ typedef struct opal_btl_usnic_component_t {
164153
/** list of usnic proc structures */
165154
opal_list_t usnic_procs;
166155

167-
#if RCACHE_VERSION == 30
168156
/** memory pool hints */
169157
char* usnic_mpool_hints;
170158

171159
/** registration cache name */
172160
char *usnic_rcache_name;
173-
#else
174-
/** name of memory pool */
175-
char* usnic_mpool_name;
176-
#endif
177161

178162
char *if_include;
179163
char *if_exclude;

0 commit comments

Comments
 (0)