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.

Reply via email to