Skip to content

Commit 44e0436

Browse files
author
Burlen Loring
committed
fix MCA variable scope in coll basic
Changes several variables scope from READONLY to ALL so that they can be set via MPI_T interface Signed-off-by: Burlen Loring <[email protected]>
1 parent fdf60ac commit 44e0436

File tree

1 file changed

+5
-4
lines changed

1 file changed

+5
-4
lines changed

ompi/mca/coll/basic/coll_basic_component.c

Lines changed: 5 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -13,6 +13,7 @@
1313
* Copyright (c) 2008 Cisco Systems, Inc. All rights reserved.
1414
* Copyright (c) 2015 Los Alamos National Security, LLC. All rights
1515
* reserved.
16+
* Copyright (c) 2024 NVIDIA CORPORATION. All rights reserved.
1617
* $COPYRIGHT$
1718
*
1819
* Additional copyrights may follow
@@ -91,16 +92,16 @@ basic_register(void)
9192
mca_coll_basic_priority = 10;
9293
(void) mca_base_component_var_register(&mca_coll_basic_component.collm_version, "priority",
9394
"Priority of the basic coll component",
94-
MCA_BASE_VAR_TYPE_INT, NULL, 0, 0,
95+
MCA_BASE_VAR_TYPE_INT, NULL, 0, MCA_BASE_VAR_FLAG_SETTABLE,
9596
OPAL_INFO_LVL_9,
96-
MCA_BASE_VAR_SCOPE_READONLY,
97+
MCA_BASE_VAR_SCOPE_ALL,
9798
&mca_coll_basic_priority);
9899
mca_coll_basic_crossover = 4;
99100
(void) mca_base_component_var_register(&mca_coll_basic_component.collm_version, "crossover",
100101
"Minimum number of processes in a communicator before using the logarithmic algorithms",
101-
MCA_BASE_VAR_TYPE_INT, NULL, 0, 0,
102+
MCA_BASE_VAR_TYPE_INT, NULL, 0, MCA_BASE_VAR_FLAG_SETTABLE,
102103
OPAL_INFO_LVL_9,
103-
MCA_BASE_VAR_SCOPE_READONLY,
104+
MCA_BASE_VAR_SCOPE_ALL,
104105
&mca_coll_basic_crossover);
105106

106107
return OMPI_SUCCESS;

0 commit comments

Comments
 (0)