diff mbox series

[v1,2/2] test/dma: add source buffer offload free test

Message ID 20230907081040.1002478-3-amitprakashs@marvell.com (mailing list archive)
State Superseded, archived
Delegated to: Thomas Monjalon
Headers
Series offload support to free dma source buffer |

Checks

Context Check Description
ci/checkpatch success coding style OK
ci/loongarch-compilation success Compilation OK
ci/loongarch-unit-testing success Unit Testing PASS
ci/Intel-compilation success Compilation OK
ci/intel-Testing success Testing PASS
ci/github-robot: build success github build: passed
ci/intel-Functional success Functional PASS
ci/iol-mellanox-Performance success Performance Testing PASS
ci/iol-broadcom-Performance success Performance Testing PASS
ci/iol-broadcom-Functional success Functional Testing PASS
ci/iol-intel-Performance success Performance Testing PASS
ci/iol-intel-Functional success Functional Testing PASS
ci/iol-compile-arm64-testing success Testing PASS
ci/iol-unit-arm64-testing success Testing PASS
ci/iol-compile-amd64-testing fail Testing issues
ci/iol-unit-amd64-testing fail Testing issues
ci/iol-sample-apps-testing success Testing PASS

Commit Message

Amit Prakash Shukla Sept. 7, 2023, 8:10 a.m. UTC 
  Add a test case to validate the functionality of drivers' dma
source buffer offload free. As part of dmadev_autotest, test case
will be executed only if the driver supports source buffer offload
free and if the test is exported by env variable DPDK_ADD_DMA_TEST.

Signed-off-by: Amit Prakash Shukla <amitprakashs@marvell.com>
---
 app/test/test_dmadev.c | 132 ++++++++++++++++++++++++++++++++++++++++-
 1 file changed, 131 insertions(+), 1 deletion(-)

Comments

Anoob Joseph Sept. 19, 2023, 11:48 a.m. UTC | #1 
Hi Amit,

Please see inline.

Thanks,
Anoob

