Skip to content

Commit 2e859fa

Browse files
author
Burlen Loring
committed
fix MCA variable scope in coll ucc
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 a9f84cc commit 2e859fa

File tree

1 file changed

+12
-12
lines changed

1 file changed

+12
-12
lines changed

ompi/mca/coll/ucc/coll_ucc_component.c

Lines changed: 12 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -60,24 +60,24 @@ static int mca_coll_ucc_register(void)
6060
mca_coll_ucc_component_t *cm = &mca_coll_ucc_component;
6161
mca_base_component_t *c = &cm->super.collm_version;
6262
mca_base_component_var_register(c, "priority", "Priority of the UCC coll component",
63-
MCA_BASE_VAR_TYPE_INT, NULL, 0, 0,
63+
MCA_BASE_VAR_TYPE_INT, NULL, 0, MCA_BASE_VAR_FLAG_SETTABLE,
6464
OPAL_INFO_LVL_9,
65-
MCA_BASE_VAR_SCOPE_READONLY, &cm->ucc_priority);
65+
MCA_BASE_VAR_SCOPE_ALL, &cm->ucc_priority);
6666

6767
mca_base_component_var_register(c, "verbose", "Verbose level of the UCC coll component",
68-
MCA_BASE_VAR_TYPE_INT, NULL, 0, 0,
68+
MCA_BASE_VAR_TYPE_INT, NULL, 0, MCA_BASE_VAR_FLAG_SETTABLE,
6969
OPAL_INFO_LVL_9,
70-
MCA_BASE_VAR_SCOPE_READONLY, &cm->ucc_verbose);
70+
MCA_BASE_VAR_SCOPE_ALL, &cm->ucc_verbose);
7171

7272
mca_base_component_var_register(c, "enable", "[0|1] Enable/Disable the UCC coll component",
73-
MCA_BASE_VAR_TYPE_INT, NULL, 0, 0,
73+
MCA_BASE_VAR_TYPE_INT, NULL, 0, MCA_BASE_VAR_FLAG_SETTABLE,
7474
OPAL_INFO_LVL_9,
75-
MCA_BASE_VAR_SCOPE_READONLY, &cm->ucc_enable);
75+
MCA_BASE_VAR_SCOPE_ALL, &cm->ucc_enable);
7676

7777
mca_base_component_var_register(c, "np", "Minimal communicator size for the UCC coll component",
78-
MCA_BASE_VAR_TYPE_INT, NULL, 0, 0,
78+
MCA_BASE_VAR_TYPE_INT, NULL, 0, MCA_BASE_VAR_FLAG_SETTABLE,
7979
OPAL_INFO_LVL_9,
80-
MCA_BASE_VAR_SCOPE_READONLY, &cm->ucc_np);
80+
MCA_BASE_VAR_SCOPE_ALL, &cm->ucc_np);
8181

8282
mca_base_component_var_register(c, MCA_COMPILETIME_VER,
8383
"Version of the libucc library with which Open MPI was compiled",
@@ -94,14 +94,14 @@ static int mca_coll_ucc_register(void)
9494
cm->cls = "";
9595
mca_base_component_var_register(c, "cls",
9696
"Comma separated list of UCC CLS to be used for team creation",
97-
MCA_BASE_VAR_TYPE_STRING, NULL, 0, 0,
98-
OPAL_INFO_LVL_6, MCA_BASE_VAR_SCOPE_READONLY, &cm->cls);
97+
MCA_BASE_VAR_TYPE_STRING, NULL, 0, MCA_BASE_VAR_FLAG_SETTABLE,
98+
OPAL_INFO_LVL_6, MCA_BASE_VAR_SCOPE_ALL, &cm->cls);
9999

100100
cm->cts = COLL_UCC_CTS_STR;
101101
mca_base_component_var_register(c, "cts",
102102
"Comma separated list of UCC coll types to be enabled",
103-
MCA_BASE_VAR_TYPE_STRING, NULL, 0, 0,
104-
OPAL_INFO_LVL_6, MCA_BASE_VAR_SCOPE_READONLY, &cm->cts);
103+
MCA_BASE_VAR_TYPE_STRING, NULL, 0, MCA_BASE_VAR_FLAG_SETTABLE,
104+
OPAL_INFO_LVL_6, MCA_BASE_VAR_SCOPE_ALL, &cm->cts);
105105
return OMPI_SUCCESS;
106106
}
107107

0 commit comments

Comments
 (0)