Re: [PATCH] net/mana: fix dependencies

[Thomas Monjalon <thomas@monjalon.net>]

10/10/2022 20:53, Long Li:
> I will send respin as soon as patch submission window reopens.

So what do we do in DPDK?
Do you agree with the patch I sent to correct wrong assumptions
in the doc?

My fear is that if we keep mana PMD as-is,
it could break compilation when libmana will be released
if there is an incompatibility with what you have in staging.

Please remember that rdma-core is also used for mlx PMDs,
and we don't want mana to break compilation when updating rdma-core.


> From: Jason Gunthorpe <jgg@nvidia.com>
> Sent: Sunday, October 9, 2022 7:08 AM
> To: thomas <thomas@monjalon.net>; Long Li <longli@microsoft.com>
> Cc: dev@dpdk.org; ferruh.yigit@amd.com; andrew.rybchenko@oktetlabs.ru; stephen@networkplumber.org
> Subject: Re: [PATCH] net/mana: fix dependencies
> 
> I'm hopeful for 6.2, but it needs a respin I haven't seen yet.
> 
> From: Thomas Monjalon <thomas@monjalon.net<mailto:thomas@monjalon.net>>
> 
> Long,
> 
> For the reasons below, we should not have merged the MANA PMD in DPDK 22.11.
> We must not assume dependencies until they are upstream.
> This is a red flag on the PMD, a revert is possible.
> 
> Jason,
> 
> Please could you tell us what is the expected date of release
> of the MANA support in Linux and rdma-core?
> 
> 
> 09/10/2022 15:53, Thomas Monjalon:
> [...]
> > The libibverbs was not detected through pkg-config.
> > The method dependency() needs to be used first.
> >
> > The support in rdma-core and Linux is not released yet,
> > so the documentation is updated.
> 
> [...]
> > -  The minimum required version of rdma-core with libmana is v43.
> > +
> > +  The support of MANA is not merged in rdma-core 42.
> >
> >  - **Kernel modules**
> >
> > @@ -56,6 +56,8 @@ and must be installed separately:
> >    - mana_ib: InifiniBand device driver.
> >    - ib_uverbs: user space driver for verbs (entry point for libibverbs).
> >
> > +  The support of MANA is planned in Linux 6.2.