On 2/16/2023 12:29 AM, Mingxia Liu wrote:
> Add support for these device ops:
> - rx_queue_start
> - tx_queue_start
>
> Signed-off-by: Mingxia Liu <mingxia....@intel.com>
<...>
> +int
> +cpfl_rx_queue_init(struct rte_eth_dev *dev, uint16_t rx_queue_id)
> +{
> + struct idpf_rx_queue *rxq;
> + int err;
> +
> + if (rx_queue_id >= dev->data->nb_rx_queues)
> + return -EINVAL;
> +
> + rxq = dev->data->rx_queues[rx_queue_id];
> +
> + if (rxq == NULL || !rxq->q_set) {
> + PMD_DRV_LOG(ERR, "RX queue %u not available or setup",
> + rx_queue_id);
> + return -EINVAL;
> + }
> +
> + if (rxq->bufq1 == NULL) {
> + /* Single queue */
What do you think to keep the queue type explicitly in the queue struct,
instead of deducing it from pointer values?
