Skip to content

Commit

Permalink
TL/UCP: fix sparse pack in hybrid a2av (openucx#825)
Browse files Browse the repository at this point in the history
  • Loading branch information
Sergei-Lebedev authored and nsarka committed Oct 24, 2023
1 parent 38450c7 commit 04186a8
Showing 1 changed file with 1 addition and 2 deletions.
3 changes: 1 addition & 2 deletions src/components/tl/ucp/alltoallv/alltoallv_hybrid.c
Original file line number Diff line number Diff line change
Expand Up @@ -303,7 +303,6 @@ static size_t pack_send_data(ucc_tl_ucp_task_t *task, int step,
int n = send_count;
int logi = 1;
void *op_metadata = task->alltoallv_hybrid.scratch_mc_header->addr;

ucc_rank_t index;
int send_len, snd_offset, i, k;
unsigned int temp_count;
Expand Down Expand Up @@ -365,7 +364,7 @@ static size_t pack_send_data(ucc_tl_ucp_task_t *task, int step,
/* if there is data for less than half the destinations, send meta data
* in sparse format.
*/
if ((sparse_cnt / 2) < send_count) {
if ((sparse_cnt * 2) < send_count) {
/* sparse format: store nonzero messages and source rank index */
((unsigned int *)packed_send_buf)[0] = sparse_cnt;
/* compute space requirement for sparse data exchange header */
Expand Down

0 comments on commit 04186a8

Please sign in to comment.