Skip to content

Commit 028e747

Browse files
committed
Do not alter ompi_coll_tuned_use_dynamic_rules.
This is set globally as an MCA parameter and should be never altered based on a single communicator setting.
1 parent 253eb80 commit 028e747

File tree

2 files changed

+0
-13
lines changed

2 files changed

+0
-13
lines changed

ompi/mca/coll/tuned/coll_tuned_component.c

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -160,7 +160,6 @@ static int tuned_register(void)
160160
MCA_BASE_VAR_SCOPE_READONLY,
161161
&ompi_coll_tuned_alltoall_intermediate_msg);
162162

163-
ompi_coll_tuned_use_dynamic_rules = false;
164163
(void) mca_base_component_var_register(&mca_coll_tuned_component.super.collm_version,
165164
"use_dynamic_rules",
166165
"Switch used to decide if we use static (compiled/if statements) or dynamic (built at runtime) decision function rules",

ompi/mca/coll/tuned/coll_tuned_module.c

Lines changed: 0 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -152,7 +152,6 @@ ompi_coll_tuned_forced_getvalues( enum COLLTYPE type,
152152
(TMOD)->com_rules[(TYPE)] = NULL; \
153153
if( 0 != (TMOD)->user_forced[(TYPE)].algorithm ) { \
154154
need_dynamic_decision = 1; \
155-
EXECUTE; \
156155
} \
157156
if( NULL != mca_coll_tuned_component.all_base_rules ) { \
158157
(TMOD)->com_rules[(TYPE)] \
@@ -164,7 +163,6 @@ ompi_coll_tuned_forced_getvalues( enum COLLTYPE type,
164163
} \
165164
if( 1 == need_dynamic_decision ) { \
166165
OPAL_OUTPUT((ompi_coll_tuned_stream,"coll:tuned: enable dynamic selection for "#TYPE)); \
167-
ompi_coll_tuned_use_dynamic_rules = true; \
168166
EXECUTE; \
169167
} \
170168
}
@@ -208,10 +206,6 @@ tuned_module_enable( mca_coll_base_module_t *module,
208206

209207
if (ompi_coll_tuned_use_dynamic_rules) {
210208
OPAL_OUTPUT((ompi_coll_tuned_stream,"coll:tuned:module_init MCW & Dynamic"));
211-
/**
212-
* Reset it to 0, it will be enabled again if we discover any need for dynamic decisions.
213-
*/
214-
ompi_coll_tuned_use_dynamic_rules = false;
215209

216210
/**
217211
* next dynamic state, recheck all forced rules as well
@@ -249,12 +243,6 @@ tuned_module_enable( mca_coll_base_module_t *module,
249243
tuned_module->super.coll_scatter = ompi_coll_tuned_scatter_intra_dec_dynamic);
250244
COLL_TUNED_EXECUTE_IF_DYNAMIC(tuned_module, SCATTERV,
251245
tuned_module->super.coll_scatterv = NULL);
252-
253-
if( false == ompi_coll_tuned_use_dynamic_rules ) {
254-
/* no real need for dynamic decisions */
255-
OPAL_OUTPUT((ompi_coll_tuned_stream, "coll:tuned:module_enable switch back to fixed"
256-
" decision by lack of dynamic rules"));
257-
}
258246
}
259247

260248
/* general n fan out tree */

0 commit comments

Comments
 (0)