Skip to content

Commit c4a93ec

Browse files
committed
Remove support for C/R
Remove the base OPAL support for C/R - frameworks: opal/mca/crs, ompi/mca/crcp - components: ompi/mca/pml/crcpw - tools: opal-checkpoint and opal-restart. Remove the FT event from all components and bump the MCA version of components and modules that lost the ft_event. Clean all the platform files and remove all references to non-existent frameworks (mostly crs, filem and snapc). This is scheduled for OMPI 5.0, we don't need to maintain backward compatibility for our internal MCA with prior versions, so the approach implemented here should be good enough. Signed-off-by: George Bosilca <[email protected]>
1 parent 8f8e9b8 commit c4a93ec

File tree

668 files changed

+520
-21182
lines changed

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

668 files changed

+520
-21182
lines changed

.gitignore

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -185,8 +185,6 @@ ompi/mca/coll/ml/coll_ml_lex.c
185185
ompi/mca/coll/self/coll-self-version.h*
186186
ompi/mca/coll/sm/coll-sm-version.h*
187187

188-
ompi/mca/crcp/ompi_crcp.7
189-
190188
ompi/mca/io/romio321/romio/adio/include/romioconf.h
191189
ompi/mca/io/romio321/romio/include/mpio.h
192190
ompi/mca/io/romio321/romio/localdefs

NEWS

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -79,6 +79,7 @@ Master (not on release branches yet)
7979
- Do not build Open SHMEM layer when there are no SPMLs available.
8080
Currently, this means the Open SHMEM layer will only build if
8181
a MXM or UCX library is found.
82+
- Remove all vestiges of the C/R support
8283

8384
4.0.5 -- August, 2020
8485
---------------------

config/opal_config_files.m4

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,9 @@
11
# -*- shell-script -*-
22
#
33
# Copyright (c) 2009-2010 Cisco Systems, Inc. All rights reserved.
4+
# Copyright (c) 2020 The University of Tennessee and The University
5+
# of Tennessee Research Foundation. All rights
6+
# reserved.
47
# $COPYRIGHT$
58
#
69
# Additional copyrights may follow
@@ -23,7 +26,5 @@ AC_DEFUN([OPAL_CONFIG_FILES],[
2326
opal/tools/wrappers/opalcc-wrapper-data.txt
2427
opal/tools/wrappers/opalc++-wrapper-data.txt
2528
opal/tools/wrappers/opal.pc
26-
opal/tools/opal-checkpoint/Makefile
27-
opal/tools/opal-restart/Makefile
2829
])
2930
])

config/opal_configure_options.m4

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -512,9 +512,6 @@ OPAL_WITH_OPTION_MIN_MAX_VALUE(port_name, 1024, 255, 2048)
512512
# Min length accroding to MPI-2.1, p. 418
513513
OPAL_WITH_OPTION_MIN_MAX_VALUE(datarep_string, 128, 64, 256)
514514

515-
AC_DEFINE_UNQUOTED([OPAL_ENABLE_CRDEBUG], [0],
516-
[Whether we want checkpoint/restart enabled debugging functionality or not])
517-
518515
# some systems don't want/like getpwuid
519516
AC_MSG_CHECKING([if want getpwuid support])
520517
AC_ARG_ENABLE([getpwuid],

config/opal_setup_ft.m4

Lines changed: 4 additions & 90 deletions
Original file line numberDiff line numberDiff line change
@@ -25,9 +25,6 @@ dnl
2525
# /* ULFM Specific sections */
2626
# #if OPAL_ENABLE_FT_MPI == 0 /* FT ULFM Disabled */
2727
# #if OPAL_ENABLE_FT_MPI == 1 /* FT ULFM Enabled */
28-
# /* CR Specific sections */
29-
# #if OPAL_ENABLE_FT_CR == 0 /* FT Ckpt/Restart Disabled */
30-
# #if OPAL_ENABLE_FT_CR == 1 /* FT Ckpt/Restart Enabled */
3128
#
3229

3330
# This macro is necessary to get the title to be displayed first. :-)
@@ -40,30 +37,9 @@ AC_DEFUN([OPAL_SETUP_FT_OPTIONS],[
4037
opal_setup_ft_options="yes"
4138
AC_ARG_WITH(ft,
4239
[AC_HELP_STRING([--with-ft=TYPE],
43-
[Specify the type of fault tolerance to enable. Options: mpi (ULFM), LAM (LAM/MPI-like), cr (Checkpoint/Restart) (default: mpi)])],
40+
[Specify the type of fault tolerance to enable. Options: mpi (ULFM), (default: mpi)])],
4441
[],
4542
[with_ft=auto]) # If not specified act as if --with-ft=mpi, but make external prte support failure only if hard requested
46-
47-
#
48-
# Checkpoint/restart enabled debugging
49-
#
50-
AC_ARG_ENABLE([crdebug],
51-
[AC_HELP_STRING([--enable-crdebug],
52-
[enable checkpoint/restart debugging functionality (default: disabled)])])
53-
54-
#
55-
# Fault Tolerance Thread
56-
#
57-
# --enable-ft-thread
58-
# #if OPAL_ENABLE_FT_THREAD == 0 /* Disabled */
59-
# #if OPAL_ENABLE_FT_THREAD == 1 /* Enabled */
60-
#
61-
AC_ARG_ENABLE([ft_thread],
62-
[AC_HELP_STRING([--disable-ft-thread],
63-
[Disable fault tolerance thread running inside all processes. Requires OPAL thread support (default: enabled)])],
64-
[enable_ft_thread="$enableval"],
65-
[enable_ft_thread="undef"])
66-
6743
])
6844

