Correct a misleading typo in the explanation comment, and clarify
another comment. This commit was SVN r3159.
Этот коммит содержится в:
родитель
51b2da657a
Коммит
353e8a7a9c
@ -151,7 +151,7 @@ int mca_coll_basic_reduce_lin_intra(void *sbuf, void *rbuf, int count,
|
|||||||
|
|
||||||
2. If B is what we get back from malloc, but we give A to
|
2. If B is what we get back from malloc, but we give A to
|
||||||
MPI_Send, then the buffer range [A,B) represents "dead space"
|
MPI_Send, then the buffer range [A,B) represents "dead space"
|
||||||
-- no data will be put there. So it's safe to give A-LB to
|
-- no data will be put there. So it's safe to give B-LB to
|
||||||
MPI_Send. More specifically, the LB is positive, so B-LB is
|
MPI_Send. More specifically, the LB is positive, so B-LB is
|
||||||
actually A.
|
actually A.
|
||||||
|
|
||||||
@ -166,9 +166,9 @@ int mca_coll_basic_reduce_lin_intra(void *sbuf, void *rbuf, int count,
|
|||||||
Hence, in all cases, we give (return_from_malloc - LB) to MPI_Send.
|
Hence, in all cases, we give (return_from_malloc - LB) to MPI_Send.
|
||||||
|
|
||||||
This works fine and dandy if we only have (count==1), which we
|
This works fine and dandy if we only have (count==1), which we
|
||||||
rarely do. ;-) So we need to then allocate ((count-1)*extent) to
|
rarely do. ;-) So we really need to allocate (true_extent +
|
||||||
get enough space for the rest. This may be more than is
|
((count - 1) * extent)) to get enough space for the rest. This may
|
||||||
necessary, but it's ok.
|
be more than is necessary, but it's ok.
|
||||||
|
|
||||||
Simple, no? :-)
|
Simple, no? :-)
|
||||||
|
|
||||||
|
Загрузка…
x
Ссылка в новой задаче
Block a user