Skip to content

Commit

Permalink
param: increase CQ read count to 16 for performance
Browse files Browse the repository at this point in the history
This patch increases CQ read count to 16.

Experiments turned out that reading up to 16 (instead of 4) CQ entries
at once improves performance.

Signed-off-by: Michael Axtmann <axtmannm@amazon.com>
(cherry picked from commit 346be40)
  • Loading branch information
maxtmann authored and AmedeoSapio committed Aug 1, 2024
1 parent 9befebf commit b5d3482
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion include/nccl_ofi_param.h
Original file line number Diff line number Diff line change
Expand Up @@ -129,7 +129,7 @@ OFI_NCCL_PARAM_INT(mr_key_size, "MR_KEY_SIZE", 2);
* Maximum number of cq entries to read in a single call to
* fi_cq_read.
*/
OFI_NCCL_PARAM_INT(cq_read_count, "CQ_READ_COUNT", 4);
OFI_NCCL_PARAM_INT(cq_read_count, "CQ_READ_COUNT", 16);

/*
* Protocol to use for send/recv operations. Valid options are
Expand Down

0 comments on commit b5d3482

Please sign in to comment.