[PATCH v7 05/21] net/cpfl: support queue start

[Liu, Mingxia]

If time permits, we will first submit a fixed patch to add a q type member in rx_queue strc.

> -----Original Message-----
> From: Liu, Mingxia
> Sent: Tuesday, February 28, 2023 11:15 AM
> To: Ferruh Yigit <ferruh.yigit at amd.com>; dev at dpdk.org; Xing, Beilei
> <beilei.xing at intel.com>; Zhang, Yuying <Yuying.Zhang at intel.com>
> Subject: RE: [PATCH v7 05/21] net/cpfl: support queue start
> 
> Thanks, Ferruh !
> It's a good idea!
> 
> As for it involves a wide range of change, including idpf common module
> and idpf pmd code, So we'd better update it in the later fix patch after rc2.
> 
> > -----Original Message-----
> > From: Ferruh Yigit <ferruh.yigit at amd.com>
> > Sent: Tuesday, February 28, 2023 5:47 AM
> > To: Liu, Mingxia <mingxia.liu at intel.com>; dev at dpdk.org; Xing, Beilei
> > <beilei.xing at intel.com>; Zhang, Yuying <yuying.zhang at intel.com>
> > Subject: Re: [PATCH v7 05/21] net/cpfl: support queue start
> >
> > 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.liu at 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?
> >