bbdev, baseband/acc: bug fixes

This series targeting 26.11 contains a set of small bug fixes for the bbdev library and the baseband/acc (ACC100/VRB) drivers.

- bbdev: remove a spurious error log printed by rte_bbdev_enqueue_status_str() when called for valid cases such as queue dump.
- baseband/acc: fix the available dequeue size computation, which could go negative when dequeuing descriptors.
- baseband/acc: fix PF2VF communication so all FFT windows can be read (previously only the first was functional) on VRB1 and VRB2.
- baseband/acc: fix the ring interrupt handler error path so a malformed ring entry is consumed instead of being reprocessed forever and blocking further ring processing (PF and VF).
- baseband/acc: explicitly zero part of the FCW when alternating TB mode.

Hernan Vargas (5):
bbdev: fix status error for ops dump baseband/acc: fix queue depth computation error baseband/acc: fix PF2VF Bug for FFT window probing baseband/acc: fix ring interrupt handler err path baseband/acc: fix bug when alternating TB mode

 drivers/baseband/acc/acc_common.h  | 5 ++++-
 drivers/baseband/acc/rte_vrb_pmd.c | 6 +++---
 lib/bbdev/rte_bbdev.c              | 1 -
 3 files changed, 7 insertions(+), 5 deletions(-)

2.34.1

The error log being reported by rte_bbdev_enqueue_status_str is displayed even without an actual error. The function may be called for such index (notably for queue dump) for valid reasons.

---
 lib/bbdev/rte_bbdev.c | 1 -
 1 file changed, 1 deletion(-)

index ea644c1f9b..3961c9deb3 100644
@@ -1236,7 +1236,6 @@ rte_bbdev_enqueue_status_str(enum rte_bbdev_enqueue_status status)
 	if ((uint8_t)status < sizeof(enq_sta_string) / sizeof(char *))
 		return enq_sta_string[status];
 
-	rte_bbdev_log(ERR, "Invalid enqueue status");
 	return NULL;
 }
 

2.34.1

Thanks,

Maxime

Fix to the computation of available dequeue size. Protects notably for possible negative scenarios when dequeuing descriptors.

---
 drivers/baseband/acc/acc_common.h | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

index 3d411cd25f..255e0971b8 100644
@@ -1602,7 +1602,7 @@ acc_ring_avail_enq(struct acc_queue *q)
 static inline uint32_t
 acc_ring_avail_deq(struct acc_queue *q)
 {
-	return (q->sw_ring_depth + q->sw_ring_head - q->sw_ring_tail) & q->sw_ring_wrap_mask;
+	return (q->sw_ring_head - q->sw_ring_tail) & q->sw_ring_wrap_mask;
 }
 
 /* Check room in AQ for the enqueues batches into Qmgr */

2.34.1

Thanks,
Maxime

Update of the PF2VF comms between bbdev and pf_bb_config to make sure all FFT windows can be read.
So far only the first window was actually used and functional. No system impact. Affects VRB1 and VRB2.

---
 drivers/baseband/acc/rte_vrb_pmd.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

index 1f85e33462..b9ff027ce8 100644
@@ -219,7 +219,7 @@ vrb_device_fft_win(struct rte_bbdev *dev)
 	/* Check from the device the first time. */
 	if (d->fft_window_width[0] == 0) {
 		for (win = 0; win < ACC_MAX_FFT_WIN; win++) {
-			vrb_vf2pf(d, ACC_VF2PF_FFT_WIN_REQUEST | win);
+			vrb_vf2pf(d, ACC_VF2PF_FFT_WIN_REQUEST | (win << 8));
 			reg = acc_reg_read(d, d->reg_addr->pf2vf_doorbell);
 			while ((time_out < ACC_STATUS_TO) && (reg == RTE_BBDEV_DEV_NOSTATUS)) {
 				usleep(ACC_STATUS_WAIT); /* Wait or VF->PF->VF Comms. */

2.34.1

Thanks,
Maxime

When get_queue_id_from_ring_info() fails to match a queue index (returning UINT16_MAX), an early return skipped clearing the ring entry and advancing the head pointer. This left the valid bit set, causing the same malformed entry to be reprocessed on every subsequent interrupt, blocking all further ring processing.

Replace the return with break so the entry is consumed and the loop continues normally. Fix applies to both PF and VF paths.

---
 drivers/baseband/acc/rte_vrb_pmd.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

index b9ff027ce8..803e0c9177 100644
@@ -454,7 +454,7 @@ vrb_dev_interrupt_handler(void *cb_arg)
 					rte_bbdev_log(ERR,
 							"Couldn't find queue: aq_id: %u, qg_id: %u, vf_id: %u",
 							aq_id, qg_id, vf_id);
-					return;
+					break;
 				}
 				rte_bbdev_pmd_callback_process(dev,
 						RTE_BBDEV_EVENT_DEQUEUE, &deq_intr_det);

@@ -482,7 +482,7 @@ vrb_dev_interrupt_handler(void *cb_arg)
 					rte_bbdev_log(ERR,
 							"Couldn't find queue: aq_id: %u, qg_id: %u",
 							aq_id, qg_id);
-					return;
+					break;
 				}
 				rte_bbdev_pmd_callback_process(dev,
 						RTE_BBDEV_EVENT_DEQUEUE, &deq_intr_det);

2.34.1

Thanks,
Maxime

Part of the FCW needs to be explicit set to zero when alternating between TB mode.

---
 drivers/baseband/acc/acc_common.h | 3 +++
 1 file changed, 3 insertions(+)

index 255e0971b8..d70a14dcbe 100644
@@ -1082,6 +1082,9 @@ acc_fcw_le_fill(const struct rte_bbdev_enc_op *op,
 	fcw->k0 = get_k0(fcw->ncb, fcw->Zc, op->ldpc_enc.basegraph,
 			op->ldpc_enc.rv_index, 0);
 	fcw->rm_e = (default_e == 0) ? op->ldpc_enc.cb_params.e : default_e;
+	fcw->rm_e_b = 0;
+	fcw->C = 0;
+	fcw->Cab = 0;
 	fcw->crc_select = check_bit(op->ldpc_enc.op_flags,
 			RTE_BBDEV_LDPC_CRC_24B_ATTACH);
 	fcw->bypass_intlv = check_bit(op->ldpc_enc.op_flags,

2.34.1

Thanks,
Maxime