> -----Original Message-----
> From: Stephen Hemminger <step...@networkplumber.org>
> Sent: Tuesday, August 22, 2023 10:03 PM
> To: Feifei Wang <feifei.wa...@arm.com>
> Cc: tho...@monjalon.net; Ferruh Yigit <ferruh.yi...@amd.com>; Andrew
> Rybchenko <andrew.rybche...@oktetlabs.ru>; dev@dpdk.org; nd
> <n...@arm.com>; Honnappa Nagarahalli <honnappa.nagaraha...@arm.com>;
> Ruifeng Wang <ruifeng.w...@arm.com>; Morten Brørup
> <m...@smartsharesystems.com>
> Subject: Re: [PATCH v11 1/4] ethdev: add API for mbufs recycle mode
>
> On Tue, 22 Aug 2023 15:27:07 +0800
> Feifei Wang <feifei.wa...@arm.com> wrote:
>
> > + if (queue_id >= dev->data->nb_rx_queues) {
> > + RTE_ETHDEV_LOG(ERR, "Invalid Rx queue_id=%u\n",
> queue_id);
> > + return -EINVAL;
> > + }
> > +
> > + if (dev->data->rx_queues == NULL ||
> > + dev->data->rx_queues[queue_id] == NULL) {
> > + RTE_ETHDEV_LOG(ERR,
> > + "Rx queue %"PRIu16" of device with port_id=%"
> > + PRIu16" has not been setup\n",
> > + queue_id, port_id);
> > + return -EINVAL;
> > + }
>
> This could be changed to something like:
>
> ret = eth_dev_validate_rx_queue(dev, queue_id);
> if (unlikely(ret != 0))
> return ret;
>
Good comments, I just know there is an API to check these. Thanks.
>
>
