Re: [PATCH v3 2/2] doc: update mlx5 required OFED version

[Adrien Mazarguil <adrien.mazarguil@6wind.com>]

On Mon, Feb 05, 2018 at 02:31:12PM +0200, Shahaf Shuler wrote:
> Signed-off-by: Shahaf Shuler <shahafs@mellanox.com>

With one minor nit, title only mentions mlx5 while mlx4 is also affected:

Acked-by: Adrien Mazarguil <adrien.mazarguil@6wind.com>

> ---
> 
> On v3:
>  - Highlighted flow counters are supported only in MLNX_OFED. 
> 
> On v2:
>  - Updated the OFED version for mlx4 guide as well.
> ---
>  doc/guides/nics/mlx4.rst | 2 +-
>  doc/guides/nics/mlx5.rst | 4 ++--
>  2 files changed, 3 insertions(+), 3 deletions(-)
> 
> diff --git a/doc/guides/nics/mlx4.rst b/doc/guides/nics/mlx4.rst
> index 88161781c..13cc6ec8a 100644
> --- a/doc/guides/nics/mlx4.rst
> +++ b/doc/guides/nics/mlx4.rst
> @@ -219,7 +219,7 @@ Current RDMA core package and Linux kernel (recommended)
>  Mellanox OFED as a fallback
>  ~~~~~~~~~~~~~~~~~~~~~~~~~~~
>  
> -- `Mellanox OFED`_ version: **4.2**.
> +- `Mellanox OFED`_ version: **4.2, 4.3**.
>  - firmware version: **2.42.5000** and above.
>  
>  .. _`Mellanox OFED`: http://www.mellanox.com/page/products_dyn?product_family=26&mtag=linux_sw_drivers
> diff --git a/doc/guides/nics/mlx5.rst b/doc/guides/nics/mlx5.rst
> index a9e4bf51a..d54ff760a 100644
> --- a/doc/guides/nics/mlx5.rst
> +++ b/doc/guides/nics/mlx5.rst
> @@ -131,7 +131,7 @@ Limitations
>  - A multi segment packet must have less than 6 segments in case the Tx burst function
>    is set to multi-packet send or Enhanced multi-packet send. Otherwise it must have
>    less than 50 segments.
> -- Count action for RTE flow is only supported in Mellanox OFED 4.2.
> +- Count action for RTE flow is **only supported in Mellanox OFED**.
>  - Flows with a VXLAN Network Identifier equal (or ends to be equal)
>    to 0 are not supported.
>  - VXLAN TSO and checksum offloads are not supported on VM.
> @@ -392,7 +392,7 @@ RMDA Core with Linux Kernel
>  Mellanox OFED
>  ^^^^^^^^^^^^^
>  
> -- Mellanox OFED version: **4.2**.
> +- Mellanox OFED version: **4.2, 4.3**.
>  - firmware version:
>  
>    - ConnectX-4: **12.21.1000** and above.
> -- 
> 2.12.0
> 

-- 
Adrien Mazarguil
6WIND