Skip to content

Commit

Permalink
temp: fix csel
Browse files Browse the repository at this point in the history
  • Loading branch information
hzhou committed Aug 23, 2024
1 parent 914755c commit b729bb4
Showing 1 changed file with 5 additions and 3 deletions.
8 changes: 5 additions & 3 deletions src/mpi/coll/src/csel.c
Original file line number Diff line number Diff line change
Expand Up @@ -643,14 +643,14 @@ static csel_node_s *prune_tree(csel_node_s * root, MPIR_Comm * comm_ptr)
else
node = node->success;
break;

/*
case CSEL_NODE_TYPE__OPERATOR__COMM_HIERARCHY:
if (comm_ptr->hierarchy_kind == node->u.comm_hierarchy.val)
node = node->success;
else
node = node->failure;
break;

*/
case CSEL_NODE_TYPE__OPERATOR__IS_NODE_CONSECUTIVE:
if (MPII_Comm_is_node_consecutive(comm_ptr) == node->u.is_node_consecutive.val)
node = node->success;
Expand Down Expand Up @@ -1188,6 +1188,7 @@ void *MPIR_Csel_search(void *csel_, MPIR_Csel_coll_sig_s coll_info)
csel_s *csel = (csel_s *) csel_;
csel_node_s *node = NULL;
MPIR_Comm *comm_ptr = coll_info.comm_ptr;
int coll_group = coll_info.coll_group;

MPIR_Assert(csel_);

Expand Down Expand Up @@ -1349,7 +1350,8 @@ void *MPIR_Csel_search(void *csel_, MPIR_Csel_coll_sig_s coll_info)
break;

case CSEL_NODE_TYPE__OPERATOR__COMM_HIERARCHY:
if (coll_info.comm_ptr->hierarchy_kind == node->u.comm_hierarchy.val)
if (node->u.comm_hierarchy.val == MPIR_COMM_HIERARCHY_KIND__PARENT &&
MPIR_Comm_is_parent_comm(comm_ptr, coll_group))
node = node->success;
else
node = node->failure;
Expand Down

0 comments on commit b729bb4

Please sign in to comment.