Thanks, applied to master
2016-08-12 9:10 GMT-07:00 Mauricio Vasquez <mauricio.vasq...@polito.it>:
> On 08/12/2016 05:57 PM, Ciara Loftus wrote:
>
>> Signed-off-by: Ciara Loftus <ciara.lof...@intel.com>
>>
> It is a necessary fix, specially if hotplug will be introduced.
> Acked-by: Mauricio Vásquez B <mauricio.vasq...@polito.it>
>
>> ---
>> lib/netdev-dpdk.c | 5 +++--
>> 1 file changed, 3 insertions(+), 2 deletions(-)
>>
>> diff --git a/lib/netdev-dpdk.c b/lib/netdev-dpdk.c
>> index 9a1f7cd..329b4ea 100644
>> --- a/lib/netdev-dpdk.c
>> +++ b/lib/netdev-dpdk.c
>> @@ -648,7 +648,7 @@ dpdk_eth_dev_init(struct netdev_dpdk *dev)
>> OVS_REQUIRES(dpdk_mutex)
>> int diag;
>> int n_rxq, n_txq;
>> - if (dev->port_id < 0 || dev->port_id >= rte_eth_dev_count()) {
>> + if (!rte_eth_dev_is_valid_port(dev->port_id)) {
>> return ENODEV;
>> }
>> @@ -2122,8 +2122,9 @@ netdev_dpdk_get_status(const struct netdev
>> *netdev, struct smap *args)
>> struct netdev_dpdk *dev = netdev_dpdk_cast(netdev);
>> struct rte_eth_dev_info dev_info;
>> - if (dev->port_id < 0)
>> + if (!rte_eth_dev_is_valid_port(dev->port_id)) {
>> return ENODEV;
>> + }
>> ovs_mutex_lock(&dev->mutex);
>> rte_eth_dev_info_get(dev->port_id, &dev_info);
>>
>
> _______________________________________________
> dev mailing list
> dev@openvswitch.org
> http://openvswitch.org/mailman/listinfo/dev
>
_______________________________________________
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/dev
