[dpdk-dev] [PATCH v2] Change rte_eal_vdev_init to update port_id

[David Marchand]

Hello Ravi, Tetsuya,

On Tue, Aug 25, 2015 at 7:59 PM, Ravi Kerur <rkerur at gmail.com> wrote:

> Let us know how you want us to fix this? To fix rte_eal_vdev_init and
> rte_eal_pci_probe_one to return allocated port_id we had 2 approaches
> mentioned in earlier discussion. In addition to those we have another
> approach with changes isolated only to rte_ether component. I am attaching
> diffs (preliminary) with this email. Please let us know your inputs since
> it involves EAL component.
>

- This patch looks like a good ethdev cleanup (even if it really lacks some
context / commit log).

I wonder just why you only take the first part of the name in
rte_eth_dev_get_port_by_name().
Would not this match, let's say, both toto and toto0 vdevs ?
Is this intended ?


- In the end, with this patch, do we still need to update eal ?
Looking at the code, I am not sure anymore.


-- 
David Marchand