Re: [PATCH v3 0/3] Mellanox ConnectX-3 PMD

[Gleb Natapov <gleb-RmZWMc9puTNJc61us3aD9laTQe2KTcn/@public.gmane.org>]

On Fri, Feb 27, 2015 at 07:38:59PM +0100, Adrien Mazarguil wrote:
> On Thu, Feb 26, 2015 at 03:49:07PM +0200, Gleb Natapov wrote:
> > On Thu, Feb 26, 2015 at 02:36:27PM +0100, Thomas Monjalon wrote:
> > > 2015-02-26 13:51, Gleb Natapov:
> > > > Did git pull today. After enabling mlnx pmd compilation fails with:
> > > > 
> > > > dpdk/lib/librte_pmd_mlx4/mlx4.c: In function ‘mlx4_pci_devinit’:
> > > > dpdk/lib/librte_pmd_mlx4/mlx4.c:4636:14: error: too few arguments to function ‘rte_eth_dev_allocate’
> > > >     eth_dev = rte_eth_dev_allocate(name);
> > > 
> > > Yes, thanks for reporting.
> > > I didn't test the disabled mlx4 after hotplug integration:
> > > 	dpdk.org/browse/dpdk/commit/?id=9f1653e7b7e1746e7c
> > > 
> > > Clearly, I have to improve my sanity checks.
> > > Sorry for the inconvenience.
> > No problem, I fixed that locally, but now I see another issue. I have
> > several PMDs statically compiled in with my application and I expect
> > dpdk to choose correct one depending on available HW, but mlnx pmd does
> > not behave nicely, if its initialization fails it kills entire
> > application:
> > 
> > EAL: PCI device 0000:03:00.0 on NUMA socket 0
> > EAL:   probe driver: 15b3:1003 librte_pmd_mlx4
> > EAL: Error - exiting with code: 1
> >   Cause: Requested device 0000:03:00.0 cannot be used
> 
> Forgot to set in-reply-to, but I just sent a patch to work around that
> issue and make mlx4 nicer:
> 
> http://dpdk.org/dev/patchwork/patch/3796/
> 
Works for me, thanks! May be better to change:
"cannot use device, are drivers up to date? To "cannot use device, are
drivers and fw up to date?". On one of my machines that was the case.

I see that some features are missing from the PMD though. Some of them
are nice to have for good performance like providing rss hash value
in mbuf (PKT_RX_RSS_HASH), but others are absolutely required for
my application to work: setting (or at least getting) rss key and
redirection table. Are there any plans to support those?

--
			Gleb.