diff mbox series

[v1,04/11] test/bbdev: handle exception for LLR generation

Message ID	20230929181328.104311-5-hernan.vargas@intel.com (mailing list archive)
State	Superseded, archived
Delegated to:	Maxime Coquelin
Headers	From: Hernan Vargas <hernan.vargas@intel.com> To: dev@dpdk.org, gakhil@marvell.com, trix@redhat.com, maxime.coquelin@redhat.com Cc: nicolas.chautru@intel.com, qi.z.zhang@intel.com, Hernan Vargas <hernan.vargas@intel.com> Subject: [PATCH v1 04/11] test/bbdev: handle exception for LLR generation Date: Fri, 29 Sep 2023 11:13:21 -0700 Message-Id: <20230929181328.104311-5-hernan.vargas@intel.com> In-Reply-To: <20230929181328.104311-1-hernan.vargas@intel.com> References: <20230929181328.104311-1-hernan.vargas@intel.com> MIME-Version: 1.0 Content-Transfer-Encoding: 8bit Precedence: list Errors-To: dev-bounces@dpdk.org
Series	test-bbdev changes for 23.11 \| [v1,00/11] test-bbdev changes for 23.11 [v1,01/11] test/bbdev: fix python script subprocess [v1,02/11] test/bbdev: update python script parameters [v1,03/11] test/bbdev: rename macros from acc200 to vrb [v1,04/11] test/bbdev: handle exception for LLR generation [v1,05/11] test/bbdev: improve test log messages [v1,06/11] test/bbdev: assert failed test for queue configure [v1,07/11] test/bbdev: ldpc encoder concatenation vector [v1,08/11] test/bbdev: increase max burst size [v1,09/11] test/bbdev: add MLD support [v1,10/11] test/bbdev: support new FFT capabilities [v1,11/11] test/bbdev: support 4 bit LLR compression

Checks

Context	Check	Description
ci/checkpatch	success	coding style OK

Commit Message

Hernan Vargas Sept. 29, 2023, 6:13 p.m. UTC

  Add range limit to prevent LLR generation greater than the data buffer
size.

Signed-off-by: Hernan Vargas <hernan.vargas@intel.com>
---
 app/test-bbdev/test_bbdev_perf.c | 6 ++++++
 1 file changed, 6 insertions(+)

Comments

Maxime Coquelin Oct. 17, 2023, 7:30 p.m. UTC | #1

On 9/29/23 20:13, Hernan Vargas wrote:
> Add range limit to prevent LLR generation greater than the data buffer
> size.

Should be a fix?

> Signed-off-by: Hernan Vargas <hernan.vargas@intel.com>
> ---
>   app/test-bbdev/test_bbdev_perf.c | 6 ++++++
>   1 file changed, 6 insertions(+)
> 
> diff --git a/app/test-bbdev/test_bbdev_perf.c b/app/test-bbdev/test_bbdev_perf.c
> index d4c001de0093..54cb2090f988 100644
> --- a/app/test-bbdev/test_bbdev_perf.c
> +++ b/app/test-bbdev/test_bbdev_perf.c
> @@ -1838,6 +1838,12 @@ generate_turbo_llr_input(uint16_t n, struct rte_bbdev_op_data *inputs,
>   	range = ref_op->turbo_dec.input.length;
>   	N0 = 1.0 / pow(10.0, get_snr() / 10.0);
>   
> +	if (range > inputs[0].data->data_len) {
> +		printf("Warning: Limiting LLR generation to first segment (%d from %d)\n",
> +				inputs[0].data->data_len, range);
> +		range = inputs[0].data->data_len;
> +	}
> +
>   	for (i = 0; i < n; ++i) {
>   		m = inputs[i].data;
>   		int8_t *llrs = rte_pktmbuf_mtod_offset(m, int8_t *, 0);

diff mbox series

Patch

diff --git a/app/test-bbdev/test_bbdev_perf.c b/app/test-bbdev/test_bbdev_perf.c
index d4c001de0093..54cb2090f988 100644
--- a/app/test-bbdev/test_bbdev_perf.c
+++ b/app/test-bbdev/test_bbdev_perf.c
@@ -1838,6 +1838,12 @@  generate_turbo_llr_input(uint16_t n, struct rte_bbdev_op_data *inputs,
 	range = ref_op->turbo_dec.input.length;
 	N0 = 1.0 / pow(10.0, get_snr() / 10.0);
 
+	if (range > inputs[0].data->data_len) {
+		printf("Warning: Limiting LLR generation to first segment (%d from %d)\n",
+				inputs[0].data->data_len, range);
+		range = inputs[0].data->data_len;
+	}
+
 	for (i = 0; i < n; ++i) {
 		m = inputs[i].data;
 		int8_t *llrs = rte_pktmbuf_mtod_offset(m, int8_t *, 0);