[PATCH v1 1/9] test/bbdev: fix TB logic

[Maxime Coquelin]

On 4/22/24 21:07, Hernan Vargas wrote:
> Fix discrepancy in logic when using large fake mbuf.
> 
> Fixes: fd96ef3787f1 ("test/bbdev: extend support for large TB")
> Cc: stable at dpdk.org
> 
> Signed-off-by: Hernan Vargas <hernan.vargas at intel.com>
> ---
>   app/test-bbdev/test_bbdev_perf.c | 3 ++-
>   1 file changed, 2 insertions(+), 1 deletion(-)
> 
> diff --git a/app/test-bbdev/test_bbdev_perf.c b/app/test-bbdev/test_bbdev_perf.c
> index dcce00aa0a17..efd046984d58 100644
> --- a/app/test-bbdev/test_bbdev_perf.c
> +++ b/app/test-bbdev/test_bbdev_perf.c
> @@ -2131,7 +2131,8 @@ validate_op_chain(struct rte_bbdev_op_data *op,
>   		uint16_t data_len = rte_pktmbuf_data_len(m) - offset;
>   		total_data_size += orig_op->segments[i].length;
>   
> -		if (orig_op->segments[i].length > RTE_BBDEV_LDPC_E_MAX_MBUF)
> +		if ((orig_op->segments[i].length + RTE_PKTMBUF_HEADROOM)
> +				> RTE_BBDEV_LDPC_E_MAX_MBUF)
>   			ignore_mbuf = true;
>   		if (!ignore_mbuf)
>   			TEST_ASSERT(orig_op->segments[i].length == data_len,

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

Thanks,
Maxime