[PATCH v2 14/16] test/bbdev: use mbuf reset function

[Maxime Coquelin]

On 2/15/23 18:09, Hernan Vargas wrote:
> Use mbuf_reset function for code consistency.
> No functional impact.
> 
> Signed-off-by: Hernan Vargas <hernan.vargas at intel.com>
> ---
>   app/test-bbdev/test_bbdev_perf.c | 14 +++++++-------
>   1 file changed, 7 insertions(+), 7 deletions(-)
> 
> diff --git a/app/test-bbdev/test_bbdev_perf.c b/app/test-bbdev/test_bbdev_perf.c
> index 5259404ff6..535b9d73dd 100644
> --- a/app/test-bbdev/test_bbdev_perf.c
> +++ b/app/test-bbdev/test_bbdev_perf.c
> @@ -3173,11 +3173,11 @@ throughput_intr_lcore_ldpc_dec(void *arg)
>   	for (j = 0; j < TEST_REPETITIONS; ++j) {
>   		for (i = 0; i < num_to_process; ++i) {
>   			if (!loopback)
> -				rte_pktmbuf_reset(ops[i]->ldpc_dec.hard_output.data);
> +				mbuf_reset(ops[i]->ldpc_dec.hard_output.data);
>   			if (hc_out || loopback)
>   				mbuf_reset(ops[i]->ldpc_dec.harq_combined_output.data);
>   			if (ops[i]->ldpc_dec.soft_output.data != NULL)
> -				rte_pktmbuf_reset(ops[i]->ldpc_dec.soft_output.data);
> +				mbuf_reset(ops[i]->ldpc_dec.soft_output.data);
>   		}
>   
>   		tp->start_time = rte_rdtsc_precise();
> @@ -3267,9 +3267,9 @@ throughput_intr_lcore_dec(void *arg)
>   
>   	for (j = 0; j < TEST_REPETITIONS; ++j) {
>   		for (i = 0; i < num_to_process; ++i) {
> -			rte_pktmbuf_reset(ops[i]->turbo_dec.hard_output.data);
> +			mbuf_reset(ops[i]->turbo_dec.hard_output.data);
>   			if (ops[i]->turbo_dec.soft_output.data != NULL)
> -				rte_pktmbuf_reset(ops[i]->turbo_dec.soft_output.data);
> +				mbuf_reset(ops[i]->turbo_dec.soft_output.data);
>   		}
>   
>   		tp->start_time = rte_rdtsc_precise();
> @@ -3356,7 +3356,7 @@ throughput_intr_lcore_enc(void *arg)
>   
>   	for (j = 0; j < TEST_REPETITIONS; ++j) {
>   		for (i = 0; i < num_to_process; ++i)
> -			rte_pktmbuf_reset(ops[i]->turbo_enc.output.data);
> +			mbuf_reset(ops[i]->turbo_enc.output.data);
>   
>   		tp->start_time = rte_rdtsc_precise();
>   		for (enqueued = 0; enqueued < num_to_process;) {
> @@ -3444,7 +3444,7 @@ throughput_intr_lcore_ldpc_enc(void *arg)
>   
>   	for (j = 0; j < TEST_REPETITIONS; ++j) {
>   		for (i = 0; i < num_to_process; ++i)
> -			rte_pktmbuf_reset(ops[i]->turbo_enc.output.data);
> +			mbuf_reset(ops[i]->turbo_enc.output.data);
>   
>   		tp->start_time = rte_rdtsc_precise();
>   		for (enqueued = 0; enqueued < num_to_process;) {
> @@ -3532,7 +3532,7 @@ throughput_intr_lcore_fft(void *arg)
>   
>   	for (j = 0; j < TEST_REPETITIONS; ++j) {
>   		for (i = 0; i < num_to_process; ++i)
> -			rte_pktmbuf_reset(ops[i]->fft.base_output.data);
> +			mbuf_reset(ops[i]->fft.base_output.data);
>   
>   		tp->start_time = rte_rdtsc_precise();
>   		for (enqueued = 0; enqueued < num_to_process;) {

Reviewed-by: Maxime Coquelin <maxime.coquelin at redhat.com>

Thanks,
Maxime