From: Raslan Darawsheh <rasland@nvidia.com>
To: Michael Baum <michaelba@nvidia.com>, "dev@dpdk.org" <dev@dpdk.org>
Cc: Matan Azrad <matan@nvidia.com>, Slava Ovsiienko <viacheslavo@nvidia.com>
Subject: RE: [PATCH v2 00/20] mlx5: refactor devargs management
Date: Mon, 21 Feb 2022 08:54:30 +0000 [thread overview]
Message-ID: <BYAPR12MB30787A98B8BAA3908D86B5C3CF3A9@BYAPR12MB3078.namprd12.prod.outlook.com> (raw)
In-Reply-To: <20220214093511.1592698-1-michaelba@nvidia.com>
Hi,
> -----Original Message-----
> From: Michael Baum <michaelba@nvidia.com>
> Sent: Monday, February 14, 2022 11:35 AM
> To: dev@dpdk.org
> Cc: Matan Azrad <matan@nvidia.com>; Raslan Darawsheh
> <rasland@nvidia.com>; Slava Ovsiienko <viacheslavo@nvidia.com>
> Subject: [PATCH v2 00/20] mlx5: refactor devargs management
>
> v2: rebase + fix coverity issue.
>
> These patches rearrange the management of the devargs on two different
> levels.
>
> The first splits the net driver's devargs into two categories,
> device-dependent devargs and port-dependent devargs.
> Arguments that depend on the device are updated once in the creation of
> the shared device context structure, and do not change even if the user
> has sent new devargs in the probe again. In contrast, the arguments that
> depend on the port are updated separately for each port.
>
> The second layer deals with the parsing of devargs in the common driver.
> The common driver once parses the devargs into a dictionary, then sends
> it to all the drivers that will use it during the their probing. Each
> driver updates within dictionary which keys it has used, then the common
> driver checks the updated dictionary and reports about unknown devargs.
>
> Michael Baum (20):
> net/mlx5: fix wrong check sibling device config mismatch
> net/mlx5: fix ineffective metadata argument adjustment
> net/mlx5: fix wrong place of ASO CT object release
> net/mlx5: fix inconsistency errno update in SH creation
> net/mlx5: remove declaration duplications
> net/mlx5: remove checking devargs duplication
> net/mlx5: remove HCA attr structure duplication
> net/mlx5: remove DevX flag duplication
> net/mlx5: remove Verbs query device duplication
> common/mlx5: share VF checking function
> net/mlx5: share realtime timestamp configure
> net/mlx5: share counter config function
> net/mlx5: add E-switch mode flag
> net/mlx5: rearrange device attribute structure
> net/mlx5: concentrate all device configurations
> net/mlx5: add share device context config structure
> net/mlx5: using function to detect operation by DevX
> net/mlx5: separate per port configuration
> common/mlx5: add check for common devargs in probing again
> common/mlx5: refactor devargs management
>
> drivers/common/mlx5/mlx5_common.c | 345 +++++++--
> drivers/common/mlx5/mlx5_common.h | 51 +-
> drivers/common/mlx5/mlx5_common_pci.c | 18 +
> drivers/common/mlx5/version.map | 3 +
> drivers/compress/mlx5/mlx5_compress.c | 38 +-
> drivers/crypto/mlx5/mlx5_crypto.c | 39 +-
> drivers/net/mlx5/linux/mlx5_flow_os.c | 3 +-
> drivers/net/mlx5/linux/mlx5_os.c | 887 +++++++++---------------
> drivers/net/mlx5/linux/mlx5_verbs.c | 9 +-
> drivers/net/mlx5/linux/mlx5_vlan_os.c | 3 +-
> drivers/net/mlx5/mlx5.c | 872 +++++++++++++++++------
> drivers/net/mlx5/mlx5.h | 216 +++---
> drivers/net/mlx5/mlx5_devx.c | 19 +-
> drivers/net/mlx5/mlx5_ethdev.c | 31 +-
> drivers/net/mlx5/mlx5_flow.c | 50 +-
> drivers/net/mlx5/mlx5_flow.h | 2 +-
> drivers/net/mlx5/mlx5_flow_dv.c | 93 ++-
> drivers/net/mlx5/mlx5_flow_flex.c | 4 +-
> drivers/net/mlx5/mlx5_flow_meter.c | 14 +-
> drivers/net/mlx5/mlx5_rxmode.c | 8 +-
> drivers/net/mlx5/mlx5_rxq.c | 49 +-
> drivers/net/mlx5/mlx5_trigger.c | 35 +-
> drivers/net/mlx5/mlx5_tx.c | 2 +-
> drivers/net/mlx5/mlx5_txpp.c | 14 +-
> drivers/net/mlx5/mlx5_txq.c | 62 +-
> drivers/net/mlx5/mlx5_vlan.c | 4 +-
> drivers/net/mlx5/windows/mlx5_flow_os.c | 2 +-
> drivers/net/mlx5/windows/mlx5_os.c | 342 +++------
> drivers/regex/mlx5/mlx5_regex.c | 3 +-
> drivers/vdpa/mlx5/mlx5_vdpa.c | 32 +-
> 30 files changed, 1842 insertions(+), 1408 deletions(-)
>
> --
> 2.25.1
Series applied to next-net-mlx,
Kindest regards,
Raslan Darawsheh
prev parent reply other threads:[~2022-02-21 8:54 UTC|newest]
Thread overview: 43+ messages / expand[flat|nested] mbox.gz Atom feed top
2022-01-27 15:39 [PATCH " Michael Baum
2022-01-27 15:39 ` [PATCH 01/20] net/mlx5: fix wrong check sibling device config mismatch Michael Baum
2022-01-27 15:39 ` [PATCH 02/20] net/mlx5: fix ineffective metadata argument adjustment Michael Baum
2022-01-27 15:39 ` [PATCH 03/20] net/mlx5: fix wrong place of ASO CT object release Michael Baum
2022-01-27 15:39 ` [PATCH 04/20] net/mlx5: fix inconsistency errno update in SH creation Michael Baum
2022-01-27 15:39 ` [PATCH 05/20] net/mlx5: remove declaration duplications Michael Baum
2022-01-27 15:39 ` [PATCH 06/20] net/mlx5: remove checking devargs duplication Michael Baum
2022-01-27 15:39 ` [PATCH 07/20] net/mlx5: remove HCA attr structure duplication Michael Baum
2022-01-27 15:39 ` [PATCH 08/20] net/mlx5: remove DevX flag duplication Michael Baum
2022-01-27 15:39 ` [PATCH 09/20] net/mlx5: remove Verbs query device duplication Michael Baum
2022-01-27 15:39 ` [PATCH 10/20] common/mlx5: share VF checking function Michael Baum
2022-01-27 15:39 ` [PATCH 11/20] net/mlx5: share realtime timestamp configure Michael Baum
2022-01-27 15:39 ` [PATCH 12/20] net/mlx5: share counter config function Michael Baum
2022-01-27 15:39 ` [PATCH 13/20] net/mlx5: add E-switch mode flag Michael Baum
2022-01-27 15:39 ` [PATCH 14/20] net/mlx5: rearrange device attribute structure Michael Baum
2022-01-27 15:39 ` [PATCH 15/20] net/mlx5: concentrate all device configurations Michael Baum
2022-01-27 15:39 ` [PATCH 16/20] net/mlx5: add share device context config structure Michael Baum
2022-01-27 15:39 ` [PATCH 17/20] net/mlx5: using function to detect operation by DevX Michael Baum
2022-01-27 15:39 ` [PATCH 18/20] net/mlx5: separate per port configuration Michael Baum
2022-01-27 15:39 ` [PATCH 19/20] common/mlx5: add check for common devargs in probing again Michael Baum
2022-01-27 15:39 ` [PATCH 20/20] common/mlx5: refactor devargs management Michael Baum
2022-02-14 9:34 ` [PATCH v2 00/20] mlx5: " Michael Baum
2022-02-14 9:34 ` [PATCH v2 01/20] net/mlx5: fix wrong check sibling device config mismatch Michael Baum
2022-02-14 9:34 ` [PATCH v2 02/20] net/mlx5: fix ineffective metadata argument adjustment Michael Baum
2022-02-14 9:34 ` [PATCH v2 03/20] net/mlx5: fix wrong place of ASO CT object release Michael Baum
2022-02-14 9:34 ` [PATCH v2 04/20] net/mlx5: fix inconsistency errno update in SH creation Michael Baum
2022-02-14 9:34 ` [PATCH v2 05/20] net/mlx5: remove declaration duplications Michael Baum
2022-02-14 9:34 ` [PATCH v2 06/20] net/mlx5: remove checking devargs duplication Michael Baum
2022-02-14 9:34 ` [PATCH v2 07/20] net/mlx5: remove HCA attr structure duplication Michael Baum
2022-02-14 9:34 ` [PATCH v2 08/20] net/mlx5: remove DevX flag duplication Michael Baum
2022-02-14 9:35 ` [PATCH v2 09/20] net/mlx5: remove Verbs query device duplication Michael Baum
2022-02-14 9:35 ` [PATCH v2 10/20] common/mlx5: share VF checking function Michael Baum
2022-02-14 9:35 ` [PATCH v2 11/20] net/mlx5: share realtime timestamp configure Michael Baum
2022-02-14 9:35 ` [PATCH v2 12/20] net/mlx5: share counter config function Michael Baum
2022-02-14 9:35 ` [PATCH v2 13/20] net/mlx5: add E-switch mode flag Michael Baum
2022-02-14 9:35 ` [PATCH v2 14/20] net/mlx5: rearrange device attribute structure Michael Baum
2022-02-14 9:35 ` [PATCH v2 15/20] net/mlx5: concentrate all device configurations Michael Baum
2022-02-14 9:35 ` [PATCH v2 16/20] net/mlx5: add share device context config structure Michael Baum
2022-02-14 9:35 ` [PATCH v2 17/20] net/mlx5: using function to detect operation by DevX Michael Baum
2022-02-14 9:35 ` [PATCH v2 18/20] net/mlx5: separate per port configuration Michael Baum
2022-02-14 9:35 ` [PATCH v2 19/20] common/mlx5: add check for common devargs in probing again Michael Baum
2022-02-14 9:35 ` [PATCH v2 20/20] common/mlx5: refactor devargs management Michael Baum
2022-02-21 8:54 ` Raslan Darawsheh [this message]
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=BYAPR12MB30787A98B8BAA3908D86B5C3CF3A9@BYAPR12MB3078.namprd12.prod.outlook.com \
--to=rasland@nvidia.com \
--cc=dev@dpdk.org \
--cc=matan@nvidia.com \
--cc=michaelba@nvidia.com \
--cc=viacheslavo@nvidia.com \
/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).