Commit 9eb0413c authored by Jacek Kalwas's avatar Jacek Kalwas Committed by Tomasz Zawadzki
Browse files

dma: remove runtime domain check 



The user shall not set callbacks on invalid (NULL) domain hence replace
with assertion. This also alignes with what was done for
_set_invalidate.

Change-Id: I9eb5f108cb5f0e883bf23eb58702c45b9d0bdf46
Signed-off-by: default avatarJacek Kalwas <jacek.kalwas@intel.com>
Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/21963


Reviewed-by: default avatarJim Harris <jim.harris@samsung.com>
Reviewed-by: default avatarAleksey Marchuk <alexeymar@nvidia.com>
Community-CI: Mellanox Build Bot
Tested-by: default avatarSPDK CI Jenkins <sys_sgci@intel.com>
parent 293f5ea8

lib/dma/dma.c

+4 −12
Original line number Diff line number Diff line
@@ -101,9 +101,7 @@ void 
spdk_memory_domain_set_translation(struct spdk_memory_domain *domain,

				   spdk_memory_domain_translate_memory_cb translate_cb)

{

	if (!domain) {

		return;

	}

	assert(domain);



	domain->translate_cb = translate_cb;

}
@@ -121,9 +119,7 @@ void 
spdk_memory_domain_set_pull(struct spdk_memory_domain *domain,

			    spdk_memory_domain_pull_data_cb pull_cb)

{

	if (!domain) {

		return;

	}

	assert(domain);



	domain->pull_cb = pull_cb;

}
@@ -132,9 +128,7 @@ void 
spdk_memory_domain_set_push(struct spdk_memory_domain *domain,

			    spdk_memory_domain_push_data_cb push_cb)

{

	if (!domain) {

		return;

	}

	assert(domain);



	domain->push_cb = push_cb;

}
@@ -143,9 +137,7 @@ void 
spdk_memory_domain_set_memzero(struct spdk_memory_domain *domain,

			       spdk_memory_domain_memzero_cb memzero_cb)

{

	if (!domain) {

		return;

	}

	assert(domain);



	domain->memzero_cb = memzero_cb;

}