[v2,3/5] common/sfc_efx/base: use the updated definitions of cidx/pidx
Checks
Commit Message
From: Abhimanyu Saini <absaini@amd.com>
Change cidx and pidx definition to mean used queue and avail
queue index respectively.
Signed-off-by: Abhimanyu Saini <absaini@amd.com>
---
v2:
* Fix checkpatch warnings
* Add a cover letter
drivers/common/sfc_efx/base/efx.h | 12 ++++++------
drivers/common/sfc_efx/base/rhead_virtio.c | 16 ++++++++--------
drivers/vdpa/sfc/sfc_vdpa_ops.c | 4 ++--
3 files changed, 16 insertions(+), 16 deletions(-)
@@ -4886,17 +4886,17 @@ typedef enum efx_virtio_vq_type_e {
typedef struct efx_virtio_vq_dyncfg_s {
/*
- * If queue is being created to be migrated then this
- * should be the FINAL_PIDX value returned by MC_CMD_VIRTIO_FINI_QUEUE
+ * If queue is being created to be migrated then this should be
+ * the FINAL_AVAIL_IDX value returned by MC_CMD_VIRTIO_FINI_QUEUE
* of the queue being migrated from. Otherwise, it should be zero.
*/
- uint32_t evvd_vq_pidx;
+ uint32_t evvd_vq_avail_idx;
/*
- * If this queue is being created to be migrated then this
- * should be the FINAL_CIDX value returned by MC_CMD_VIRTIO_FINI_QUEUE
+ * If queue is being created to be migrated then this should be
+ * the FINAL_USED_IDX value returned by MC_CMD_VIRTIO_FINI_QUEUE
* of the queue being migrated from. Otherwise, it should be zero.
*/
- uint32_t evvd_vq_cidx;
+ uint32_t evvd_vq_used_idx;
} efx_virtio_vq_dyncfg_t;
/*
@@ -95,10 +95,10 @@ rhead_virtio_qstart(
evvcp->evcc_features >> 32);
if (evvdp != NULL) {
- MCDI_IN_SET_DWORD(req, VIRTIO_INIT_QUEUE_REQ_INITIAL_PIDX,
- evvdp->evvd_vq_pidx);
- MCDI_IN_SET_DWORD(req, VIRTIO_INIT_QUEUE_REQ_INITIAL_CIDX,
- evvdp->evvd_vq_cidx);
+ MCDI_IN_SET_DWORD(req, VIRTIO_INIT_QUEUE_REQ_INITIAL_AVAIL_IDX,
+ evvdp->evvd_vq_avail_idx);
+ MCDI_IN_SET_DWORD(req, VIRTIO_INIT_QUEUE_REQ_INITIAL_USED_IDX,
+ evvdp->evvd_vq_used_idx);
}
MCDI_IN_SET_DWORD(req, VIRTIO_INIT_QUEUE_REQ_MPORT_SELECTOR,
@@ -161,10 +161,10 @@ rhead_virtio_qstop(
}
if (evvdp != NULL) {
- evvdp->evvd_vq_pidx =
- MCDI_OUT_DWORD(req, VIRTIO_FINI_QUEUE_RESP_FINAL_PIDX);
- evvdp->evvd_vq_cidx =
- MCDI_OUT_DWORD(req, VIRTIO_FINI_QUEUE_RESP_FINAL_CIDX);
+ evvdp->evvd_vq_avail_idx =
+ MCDI_OUT_DWORD(req, VIRTIO_FINI_QUEUE_RESP_FINAL_AVAIL_IDX);
+ evvdp->evvd_vq_used_idx =
+ MCDI_OUT_DWORD(req, VIRTIO_FINI_QUEUE_RESP_FINAL_USED_IDX);
}
return (0);
@@ -258,8 +258,8 @@ sfc_vdpa_virtq_start(struct sfc_vdpa_ops_data *ops_data, int vq_num)
vq_cfg.evvc_used_ring_addr = vring.used;
vq_cfg.evvc_vq_size = vring.size;
- vq_dyncfg.evvd_vq_pidx = vring.last_used_idx;
- vq_dyncfg.evvd_vq_cidx = vring.last_avail_idx;
+ vq_dyncfg.evvd_vq_used_idx = vring.last_used_idx;
+ vq_dyncfg.evvd_vq_avail_idx = vring.last_avail_idx;
/* MSI-X vector is function-relative */
vq_cfg.evvc_msix_vector = RTE_INTR_VEC_RXTX_OFFSET + vq_num;