Skip to content

Commit

Permalink
fabtests/efa: Remove rnr cq error message check
Browse files Browse the repository at this point in the history
The current check for rnr cq error message string
is unnecessary and fragile as we may adjust
the error message. This patch removes such error
message check and relies on the error code check.

Signed-off-by: Shi Jin <sjina@amazon.com>
  • Loading branch information
shijin-aws committed Oct 18, 2024
1 parent 12131e9 commit 115660c
Showing 1 changed file with 0 additions and 11 deletions.
11 changes: 0 additions & 11 deletions fabtests/prov/efa/src/rdm_rnr_read_cq_error.c
Original file line number Diff line number Diff line change
Expand Up @@ -43,7 +43,6 @@ static int rnr_read_cq_error(void)
{
int total_send, expected_rnr_error;
int ret, i, cnt, rnr_flag;
const char *prov_errmsg;

expected_rnr_error = fi->rx_attr->size;
rnr_flag = 0;
Expand Down Expand Up @@ -89,16 +88,6 @@ static int rnr_read_cq_error(void)
rnr_flag = 1;
printf("Got RNR error CQ entry as expected: %d, %s\n",
comp_err.err, fi_strerror(comp_err.err));
prov_errmsg = fi_cq_strerror(txcq, comp_err.prov_errno,
comp_err.err_data,
comp_err.buf,
comp_err.len);
if (strstr(prov_errmsg, "Destination resource not ready") == NULL) {
printf("Got unexpected provider error message.\n");
printf(" Expected error message to have \"Destination resource not ready\" in it\n");
printf(" Got: %s\n", prov_errmsg);
return -FI_EINVAL;
}
} else {
printf("Got non-RNR error CQ entry: %d, %s\n",
comp_err.err, fi_strerror(comp_err.err));
Expand Down

0 comments on commit 115660c

Please sign in to comment.