On 10/23/20 4:42 PM, Nicolas Chautru wrote: > bler test results are not valid when LLR compression > is used or for loopback scenarios. Skipping these. > > Signed-off-by: Nicolas Chautru <nicolas.chau...@intel.com> > Acked-by: Aidan Goddard <aidan.godd...@accelercomm.com> > Acked-by: Dave Burley <dave.bur...@accelercomm.com> > --- > app/test-bbdev/test_bbdev_perf.c | 6 +++++- > 1 file changed, 5 insertions(+), 1 deletion(-) > > diff --git a/app/test-bbdev/test_bbdev_perf.c > b/app/test-bbdev/test_bbdev_perf.c > index 39f06db..a15ea69 100644 > --- a/app/test-bbdev/test_bbdev_perf.c > +++ b/app/test-bbdev/test_bbdev_perf.c > @@ -3719,7 +3719,11 @@ typedef int (test_case_function)(struct active_device > *ad, > RTE_ALIGN(sizeof(struct thread_params) * num_lcores, > RTE_CACHE_LINE_SIZE)); > > - if (test_vector.op_type == RTE_BBDEV_OP_LDPC_DEC) > + if ((test_vector.op_type == RTE_BBDEV_OP_LDPC_DEC) && > + !check_bit(test_vector.ldpc_dec.op_flags, > + RTE_BBDEV_LDPC_INTERNAL_HARQ_MEMORY_LOOPBACK) > + && !check_bit(test_vector.ldpc_dec.op_flags, > + RTE_BBDEV_LDPC_LLR_COMPRESSION)) > bler_function = bler_pmd_lcore_ldpc_dec; > else > return TEST_SKIPPED;
Looks ok. Reviewed-by: Tom Rix <t...@redhat.com>