> -----Original Message-----
> From: Amit Prakash Shukla <amitprakashs@marvell.com>
> Sent: Thursday, September 7, 2023 1:41 PM
> To: Chengwen Feng <fengchengwen@huawei.com>; Kevin Laatz
> <kevin.laatz@intel.com>; Bruce Richardson <bruce.richardson@intel.com>
> Cc: dev@dpdk.org; Jerin Jacob Kollanukkaran <jerinj@marvell.com>;
> conor.walsh@intel.com; Vamsi Krishna Attunuru <vattunuru@marvell.com>;
> g.singh@nxp.com; sachin.saxena@oss.nxp.com; hemant.agrawal@nxp.com;
> cheng1.jiang@intel.com; Nithin Kumar Dabilpuram
> <ndabilpuram@marvell.com>; Anoob Joseph <anoobj@marvell.com>; Amit
> Prakash Shukla <amitprakashs@marvell.com>
> Subject: [PATCH v1 2/2] test/dma: add source buffer offload free test
> 
> Add a test case to validate the functionality of drivers' dma source buffer
> offload free. As part of dmadev_autotest, test case will be executed only if
> the driver supports source buffer offload free and if the test is exported by
> env variable DPDK_ADD_DMA_TEST.
> 
> Signed-off-by: Amit Prakash Shukla <amitprakashs@marvell.com>
> ---
>  app/test/test_dmadev.c | 132
> ++++++++++++++++++++++++++++++++++++++++-
>  1 file changed, 131 insertions(+), 1 deletion(-)
> 
> diff --git a/app/test/test_dmadev.c b/app/test/test_dmadev.c index
> 6ef875e545..48da4664ae 100644
> --- a/app/test/test_dmadev.c
> +++ b/app/test/test_dmadev.c
> @@ -18,11 +18,26 @@
> 
>  #define ERR_RETURN(...) do { print_err(__func__, __LINE__,
> __VA_ARGS__); return -1; } while (0)
> 
> +#define TEST_RINGSIZE 512
>  #define COPY_LEN 1024
> 
>  static struct rte_mempool *pool;
>  static uint16_t id_count;
> 
> +enum {
> +	TEST_SRC_BUF_FREE = 0,
> +	TEST_MAX,
> +};
> +
> +struct dma_add_test {
> +	const char *name;
> +	bool enabled;
> +};
> +
> +struct dma_add_test dma_add_test[] = {
> +	[TEST_SRC_BUF_FREE] = {.name = "sbuf_free", .enabled = false}, };
> +
>  static void
>  __rte_format_printf(3, 4)
>  print_err(const char *func, int lineno, const char *format, ...) @@ -797,10
> +812,93 @@ test_burst_capacity(int16_t dev_id, uint16_t vchan)
>  	return 0;
>  }
> 
> +static int
> +test_sbuf_free(int16_t dev_id, uint16_t vchan) { #define NR_MBUF 256
> +	int i, ret = 0;
> +	int retry = 100;
> +	uint16_t nb_done = 0;
> +	bool dma_err = false;
> +	uint32_t buf_cnt1, buf_cnt2;
> +	struct rte_mempool_ops *ops;
> +	uint64_t remote_addr = 0x40000000ull;

[Anoob] Can you make remote_addr as a variable read from env variable? Hard coding may not be the right approach.

> +	struct rte_mbuf *src[NR_MBUF], *dst[NR_MBUF];
> +	const struct rte_dma_vchan_conf qconf = {
> +		.direction = RTE_DMA_DIR_MEM_TO_DEV,
> +		.nb_desc = TEST_RINGSIZE,
> +		.mem_to_dev_src_buf_pool = pool,
> +		.dst_port.port_type = RTE_DMA_PORT_PCIE,
> +		/* Assuming pemid as 0. */
> +		.dst_port.pcie.coreid = 0,
> +	};
> +	static int dev_init;

[Anoob] Can you use bool instead of int for dev_init?

> +
> +	if (!dev_init) {
> +		/* Stop the device to reconfigure vchan. */
> +		if (rte_dma_stop(dev_id) < 0)
> +			ERR_RETURN("Error stopping device %u\n", dev_id);
> +
> +		if (rte_dma_vchan_setup(dev_id, vchan, &qconf) < 0)
> +			ERR_RETURN("Error with queue configuration\n");
> +
> +		if (rte_dma_start(dev_id) != 0)
> +			ERR_RETURN("Error with rte_dma_start()\n");
> +
> +		dev_init++;
> +	}
> +
> +	if (rte_pktmbuf_alloc_bulk(pool, dst, NR_MBUF) != 0)
> +		ERR_RETURN("alloc dst mbufs failed.\n");
> +
> +	for (i = 0; i < NR_MBUF; i++) {
> +		/* Using mbuf structure to hold remote iova address. */
> +		rte_mbuf_iova_set(dst[i], (rte_iova_t)remote_addr);
> +		dst[i]->data_off = 0;
> +	}
> +
> +	/* Capture buffer count before allocating source buffer. */
> +	ops = rte_mempool_get_ops(pool->ops_index);
> +	buf_cnt1 = ops->get_count(pool);
> +
> +	if (rte_pktmbuf_alloc_bulk(pool, src, NR_MBUF) != 0)
> +		ERR_RETURN("alloc src mbufs failed.\n");

[Anoob] The memory is not freed in case of errors. May be you can free them in the end and use goto as required.

> +
> +	if ((buf_cnt1 - NR_MBUF) != ops->get_count(pool))
> +		ERR_RETURN("Buffer count check failed.\n");
> +
> +	for (i = 0; i < NR_MBUF; i++) {
> +		ret = rte_dma_copy(dev_id, vchan,
> rte_mbuf_data_iova(src[i]),
> +				rte_mbuf_data_iova(dst[i]), COPY_LEN,
> +				RTE_DMA_OP_FLAG_FREE_SBUF);
> +
> +		if (ret < 0)
> +			ERR_RETURN("rte_dma_copy returned error.\n");
> +	}
> +
> +	rte_dma_submit(dev_id, vchan);
> +	nb_done = 0;
> +	do {
> +		nb_done += rte_dma_completed(dev_id, vchan, (NR_MBUF
> - nb_done), NULL, &dma_err);
> +		if (dma_err)
> +			break;
> +		/* Sleep for 1 millisecond */
> +		rte_delay_us_sleep(1000);
> +	} while (retry-- && (nb_done < NR_MBUF));
> +
> +	buf_cnt2 = ops->get_count(pool);
> +	if ((buf_cnt1 != buf_cnt2) || dma_err)
> +		ERR_RETURN("Free source buffer test failed.\n");
> +
> +	/* If the test passes source buffer will be freed in hardware. */
> +	rte_pktmbuf_free_bulk(dst, NR_MBUF);
> +
> +	return 0;
> +}
> +
>  static int
>  test_dmadev_instance(int16_t dev_id)
>  {
> -#define TEST_RINGSIZE 512
>  #define CHECK_ERRS    true
>  	struct rte_dma_stats stats;
>  	struct rte_dma_info info;
> @@ -890,6 +988,12 @@ test_dmadev_instance(int16_t dev_id)
>  	else if (runtest("fill", test_enqueue_fill, 1, dev_id, vchan,
> CHECK_ERRS) < 0)
>  		goto err;
> 
> +	if ((info.dev_capa &
> RTE_DMA_CAPA_MEM_TO_DEV_SOURCE_BUFFER_FREE) &&
> +	    dma_add_test[TEST_SRC_BUF_FREE].enabled == true) {
> +		if (runtest("sbuf_free", test_sbuf_free, 128, dev_id, vchan,
> CHECK_ERRS) < 0)
> +			goto err;
> +	}
> +
>  	rte_mempool_free(pool);
> 
>  	if (rte_dma_stop(dev_id) < 0)
> @@ -922,11 +1026,37 @@ test_apis(void)
>  	return ret;
>  }
> 
> +static void
> +parse_dma_env_var(void)
> +{
> +	char *dma_env_str = getenv("DPDK_ADD_DMA_TEST");
> +	char *tests[32] = {0};
> +	int n_tests = 0;
> +	int i, j;
> +
> +	if (dma_env_str && strlen(dma_env_str) > 0) {
> +		char *additional_test = strdup(dma_env_str);
> +		if (additional_test) {
> +			n_tests = rte_strsplit(additional_test,
> strlen(additional_test), tests,
> +				       RTE_DIM(tests), ',');
> +			for (i = 0; i < n_tests; i++) {
> +				for (j = 0; j < TEST_MAX; j++) {
> +					if (!strcmp(tests[i],
> dma_add_test[j].name))
> +						dma_add_test[j].enabled =
> true;
> +				}
> +			}
> +		}
> +		free(additional_test);
> +	}
> +}
> +
>  static int
>  test_dma(void)
>  {
>  	int i;
> 
> +	parse_dma_env_var();
> +
>  	/* basic sanity on dmadev infrastructure */
>  	if (test_apis() < 0)
>  		ERR_RETURN("Error performing API tests\n");
> --
> 2.25.1
Amit Prakash Shukla Sept. 26, 2023, 8:11 a.m. UTC | #2 
Hi Anoob,

Thanks for the review and feedback. I will make the suggested changes in next version of the patch.

Thanks,
Amit Shukla

> -----Original Message-----
> From: Anoob Joseph <anoobj@marvell.com>
> Sent: Tuesday, September 19, 2023 5:19 PM
> To: Amit Prakash Shukla <amitprakashs@marvell.com>; Chengwen Feng
> <fengchengwen@huawei.com>; Kevin Laatz <kevin.laatz@intel.com>; Bruce
> Richardson <bruce.richardson@intel.com>
> Cc: dev@dpdk.org; Jerin Jacob Kollanukkaran <jerinj@marvell.com>;
> conor.walsh@intel.com; Vamsi Krishna Attunuru <vattunuru@marvell.com>;
> g.singh@nxp.com; sachin.saxena@oss.nxp.com; hemant.agrawal@nxp.com;
> cheng1.jiang@intel.com; Nithin Kumar Dabilpuram
> <ndabilpuram@marvell.com>; Amit Prakash Shukla
> <amitprakashs@marvell.com>
> Subject: RE: [PATCH v1 2/2] test/dma: add source buffer offload free test
> 
> Hi Amit,
> 
> Please see inline.
> 
> Thanks,
> Anoob
> 
> > -----Original Message-----
> > From: Amit Prakash Shukla <amitprakashs@marvell.com>
> > Sent: Thursday, September 7, 2023 1:41 PM
> > To: Chengwen Feng <fengchengwen@huawei.com>; Kevin Laatz
> > <kevin.laatz@intel.com>; Bruce Richardson <bruce.richardson@intel.com>
> > Cc: dev@dpdk.org; Jerin Jacob Kollanukkaran <jerinj@marvell.com>;
> > conor.walsh@intel.com; Vamsi Krishna Attunuru
> <vattunuru@marvell.com>;
> > g.singh@nxp.com; sachin.saxena@oss.nxp.com;
> hemant.agrawal@nxp.com;
> > cheng1.jiang@intel.com; Nithin Kumar Dabilpuram
> > <ndabilpuram@marvell.com>; Anoob Joseph <anoobj@marvell.com>;
> Amit
> > Prakash Shukla <amitprakashs@marvell.com>
> > Subject: [PATCH v1 2/2] test/dma: add source buffer offload free test
> >
> > Add a test case to validate the functionality of drivers' dma source
> > buffer offload free. As part of dmadev_autotest, test case will be
> > executed only if the driver supports source buffer offload free and if
> > the test is exported by env variable DPDK_ADD_DMA_TEST.
> >
> > Signed-off-by: Amit Prakash Shukla <amitprakashs@marvell.com>
> > ---
> >  app/test/test_dmadev.c | 132
> > ++++++++++++++++++++++++++++++++++++++++-
> >  1 file changed, 131 insertions(+), 1 deletion(-)
> >
> > diff --git a/app/test/test_dmadev.c b/app/test/test_dmadev.c index
> > 6ef875e545..48da4664ae 100644
> > --- a/app/test/test_dmadev.c
> > +++ b/app/test/test_dmadev.c
> > @@ -18,11 +18,26 @@
> >
> >  #define ERR_RETURN(...) do { print_err(__func__, __LINE__,
> > __VA_ARGS__); return -1; } while (0)
> >
> > +#define TEST_RINGSIZE 512
> >  #define COPY_LEN 1024
> >
> >  static struct rte_mempool *pool;
> >  static uint16_t id_count;
> >
> > +enum {
> > +	TEST_SRC_BUF_FREE = 0,
> > +	TEST_MAX,
> > +};
> > +
> > +struct dma_add_test {
> > +	const char *name;
> > +	bool enabled;
> > +};
> > +
> > +struct dma_add_test dma_add_test[] = {
> > +	[TEST_SRC_BUF_FREE] = {.name = "sbuf_free", .enabled = false}, };
> > +
> >  static void
> >  __rte_format_printf(3, 4)
> >  print_err(const char *func, int lineno, const char *format, ...) @@
> > -797,10
> > +812,93 @@ test_burst_capacity(int16_t dev_id, uint16_t vchan)
> >  	return 0;
> >  }
> >
> > +static int
> > +test_sbuf_free(int16_t dev_id, uint16_t vchan) { #define NR_MBUF 256
> > +	int i, ret = 0;
> > +	int retry = 100;
> > +	uint16_t nb_done = 0;
> > +	bool dma_err = false;
> > +	uint32_t buf_cnt1, buf_cnt2;
> > +	struct rte_mempool_ops *ops;
> > +	uint64_t remote_addr = 0x40000000ull;
> 
> [Anoob] Can you make remote_addr as a variable read from env variable?
> Hard coding may not be the right approach.
> 
> > +	struct rte_mbuf *src[NR_MBUF], *dst[NR_MBUF];
> > +	const struct rte_dma_vchan_conf qconf = {
> > +		.direction = RTE_DMA_DIR_MEM_TO_DEV,
> > +		.nb_desc = TEST_RINGSIZE,
> > +		.mem_to_dev_src_buf_pool = pool,
> > +		.dst_port.port_type = RTE_DMA_PORT_PCIE,
> > +		/* Assuming pemid as 0. */
> > +		.dst_port.pcie.coreid = 0,
> > +	};
> > +	static int dev_init;
> 
> [Anoob] Can you use bool instead of int for dev_init?
> 
> > +
> > +	if (!dev_init) {
> > +		/* Stop the device to reconfigure vchan. */
> > +		if (rte_dma_stop(dev_id) < 0)
> > +			ERR_RETURN("Error stopping device %u\n", dev_id);
> > +
> > +		if (rte_dma_vchan_setup(dev_id, vchan, &qconf) < 0)
> > +			ERR_RETURN("Error with queue configuration\n");
> > +
> > +		if (rte_dma_start(dev_id) != 0)
> > +			ERR_RETURN("Error with rte_dma_start()\n");
> > +
> > +		dev_init++;
> > +	}
> > +
> > +	if (rte_pktmbuf_alloc_bulk(pool, dst, NR_MBUF) != 0)
> > +		ERR_RETURN("alloc dst mbufs failed.\n");
> > +
> > +	for (i = 0; i < NR_MBUF; i++) {
> > +		/* Using mbuf structure to hold remote iova address. */
> > +		rte_mbuf_iova_set(dst[i], (rte_iova_t)remote_addr);
> > +		dst[i]->data_off = 0;
> > +	}
> > +
> > +	/* Capture buffer count before allocating source buffer. */
> > +	ops = rte_mempool_get_ops(pool->ops_index);
> > +	buf_cnt1 = ops->get_count(pool);
> > +
> > +	if (rte_pktmbuf_alloc_bulk(pool, src, NR_MBUF) != 0)
> > +		ERR_RETURN("alloc src mbufs failed.\n");
> 
> [Anoob] The memory is not freed in case of errors. May be you can free
> them in the end and use goto as required.
> 
> > +
> > +	if ((buf_cnt1 - NR_MBUF) != ops->get_count(pool))
> > +		ERR_RETURN("Buffer count check failed.\n");
> > +
> > +	for (i = 0; i < NR_MBUF; i++) {
> > +		ret = rte_dma_copy(dev_id, vchan,
> > rte_mbuf_data_iova(src[i]),
> > +				rte_mbuf_data_iova(dst[i]), COPY_LEN,
> > +				RTE_DMA_OP_FLAG_FREE_SBUF);
> > +
> > +		if (ret < 0)
> > +			ERR_RETURN("rte_dma_copy returned error.\n");
> > +	}
> > +
> > +	rte_dma_submit(dev_id, vchan);
> > +	nb_done = 0;
> > +	do {
> > +		nb_done += rte_dma_completed(dev_id, vchan, (NR_MBUF
> > - nb_done), NULL, &dma_err);
> > +		if (dma_err)
> > +			break;
> > +		/* Sleep for 1 millisecond */
> > +		rte_delay_us_sleep(1000);
> > +	} while (retry-- && (nb_done < NR_MBUF));
> > +
> > +	buf_cnt2 = ops->get_count(pool);
> > +	if ((buf_cnt1 != buf_cnt2) || dma_err)
> > +		ERR_RETURN("Free source buffer test failed.\n");
> > +
> > +	/* If the test passes source buffer will be freed in hardware. */
> > +	rte_pktmbuf_free_bulk(dst, NR_MBUF);
> > +
> > +	return 0;
> > +}
> > +
> >  static int
> >  test_dmadev_instance(int16_t dev_id)
> >  {
> > -#define TEST_RINGSIZE 512
> >  #define CHECK_ERRS    true
> >  	struct rte_dma_stats stats;
> >  	struct rte_dma_info info;
> > @@ -890,6 +988,12 @@ test_dmadev_instance(int16_t dev_id)
> >  	else if (runtest("fill", test_enqueue_fill, 1, dev_id, vchan,
> > CHECK_ERRS) < 0)
> >  		goto err;
> >
> > +	if ((info.dev_capa &
> > RTE_DMA_CAPA_MEM_TO_DEV_SOURCE_BUFFER_FREE) &&
> > +	    dma_add_test[TEST_SRC_BUF_FREE].enabled == true) {
> > +		if (runtest("sbuf_free", test_sbuf_free, 128, dev_id, vchan,
> > CHECK_ERRS) < 0)
> > +			goto err;
> > +	}
> > +
> >  	rte_mempool_free(pool);
> >
> >  	if (rte_dma_stop(dev_id) < 0)
> > @@ -922,11 +1026,37 @@ test_apis(void)
> >  	return ret;
> >  }
> >
> > +static void
> > +parse_dma_env_var(void)
> > +{
> > +	char *dma_env_str = getenv("DPDK_ADD_DMA_TEST");
> > +	char *tests[32] = {0};
> > +	int n_tests = 0;
> > +	int i, j;
> > +
> > +	if (dma_env_str && strlen(dma_env_str) > 0) {
> > +		char *additional_test = strdup(dma_env_str);
> > +		if (additional_test) {
> > +			n_tests = rte_strsplit(additional_test,
> > strlen(additional_test), tests,
> > +				       RTE_DIM(tests), ',');
> > +			for (i = 0; i < n_tests; i++) {
> > +				for (j = 0; j < TEST_MAX; j++) {
> > +					if (!strcmp(tests[i],
> > dma_add_test[j].name))
> > +						dma_add_test[j].enabled =
> > true;
> > +				}
> > +			}
> > +		}
> > +		free(additional_test);
> > +	}
> > +}
> > +
> >  static int
> >  test_dma(void)
> >  {
> >  	int i;
> >
> > +	parse_dma_env_var();
> > +
> >  	/* basic sanity on dmadev infrastructure */
> >  	if (test_apis() < 0)
> >  		ERR_RETURN("Error performing API tests\n");
> > --
> > 2.25.1
diff mbox series

Patch

diff --git a/app/test/test_dmadev.c b/app/test/test_dmadev.c
index 6ef875e545..48da4664ae 100644
--- a/app/test/test_dmadev.c
+++ b/app/test/test_dmadev.c
@@ -18,11 +18,26 @@ 
 
 #define ERR_RETURN(...) do { print_err(__func__, __LINE__, __VA_ARGS__); return -1; } while (0)
 
+#define TEST_RINGSIZE 512
 #define COPY_LEN 1024
 
 static struct rte_mempool *pool;
 static uint16_t id_count;
 
+enum {
+	TEST_SRC_BUF_FREE = 0,
+	TEST_MAX,
+};
+
+struct dma_add_test {
+	const char *name;
+	bool enabled;
+};
+
+struct dma_add_test dma_add_test[] = {
+	[TEST_SRC_BUF_FREE] = {.name = "sbuf_free", .enabled = false},
+};
+
 static void
 __rte_format_printf(3, 4)
 print_err(const char *func, int lineno, const char *format, ...)
@@ -797,10 +812,93 @@  test_burst_capacity(int16_t dev_id, uint16_t vchan)
 	return 0;
 }
 
