Skip to content

Commit 7100bed

Browse files
committed
accelerator: build components as dso's by default
also need to switch rcache/gpsum and rcache/rgpusum to DSO by default. Fix a problem in opal_mca.m4 where the enable-mca-dso list wasn't being processed correctly starting with 5.0.0. related to open-mpi#12036 Signed-off-by: Howard Pritchard <[email protected]>
1 parent 554a8eb commit 7100bed

File tree

1 file changed

+11
-4
lines changed

1 file changed

+11
-4
lines changed

config/opal_mca.m4

Lines changed: 11 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -167,29 +167,36 @@ of type-component pairs. For example, --enable-mca-no-build=pml-ob1])
167167
# Second, set the DSO_all and STATIC_all variables. conflict
168168
# resolution (prefer static) is done in the big loop below
169169
#
170+
# Exception here is the components of the accelerator framework,
171+
# which by default are built to be dynamic.
172+
#
170173
AC_MSG_CHECKING([which components should be run-time loadable])
171174
if test "$enable_static" != "no"; then
172175
DSO_all=0
173176
msg="none (static libraries built)"
174177
elif test "$OPAL_ENABLE_DLOPEN_SUPPORT" = 0; then
175178
DSO_all=0
176179
msg="none (dlopen disabled)"
177-
elif test -z "$enable_mca_dso"; then
178-
DSO_all=0
179-
msg=default
180180
elif test "$enable_mca_dso" = "no"; then
181181
DSO_all=0
182182
msg=none
183183
elif test "$enable_mca_dso" = "yes"; then
184184
DSO_all=1
185185
msg=all
186186
else
187+
if test -z "$enable_mca_dso"; then
188+
enable_mca_dso="accelerator,btl-smcuda,rcache-gpusm,rcache-rgpusm"
189+
AC_MSG_NOTICE([building accelerator framework compnents as DSOs (default)])
190+
else
191+
enable_mca_dso=$enable_mca_dso
192+
fi
187193
DSO_all=0
188194
ifs_save="$IFS"
189195
IFS="${IFS}$PATH_SEPARATOR,"
190196
msg=
191197
for item in $enable_mca_dso; do
192-
AS_VAR_SET([AS_TR_SH([DSO_$item])], [1])
198+
str="`echo DSO_$item=1 | sed s/-/_/g`"
199+
eval $str
193200
msg="$item $msg"
194201
done
195202
IFS="$ifs_save"

0 commit comments

Comments
 (0)