From: Slava Ovsiienko <viacheslavo@mellanox.com>
To: Thomas Monjalon <thomas@monjalon.net>, "dev@dpdk.org" <dev@dpdk.org>
Cc: Gal Cohen <galco@mellanox.com>, Matan Azrad <matan@mellanox.com>,
Shahaf Shuler <shahafs@mellanox.com>,
John McNamara <john.mcnamara@intel.com>,
Marko Kovacevic <marko.kovacevic@intel.com>
Subject: Re: [dpdk-dev] [PATCH] doc: fix naming of Mellanox devices
Date: Mon, 24 Feb 2020 08:59:40 +0000 [thread overview]
Message-ID: <AM4PR05MB3265C48B54BD1224149986D1D2EC0@AM4PR05MB3265.eurprd05.prod.outlook.com> (raw)
In-Reply-To: <20200221145927.345130-1-thomas@monjalon.net>
> -----Original Message-----
> From: Thomas Monjalon <thomas@monjalon.net>
> Sent: Friday, February 21, 2020 16:59
> To: dev@dpdk.org
> Cc: Gal Cohen <galco@mellanox.com>; Matan Azrad <matan@mellanox.com>;
> Shahaf Shuler <shahafs@mellanox.com>; Slava Ovsiienko
> <viacheslavo@mellanox.com>; John McNamara <john.mcnamara@intel.com>;
> Marko Kovacevic <marko.kovacevic@intel.com>
> Subject: [PATCH] doc: fix naming of Mellanox devices
>
> The devices of the family ConnectX may have two letters as suffix.
> Such suffix is preceded with a space and the second x is lowercase:
> - ConnectX-4 Lx
> - ConnectX-5 Ex
> - ConnectX-6 Dx
>
> Uppercase of the device family name BlueField is also fixed.
>
> Signed-off-by: Thomas Monjalon <thomas@monjalon.net>
Acked-by: Viacheslav Ovsiienko <viacheslavo@mellanox.com>
> ---
> config/common_base | 4 ++--
> doc/guides/nics/mlx5.rst | 24 ++++++++++++------------
> doc/guides/rel_notes/release_18_08.rst | 2 +-
> doc/guides/rel_notes/release_19_02.rst | 2 +-
> doc/guides/rel_notes/release_19_11.rst | 2 +-
> doc/guides/vdpadevs/mlx5.rst | 6 +++---
> drivers/net/mlx5/mlx5.c | 2 +-
> drivers/net/mlx5/mlx5_rxtx.c | 2 +-
> drivers/net/mlx5/mlx5_txq.c | 2 +-
> 9 files changed, 23 insertions(+), 23 deletions(-)
>
> diff --git a/config/common_base b/config/common_base index
> 6ea9c63cc3..7ca2f28b19 100644
> --- a/config/common_base
> +++ b/config/common_base
> @@ -361,13 +361,13 @@ CONFIG_RTE_LIBRTE_MLX4_DEBUG=n
>
> #
> # Compile burst-oriented Mellanox ConnectX-4, ConnectX-5, -# ConnectX-6 &
> Bluefield (MLX5) PMD
> +# ConnectX-6 & BlueField (MLX5) PMD
> #
> CONFIG_RTE_LIBRTE_MLX5_PMD=n
> CONFIG_RTE_LIBRTE_MLX5_DEBUG=n
>
> #
> -# Compile vdpa-oriented Mellanox ConnectX-6 & Bluefield (MLX5) PMD
> +# Compile vdpa-oriented Mellanox ConnectX-6 & BlueField (MLX5) PMD
> #
> CONFIG_RTE_LIBRTE_MLX5_VDPA_PMD=n
>
> diff --git a/doc/guides/nics/mlx5.rst b/doc/guides/nics/mlx5.rst index
> 5ab7c07165..748a87d8f3 100644
> --- a/doc/guides/nics/mlx5.rst
> +++ b/doc/guides/nics/mlx5.rst
> @@ -7,7 +7,7 @@ MLX5 poll mode driver
>
> The MLX5 poll mode driver library (**librte_pmd_mlx5**) provides support
> for **Mellanox ConnectX-4**, **Mellanox ConnectX-4 Lx** , **Mellanox -
> ConnectX-5**, **Mellanox ConnectX-6**, **Mellanox ConnectX-6DX** and
> +ConnectX-5**, **Mellanox ConnectX-6**, **Mellanox ConnectX-6 Dx** and
> **Mellanox BlueField** families of 10/25/40/50/100/200 Gb/s adapters as
> well as their virtual functions (VF) in SR-IOV context.
>
> @@ -324,9 +324,9 @@ Run-time configuration
>
> Supported on:
>
> - - x86_64 with ConnectX-4, ConnectX-4 LX, ConnectX-5, ConnectX-6,
> ConnectX-6 DX
> + - x86_64 with ConnectX-4, ConnectX-4 Lx, ConnectX-5, ConnectX-6,
> + ConnectX-6 Dx
> and BlueField.
> - - POWER9 and ARMv8 with ConnectX-4 LX, ConnectX-5, ConnectX-6,
> ConnectX-6 DX
> + - POWER9 and ARMv8 with ConnectX-4 Lx, ConnectX-5, ConnectX-6,
> + ConnectX-6 Dx
> and BlueField.
>
> - ``rxq_cqe_pad_en`` parameter [int]
> @@ -357,9 +357,9 @@ Run-time configuration
>
> Supported on:
>
> - - x86_64 with ConnectX-4, ConnectX-4 LX, ConnectX-5, ConnectX-6,
> ConnectX-6 DX
> + - x86_64 with ConnectX-4, ConnectX-4 Lx, ConnectX-5, ConnectX-6,
> + ConnectX-6 Dx
> and BlueField.
> - - POWER8 and ARMv8 with ConnectX-4 LX, ConnectX-5, ConnectX-6,
> ConnectX-6 DX
> + - POWER8 and ARMv8 with ConnectX-4 Lx, ConnectX-5, ConnectX-6,
> + ConnectX-6 Dx
> and BlueField.
>
> - ``mprq_en`` parameter [int]
> @@ -462,14 +462,14 @@ Run-time configuration
> If ``txq_inline_min`` key is not present, the value may be queried by the
> driver from the NIC via DevX if this feature is available. If there is no DevX
> enabled/supported the value 18 (supposing L2 header including VLAN) is set
> - for ConnectX-4 and ConnectX-4LX, and 0 is set by default for ConnectX-5
> + for ConnectX-4 and ConnectX-4 Lx, and 0 is set by default for
> + ConnectX-5
> and newer NICs. If packet is shorter the ``txq_inline_min`` value, the entire
> packet is inlined.
>
> For ConnectX-4 NIC, driver does not allow specifying value below 18
> (minimal L2 header, including VLAN), error will be raised.
>
> - For ConnectX-4LX NIC, it is allowed to specify values below 18, but
> + For ConnectX-4 Lx NIC, it is allowed to specify values below 18, but
> it is not recommended and may prevent NIC from sending packets over
> some configurations.
>
> @@ -552,7 +552,7 @@ Run-time configuration
> - ``txq_mpw_en`` parameter [int]
>
> A nonzero value enables Enhanced Multi-Packet Write (eMPW) for
> ConnectX-5,
> - ConnectX-6, ConnectX-6 DX and BlueField. eMPW allows the TX burst
> function to pack
> + ConnectX-6, ConnectX-6 Dx and BlueField. eMPW allows the TX burst
> + function to pack
> up multiple packets in a single descriptor session in order to save PCI
> bandwidth
> and improve performance at the cost of a slightly higher CPU usage. When
> ``txq_inline_mpw`` is set along with ``txq_mpw_en``, TX burst function
> copies @@ -598,7 +598,7 @@ Run-time configuration
>
> - ``tx_vec_en`` parameter [int]
>
> - A nonzero value enables Tx vector on ConnectX-5, ConnectX-6, ConnectX-6
> DX
> + A nonzero value enables Tx vector on ConnectX-5, ConnectX-6,
> + ConnectX-6 Dx
> and BlueField NICs if the number of global Tx queues on the port is less
> than
> ``txqs_max_vec``. The parameter is deprecated and ignored.
>
> @@ -899,7 +899,7 @@ Mellanox OFED/EN
> - ConnectX-5: **16.21.1000** and above.
> - ConnectX-5 Ex: **16.21.1000** and above.
> - ConnectX-6: **20.99.5374** and above.
> - - ConnectX-6 DX: **22.27.0090** and above.
> + - ConnectX-6 Dx: **22.27.0090** and above.
> - BlueField: **18.25.1010** and above.
>
> While these libraries and kernel modules are available on OpenFabrics @@ -
> 944,8 +944,8 @@ Supported NICs
> * Mellanox(R) ConnectX(R)-5 100G MCX556A-ECAT (2x100G)
> * Mellanox(R) ConnectX(R)-5 Ex EN 100G MCX516A-CDAT (2x100G)
> * Mellanox(R) ConnectX(R)-6 200G MCX654106A-HCAT (4x200G)
> -* Mellanox(R) ConnectX(R)-6DX EN 100G MCX623106AN-CDAT (2*100g)
> -* Mellanox(R) ConnectX(R)-6DX EN 200G MCX623105AN-VDAT (1*200g)
> +* Mellanox(R) ConnectX(R)-6 Dx EN 100G MCX623106AN-CDAT (2*100g)
> +* Mellanox(R) ConnectX(R)-6 Dx EN 200G MCX623105AN-VDAT (1*200g)
>
> Quick Start Guide on OFED/EN
> ----------------------------
> diff --git a/doc/guides/rel_notes/release_18_08.rst
> b/doc/guides/rel_notes/release_18_08.rst
> index 8a09dee95c..4ae388c331 100644
> --- a/doc/guides/rel_notes/release_18_08.rst
> +++ b/doc/guides/rel_notes/release_18_08.rst
> @@ -546,4 +546,4 @@ Tested Platforms
> * Mellanox MLNX_OFED 4.2-1.4.21.0
>
> * DPDK application running on ARM cores inside SmartNIC
> - * Bluefield representors support planned for next release.
> + * BlueField representors support planned for next release.
> diff --git a/doc/guides/rel_notes/release_19_02.rst
> b/doc/guides/rel_notes/release_19_02.rst
> index ace1534eff..87dfbf5c7d 100644
> --- a/doc/guides/rel_notes/release_19_02.rst
> +++ b/doc/guides/rel_notes/release_19_02.rst
> @@ -109,7 +109,7 @@ New Features
> ``CONFIG_RTE_IBVERBS_LINK_DLOPEN`` for make and ``ibverbs_link`` for
> meson.
> * Added static linkage of ``mlx`` dependency.
> * Improved stability of E-Switch flow driver.
> - * Added new make build configuration to set the cacheline size for Bluefield
> + * Added new make build configuration to set the cacheline size for
> + BlueField
> correctly - ``arm64-bluefield-linux-gcc``.
>
> * **Updated the enic driver.**
> diff --git a/doc/guides/rel_notes/release_19_11.rst
> b/doc/guides/rel_notes/release_19_11.rst
> index 84aa03a1f2..f0aee29477 100644
> --- a/doc/guides/rel_notes/release_19_11.rst
> +++ b/doc/guides/rel_notes/release_19_11.rst
> @@ -206,7 +206,7 @@ New Features
> * Added support for VLAN set VID offload command.
> * Added support for matching on packets withe Geneve tunnel header.
> * Added hairpin support.
> - * Added ConnectX6-DX support.
> + * Added ConnectX-6 Dx support.
> * Flow engine selected based on RDMA Core library version.
> DV flow engine selected if version is rdma-core-24.0 or higher.
> Verbs flow engine selected otherwise.
> diff --git a/doc/guides/vdpadevs/mlx5.rst b/doc/guides/vdpadevs/mlx5.rst
> index ce7c8a7415..cf878e1917 100644
> --- a/doc/guides/vdpadevs/mlx5.rst
> +++ b/doc/guides/vdpadevs/mlx5.rst
> @@ -6,7 +6,7 @@ MLX5 vDPA driver
>
> The MLX5 vDPA (vhost data path acceleration) driver library
> (**librte_pmd_mlx5_vdpa**) provides support for **Mellanox ConnectX-6**,
> -**Mellanox ConnectX-6DX** and **Mellanox BlueField** families of
> +**Mellanox ConnectX-6 Dx** and **Mellanox BlueField** families of
> 10/25/40/50/100/200 Gb/s adapters as well as their virtual functions (VF) in
> SR-IOV context.
>
> @@ -49,8 +49,8 @@ Supported NICs
> --------------
>
> * Mellanox(R) ConnectX(R)-6 200G MCX654106A-HCAT (4x200G)
> -* Mellanox(R) ConnectX(R)-6DX EN 100G MCX623106AN-CDAT (2*100G)
> -* Mellanox(R) ConnectX(R)-6DX EN 200G MCX623105AN-VDAT (1*200G)
> +* Mellanox(R) ConnectX(R)-6 Dx EN 100G MCX623106AN-CDAT (2*100G)
> +* Mellanox(R) ConnectX(R)-6 Dx EN 200G MCX623105AN-VDAT (1*200G)
> * Mellanox(R) BlueField SmartNIC 25G MBF1M332A-ASCAT (2*25G)
>
> Prerequisites
> diff --git a/drivers/net/mlx5/mlx5.c b/drivers/net/mlx5/mlx5.c index
> 4803760cab..94aaa60579 100644
> --- a/drivers/net/mlx5/mlx5.c
> +++ b/drivers/net/mlx5/mlx5.c
> @@ -1724,7 +1724,7 @@ mlx5_init_once(void)
> * key is specified in devargs
> * - if DevX is enabled the inline mode is queried from the
> * device (HCA attributes and NIC vport context if needed).
> - * - otherwise L2 mode (18 bytes) is assumed for ConnectX-4/4LX
> + * - otherwise L2 mode (18 bytes) is assumed for ConnectX-4/4 Lx
> * and none (0 bytes) for other NICs
> *
> * @param spawn
> diff --git a/drivers/net/mlx5/mlx5_rxtx.c b/drivers/net/mlx5/mlx5_rxtx.c index
> 2b4fc2ae06..5ac63da803 100644
> --- a/drivers/net/mlx5/mlx5_rxtx.c
> +++ b/drivers/net/mlx5/mlx5_rxtx.c
> @@ -5213,7 +5213,7 @@ MLX5_TXOFF_DECL(iv,
>
> /*
> * Generate routines with Legacy Multi-Packet Write support.
> - * This mode is supported by ConnectX-4LX only and imposes
> + * This mode is supported by ConnectX-4 Lx only and imposes
> * offload limitations, not supported:
> * - ACL/Flows (metadata are becoming meaningless)
> * - WQE Inline headers
> diff --git a/drivers/net/mlx5/mlx5_txq.c b/drivers/net/mlx5/mlx5_txq.c index
> 2ad849a229..57bc116450 100644
> --- a/drivers/net/mlx5/mlx5_txq.c
> +++ b/drivers/net/mlx5/mlx5_txq.c
> @@ -983,7 +983,7 @@ txq_set_params(struct mlx5_txq_ctrl *txq_ctrl)
> * If there is requested minimal amount of data to inline
> * we MUST enable inlining. This is a case for ConnectX-4
> * which usually requires L2 inlined for correct operating
> - * and ConnectX-4LX which requires L2-L4 inlined to
> + * and ConnectX-4 Lx which requires L2-L4 inlined to
> * support E-Switch Flows.
> */
> if (inlen_mode) {
> --
> 2.25.0
next prev parent reply other threads:[~2020-02-24 8:59 UTC|newest]
Thread overview: 5+ messages / expand[flat|nested] mbox.gz Atom feed top
2020-02-21 14:59 Thomas Monjalon
2020-02-24 8:59 ` Slava Ovsiienko [this message]
2020-02-24 19:56 ` Thomas Monjalon
2020-02-24 19:52 Thomas Monjalon
2020-02-25 14:55 ` Thomas Monjalon
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=AM4PR05MB3265C48B54BD1224149986D1D2EC0@AM4PR05MB3265.eurprd05.prod.outlook.com \
--to=viacheslavo@mellanox.com \
--cc=dev@dpdk.org \
--cc=galco@mellanox.com \
--cc=john.mcnamara@intel.com \
--cc=marko.kovacevic@intel.com \
--cc=matan@mellanox.com \
--cc=shahafs@mellanox.com \
--cc=thomas@monjalon.net \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).