From af2f4ca8926b2af561a5563d4043765ff8e42c7f Mon Sep 17 00:00:00 2001
From: Nicholas Sielicki <nslick@amazon.com>
Date: Mon, 11 Mar 2024 12:28:55 -0700
Subject: [PATCH] fix(build): tuner: prefer ncclInvalidArgument

ncclInvalidArgument is present both in neuron and nvidia whereas
ncclInvalidUsage is only present in neuron headers. Callsite just wraps
the value in ncclcheck and the meaning isn't meaningfully changed, so
prefer the portable one.
---
 src/tuner/nccl_ofi_tuner.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/tuner/nccl_ofi_tuner.c b/src/tuner/nccl_ofi_tuner.c
index 2f5540899..1a82d12fb 100644
--- a/src/tuner/nccl_ofi_tuner.c
+++ b/src/tuner/nccl_ofi_tuner.c
@@ -21,7 +21,7 @@ ncclResult_t nccl_ofi_tuner_init(size_t nRanks, size_t nNodes, ncclDebugLogger_t
 	 */
 	if (getenv("NCCL_ALGO") || getenv("NCCL_PROTO")) {
 		NCCL_OFI_WARN("The tuner plugin can not be loaded when explicitly choosing an algorithm or protocol with NCCL_ALGO/NCCL_PROTO");
-		return ncclInvalidUsage;
+		return ncclInvalidArgument;
 	}
 
 	struct nccl_ofi_tuner_model_params params = {