Skip to content

Commit

Permalink
rdma: Disable eager by default
Browse files Browse the repository at this point in the history
We don't think we need eager with the performance improvements on P5 and
P5en, so set the default to -1 (disabled).

Signed-off-by: Brian Barrett <bbarrett@amazon.com>
  • Loading branch information
bwbarrett authored and xwangxin committed Feb 24, 2025
1 parent a8544ed commit a4832e0
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 @@ -319,7 +319,7 @@ OFI_NCCL_PARAM_INT(net_latency, "NET_LATENCY", -1);
* Eager message size limit when using RDMA protocol. Message sizes greater than
* this limit will always be sent using RDMA write instead of eagerly.
*/
OFI_NCCL_PARAM_INT(eager_max_size, "EAGER_MAX_SIZE", 8192);
OFI_NCCL_PARAM_INT(eager_max_size, "EAGER_MAX_SIZE", -1);

/*
* Decide whether or not mutexes should default to errorcheck mode.
Expand Down

0 comments on commit a4832e0

Please sign in to comment.