Commit 103d7cba authored by Alexey Marchuk's avatar Alexey Marchuk Committed by Konrad Sztyber
Browse files

test_dma: Update condition to corrupt mkey 



Use `>=` instead of `>`, otherwise we corrupt mkey later
than expected.
Increment counter before the check if the mkey corruption
is needed.

Suggested-by: default avatarJim Harris <jim.harris@samsung.com>
Signed-off-by: default avatarAlexey Marchuk <alexeymar@nvidia.com>
Change-Id: I5b948cd567a03498c44faaf81989e0b4b2341933
Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/24714


Reviewed-by: default avatarKonrad Sztyber <konrad.sztyber@intel.com>
Community-CI: Broadcom CI <spdk-ci.pdl@broadcom.com>
Reviewed-by: default avatarShuhei Matsumoto <smatsumoto@nvidia.com>
Tested-by: default avatarSPDK CI Jenkins <sys_sgci@intel.com>
parent 23dc652d

test/dma/test_dma/test_dma.c

+3 −3
Original line number Diff line number Diff line
@@ -480,15 +480,15 @@ dma_test_translate_memory_cb(struct spdk_memory_domain *src_domain, void *src_do 
	result->rdma.rkey = req->mr->rkey;

	result->dst_domain = dst_domain;



	if (g_corrupt_mkey_counter && task->num_translations > g_corrupt_mkey_counter &&

	task->num_translations++;



	if (g_corrupt_mkey_counter && task->num_translations >= g_corrupt_mkey_counter &&

	    task->num_translations % g_corrupt_mkey_counter == 0) {

		SPDK_NOTICELOG("Corrupt mkey on core %u\n", task->lcore);

		result->rdma.lkey = 0xffffffff;

		result->rdma.rkey = 0xffffffff;

	}



	task->num_translations++;



	return 0;

}