[dpdk-dev] [PATCH v1 04/48] net/mlx4: remove useless compilation checks

Adrien Mazarguil adrien.mazarguil at 6wind.com
Fri Sep 1 12:19:45 CEST 2017


Hi Ferruh,

On Fri, Aug 18, 2017 at 02:39:18PM +0100, Ferruh Yigit wrote:
> On 8/1/2017 5:53 PM, Adrien Mazarguil wrote:
> > Verbs support for RSS, inline receive and extended device query calls has
> > not been optional for a while. Their absence is untested and is therefore
> > unsupported.
> > 
> > Remove the related compilation checks and assume Mellanox OFED is up to
> > date, as described in the documentation.
> 
> So this requires Mellanox OFED 4.1 is there,

Well, the PMD most likely works with versions older than that, but they are
not officially supported anymore. These changes assert that no effort is
made to maintain compatibility.

> is there a check for the OFED version, or do you think does it required?

I think it's not necessary. You know, the existing checks haven't been
validated for a very long time, mlx4 probably does not even compile anymore
without these features. Even if it does, it might be unable to perform TX/RX
at all. Getting a compilation failure at least makes things clear.

> 
> > 
> > Use this opportunity to remove a few useless data path debugging messages
> > behind compilation checks on never defined macros.
> > 
> > Signed-off-by: Adrien Mazarguil <adrien.mazarguil at 6wind.com>
> 
> <...>
> 

-- 
Adrien Mazarguil
6WIND


More information about the dev mailing list