RE: [PATCH] net/mana: fix dependencies

[Long Li <longli@microsoft.com>]

> Subject: Re: [PATCH] net/mana: fix dependencies
> 
> 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?

Yes, the patch looks good to me.

> 
> 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.

I hope that will not happen. After MANA PMD is merged, any changes in libmana while in staging will not break the PMD. (At minimum, it will guarantee not to break compilation)

> 
> 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.
> 
>