On 9/7/21 6:39 PM, Nicolas Chautru wrote:
This patch fixes the issue by adjusting the outbound size after turbodecoding when the appended CRC is meant to be dropped. Fixes: f404dfe35cc3 ("baseband/acc100: support 4G processing") Cc: [email protected] Signed-off-by: Nicolas Chautru <[email protected]> --- drivers/baseband/acc100/rte_acc100_pmd.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/drivers/baseband/acc100/rte_acc100_pmd.c b/drivers/baseband/acc100/rte_acc100_pmd.c index 68ba523..b0485e7 100644 --- a/drivers/baseband/acc100/rte_acc100_pmd.c +++ b/drivers/baseband/acc100/rte_acc100_pmd.c @@ -1744,7 +1744,8 @@next_triplet = acc100_dma_fill_blk_type_out(desc, h_output, *h_out_offset, - k >> 3, next_triplet, ACC100_DMA_BLKID_OUT_HARD); + (k - crc24_overlap) >> 3, next_triplet, + ACC100_DMA_BLKID_OUT_HARD); if (unlikely(next_triplet < 0)) { rte_bbdev_log(ERR, "Mismatch between data to process and mbuf data length in bbdev_op: %p",
Thanks for this change, looks good. Reviewed-by: Tom Rix <[email protected]>
