fabtests: Reorganize the FI_RX_CQ_DATA set.

FI_RX_CQ_DATA mode shouldn't be set per operation. It should be set based on whether the code supports it in the implementation. This patch adds the FI_RX_CQ_DATA to the hints setup phase among the code that supports FI_RX_CQ_DATA in implementation, like FI_CONTEXT and FI_CONTEXT2. Signed-off-by: Shi Jin <sjina@amazon.com>
shijin-aws · Feb 6, 2025 · f66367d · f66367d
1 parent c3420a7
commit f66367d
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/fabtests/benchmarks/rma_bw.c b/fabtests/benchmarks/rma_bw.c
@@ -93,7 +93,7 @@ int main(int argc, char **argv)
 
 	hints->caps = FI_MSG | FI_RMA;
 	hints->domain_attr->resource_mgmt = FI_RM_ENABLED;
-	hints->mode = FI_CONTEXT | FI_CONTEXT2;
+	hints->mode = FI_CONTEXT | FI_CONTEXT2 | FI_RX_CQ_DATA;
 	hints->domain_attr->threading = FI_THREAD_DOMAIN;
 	hints->addr_format = opts.address_format;
 

diff --git a/fabtests/benchmarks/rma_pingpong.c b/fabtests/benchmarks/rma_pingpong.c
@@ -93,7 +93,7 @@ int main(int argc, char **argv)
 
 	hints->caps = FI_MSG | FI_RMA | FI_WRITE | FI_REMOTE_WRITE;
 	hints->domain_attr->resource_mgmt = FI_RM_ENABLED;
-	hints->mode = FI_CONTEXT | FI_CONTEXT2;
+	hints->mode = FI_CONTEXT | FI_CONTEXT2 | FI_RX_CQ_DATA;
 	hints->addr_format = opts.address_format;
 
 	while ((op = getopt_long(argc, argv, "Uh" CS_OPTS INFO_OPTS API_OPTS