2015-07-24 10:06, Stephen Hemminger: > --- a/lib/librte_ether/rte_ethdev.c > +++ b/lib/librte_ether/rte_ethdev.c > @@ -122,6 +122,7 @@ > } \ > } while (0) > > +
checkpatch would warn about this whitespace change > static const char *MZ_RTE_ETH_DEV_DATA = "rte_eth_dev_data"; > struct rte_eth_dev rte_eth_devices[RTE_MAX_ETHPORTS]; > static struct rte_eth_dev_data *rte_eth_dev_data; > @@ -2910,6 +2911,11 @@ rte_eth_rx_queue_count(uint8_t port_id, uint16_t > queue_id) > > VALID_PORTID_OR_ERR_RET(port_id, 0); > > + if (queue_id >= dev->data->nb_rx_queues) { > + PMD_RX_LOG(ERR, "Invalid RX queue id=%d", queue_id); > + return 0; > + } PMD_RX_LOG is not defined in ethdev.