Skip to content
This repository was archived by the owner on Sep 30, 2022. It is now read-only.

Commit 7f6ce53

Browse files
committed
Merge pull request #1214 from ggouaillardet/topic/v2.x/mca_coll_basic_bcast_log_intra
coll/basic: fix mca_coll_basic_bcast_log_intra()
2 parents 6011906 + 1953a53 commit 7f6ce53

File tree

1 file changed

+4
-4
lines changed

1 file changed

+4
-4
lines changed

ompi/mca/coll/basic/coll_basic_bcast.c

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -9,7 +9,9 @@
99
* University of Stuttgart. All rights reserved.
1010
* Copyright (c) 2004-2005 The Regents of the University of California.
1111
* All rights reserved.
12-
* Copyright (c) 2015 Cisco Systems, Inc. All rights reserved.
12+
* Copyright (c) 2015 Cisco Systems, Inc. All rights reserved.
13+
* Copyright (c) 2016 Research Organization for Information Science
14+
* and Technology (RIST). All rights reserved.
1315
* $COPYRIGHT$
1416
*
1517
* Additional copyrights may follow
@@ -61,14 +63,12 @@ mca_coll_basic_bcast_log_intra(void *buff, int count,
6163

6264
dim = comm->c_cube_dim;
6365
hibit = opal_hibit(vrank, dim);
64-
if (hibit < 0) {
65-
return MPI_ERR_OTHER;
66-
}
6766
--dim;
6867

6968
/* Receive data from parent in the tree. */
7069

7170
if (vrank > 0) {
71+
assert(hibit >= 0);
7272
peer = ((vrank & ~(1 << hibit)) + root) % size;
7373

7474
err = MCA_PML_CALL(recv(buff, count, datatype, peer,

0 commit comments

Comments
 (0)