From: Ferruh Yigit <ferruh.yigit@amd.com>
To: Chaoyong He <chaoyong.he@corigine.com>,
Andrew Rybchenko <andrew.rybchenko@oktetlabs.ru>,
Ori Kam <orika@nvidia.com>
Cc: oss-drivers@corigine.com, niklas.soderlund@corigine.com, dev@dpdk.org
Subject: Re: [PATCH v5 13/25] net/nfp: support SRC MAC flow action
Date: Thu, 20 Oct 2022 12:12:32 +0100 [thread overview]
Message-ID: <19e889dc-701b-c3fc-3194-a792eccd4f7c@amd.com> (raw)
In-Reply-To: <1666232391-29152-14-git-send-email-chaoyong.he@corigine.com>
On 10/20/2022 3:19 AM, Chaoyong He wrote:
> Add the corresponding data structure and logics, to support
> the offload of set source MAC action.
>
> Signed-off-by: Chaoyong He <chaoyong.he@corigine.com>
> Reviewed-by: Niklas Söderlund <niklas.soderlund@corigine.com>
<...>
> + case RTE_FLOW_ACTION_TYPE_SET_MAC_SRC:
> + PMD_DRV_LOG(DEBUG, "Process RTE_FLOW_ACTION_TYPE_SET_MAC_SRC");
> + nfp_flow_action_set_mac(position, action, true, mac_set_flag);
> + if (!mac_set_flag) {
> + position += sizeof(struct nfp_fl_act_set_eth);
> + mac_set_flag = true;
> + }
Hi Andrew, Ori,
I can see 'RTE_FLOW_ACTION_TYPE_SET_MAC_SRC' and many other marked as
legacy and reference to 'RTE_FLOW_ACTION_TYPE_MODIFY_FIELD'.
What is the expectation from PMD developers for this?
User still can provide these legacy actions, right? So should PMD
implement both legacy and new ones?
next prev parent reply other threads:[~2022-10-20 11:12 UTC|newest]
Thread overview: 96+ messages / expand[flat|nested] mbox.gz Atom feed top
2022-10-18 11:26 [PATCH v4 00/25] add the basic rte_flow offload support of nfp PMD Chaoyong He
2022-10-18 11:26 ` [PATCH v4 01/25] net/nfp: fix the requirement of cpp bridge service Chaoyong He
2022-10-18 12:39 ` Ferruh Yigit
2022-10-18 11:26 ` [PATCH v4 02/25] net/nfp: add the structures and functions for flow offload Chaoyong He
2022-10-18 11:26 ` [PATCH v4 03/25] net/nfp: add the stats process logic in ctrl VNIC service Chaoyong He
2022-10-18 11:26 ` [PATCH v4 04/25] net/nfp: add the flow APIs of nfp PMD Chaoyong He
2022-10-18 11:26 ` [PATCH v4 05/25] net/nfp: support basic flow items Chaoyong He
2022-10-18 11:26 ` [PATCH v4 06/25] net/nfp: support basic flow actions Chaoyong He
2022-10-18 11:26 ` [PATCH v4 07/25] net/nfp: support VLAN flow item Chaoyong He
2022-10-18 11:26 ` [PATCH v4 08/25] net/nfp: support IPv4 " Chaoyong He
2022-10-18 11:26 ` [PATCH v4 09/25] net/nfp: support IPv6 " Chaoyong He
2022-10-18 11:26 ` [PATCH v4 10/25] net/nfp: support TCP " Chaoyong He
2022-10-18 11:27 ` [PATCH v4 11/25] net/nfp: support UDP " Chaoyong He
2022-10-18 11:27 ` [PATCH v4 12/25] net/nfp: support SCTP " Chaoyong He
2022-10-18 11:27 ` [PATCH v4 13/25] net/nfp: support SRC MAC flow action Chaoyong He
2022-10-18 11:27 ` [PATCH v4 14/25] net/nfp: support DST " Chaoyong He
2022-10-18 11:27 ` [PATCH v4 15/25] net/nfp: support pop VLAN " Chaoyong He
2022-10-18 11:27 ` [PATCH v4 16/25] net/nfp: support push " Chaoyong He
2022-10-18 11:27 ` [PATCH v4 17/25] net/nfp: support SRC IPv4 " Chaoyong He
2022-10-18 11:27 ` [PATCH v4 18/25] net/nfp: support DST " Chaoyong He
2022-10-18 11:27 ` [PATCH v4 19/25] net/nfp: support SRC IPv6 " Chaoyong He
2022-10-18 11:27 ` [PATCH v4 20/25] net/nfp: support DST " Chaoyong He
2022-10-18 11:27 ` [PATCH v4 21/25] net/nfp: support TP SRC " Chaoyong He
2022-10-18 11:27 ` [PATCH v4 22/25] net/nfp: support TP DST " Chaoyong He
2022-10-18 11:27 ` [PATCH v4 23/25] net/nfp: support TTL " Chaoyong He
2022-10-18 11:27 ` [PATCH v4 24/25] net/nfp: support IPv4 DSCP " Chaoyong He
2022-10-18 11:27 ` [PATCH v4 25/25] net/nfp: support IPv6 " Chaoyong He
2022-10-18 12:27 ` [PATCH v4 00/25] add the basic rte_flow offload support of nfp PMD Ferruh Yigit
2022-10-20 2:19 ` [PATCH v5 " Chaoyong He
2022-10-20 2:19 ` [PATCH v5 01/25] net/nfp: fix CPP bridge service requirement Chaoyong He
2022-10-20 2:19 ` [PATCH v5 02/25] net/nfp: add the structures and functions for flow offload Chaoyong He
2022-10-20 2:19 ` [PATCH v5 03/25] net/nfp: add the stats process logic in ctrl VNIC service Chaoyong He
2022-10-20 2:19 ` [PATCH v5 04/25] net/nfp: add the flow APIs of nfp PMD Chaoyong He
2022-10-20 11:09 ` Ferruh Yigit
2022-10-20 11:42 ` Chaoyong He
2022-10-20 2:19 ` [PATCH v5 05/25] net/nfp: support basic flow items Chaoyong He
2022-10-20 11:11 ` Ferruh Yigit
2022-10-20 11:43 ` Chaoyong He
2022-10-20 2:19 ` [PATCH v5 06/25] net/nfp: support basic flow actions Chaoyong He
2022-10-20 11:12 ` Ferruh Yigit
2022-10-20 11:41 ` Chaoyong He
2022-10-20 2:19 ` [PATCH v5 07/25] net/nfp: support VLAN flow item Chaoyong He
2022-10-20 2:19 ` [PATCH v5 08/25] net/nfp: support IPv4 " Chaoyong He
2022-10-20 2:19 ` [PATCH v5 09/25] net/nfp: support IPv6 " Chaoyong He
2022-10-20 2:19 ` [PATCH v5 10/25] net/nfp: support TCP " Chaoyong He
2022-10-20 2:19 ` [PATCH v5 11/25] net/nfp: support UDP " Chaoyong He
2022-10-20 2:19 ` [PATCH v5 12/25] net/nfp: support SCTP " Chaoyong He
2022-10-20 2:19 ` [PATCH v5 13/25] net/nfp: support SRC MAC flow action Chaoyong He
2022-10-20 11:12 ` Ferruh Yigit [this message]
2022-10-20 11:48 ` Chaoyong He
2022-10-20 11:55 ` Ferruh Yigit
2022-10-20 2:19 ` [PATCH v5 14/25] net/nfp: support DST " Chaoyong He
2022-10-20 2:19 ` [PATCH v5 15/25] net/nfp: support pop VLAN " Chaoyong He
2022-10-20 2:19 ` [PATCH v5 16/25] net/nfp: support push " Chaoyong He
2022-10-20 2:19 ` [PATCH v5 17/25] net/nfp: support SRC IPv4 " Chaoyong He
2022-10-20 2:19 ` [PATCH v5 18/25] net/nfp: support DST " Chaoyong He
2022-10-20 2:19 ` [PATCH v5 19/25] net/nfp: support SRC IPv6 " Chaoyong He
2022-10-20 2:19 ` [PATCH v5 20/25] net/nfp: support DST " Chaoyong He
2022-10-20 2:19 ` [PATCH v5 21/25] net/nfp: support TP SRC " Chaoyong He
2022-10-20 2:19 ` [PATCH v5 22/25] net/nfp: support TP DST " Chaoyong He
2022-10-20 2:19 ` [PATCH v5 23/25] net/nfp: support TTL " Chaoyong He
2022-10-20 2:19 ` [PATCH v5 24/25] net/nfp: support IPv4 DSCP " Chaoyong He
2022-10-20 2:19 ` [PATCH v5 25/25] net/nfp: support IPv6 " Chaoyong He
2022-10-21 8:01 ` [PATCH v6 00/27] add the basic rte_flow offload support of nfp PMD Chaoyong He
2022-10-21 8:01 ` [PATCH v6 01/27] net/nfp: fix CPP bridge service requirement Chaoyong He
2022-10-21 8:01 ` [PATCH v6 02/27] net/nfp: fix the promiscuous mode control functions Chaoyong He
2022-10-21 13:03 ` Ferruh Yigit
2022-10-22 6:49 ` Chaoyong He
2022-10-22 8:11 ` [PATCH v7] net/nfp: fix the promiscuous mode functions Chaoyong He
2022-10-24 12:31 ` Ferruh Yigit
2022-10-21 8:01 ` [PATCH v6 03/27] net/nfp: fix the service stuck the app end Chaoyong He
2022-10-21 8:01 ` [PATCH v6 04/27] net/nfp: add the structures and functions for flow offload Chaoyong He
2022-10-21 8:01 ` [PATCH v6 05/27] net/nfp: add the stats process logic in ctrl VNIC service Chaoyong He
2022-10-21 8:01 ` [PATCH v6 06/27] net/nfp: add the flow APIs of nfp PMD Chaoyong He
2022-10-21 8:01 ` [PATCH v6 07/27] net/nfp: support basic flow items Chaoyong He
2022-10-21 8:01 ` [PATCH v6 08/27] net/nfp: support basic flow actions Chaoyong He
2022-10-21 8:01 ` [PATCH v6 09/27] net/nfp: support VLAN flow item Chaoyong He
2022-10-21 8:01 ` [PATCH v6 10/27] net/nfp: support IPv4 " Chaoyong He
2022-10-21 8:01 ` [PATCH v6 11/27] net/nfp: support IPv6 " Chaoyong He
2022-10-21 8:01 ` [PATCH v6 12/27] net/nfp: support TCP " Chaoyong He
2022-10-21 8:01 ` [PATCH v6 13/27] net/nfp: support UDP " Chaoyong He
2022-10-21 8:01 ` [PATCH v6 14/27] net/nfp: support SCTP " Chaoyong He
2022-10-21 8:01 ` [PATCH v6 15/27] net/nfp: support SRC MAC flow action Chaoyong He
2022-10-21 8:01 ` [PATCH v6 16/27] net/nfp: support DST " Chaoyong He
2022-10-21 8:01 ` [PATCH v6 17/27] net/nfp: support pop VLAN " Chaoyong He
2022-10-21 8:01 ` [PATCH v6 18/27] net/nfp: support push " Chaoyong He
2022-10-21 8:01 ` [PATCH v6 19/27] net/nfp: support SRC IPv4 " Chaoyong He
2022-10-21 8:01 ` [PATCH v6 20/27] net/nfp: support DST " Chaoyong He
2022-10-21 8:01 ` [PATCH v6 21/27] net/nfp: support SRC IPv6 " Chaoyong He
2022-10-21 8:01 ` [PATCH v6 22/27] net/nfp: support DST " Chaoyong He
2022-10-21 8:01 ` [PATCH v6 23/27] net/nfp: support TP SRC " Chaoyong He
2022-10-21 8:02 ` [PATCH v6 24/27] net/nfp: support TP DST " Chaoyong He
2022-10-21 8:02 ` [PATCH v6 25/27] net/nfp: support TTL " Chaoyong He
2022-10-21 8:02 ` [PATCH v6 26/27] net/nfp: support IPv4 DSCP " Chaoyong He
2022-10-21 8:02 ` [PATCH v6 27/27] net/nfp: support IPv6 " Chaoyong He
2022-10-21 13:04 ` [PATCH v6 00/27] add the basic rte_flow offload support of nfp PMD Ferruh Yigit
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=19e889dc-701b-c3fc-3194-a792eccd4f7c@amd.com \
--to=ferruh.yigit@amd.com \
--cc=andrew.rybchenko@oktetlabs.ru \
--cc=chaoyong.he@corigine.com \
--cc=dev@dpdk.org \
--cc=niklas.soderlund@corigine.com \
--cc=orika@nvidia.com \
--cc=oss-drivers@corigine.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).