[1/2] net/mana: add version information for dependencies

Message ID 1674181168-31167-1-git-send-email-longli@linuxonhyperv.com (mailing list archive)
State Accepted, archived
Delegated to: Ferruh Yigit
Headers
Series [1/2] net/mana: add version information for dependencies |

Checks

Context Check Description
ci/checkpatch success coding style OK

Commit Message

Long Li Jan. 20, 2023, 2:19 a.m. UTC
  From: Long Li <longli@microsoft.com>

The required dependencies for mana from rdma-core and Linux kernel have
been released. Add those information to the doc.

Fixes: 517ed6e2d590 ("net/mana: add basic driver with build environment")
Signed-off-by: Long Li <longli@microsoft.com>
---
 doc/guides/nics/mana.rst | 8 +++-----
 1 file changed, 3 insertions(+), 5 deletions(-)
  

Comments

Ferruh Yigit Feb. 20, 2023, 3:52 p.m. UTC | #1
On 1/20/2023 2:19 AM, longli@linuxonhyperv.com wrote:

> From: Long Li <longli@microsoft.com>
> 
> The required dependencies for mana from rdma-core and Linux kernel have
> been released. Add those information to the doc.
> 
> Fixes: 517ed6e2d590 ("net/mana: add basic driver with build environment")
> Signed-off-by: Long Li <longli@microsoft.com>
> ---
>  doc/guides/nics/mana.rst | 8 +++-----
>  1 file changed, 3 insertions(+), 5 deletions(-)
> 
> diff --git a/doc/guides/nics/mana.rst b/doc/guides/nics/mana.rst
> index 005c0b2ca7..341146c4e7 100644
> --- a/doc/guides/nics/mana.rst
> +++ b/doc/guides/nics/mana.rst
> @@ -29,6 +29,7 @@ and must be installed separately:
>    It allows slow and privileged operations
>    (context initialization, hardware resources allocations)
>    to be managed by the kernel and fast operations to never leave user space.
> +  The minimum required rdma-core version is v44.
> 
>    In most cases, rdma-core is shipped as a package with an OS distribution.
>    User can also install the upstream version of the rdma-core from
> @@ -39,15 +40,14 @@ and must be installed separately:
>    Low-level user space driver library
>    for Microsoft Azure Network Adapter devices,
>    it is automatically loaded by libibverbs.
> -
> -  The support of MANA is not merged in rdma-core 42.
> +  The minimum required version of rdma-core with libmana is v44.
> 
>  - **Kernel modules**
> 
>    They provide the kernel-side verbs API and low level device drivers
>    that manage actual hardware initialization
>    and resources sharing with user space processes.
> -  The minimum required Linux kernel version is 6.1.
> +  The minimum required Linux kernel version is 6.2.
> 

Hi Long,

Linux 6.2 seems release, can you please confirm required dependency is
part of 6.2 release.

>    Unlike most other PMDs, these modules must remain loaded
>    and bound to their devices:
> @@ -56,8 +56,6 @@ 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.
> -
>  Driver compilation and testing
>  ------------------------------
> 
> --
> 2.17.1
>
  
Long Li Feb. 21, 2023, 5:26 p.m. UTC | #2
> Subject: Re: [PATCH 1/2] net/mana: add version information for dependencies
> 
> On 1/20/2023 2:19 AM, longli@linuxonhyperv.com wrote:
> 
> > From: Long Li <longli@microsoft.com>
> >
> > The required dependencies for mana from rdma-core and Linux kernel
> > have been released. Add those information to the doc.
> >
> > Fixes: 517ed6e2d590 ("net/mana: add basic driver with build
> > environment")
> > Signed-off-by: Long Li <longli@microsoft.com>
> > ---
> >  doc/guides/nics/mana.rst | 8 +++-----
> >  1 file changed, 3 insertions(+), 5 deletions(-)
> >
> > diff --git a/doc/guides/nics/mana.rst b/doc/guides/nics/mana.rst index
> > 005c0b2ca7..341146c4e7 100644
> > --- a/doc/guides/nics/mana.rst
> > +++ b/doc/guides/nics/mana.rst
> > @@ -29,6 +29,7 @@ and must be installed separately:
> >    It allows slow and privileged operations
> >    (context initialization, hardware resources allocations)
> >    to be managed by the kernel and fast operations to never leave user space.
> > +  The minimum required rdma-core version is v44.
> >
> >    In most cases, rdma-core is shipped as a package with an OS distribution.
> >    User can also install the upstream version of the rdma-core from @@
> > -39,15 +40,14 @@ and must be installed separately:
> >    Low-level user space driver library
> >    for Microsoft Azure Network Adapter devices,
> >    it is automatically loaded by libibverbs.
> > -
> > -  The support of MANA is not merged in rdma-core 42.
> > +  The minimum required version of rdma-core with libmana is v44.
> >
> >  - **Kernel modules**
> >
> >    They provide the kernel-side verbs API and low level device drivers
> >    that manage actual hardware initialization
> >    and resources sharing with user space processes.
> > -  The minimum required Linux kernel version is 6.1.
> > +  The minimum required Linux kernel version is 6.2.
> >
> 
> Hi Long,
> 
> Linux 6.2 seems release, can you please confirm required dependency is part
> of 6.2 release.

Hi Ferruh,

I can confirm the required dependency is in Linux 6.2.

Thanks,
Long

> 
> >    Unlike most other PMDs, these modules must remain loaded
> >    and bound to their devices:
> > @@ -56,8 +56,6 @@ 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.
> > -
> >  Driver compilation and testing
> >  ------------------------------
> >
> > --
> > 2.17.1
> >
  
Ferruh Yigit Feb. 22, 2023, 12:09 a.m. UTC | #3
On 1/20/2023 2:19 AM, longli@linuxonhyperv.com wrote:
> From: Long Li <longli@microsoft.com>
> 
> The required dependencies for mana from rdma-core and Linux kernel have
> been released. Add those information to the doc.
> 
> Fixes: 517ed6e2d590 ("net/mana: add basic driver with build environment")
> Signed-off-by: Long Li <longli@microsoft.com>

For series,
Acked-by: Ferruh Yigit <ferruh.yigit@amd.com>

Series applied to dpdk-next-net/main, thanks.
  

Patch

diff --git a/doc/guides/nics/mana.rst b/doc/guides/nics/mana.rst
index 005c0b2ca7..341146c4e7 100644
--- a/doc/guides/nics/mana.rst
+++ b/doc/guides/nics/mana.rst
@@ -29,6 +29,7 @@  and must be installed separately:
   It allows slow and privileged operations
   (context initialization, hardware resources allocations)
   to be managed by the kernel and fast operations to never leave user space.
+  The minimum required rdma-core version is v44.
 
   In most cases, rdma-core is shipped as a package with an OS distribution.
   User can also install the upstream version of the rdma-core from
@@ -39,15 +40,14 @@  and must be installed separately:
   Low-level user space driver library
   for Microsoft Azure Network Adapter devices,
   it is automatically loaded by libibverbs.
-
-  The support of MANA is not merged in rdma-core 42.
+  The minimum required version of rdma-core with libmana is v44.
 
 - **Kernel modules**
 
   They provide the kernel-side verbs API and low level device drivers
   that manage actual hardware initialization
   and resources sharing with user space processes.
-  The minimum required Linux kernel version is 6.1.
+  The minimum required Linux kernel version is 6.2.
 
   Unlike most other PMDs, these modules must remain loaded
   and bound to their devices:
@@ -56,8 +56,6 @@  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.
-
 Driver compilation and testing
 ------------------------------