+static int
+test_sbuf_free(int16_t dev_id, uint16_t vchan)
+{
+#define NR_MBUF 256
+	int i, ret = 0;
+	int retry = 100;
+	uint16_t nb_done = 0;
+	bool dma_err = false;
+	uint32_t buf_cnt1, buf_cnt2;
+	struct rte_mempool_ops *ops;
+	uint64_t remote_addr = 0x40000000ull;
+	struct rte_mbuf *src[NR_MBUF], *dst[NR_MBUF];
+	const struct rte_dma_vchan_conf qconf = {
+		.direction = RTE_DMA_DIR_MEM_TO_DEV,
+		.nb_desc = TEST_RINGSIZE,
+		.mem_to_dev_src_buf_pool = pool,
+		.dst_port.port_type = RTE_DMA_PORT_PCIE,
+		/* Assuming pemid as 0. */
+		.dst_port.pcie.coreid = 0,
+	};
+	static int dev_init;
+
+	if (!dev_init) {
+		/* Stop the device to reconfigure vchan. */
+		if (rte_dma_stop(dev_id) < 0)
+			ERR_RETURN("Error stopping device %u\n", dev_id);
+
+		if (rte_dma_vchan_setup(dev_id, vchan, &qconf) < 0)
+			ERR_RETURN("Error with queue configuration\n");
+
+		if (rte_dma_start(dev_id) != 0)
+			ERR_RETURN("Error with rte_dma_start()\n");
+
+		dev_init++;
+	}
+
+	if (rte_pktmbuf_alloc_bulk(pool, dst, NR_MBUF) != 0)
+		ERR_RETURN("alloc dst mbufs failed.\n");
+
+	for (i = 0; i < NR_MBUF; i++) {
+		/* Using mbuf structure to hold remote iova address. */
+		rte_mbuf_iova_set(dst[i], (rte_iova_t)remote_addr);
+		dst[i]->data_off = 0;
+	}
+
+	/* Capture buffer count before allocating source buffer. */
+	ops = rte_mempool_get_ops(pool->ops_index);
+	buf_cnt1 = ops->get_count(pool);
+
+	if (rte_pktmbuf_alloc_bulk(pool, src, NR_MBUF) != 0)
+		ERR_RETURN("alloc src mbufs failed.\n");
+
+	if ((buf_cnt1 - NR_MBUF) != ops->get_count(pool))
+		ERR_RETURN("Buffer count check failed.\n");
+
+	for (i = 0; i < NR_MBUF; i++) {
+		ret = rte_dma_copy(dev_id, vchan, rte_mbuf_data_iova(src[i]),
+				rte_mbuf_data_iova(dst[i]), COPY_LEN,
+				RTE_DMA_OP_FLAG_FREE_SBUF);
+
+		if (ret < 0)
+			ERR_RETURN("rte_dma_copy returned error.\n");
+	}
+
+	rte_dma_submit(dev_id, vchan);
+	nb_done = 0;
+	do {
+		nb_done += rte_dma_completed(dev_id, vchan, (NR_MBUF - nb_done), NULL, &dma_err);
+		if (dma_err)
+			break;
+		/* Sleep for 1 millisecond */
+		rte_delay_us_sleep(1000);
+	} while (retry-- && (nb_done < NR_MBUF));
+
+	buf_cnt2 = ops->get_count(pool);
+	if ((buf_cnt1 != buf_cnt2) || dma_err)
+		ERR_RETURN("Free source buffer test failed.\n");
+
+	/* If the test passes source buffer will be freed in hardware. */
+	rte_pktmbuf_free_bulk(dst, NR_MBUF);
+
+	return 0;
+}
+
 static int
 test_dmadev_instance(int16_t dev_id)
 {
-#define TEST_RINGSIZE 512
 #define CHECK_ERRS    true
 	struct rte_dma_stats stats;
 	struct rte_dma_info info;
@@ -890,6 +988,12 @@  test_dmadev_instance(int16_t dev_id)
 	else if (runtest("fill", test_enqueue_fill, 1, dev_id, vchan, CHECK_ERRS) < 0)
 		goto err;
 
+	if ((info.dev_capa & RTE_DMA_CAPA_MEM_TO_DEV_SOURCE_BUFFER_FREE) &&
+	    dma_add_test[TEST_SRC_BUF_FREE].enabled == true) {
+		if (runtest("sbuf_free", test_sbuf_free, 128, dev_id, vchan, CHECK_ERRS) < 0)
+			goto err;
+	}
+
 	rte_mempool_free(pool);
 
 	if (rte_dma_stop(dev_id) < 0)
@@ -922,11 +1026,37 @@  test_apis(void)
 	return ret;
 }
 
+static void
+parse_dma_env_var(void)
+{
+	char *dma_env_str = getenv("DPDK_ADD_DMA_TEST");
+	char *tests[32] = {0};
+	int n_tests = 0;
+	int i, j;
+
+	if (dma_env_str && strlen(dma_env_str) > 0) {
+		char *additional_test = strdup(dma_env_str);
+		if (additional_test) {
+			n_tests = rte_strsplit(additional_test, strlen(additional_test), tests,
+				       RTE_DIM(tests), ',');
+			for (i = 0; i < n_tests; i++) {
+				for (j = 0; j < TEST_MAX; j++) {
+					if (!strcmp(tests[i], dma_add_test[j].name))
+						dma_add_test[j].enabled = true;
+				}
+			}
+		}
+		free(additional_test);
+	}
+}
+
 static int
 test_dma(void)
 {
 	int i;
 
+	parse_dma_env_var();
+
 	/* basic sanity on dmadev infrastructure */
 	if (test_apis() < 0)
 		ERR_RETURN("Error performing API tests\n");