On Thu, 21 May 2015 16:56:00 +0800 Cunming Liang <cunming.liang at intel.com> wrote:
> + if (!rte_eth_dev_is_valid_port(port_id)) { > + PMD_DEBUG_TRACE("Invalid port_id=%d\n", port_id); > + return -ENODEV; > + } > + > + dev = &rte_eth_devices[port_id]; > + if (dev == NULL) { > + PMD_DEBUG_TRACE("Invalid port device\n"); > + return -ENODEV; > + } This check is not needed, rte_eth_dev_is_valid_port already checked that.