From: Jerin Jacob <jerinjacobk@gmail.com>
To: Satheesh Paul <psatheesh@marvell.com>,
ferruh.yigit@xilinx.com,
Andrew Rybchenko <andrew.rybchenko@oktetlabs.ru>
Cc: Nithin Dabilpuram <ndabilpuram@marvell.com>,
Kiran Kumar K <kirankumark@marvell.com>,
Sunil Kumar Kori <skori@marvell.com>,
Satha Rao <skoteshwar@marvell.com>, dpdk-dev <dev@dpdk.org>
Subject: Re: [dpdk-dev] [PATCH 1/2] common/cnxk: support for CPT second pass flow rules
Date: Sun, 24 Apr 2022 09:24:25 +0530 [thread overview]
Message-ID: <CALBAE1ObJ_oDiZ3+iv+06KfjpZfUcM=c_213-kUu8qce-uFX-Q@mail.gmail.com> (raw)
In-Reply-To: <20220228045322.1841812-1-psatheesh@marvell.com>
On Mon, Feb 28, 2022 at 10:23 AM <psatheesh@marvell.com> wrote:
>
> From: Satheesh Paul <psatheesh@marvell.com>
>
> Added support to create flow rules to match packets
> from CPT(second pass packets). With this change, ingress
> rules will be created with bits 10 and 11 of channel field
> in the MCAM ignored by default. For rules specific to
> second pass packets, the CPT channel bits will be set
> in the MCAM.
>
> Signed-off-by: Satheesh Paul <psatheesh@marvell.com>
Series Acked-by: Jerin Jacob <jerinj@marvell.com>
Series applied to dpdk-next-net-mrvl/for-next-net. Thanks.
> ---
> drivers/common/cnxk/hw/nix.h | 7 +++-
> drivers/common/cnxk/roc_npc.c | 9 +++--
> drivers/common/cnxk/roc_npc.h | 1 +
> drivers/common/cnxk/roc_npc_mcam.c | 60 ++++++++++++++++++++---------
> drivers/common/cnxk/roc_npc_parse.c | 14 +++++++
> drivers/common/cnxk/roc_npc_priv.h | 2 +
> 6 files changed, 69 insertions(+), 24 deletions(-)
>
> diff --git a/drivers/common/cnxk/hw/nix.h b/drivers/common/cnxk/hw/nix.h
> index 1cc0c8dfb8..5863e358e0 100644
> --- a/drivers/common/cnxk/hw/nix.h
> +++ b/drivers/common/cnxk/hw/nix.h
> @@ -830,7 +830,7 @@
> #define NIX_CHAN_LBKX_CHX(a, b) \
> (0x000ull | ((uint64_t)(a) << 8) | (uint64_t)(b))
> #define NIX_CHAN_CPT_CH_END (0x4ffull) /* [CN10K, .) */
> -#define NIX_CHAN_CPT_CH_START (0x400ull) /* [CN10K, .) */
> +#define NIX_CHAN_CPT_CH_START (0x800ull) /* [CN10K, .) */
> #define NIX_CHAN_R4 (0x400ull) /* [CN9K, CN10K) */
> #define NIX_CHAN_R5 (0x500ull)
> #define NIX_CHAN_R6 (0x600ull)
> @@ -843,6 +843,11 @@
> #define NIX_CHAN_RPMX_LMACX_CHX(a, b, c) \
> (0x800ull | ((uint64_t)(a) << 8) | ((uint64_t)(b) << 4) | (uint64_t)(c))
>
> +/* The mask is to extract lower 10-bits of channel number
> + * which CPT will pass to X2P.
> + */
> +#define NIX_CHAN_CPT_X2P_MASK (0x3ffull)
> +
> #define NIX_INTF_SDP (0x4ull)
> #define NIX_INTF_CGX0 (0x0ull) /* [CN9K, CN10K) */
> #define NIX_INTF_CGX1 (0x1ull) /* [CN9K, CN10K) */
> diff --git a/drivers/common/cnxk/roc_npc.c b/drivers/common/cnxk/roc_npc.c
> index fc88fd58bc..51e36f141f 100644
> --- a/drivers/common/cnxk/roc_npc.c
> +++ b/drivers/common/cnxk/roc_npc.c
> @@ -570,10 +570,11 @@ npc_parse_pattern(struct npc *npc, const struct roc_npc_item_info pattern[],
> struct roc_npc_flow *flow, struct npc_parse_state *pst)
> {
> npc_parse_stage_func_t parse_stage_funcs[] = {
> - npc_parse_meta_items, npc_parse_pre_l2, npc_parse_cpt_hdr,
> - npc_parse_higig2_hdr, npc_parse_la, npc_parse_lb,
> - npc_parse_lc, npc_parse_ld, npc_parse_le,
> - npc_parse_lf, npc_parse_lg, npc_parse_lh,
> + npc_parse_meta_items, npc_parse_mark_item, npc_parse_pre_l2,
> + npc_parse_cpt_hdr, npc_parse_higig2_hdr, npc_parse_la,
> + npc_parse_lb, npc_parse_lc, npc_parse_ld,
> + npc_parse_le, npc_parse_lf, npc_parse_lg,
> + npc_parse_lh,
> };
> uint8_t layer = 0;
> int key_offset;
> diff --git a/drivers/common/cnxk/roc_npc.h b/drivers/common/cnxk/roc_npc.h
> index 6204139396..aecea37b3d 100644
> --- a/drivers/common/cnxk/roc_npc.h
> +++ b/drivers/common/cnxk/roc_npc.h
> @@ -37,6 +37,7 @@ enum roc_npc_item_type {
> ROC_NPC_ITEM_TYPE_L3_CUSTOM,
> ROC_NPC_ITEM_TYPE_QINQ,
> ROC_NPC_ITEM_TYPE_RAW,
> + ROC_NPC_ITEM_TYPE_MARK,
> ROC_NPC_ITEM_TYPE_END,
> };
>
> diff --git a/drivers/common/cnxk/roc_npc_mcam.c b/drivers/common/cnxk/roc_npc_mcam.c
> index 9c5ff5e60a..3447b59344 100644
> --- a/drivers/common/cnxk/roc_npc_mcam.c
> +++ b/drivers/common/cnxk/roc_npc_mcam.c
> @@ -497,6 +497,38 @@ npc_mcam_fetch_kex_cfg(struct npc *npc)
> return rc;
> }
>
> +static void
> +npc_mcam_set_channel(struct roc_npc_flow *flow,
> + struct npc_mcam_write_entry_req *req, uint16_t channel,
> + uint16_t chan_mask, bool is_second_pass)
> +{
> + uint16_t chan = 0, mask = 0;
> +
> + req->entry_data.kw[0] &= ~(GENMASK(11, 0));
> + req->entry_data.kw_mask[0] &= ~(GENMASK(11, 0));
> + flow->mcam_data[0] &= ~(GENMASK(11, 0));
> + flow->mcam_mask[0] &= ~(GENMASK(11, 0));
> +
> + if (is_second_pass) {
> + chan = (channel | NIX_CHAN_CPT_CH_START);
> + mask = (chan_mask | NIX_CHAN_CPT_CH_START);
> + } else {
> + /*
> + * Clear bits 10 & 11 corresponding to CPT
> + * channel. By default, rules should match
> + * both first pass packets and second pass
> + * packets from CPT.
> + */
> + chan = (channel & NIX_CHAN_CPT_X2P_MASK);
> + mask = (chan_mask & NIX_CHAN_CPT_X2P_MASK);
> + }
> +
> + req->entry_data.kw[0] |= (uint64_t)chan;
> + req->entry_data.kw_mask[0] |= (uint64_t)mask;
> + flow->mcam_data[0] |= (uint64_t)chan;
> + flow->mcam_mask[0] |= (uint64_t)mask;
> +}
> +
> int
> npc_mcam_alloc_and_write(struct npc *npc, struct roc_npc_flow *flow,
> struct npc_parse_state *pst)
> @@ -564,32 +596,22 @@ npc_mcam_alloc_and_write(struct npc *npc, struct roc_npc_flow *flow,
> if (flow->nix_intf == NIX_INTF_RX) {
> if (inl_dev && inl_dev->is_multi_channel &&
> (flow->npc_action & NIX_RX_ACTIONOP_UCAST_IPSEC)) {
> - req->entry_data.kw[0] |= (uint64_t)inl_dev->channel;
> - req->entry_data.kw_mask[0] |=
> - (uint64_t)inl_dev->chan_mask;
> pf_func = nix_inl_dev_pffunc_get();
> req->entry_data.action &= ~(GENMASK(19, 4));
> req->entry_data.action |= (uint64_t)pf_func << 4;
> -
> flow->npc_action &= ~(GENMASK(19, 4));
> flow->npc_action |= (uint64_t)pf_func << 4;
> - flow->mcam_data[0] |= (uint64_t)inl_dev->channel;
> - flow->mcam_mask[0] |= (uint64_t)inl_dev->chan_mask;
> +
> + npc_mcam_set_channel(flow, req, inl_dev->channel,
> + inl_dev->chan_mask, false);
> } else if (npc->is_sdp_link) {
> - req->entry_data.kw[0] &= ~(GENMASK(11, 0));
> - req->entry_data.kw_mask[0] &= ~(GENMASK(11, 0));
> - req->entry_data.kw[0] |= (uint64_t)npc->sdp_channel;
> - req->entry_data.kw_mask[0] |=
> - (uint64_t)npc->sdp_channel_mask;
> - flow->mcam_data[0] &= ~(GENMASK(11, 0));
> - flow->mcam_mask[0] &= ~(GENMASK(11, 0));
> - flow->mcam_data[0] |= (uint64_t)npc->sdp_channel;
> - flow->mcam_mask[0] |= (uint64_t)npc->sdp_channel_mask;
> + npc_mcam_set_channel(flow, req, npc->sdp_channel,
> + npc->sdp_channel_mask,
> + pst->is_second_pass_rule);
> } else {
> - req->entry_data.kw[0] |= (uint64_t)npc->channel;
> - req->entry_data.kw_mask[0] |= (BIT_ULL(12) - 1);
> - flow->mcam_data[0] |= (uint64_t)npc->channel;
> - flow->mcam_mask[0] |= (BIT_ULL(12) - 1);
> + npc_mcam_set_channel(flow, req, npc->channel,
> + (BIT_ULL(12) - 1),
> + pst->is_second_pass_rule);
> }
> } else {
> uint16_t pf_func = (flow->npc_action >> 4) & 0xffff;
> diff --git a/drivers/common/cnxk/roc_npc_parse.c b/drivers/common/cnxk/roc_npc_parse.c
> index b849326a19..364a846963 100644
> --- a/drivers/common/cnxk/roc_npc_parse.c
> +++ b/drivers/common/cnxk/roc_npc_parse.c
> @@ -21,6 +21,20 @@ npc_parse_meta_items(struct npc_parse_state *pst)
> return 0;
> }
>
> +int
> +npc_parse_mark_item(struct npc_parse_state *pst)
> +{
> + if (pst->pattern->type == ROC_NPC_ITEM_TYPE_MARK) {
> + if (pst->flow->nix_intf != NIX_INTF_RX)
> + return -EINVAL;
> +
> + pst->is_second_pass_rule = true;
> + pst->pattern++;
> + }
> +
> + return 0;
> +}
> +
> static int
> npc_flow_raw_item_prepare(const struct roc_npc_flow_item_raw *raw_spec,
> const struct roc_npc_flow_item_raw *raw_mask,
> diff --git a/drivers/common/cnxk/roc_npc_priv.h b/drivers/common/cnxk/roc_npc_priv.h
> index e78d96e876..78d6ee844d 100644
> --- a/drivers/common/cnxk/roc_npc_priv.h
> +++ b/drivers/common/cnxk/roc_npc_priv.h
> @@ -189,6 +189,7 @@ struct npc_parse_state {
> /* adjust ltype in MCAM to match at least one vlan */
> bool set_vlan_ltype_mask;
> bool set_ipv6ext_ltype_mask;
> + bool is_second_pass_rule;
> };
>
> enum npc_kpu_parser_flag {
> @@ -421,6 +422,7 @@ void npc_get_hw_supp_mask(struct npc_parse_state *pst,
> int npc_parse_item_basic(const struct roc_npc_item_info *item,
> struct npc_parse_item_info *info);
> int npc_parse_meta_items(struct npc_parse_state *pst);
> +int npc_parse_mark_item(struct npc_parse_state *pst);
> int npc_parse_pre_l2(struct npc_parse_state *pst);
> int npc_parse_higig2_hdr(struct npc_parse_state *pst);
> int npc_parse_cpt_hdr(struct npc_parse_state *pst);
> --
> 2.25.4
>
prev parent reply other threads:[~2022-04-24 3:54 UTC|newest]
Thread overview: 3+ messages / expand[flat|nested] mbox.gz Atom feed top
2022-02-28 4:53 psatheesh
2022-02-28 4:53 ` [dpdk-dev] [PATCH 2/2] net/cnxk: support for mark pattern item type psatheesh
2022-04-24 3:54 ` Jerin Jacob [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='CALBAE1ObJ_oDiZ3+iv+06KfjpZfUcM=c_213-kUu8qce-uFX-Q@mail.gmail.com' \
--to=jerinjacobk@gmail.com \
--cc=andrew.rybchenko@oktetlabs.ru \
--cc=dev@dpdk.org \
--cc=ferruh.yigit@xilinx.com \
--cc=kirankumark@marvell.com \
--cc=ndabilpuram@marvell.com \
--cc=psatheesh@marvell.com \
--cc=skori@marvell.com \
--cc=skoteshwar@marvell.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).