Re: [dpdk-dev] [PATCH] ethdev: check if queue setupped in queue-related APIs

2020-10-11 Thread Wei Hu (Xavier)
Hi, Kalesh Anakkur Purayil On 2020/10/11 0:38, Kalesh Anakkur Purayil wrote: On Sat, Oct 10, 2020 at 12:42 PM Wei Hu (Xavier) mailto:huwei...@chinasoftinc.com>> wrote: From: Chengchang Tang mailto:tangchengch...@huawei.com>> This patch adds checking whether the related Tx or Rx que

Re: [dpdk-dev] [PATCH] ethdev: check if queue setupped in queue-related APIs

2020-10-11 Thread Wei Hu (Xavier)
Hi, Stephen Hemminger On 2020/10/10 23:24, Stephen Hemminger wrote: On Sat, 10 Oct 2020 15:12:12 +0800 "Wei Hu (Xavier)" wrote: + if (dev->data->rx_queues[rx_queue_id] == NULL) { + RTE_ETHDEV_LOG(ERR, "Rx queue %"PRIu16" of device with port_id=%" +

Re: [dpdk-dev] [PATCH] ethdev: check if queue setupped in queue-related APIs

2020-10-10 Thread Kalesh Anakkur Purayil
On Sat, Oct 10, 2020 at 12:42 PM Wei Hu (Xavier) wrote: > From: Chengchang Tang > > This patch adds checking whether the related Tx or Rx queue has been > setupped in the queue-related API functions to avoid illegal address > access. And validity check of the queue_id is also added in the API >

Re: [dpdk-dev] [PATCH] ethdev: check if queue setupped in queue-related APIs

2020-10-10 Thread Stephen Hemminger
On Sat, 10 Oct 2020 15:12:12 +0800 "Wei Hu (Xavier)" wrote: > + if (dev->data->rx_queues[rx_queue_id] == NULL) { > + RTE_ETHDEV_LOG(ERR, "Rx queue %"PRIu16" of device with > port_id=%" > + PRIu16" has not been setupped\n", > +

[dpdk-dev] [PATCH] ethdev: check if queue setupped in queue-related APIs

2020-10-10 Thread Wei Hu (Xavier)
From: Chengchang Tang This patch adds checking whether the related Tx or Rx queue has been setupped in the queue-related API functions to avoid illegal address access. And validity check of the queue_id is also added in the API functions rte_eth_dev_rx_intr_enable and rte_eth_dev_rx_intr_disable.