From: Tonghao Zhang <xiangxia.m.yue@gmail.com>
To: Pablo Neira Ayuso <pablo@netfilter.org>
Cc: Linux Kernel Network Developers <netdev@vger.kernel.org>,
David Miller <davem@davemloft.net>,
thomas.lendacky@amd.com, f.fainelli@gmail.com,
ariel.elior@cavium.com, michael.chan@broadcom.com,
santosh@chelsio.com, madalin.bucur@nxp.com,
yisen.zhuang@huawei.com, salil.mehta@huawei.com,
jeffrey.t.kirsher@intel.com, tariqt@mellanox.com,
Saeed Mahameed <saeedm@mellanox.com>,
jiri@mellanox.com, idosch@mellanox.com,
jakub.kicinski@netronome.com, peppe.cavallaro@st.com,
grygorii.strashko@ti.com, andrew@lunn.ch,
vivien.didelot@savoirfairelinux.com, alexandre.torgue@st.com,
joabreu@synopsys.com, linux-net-drivers@solarflare.com,
ganeshgr@chelsio.com, Or Gerlitz <ogerlitz@mellanox.com>,
Manish.Chopra@cavium.com, marcelo.leitner@gmail.com,
mkubecek@suse.cz, venkatkumar.duvvuru@broadcom.com,
julia.lawall@lip6.fr, john.fastabend@gmail.com,
netfilter-devel@vger.kernel.org, cphealy@gmail.com
Subject: Re: [PATCH 02/12 net-next,v7] net/mlx5e: support for two independent packet edit actions
Date: Sat, 2 Feb 2019 23:09:47 +0800 [thread overview]
Message-ID: <CAMDZJNVv20HWiv4zT__jhYv8RWfTdUJ+bnG1QsgJZTDL6XfV4A@mail.gmail.com> (raw)
In-Reply-To: <20190202115054.4880-3-pablo@netfilter.org>
On Sat, Feb 2, 2019 at 8:11 PM Pablo Neira Ayuso <pablo@netfilter.org> wrote:
>
> This patch adds pedit_headers_action structure to store the result of
> parsing tc pedit actions. Then, it calls alloc_tc_pedit_action() to
> populate the mlx5e hardware intermediate representation once all actions
> have been parsed.
>
> This patch comes in preparation for the new flow_action infrastructure,
> where each packet mangling comes in an separated action, ie. not packed
> as in tc pedit.
>
> Signed-off-by: Pablo Neira Ayuso <pablo@netfilter.org>
> Acked-by: Jiri Pirko <jiri@mellanox.com>
> Acked-by: Saeed Mahameed <saeedm@mellanox.com>
> ---
> v7: rebase on top of net-next.
>
> drivers/net/ethernet/mellanox/mlx5/core/en_tc.c | 81 ++++++++++++++++++-------
> 1 file changed, 59 insertions(+), 22 deletions(-)
>
> diff --git a/drivers/net/ethernet/mellanox/mlx5/core/en_tc.c b/drivers/net/ethernet/mellanox/mlx5/core/en_tc.c
> index cd289ce0582d..1c8e8da166bd 100644
> --- a/drivers/net/ethernet/mellanox/mlx5/core/en_tc.c
> +++ b/drivers/net/ethernet/mellanox/mlx5/core/en_tc.c
> @@ -1804,6 +1804,12 @@ struct pedit_headers {
> struct udphdr udp;
> };
>
> +struct pedit_headers_action {
> + struct pedit_headers vals;
> + struct pedit_headers masks;
> + u32 pedits;
> +};
> +
> static int pedit_header_offsets[] = {
> [TCA_PEDIT_KEY_EX_HDR_TYPE_ETH] = offsetof(struct pedit_headers, eth),
> [TCA_PEDIT_KEY_EX_HDR_TYPE_IP4] = offsetof(struct pedit_headers, ip4),
> @@ -1815,16 +1821,15 @@ static int pedit_header_offsets[] = {
> #define pedit_header(_ph, _htype) ((void *)(_ph) + pedit_header_offsets[_htype])
>
> static int set_pedit_val(u8 hdr_type, u32 mask, u32 val, u32 offset,
> - struct pedit_headers *masks,
> - struct pedit_headers *vals)
> + struct pedit_headers_action *hdrs)
> {
> u32 *curr_pmask, *curr_pval;
>
> if (hdr_type >= __PEDIT_HDR_TYPE_MAX)
> goto out_err;
>
> - curr_pmask = (u32 *)(pedit_header(masks, hdr_type) + offset);
> - curr_pval = (u32 *)(pedit_header(vals, hdr_type) + offset);
> + curr_pmask = (u32 *)(pedit_header(&hdrs->masks, hdr_type) + offset);
> + curr_pval = (u32 *)(pedit_header(&hdrs->vals, hdr_type) + offset);
>
> if (*curr_pmask & mask) /* disallow acting twice on the same location */
> goto out_err;
> @@ -1880,8 +1885,7 @@ static struct mlx5_fields fields[] = {
> * max from the SW pedit action. On success, it says how many HW actions were
> * actually parsed.
> */
> -static int offload_pedit_fields(struct pedit_headers *masks,
> - struct pedit_headers *vals,
> +static int offload_pedit_fields(struct pedit_headers_action *hdrs,
> struct mlx5e_tc_flow_parse_attr *parse_attr,
> struct netlink_ext_ack *extack)
> {
> @@ -1896,10 +1900,10 @@ static int offload_pedit_fields(struct pedit_headers *masks,
> __be16 mask_be16;
> void *action;
>
> - set_masks = &masks[TCA_PEDIT_KEY_EX_CMD_SET];
> - add_masks = &masks[TCA_PEDIT_KEY_EX_CMD_ADD];
> - set_vals = &vals[TCA_PEDIT_KEY_EX_CMD_SET];
> - add_vals = &vals[TCA_PEDIT_KEY_EX_CMD_ADD];
> + set_masks = &hdrs[TCA_PEDIT_KEY_EX_CMD_SET].masks;
> + add_masks = &hdrs[TCA_PEDIT_KEY_EX_CMD_ADD].masks;
> + set_vals = &hdrs[TCA_PEDIT_KEY_EX_CMD_SET].vals;
> + add_vals = &hdrs[TCA_PEDIT_KEY_EX_CMD_ADD].vals;
>
> action_size = MLX5_UN_SZ_BYTES(set_action_in_add_action_in_auto);
> action = parse_attr->mod_hdr_actions;
> @@ -1995,12 +1999,14 @@ static int offload_pedit_fields(struct pedit_headers *masks,
> }
>
> static int alloc_mod_hdr_actions(struct mlx5e_priv *priv,
> - const struct tc_action *a, int namespace,
> + struct pedit_headers_action *hdrs,
> + int namespace,
> struct mlx5e_tc_flow_parse_attr *parse_attr)
> {
> int nkeys, action_size, max_actions;
>
> - nkeys = tcf_pedit_nkeys(a);
> + nkeys = hdrs[TCA_PEDIT_KEY_EX_CMD_SET].pedits +
> + hdrs[TCA_PEDIT_KEY_EX_CMD_ADD].pedits;
> action_size = MLX5_UN_SZ_BYTES(set_action_in_add_action_in_auto);
>
> if (namespace == MLX5_FLOW_NAMESPACE_FDB) /* FDB offloading */
> @@ -2024,18 +2030,15 @@ static const struct pedit_headers zero_masks = {};
> static int parse_tc_pedit_action(struct mlx5e_priv *priv,
> const struct tc_action *a, int namespace,
> struct mlx5e_tc_flow_parse_attr *parse_attr,
> + struct pedit_headers_action *hdrs,
> struct netlink_ext_ack *extack)
> {
> - struct pedit_headers masks[__PEDIT_CMD_MAX], vals[__PEDIT_CMD_MAX], *cmd_masks;
> int nkeys, i, err = -EOPNOTSUPP;
> u32 mask, val, offset;
> u8 cmd, htype;
>
> nkeys = tcf_pedit_nkeys(a);
>
> - memset(masks, 0, sizeof(struct pedit_headers) * __PEDIT_CMD_MAX);
> - memset(vals, 0, sizeof(struct pedit_headers) * __PEDIT_CMD_MAX);
> -
> for (i = 0; i < nkeys; i++) {
> htype = tcf_pedit_htype(a, i);
> cmd = tcf_pedit_cmd(a, i);
> @@ -2056,21 +2059,37 @@ static int parse_tc_pedit_action(struct mlx5e_priv *priv,
> val = tcf_pedit_val(a, i);
> offset = tcf_pedit_offset(a, i);
>
> - err = set_pedit_val(htype, ~mask, val, offset, &masks[cmd], &vals[cmd]);
> + err = set_pedit_val(htype, ~mask, val, offset, &hdrs[cmd]);
> if (err)
> goto out_err;
> +
> + hdrs[cmd].pedits++;
> }
>
> - err = alloc_mod_hdr_actions(priv, a, namespace, parse_attr);
> + return 0;
> +out_err:
> + return err;
> +}
> +
> +static int alloc_tc_pedit_action(struct mlx5e_priv *priv, int namespace,
> + struct mlx5e_tc_flow_parse_attr *parse_attr,
> + struct pedit_headers_action *hdrs,
> + struct netlink_ext_ack *extack)
> +{
> + struct pedit_headers *cmd_masks;
> + int err;
> + u8 cmd;
> +
> + err = alloc_mod_hdr_actions(priv, hdrs, namespace, parse_attr);
> if (err)
> goto out_err;
>
> - err = offload_pedit_fields(masks, vals, parse_attr, extack);
> + err = offload_pedit_fields(hdrs, parse_attr, extack);
> if (err < 0)
> goto out_dealloc_parsed_actions;
>
> for (cmd = 0; cmd < __PEDIT_CMD_MAX; cmd++) {
> - cmd_masks = &masks[cmd];
> + cmd_masks = &hdrs[cmd].masks;
> if (memcmp(cmd_masks, &zero_masks, sizeof(zero_masks))) {
> NL_SET_ERR_MSG_MOD(extack,
> "attempt to offload an unsupported field");
> @@ -2211,6 +2230,7 @@ static int parse_tc_nic_actions(struct mlx5e_priv *priv, struct tcf_exts *exts,
> struct mlx5e_tc_flow *flow,
> struct netlink_ext_ack *extack)
> {
> + struct pedit_headers_action hdrs[__PEDIT_CMD_MAX] = {};
> struct mlx5_nic_flow_attr *attr = flow->nic_attr;
> const struct tc_action *a;
> u32 action = 0;
> @@ -2232,7 +2252,7 @@ static int parse_tc_nic_actions(struct mlx5e_priv *priv, struct tcf_exts *exts,
>
> if (is_tcf_pedit(a)) {
> err = parse_tc_pedit_action(priv, a, MLX5_FLOW_NAMESPACE_KERNEL,
> - parse_attr, extack);
> + parse_attr, hdrs, extack);
> if (err)
> return err;
>
> @@ -2286,6 +2306,14 @@ static int parse_tc_nic_actions(struct mlx5e_priv *priv, struct tcf_exts *exts,
> return -EINVAL;
> }
>
> + if (hdrs[TCA_PEDIT_KEY_EX_CMD_SET].pedits ||
> + hdrs[TCA_PEDIT_KEY_EX_CMD_ADD].pedits) {
> + err = alloc_tc_pedit_action(priv, MLX5_FLOW_NAMESPACE_KERNEL,
> + parse_attr, hdrs, extack);
> + if (err)
> + return err;
> + }
> +
> attr->action = action;
> if (!actions_match_supported(priv, exts, parse_attr, flow, extack))
> return -EOPNOTSUPP;
> @@ -2446,6 +2474,7 @@ static int parse_tc_fdb_actions(struct mlx5e_priv *priv, struct tcf_exts *exts,
> struct mlx5e_tc_flow *flow,
> struct netlink_ext_ack *extack)
> {
> + struct pedit_headers_action hdrs[__PEDIT_CMD_MAX] = {};
> struct mlx5_eswitch *esw = priv->mdev->priv.eswitch;
> struct mlx5_esw_flow_attr *attr = flow->esw_attr;
> struct mlx5e_rep_priv *rpriv = priv->ppriv;
> @@ -2470,7 +2499,7 @@ static int parse_tc_fdb_actions(struct mlx5e_priv *priv, struct tcf_exts *exts,
>
> if (is_tcf_pedit(a)) {
> err = parse_tc_pedit_action(priv, a, MLX5_FLOW_NAMESPACE_FDB,
> - parse_attr, extack);
> + parse_attr, hdrs, extack);
> if (err)
> return err;
>
> @@ -2605,6 +2634,14 @@ static int parse_tc_fdb_actions(struct mlx5e_priv *priv, struct tcf_exts *exts,
> return -EINVAL;
> }
>
> + if (hdrs[TCA_PEDIT_KEY_EX_CMD_SET].pedits ||
> + hdrs[TCA_PEDIT_KEY_EX_CMD_ADD].pedits) {
> + err = alloc_tc_pedit_action(priv, MLX5_FLOW_NAMESPACE_KERNEL,
> + parse_attr, hdrs, extack);
> + if (err)
> + return err;
> + }
> +
> attr->action = action;
> if (!actions_match_supported(priv, exts, parse_attr, flow, extack))
> return -EOPNOTSUPP;
> --
> 2.11.0
>
The patch [1] is ready too. David may decide which one will be applied
firstly. and other is rebased ?.
[1] http://patchwork.ozlabs.org/patch/1032952/
next prev parent reply other threads:[~2019-02-02 15:10 UTC|newest]
Thread overview: 20+ messages / expand[flat|nested] mbox.gz Atom feed top
2019-02-02 11:50 [PATCH 00/12 net-next,v7] add flow_rule infrastructure Pablo Neira Ayuso
2019-02-02 11:50 ` [PATCH 01/12 net-next,v7] flow_offload: add flow_rule and flow_match structures and use them Pablo Neira Ayuso
2019-02-02 15:19 ` Jiri Pirko
2019-02-02 11:50 ` [PATCH 02/12 net-next,v7] net/mlx5e: support for two independent packet edit actions Pablo Neira Ayuso
2019-02-02 15:09 ` Tonghao Zhang [this message]
2019-02-02 21:40 ` Or Gerlitz
2019-02-02 21:43 ` Or Gerlitz
2019-02-02 11:50 ` [PATCH 03/12 net-next,v7] flow_offload: add flow action infrastructure Pablo Neira Ayuso
2019-02-02 11:50 ` [PATCH 04/12 net-next,v7] cls_api: add translator to flow_action representation Pablo Neira Ayuso
2019-02-02 11:50 ` [PATCH 05/12 net-next,v7] flow_offload: add statistics retrieval infrastructure and use it Pablo Neira Ayuso
2019-02-02 11:50 ` [PATCH 06/12 net-next,v7] drivers: net: use flow action infrastructure Pablo Neira Ayuso
2019-02-02 15:44 ` Jiri Pirko
2019-02-02 11:50 ` [PATCH 07/12 net-next,v7] cls_flower: don't expose TC actions to drivers anymore Pablo Neira Ayuso
2019-02-02 11:50 ` [PATCH 08/12 net-next,v7] flow_offload: add wake-up-on-lan and queue to flow_action Pablo Neira Ayuso
2019-02-02 11:50 ` [PATCH 09/12 net-next,v7] ethtool: add ethtool_rx_flow_spec to flow_rule structure translator Pablo Neira Ayuso
2019-02-02 11:50 ` [PATCH 10/12 net-next,v7] dsa: bcm_sf2: use flow_rule infrastructure Pablo Neira Ayuso
2019-02-02 11:50 ` [PATCH 11/12 net-next,v7] qede: place ethtool_rx_flow_spec after code after TC flower codebase Pablo Neira Ayuso
2019-02-02 11:50 ` [PATCH 12/12 net-next,v7] qede: use ethtool_rx_flow_rule() to remove duplicated parser code Pablo Neira Ayuso
2019-02-06 18:39 ` [PATCH 00/12 net-next,v7] add flow_rule infrastructure David Miller
2019-02-06 20:44 ` Florian Fainelli
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=CAMDZJNVv20HWiv4zT__jhYv8RWfTdUJ+bnG1QsgJZTDL6XfV4A@mail.gmail.com \
--to=xiangxia.m.yue@gmail.com \
--cc=Manish.Chopra@cavium.com \
--cc=alexandre.torgue@st.com \
--cc=andrew@lunn.ch \
--cc=ariel.elior@cavium.com \
--cc=cphealy@gmail.com \
--cc=davem@davemloft.net \
--cc=f.fainelli@gmail.com \
--cc=ganeshgr@chelsio.com \
--cc=grygorii.strashko@ti.com \
--cc=idosch@mellanox.com \
--cc=jakub.kicinski@netronome.com \
--cc=jeffrey.t.kirsher@intel.com \
--cc=jiri@mellanox.com \
--cc=joabreu@synopsys.com \
--cc=john.fastabend@gmail.com \
--cc=julia.lawall@lip6.fr \
--cc=linux-net-drivers@solarflare.com \
--cc=madalin.bucur@nxp.com \
--cc=marcelo.leitner@gmail.com \
--cc=michael.chan@broadcom.com \
--cc=mkubecek@suse.cz \
--cc=netdev@vger.kernel.org \
--cc=netfilter-devel@vger.kernel.org \
--cc=ogerlitz@mellanox.com \
--cc=pablo@netfilter.org \
--cc=peppe.cavallaro@st.com \
--cc=saeedm@mellanox.com \
--cc=salil.mehta@huawei.com \
--cc=santosh@chelsio.com \
--cc=tariqt@mellanox.com \
--cc=thomas.lendacky@amd.com \
--cc=venkatkumar.duvvuru@broadcom.com \
--cc=vivien.didelot@savoirfairelinux.com \
--cc=yisen.zhuang@huawei.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).