This commit adds support to BBDEV for LDPC Code Block Group Transmission (CBGT) as defined in 3GPP TS 38.214. The following changes facilitate this: - add RTE_BBDEV_LDPC_[ENC/DEC]_CBGT feature flag - add CBGT input parameters to LDPC encode transport block op structure - add CBGT input and output parameters to LDPC decode transport block op - add support for reading these parameters from test vector files - add sanity tests for RTE_BBDEV_LDPC_[ENC/DEC]_CBGT flag in test vector - update user guide with the flags and parameters
CBGT parameters are only required when the RTE_BBDEV_LDPC_[ENC/DEC]_CBGT op flag is specified. Reported-by: Rob Maunder <rob.maun...@accelercomm.com> Signed-off-by: Aidan Goddard <aidan.godd...@accelercomm.com> Acked-by: Dave Burley <dave.bur...@accelercomm.com> --- app/test-bbdev/test_bbdev_vector.c | 46 +++++++++++++++++++--- app/test-bbdev/test_bbdev_vector.h | 3 ++ doc/guides/prog_guide/bbdev.rst | 29 ++++++++++++++ lib/librte_bbdev/rte_bbdev_op.h | 80 +++++++++++++++++++++++++++++++++++++- 4 files changed, 151 insertions(+), 7 deletions(-) diff --git a/app/test-bbdev/test_bbdev_vector.c b/app/test-bbdev/test_bbdev_vector.c index 50d1da0..e5808b2 100644 --- a/app/test-bbdev/test_bbdev_vector.c +++ b/app/test-bbdev/test_bbdev_vector.c @@ -200,6 +200,8 @@ else if (!strcmp(token, "RTE_BBDEV_LDPC_INTERNAL_HARQ_MEMORY_LOOPBACK")) *op_flag_value = RTE_BBDEV_LDPC_INTERNAL_HARQ_MEMORY_LOOPBACK; + else if (!strcmp(token, "RTE_BBDEV_LDPC_DEC_CBGT")) + *op_flag_value = RTE_BBDEV_LDPC_DEC_CBGT; else { printf("The given value is not a LDPC decoder flag\n"); return -1; @@ -248,8 +250,10 @@ *op_flag_value = RTE_BBDEV_LDPC_ENC_INTERRUPTS; else if (!strcmp(token, "RTE_BBDEV_LDPC_ENC_SCATTER_GATHER")) *op_flag_value = RTE_BBDEV_LDPC_ENC_SCATTER_GATHER; + else if (!strcmp(token, "RTE_BBDEV_LDPC_ENC_CBGT")) + *op_flag_value = RTE_BBDEV_LDPC_ENC_CBGT; else { - printf("The given value is not a turbo encoder flag\n"); + printf("The given value is not a LDPC encoder flag\n"); return -1; } @@ -718,6 +722,14 @@ vector->mask |= TEST_BBDEV_VF_CODE_BLOCK_MODE; ldpc_enc->code_block_mode = (uint8_t) strtoul(token, &err, 0); ret = ((err == NULL) || (*err != '\0')) ? -1 : 0; + } else if (!strcmp(key_token, "max_cbg")) { + vector->mask |= TEST_BBDEV_VF_MAX_CBG_PER_TB; + ldpc_enc->tb_params.max_cbg = (uint8_t) strtoul(token, &err, 0); + ret = ((err == NULL) || (*err != '\0')) ? -1 : 0; + } else if (!strcmp(key_token, "cbgti")) { + vector->mask |= TEST_BBDEV_VF_CBGTI; + ldpc_enc->tb_params.cbgti = (uint8_t) strtoul(token, &err, 0); + ret = ((err == NULL) || (*err != '\0')) ? -1 : 0; } else if (!strcmp(key_token, "op_flags")) { vector->mask |= TEST_BBDEV_VF_OP_FLAGS; ret = parse_turbo_flags(token, &op_flags, vector->op_type); @@ -828,6 +840,18 @@ vector->mask |= TEST_BBDEV_VF_CODE_BLOCK_MODE; ldpc_dec->code_block_mode = (uint8_t) strtoul(token, &err, 0); ret = ((err == NULL) || (*err != '\0')) ? -1 : 0; + } else if (!strcmp(key_token, "max_cbg")) { + vector->mask |= TEST_BBDEV_VF_MAX_CBG_PER_TB; + ldpc_dec->tb_params.max_cbg = (uint8_t) strtoul(token, &err, 0); + ret = ((err == NULL) || (*err != '\0')) ? -1 : 0; + } else if (!strcmp(key_token, "cbgti")) { + vector->mask |= TEST_BBDEV_VF_CBGTI; + ldpc_dec->tb_params.cbgti = (uint8_t) strtoul(token, &err, 0); + ret = ((err == NULL) || (*err != '\0')) ? -1 : 0; + } else if (!strcmp(key_token, "cbgfi")) { + vector->mask |= TEST_BBDEV_VF_CBGFI; + ldpc_dec->tb_params.cbgfi = (uint8_t) strtoul(token, &err, 0); + ret = ((err == NULL) || (*err != '\0')) ? -1 : 0; } else if (!strcmp(key_token, "op_flags")) { vector->mask |= TEST_BBDEV_VF_OP_FLAGS; ret = parse_turbo_flags(token, &op_flags, vector->op_type); @@ -1162,6 +1186,12 @@ if (!(mask & TEST_BBDEV_VF_R)) printf( "WARNING: r was not specified in vector file and will be set to 0\n"); + if (!(mask & TEST_BBDEV_VF_MAX_CBG_PER_TB) && (ldpc_dec->op_flags & + RTE_BBDEV_LDPC_DEC_CBGT)) { + printf( + "WARNING: max_cbg was not specified in vector file and will be set to 1\n"); + ldpc_dec->tb_params.max_cbg = 1; + } } else { if (!(mask & TEST_BBDEV_VF_E)) printf( @@ -1298,12 +1328,18 @@ if (!(mask & TEST_BBDEV_VF_CODE_BLOCK_MODE)) { printf( "WARNING: code_block_mode was not specified in vector file and will be set to 1\n"); - vector->turbo_enc.code_block_mode = 1; + vector->ldpc_enc.code_block_mode = 1; } - if (vector->turbo_enc.code_block_mode == 0) { + if (vector->ldpc_enc.code_block_mode == 0) { + if (!(mask & TEST_BBDEV_VF_MAX_CBG_PER_TB) && (vector->ldpc_enc.op_flags & + RTE_BBDEV_LDPC_ENC_CBGT)) { + printf( + "WARNING: max_cbg was not specified in vector file and will be set to 1\n"); + vector->ldpc_enc.tb_params.max_cbg = 1; + } } else { - if (!(mask & TEST_BBDEV_VF_E) && (vector->turbo_enc.op_flags & - RTE_BBDEV_TURBO_RATE_MATCH)) + if (!(mask & TEST_BBDEV_VF_E) && (vector->ldpc_enc.op_flags & + RTE_BBDEV_LDPC_RATE_MATCH)) printf( "WARNING: e was not specified in vector file and will be set to 0\n"); if (!(mask & TEST_BBDEV_VF_NCB)) diff --git a/app/test-bbdev/test_bbdev_vector.h b/app/test-bbdev/test_bbdev_vector.h index 4e5dbf5..69f5b9c 100644 --- a/app/test-bbdev/test_bbdev_vector.h +++ b/app/test-bbdev/test_bbdev_vector.h @@ -35,6 +35,9 @@ enum { TEST_BBDEV_VF_CODE_BLOCK_MODE = (1ULL << 23), TEST_BBDEV_VF_OP_FLAGS = (1ULL << 24), TEST_BBDEV_VF_EXPECTED_STATUS = (1ULL << 25), + TEST_BBDEV_VF_MAX_CBG_PER_TB = (1ULL << 26), + TEST_BBDEV_VF_CBGTI = (1ULL << 27), + TEST_BBDEV_VF_CBGFI = (1ULL << 28), }; enum op_data_type { diff --git a/doc/guides/prog_guide/bbdev.rst b/doc/guides/prog_guide/bbdev.rst index 6b2bd54..0ff4039 100644 --- a/doc/guides/prog_guide/bbdev.rst +++ b/doc/guides/prog_guide/bbdev.rst @@ -747,6 +747,9 @@ given below. |RTE_BBDEV_LDPC_ENC_CONCATENATION | | Set if a device supports concatenation of non byte aligned output | +--------------------------------------------------------------------+ +|RTE_BBDEV_LDPC_ENC_CBGT | +| Set if a device supports CB group transmission | ++--------------------------------------------------------------------+ The structure passed for each LDPC encode operation is given below, with the operation flags forming a bitmask in the ``op_flags`` field. @@ -815,6 +818,10 @@ The LDPC encode parameters are set out in the table below. +----------------+------------+-------------------------------------------------------+ | |eb |Eb, length of the RM output sequence in bits, r >= cab | +----------------+------------+-------------------------------------------------------+ +| |max_cbg |maximum number of CB groups per TB: 1-8 | ++----------------+------------+-------------------------------------------------------+ +| |cbgti |CB group transmission information bitfield | ++----------------+------------+-------------------------------------------------------+ The mbuf input ``input`` is mandatory for all BBDEV PMDs and is the incoming code block or transport block data. @@ -869,6 +876,11 @@ Figure :numref:`figure_turbo_tb_encode` above showing the Turbo encoding of CBs using BBDEV interface in TB-mode is also valid for LDPC encode. +In TB-mode, If the ``RTE_BBDEV_LDPC_ENC_CBGT`` flag is specified the +``max_cbg`` and ``cbgti`` parameters are used for CB group transmission. If +the ``RTE_BBDEV_LDPC_ENC_CBGT`` flag is not specified, these parameters +are ignored. + BBDEV LDPC Decode Operation ~~~~~~~~~~~~~~~~~~~~~~~~~~~~ @@ -942,6 +954,9 @@ given below. |RTE_BBDEV_LDPC_INTERNAL_HARQ_MEMORY_LOOPBACK | | Set if a device supports loopback access to HARQ internal memory | +--------------------------------------------------------------------+ +|RTE_BBDEV_LDPC_DEC_CBGT | +| Set if a device supports CB group transmission | ++--------------------------------------------------------------------+ The structure passed for each LDPC decode operation is given below, with the operation flags forming a bitmask in the ``op_flags`` field. @@ -1027,6 +1042,14 @@ The LDPC decode parameters are set out in the table below. +----------------+------------+-------------------------------------------------------+ | |eb |Eb, length of the RM output sequence in bits r >= cab | +----------------+------------+-------------------------------------------------------+ +| |max_cbg |maximum number of CB groups per TB: 1-8 | ++----------------+------------+-------------------------------------------------------+ +| |cbgti |CB group transmission information bitfield | ++----------------+------------+-------------------------------------------------------+ +| |cbgfi |CB group Flushing out Information (CBGFI) | ++----------------+------------+-------------------------------------------------------+ +| |cbg_crc_err |CB group CRC failure bitfield (output) | ++----------------+------------+-------------------------------------------------------+ The mbuf input ``input`` encoded CB data is mandatory for all BBDEV PMDs and is the Virtual Circular Buffer data stream with null padding. @@ -1084,6 +1107,12 @@ Figure :numref:`figure_turbo_tb_decode` above showing the Turbo decoding of CBs using BBDEV interface in TB-mode is also valid for LDPC decode. +In TB-mode, If the ``RTE_BBDEV_LDPC_DEC_CBGT`` flag is specified the +``max_cbg`` and ``cbgti`` parameters are used for CB group transmission. The +``cbg_crc_err`` output parameter is a bitfield used to indicate crc failures +in the respective CB groups. If the ``RTE_BBDEV_LDPC_DEC_CBGT`` flag is not +specified, these parameters are ignored. + Sample code ----------- diff --git a/lib/librte_bbdev/rte_bbdev_op.h b/lib/librte_bbdev/rte_bbdev_op.h index f726d73..f3d00dc 100644 --- a/lib/librte_bbdev/rte_bbdev_op.h +++ b/lib/librte_bbdev/rte_bbdev_op.h @@ -186,7 +186,9 @@ enum rte_bbdev_op_ldpcdec_flag_bitmasks { * for HARQ memory. If not set, it is assumed the filler bits are not * in HARQ memory and handled directly by the LDPC decoder. */ - RTE_BBDEV_LDPC_INTERNAL_HARQ_MEMORY_FILLERS = (1ULL << 18) + RTE_BBDEV_LDPC_INTERNAL_HARQ_MEMORY_FILLERS = (1ULL << 18), + /** Set if a device supports CB group transmission. */ + RTE_BBDEV_LDPC_DEC_CBGT = (1ULL << 19) }; /** Flags for LDPC encoder operation and capability structure */ @@ -206,7 +208,9 @@ enum rte_bbdev_op_ldpcenc_flag_bitmasks { /** Set if a device supports scatter-gather functionality. */ RTE_BBDEV_LDPC_ENC_SCATTER_GATHER = (1ULL << 6), /** Set if a device supports concatenation of non byte aligned output */ - RTE_BBDEV_LDPC_ENC_CONCATENATION = (1ULL << 7) + RTE_BBDEV_LDPC_ENC_CONCATENATION = (1ULL << 7), + /** Set if a device supports CB group transmission. */ + RTE_BBDEV_LDPC_ENC_CBGT = (1ULL << 8) }; /** Data input and output buffer for BBDEV operations */ @@ -334,6 +338,56 @@ struct rte_bbdev_op_dec_ldpc_tb_params { uint8_t r; /** The number of CBs that use Ea before switching to Eb, [0:63] */ uint8_t cab; + /** If the RTE_BBDEV_LDPC_DEC_CBGT capability is not asserted, then + * the value of max_cbg is ignored. Otherwise, a max_cbg value of + * 0 or 1 indicates that codeBlockGroupTransmission is disabled, + * as defined in 3GPP TS 38.214. + * + * A max_cbg value of 2, 4, 6, or 8 sets the value of + * maxCodeBlockGroupsPerTransportBlock and indicates that + * codeBlockGroupTransmission is enabled, as defined in 3GPP TS 38.214. + */ + uint8_t max_cbg; + /** If codeBlockGroupTransmission is disabled, then the value of cbgti + * is ignored. Otherwise, cbgti represents the Code Block Group + * Transmission Information (CBGTI), as defined in 3GPP TS 38.214. + * In this case, the M = min(C, max_cbg) number of Most Significant + * Bits (MSBs) of the uint8_t cbgti have an in-order one-to-one mapping + * with the M code block groups (CBGs) of the transport block, with the + * MSB mapped to CBG#0, as detailed in 3GPP TS 38.214. + * + * Here, C is the total number of code blocks in the full transport block, + * as defined in 3GPP TS 38.212. + */ + uint8_t cbgti; + /** If codeBlockGroupTransmission is disabled, then the value of cbgfi + * should be ignored. Otherwise, cbgfi represents the Code Block Group + * Flushing out Information (CBGFI), as defined in 3GPP TS 38.214. In this + * case, if the LSB of the uint8_t cbgfi is set to 0, this indicates that + * the earlier received instances of the same CBGs being decoded may be + * corrupted and that the corresponding contents of the HARQ memory should + * be flushed and not combined with the present CBGs being decoded. If the + * LSB of the uint8_t cbgfi is set to 1, this indicates that the earlier + * received instances of the same CBGs being decoded should be combined with + * the present CBGs being decoded. + */ + uint8_t cbgfi; + + /** If codeBlockGroupTransmission is disabled or if + * RTE_BBDEV_LDPC_CRC_TYPE_24B_CHECK is not asserted, then the value of + * cbg_crc_err should be ignored. Otherwise, cbg_crc_err reports whether + * any CRC24B failures have been encountered in each of the code block + * groups (CBGs). In this case, the M = min(C, max_cbg) number of Most + * Significant Bits (MSBs) of the uint8_t cbg_crc_error have an in-order + * one-to-one mapping with the M CBGs of the transport block, with the MSB + * mapped to CBG#0, as detailed in 3GPP TS 38.214. An asserted bit + * indicates that a CRC24B failure was encountered among the code blocks + * of the corresponding CBG. + * + * Here, C is the total number of code blocks in the full transport block, + * as defined in 3GPP TS 38.212. + */ + uint8_t cbg_crc_err; }; /** Operation structure for Turbo decode. @@ -584,6 +638,28 @@ struct rte_bbdev_op_enc_ldpc_tb_params { uint8_t r; /** The number of CBs that use Ea before switching to Eb, [0:63] */ uint8_t cab; + /** If the RTE_BBDEV_LDPC_ENC_CBGT capability is not asserted, then + * the value of max_cbg is ignored. Otherwise, a max_cbg value of + * 0 or 1 indicates that codeBlockGroupTransmission is disabled, + * as defined in 3GPP TS 38.214. + * + * A max_cbg value of 2, 4, 6, or 8 sets the value of + * maxCodeBlockGroupsPerTransportBlock and indicates that + * codeBlockGroupTransmission is enabled, as defined in 3GPP TS 38.214. + */ + uint8_t max_cbg; + /** If codeBlockGroupTransmission is disabled, then the value of cbgti + * is ignored. Otherwise, cbgti represents the Code Block Group + * Transmission Information (CBGTI), as defined in 3GPP TS 38.214. + * In this case, the M = min(C, max_cbg) number of Most Significant + * Bits (MSBs) of the uint8_t cbgti have an in-order one-to-one mapping + * with the M code block groups (CBGs) of the transport block, with the + * MSB mapped to CBG#0, as detailed in 3GPP TS 38.214. + * + * Here, C is the total number of code blocks in the full transport block, + * as defined in 3GPP TS 38.212. + */ + uint8_t cbgti; }; /** Operation structure for Turbo encode. -- 1.8.3.1