6945
AC_DEFUN([OPAL_SETUP_FT],[
@@ -74,7 +50,6 @@ AC_DEFUN([OPAL_SETUP_FT],[
7450
7551
if test x"$with_ft" != "xno"; then
7652
opal_want_ft=1
77-
opal_want_ft_cr=0
7853
opal_want_ft_mpi=0
7954
opal_want_ft_type=none
8055
@@ -83,6 +58,9 @@ AC_DEFUN([OPAL_SETUP_FT],[
8358
for opt in $with_ft; do
8459
IFS=$as_save_IFS
8560
61+
if test "$opt" = "LAM" || test "$opt" = "lam" || test "$opt" = "CR" || test "$opt" = "cr"; then
62+
AC_MSG_RESULT([Support for C/R FT has been removed in OMPI 5.0])
63+
fi
8664
# Default value
8765
if test "$opt" = "auto"; then
8866
opal_want_ft_mpi=1
@@ -96,23 +74,13 @@ AC_DEFUN([OPAL_SETUP_FT],[
9674
opal_want_ft_mpi=1
9775
elif test "$opt" = "mpi"; then
9876
opal_want_ft_mpi=1
99-
elif test "$opt" = "LAM"; then
100-
opal_want_ft_cr=1
101-
elif test "$opt" = "lam"; then
102-
opal_want_ft_cr=1
103-
elif test "$opt" = "CR"; then
104-
opal_want_ft_cr=1
105-
elif test "$opt" = "cr"; then
106-
opal_want_ft_cr=1
10777
else
10878
AC_MSG_RESULT([Unrecognized FT TYPE: $opt])
10979
AC_MSG_ERROR([Cannot continue])
11080
fi
11181
done
11282
if test "$opal_want_ft_mpi" = 1; then
11383
opal_want_ft_type="mpi"
114-
elif test "$opal_want_ft_cr" = 1; then
115-
opal_want_ft_type="cr"
11684
fi
11785
11886
# If we use external PRTE, does it support FT?
@@ -130,7 +98,6 @@ AC_DEFUN([OPAL_SETUP_FT],[
13098
AC_MSG_WARN([*** DISABLING FAULT TOLERANCE SUPPORT. *])
13199
AC_MSG_WARN([**************************************************])
132100
opal_want_ft_mpi=0
133-
opal_want_ft_cr=0
134101
opal_want_ft_type="none"
135102
])
136103
])
@@ -147,7 +114,6 @@ AC_DEFUN([OPAL_SETUP_FT],[
147114
else
148115
opal_want_ft=0
149116
opal_want_ft_mpi=0
150-
opal_want_ft_cr=0
151117
if test "$opal_setup_ft_options" = "yes"; then
152118
AC_MSG_RESULT([Disabled fault tolerance])
153119
fi
@@ -156,73 +122,21 @@ AC_DEFUN([OPAL_SETUP_FT],[
156122
[Enable fault tolerance general components and logic])
157123
AC_DEFINE_UNQUOTED([OPAL_ENABLE_FT_MPI], [$opal_want_ft_mpi],
158124
[Enable fault tolerance MPI ULFM components and logic])
159-
AC_DEFINE_UNQUOTED([OPAL_ENABLE_FT_CR], [$opal_want_ft_cr],
160-
[Enable fault tolerance checkpoint/restart components and logic])
161125
AM_CONDITIONAL(WANT_FT, test "$opal_want_ft" = "1")
162126
AM_CONDITIONAL(WANT_FT_MPI, test "$opal_want_ft_mpi" = "1")
163-
AM_CONDITIONAL(WANT_FT_CR, test "$opal_want_ft_cr" = "1")
164127
165128
if test "$opal_setup_ft_options" = "yes"; then
166129
AC_MSG_CHECKING([if want checkpoint/restart enabled debugging option])
167130
fi
168131
if test "$opal_want_ft" = "0"; then
169-
opal_want_prd=0
170132
if test "$opal_setup_ft_options" = "yes"; then
171133
AC_MSG_RESULT([Disabled (fault tolerance disabled --without-ft)])
172134
fi
173-
elif test "$enable_crdebug" = "yes"; then
174-
opal_want_prd=1
175-
AC_MSG_RESULT([Enabled])
176135
else
177-
opal_want_prd=0
178136
if test "$opal_setup_ft_options" = "yes"; then
179137
AC_MSG_RESULT([Disabled])
180138
fi
181139
fi
182-
AC_DEFINE_UNQUOTED([OPAL_ENABLE_CRDEBUG], [$opal_want_prd],
183-
[Whether we want checkpoint/restart enabled debugging functionality or not])
184140
185-
if test "$opal_setup_ft_options" = "yes"; then
186-
AC_MSG_CHECKING([if want fault tolerance thread])
187-
fi
188-
# if they do not want FT support, then they do not want this thread either
189-
if test "$opal_want_ft" = "0"; then
190-
opal_want_ft_thread=0
191-
if test "$opal_setup_ft_options" = "yes"; then
192-
AC_MSG_RESULT([Disabled (fault tolerance disabled --without-ft)])
193-
fi
194-
# if --disable-ft-thread
195-
elif test "$enable_ft_thread" = "no"; then
196-
opal_want_ft_thread=0
197-
AC_MSG_RESULT([Disabled])
198-
# if default, and no progress or MPI threads
199-
elif test "$enable_ft_thread" = "undef" && test "$enable_opal_multi_threads" = "no" ; then
200-
opal_want_ft_thread=0
201-
AC_MSG_RESULT([Disabled (OPAL Thread Support Disabled)])
202-
# if default, and MPI threads enabled for C/R only
203-
elif test "$opal_want_ft_cr" = 1; then
204-
# Default: Enable
205-
# Make sure we have OPAL Threads enabled
206-
if test "$enable_opal_multi_threads" = "no"; then
207-
AC_MSG_RESULT([Must enable OPAL basic thread support to use this option])
208-
AC_MSG_ERROR([Cannot continue])
209-
else
210-
AC_MSG_RESULT([yes])
211-
opal_want_ft_thread=1
212-
AC_MSG_WARN([**************************************************])
213-
AC_MSG_WARN([*** Fault Tolerance with a thread in Open MPI *])
214-
AC_MSG_WARN([*** is an experimental, research quality option. *])
215-
AC_MSG_WARN([*** It requires OPAL thread support and care *])
216-
AC_MSG_WARN([*** should be used when enabling these options. *])
217-
AC_MSG_WARN([**************************************************])
218-
fi
219-
# Otherwise disabled
220-
else
221-
opal_want_ft_thread=0
222-
AC_MSG_RESULT([Disabled (Non-C/R Fault Tolerance enabled)])
223-
fi
224-
AC_DEFINE_UNQUOTED([OPAL_ENABLE_FT_THREAD], [$opal_want_ft_thread],
225-
[Enable fault tolerance thread in Open PAL])
226-
AM_CONDITIONAL(WANT_FT_THREAD, test "$opal_want_ft_thread" = "1")
227141
OPAL_SUMMARY_ADD([[Miscellaneous]],[[Fault Tolerance support]],[unnecessary], [$opal_want_ft_type])
228142
])

contrib/Makefile.am

Lines changed: 1 addition & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,7 @@
22
# Copyright (c) 2004-2010 The Trustees of Indiana University and Indiana
33
# University Research and Technology
44
# Corporation. All rights reserved.
5-
# Copyright (c) 2004-2005 The University of Tennessee and The University
5+
# Copyright (c) 2004-2020 The University of Tennessee and The University
66
# of Tennessee Research Foundation. All rights
77
# reserved.
88
# Copyright (c) 2004-2009 High Performance Computing Center Stuttgart,
@@ -31,12 +31,6 @@ if WANT_FT_MPI
3131
dist_amca_param_DATA += amca-param-sets/ft-mpi
3232
endif # WANT_FT_MPI
3333

34-
if WANT_FT_CR
35-
dist_amca_param_DATA += \
36-
amca-param-sets/ft-enable-cr \
37-
amca-param-sets/ft-enable-cr-recovery
38-
endif # WANT_FT_CR
39-
4034
EXTRA_DIST = \
4135
completion/mpirun.sh \
4236
completion/mpirun.zsh \

contrib/amca-param-sets/ft-enable-cr

Lines changed: 0 additions & 66 deletions
This file was deleted.

contrib/amca-param-sets/ft-enable-cr-recovery

Lines changed: 0 additions & 81 deletions
This file was deleted.

0 commit comments

Comments
 (0)