Skip to content

Commit

Permalink
baseband/acc: fix ACC100 HARQ input alignment
Browse files Browse the repository at this point in the history
[ upstream commit 7d048916644d7777872f957901222f2654b234dd ]

Some constraints are imposed onto the ACC100 HARQ input size,
but that value is incorrectly aligned down when getting close to
max (Ncb-F) which is not required.
The wireless performance impact is negligeable but still causes a
few LLRs no to be combined at the very end of the circular buffer.

Fixes: 5802f36 ("baseband/acc100: enforce additional check on FCW")

Signed-off-by: Hernan Vargas <hernan.vargas@intel.com>
Reviewed-by: Maxime Coquelin <maxime.coquelin@redhat.com>
  • Loading branch information
Hernanlv authored and kevintraynor committed Oct 31, 2023
1 parent 8816d35 commit e166b7f
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion drivers/baseband/acc100/rte_acc100_pmd.c
Original file line number Diff line number Diff line change
Expand Up @@ -1375,7 +1375,7 @@ acc100_fcw_ld_fill(struct rte_bbdev_dec_op *op, struct acc100_fcw_ld *fcw,
- op->ldpc_dec.n_filler);

/* Alignment on next 64B - Already enforced from HC output */
harq_in_length = RTE_ALIGN_FLOOR(harq_in_length, ACC100_HARQ_ALIGN_64B);
harq_in_length = RTE_ALIGN_CEIL(harq_in_length, ACC100_HARQ_ALIGN_64B);

/* Stronger alignment requirement when in decompression mode */
if (fcw->hcin_decomp_mode > 0)
Expand Down

0 comments on commit e166b7f

Please sign in to comment.