Re: [PATCH] lib/lib_eal:add mellanox kernel driver type

[Thomas Monjalon <thomas@monjalon.net>]

23/08/2017 10:04, Gaëtan Rivet:
> Hi,
> 
> On Wed, Aug 23, 2017 at 10:48:01AM +0800, Rongqiang XIE wrote:
> > When use bond function in mellanox driver environment, we call the
> > find_port_id_by_pci_addr() function,if we don't add mellanox kernel
> > driver type in enum rte_kernel_driver, the function will return -1
> > because kdrv unknown, so we add the mellanox driver type, and when
> > scan the pci, fill the kdrv to fix this problem.
> > 
> 
> The issue is not that MLX devices are not properly represented, but
> that the find_port_id_by_pci_addr function has a crappy way to recognize
> PCI devices.
> 
> The find_device bus operator should be used to find a device having
> a specific rte_pci_addr. I described something similar not too long
> ago[1]. This matching function would benefit from being properly
> rewritten.
> 
> [1]: http://dpdk.org/ml/archives/dev/2017-July/072172.html
> 
> > Signed-off-by: Rongqiang XIE <xie.rongqiang@zte.com.cn>

This issue has been fixed as a result of the thread which
Gaetan is referencing:
	http://dpdk.org/browse/dpdk/commit/?id=c848b518bb