Re: [PATCH v1 11/11] test/bbdev: support 4 bit LLR compression

[Maxime Coquelin]

On 9/29/23 20:13, Hernan Vargas wrote:

Add support to test LDPC UL operation for new capability.
Option to compress HARQ memory to 4 bits per LLR.

Signed-off-by: Hernan Vargas <hernan.var...@intel.com>
---
  app/test-bbdev/test_bbdev_perf.c   | 3 ++-
  app/test-bbdev/test_bbdev_vector.c | 2 ++
  2 files changed, 4 insertions(+), 1 deletion(-)

diff --git a/app/test-bbdev/test_bbdev_perf.c b/app/test-bbdev/test_bbdev_perf.c
index 82deb9b1b4b9..149c7a1f504c 100644
--- a/app/test-bbdev/test_bbdev_perf.c
+++ b/app/test-bbdev/test_bbdev_perf.c
@@ -2240,7 +2240,8 @@ validate_op_harq_chain(struct rte_bbdev_op_data *op,

/* Cannot compare HARQ output data for such cases */

                if ((ldpc_llr_decimals > 1) && ((ops_ld->op_flags & 
RTE_BBDEV_LDPC_LLR_COMPRESSION)
-                               || (ops_ld->op_flags & 
RTE_BBDEV_LDPC_HARQ_6BIT_COMPRESSION)))
+                               || (ops_ld->op_flags & 
RTE_BBDEV_LDPC_HARQ_6BIT_COMPRESSION)
+                               || (ops_ld->op_flags & 
RTE_BBDEV_LDPC_HARQ_4BIT_COMPRESSION)))
                        break;

if (!(ldpc_cap_flags &

diff --git a/app/test-bbdev/test_bbdev_vector.c 
b/app/test-bbdev/test_bbdev_vector.c
index 56b882533cda..42fa630041e9 100644
--- a/app/test-bbdev/test_bbdev_vector.c
+++ b/app/test-bbdev/test_bbdev_vector.c
@@ -196,6 +196,8 @@ op_ldpc_decoder_flag_strtoul(char *token, uint32_t 
*op_flag_value)
                *op_flag_value = RTE_BBDEV_LDPC_DEC_SCATTER_GATHER;
        else if (!strcmp(token, "RTE_BBDEV_LDPC_HARQ_6BIT_COMPRESSION"))
                *op_flag_value = RTE_BBDEV_LDPC_HARQ_6BIT_COMPRESSION;
+       else if (!strcmp(token, "RTE_BBDEV_LDPC_HARQ_4BIT_COMPRESSION"))
+               *op_flag_value = RTE_BBDEV_LDPC_HARQ_4BIT_COMPRESSION;
        else if (!strcmp(token, "RTE_BBDEV_LDPC_LLR_COMPRESSION"))
                *op_flag_value = RTE_BBDEV_LDPC_LLR_COMPRESSION;
        else if (!strcmp(token,

Reviewed-by: Maxime Coquelin <maxime.coque...@redhat.com>

Thanks,
Maxime