From: Jerin Jacob <jerinjacobk@gmail.com>
To: Satheesh Paul <psatheesh@marvell.com>
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 3/3] common/cnxk: update MKEX capability flags
Date: Mon, 29 Aug 2022 10:52:48 +0530 [thread overview]
Message-ID: <CALBAE1MzZh5G7tmAYDguS0f7P0SksOJXdK3cE3PSqQXvXXc+sA@mail.gmail.com> (raw)
In-Reply-To: <20220801061629.1862969-3-psatheesh@marvell.com>
On Mon, Aug 1, 2022 at 11:47 AM <psatheesh@marvell.com> wrote:
>
> From: Satheesh Paul <psatheesh@marvell.com>
>
> Update MKEX capability flags to enable parsing
> VLAN PCP, DSCP and GTPv1 TEID.
>
> Signed-off-by: Satheesh Paul <psatheesh@marvell.com>
> Reviewed-by: Kiran Kumar K <kirankumark@marvell.com>
> ---
> drivers/common/cnxk/roc_npc_mcam.c | 86 ++++++++++++------------------
> drivers/common/cnxk/roc_npc_priv.h | 12 +++--
> 2 files changed, 44 insertions(+), 54 deletions(-)
Build issues clang
[792/3405] Compiling C object
drivers/libtmp_rte_common_cnxk.a.p/common_cnxk_roc_npc_mcam.c.o
FAILED: drivers/libtmp_rte_common_cnxk.a.p/common_cnxk_roc_npc_mcam.c.o
ccache clang -Idrivers/libtmp_rte_common_cnxk.a.p -Idrivers
-I../drivers -Idrivers/common/cnxk -I../drivers/common/cnxk
-Idrivers/bus/pci -I../drivers/bus/pci -Ilib/net -I../lib/net
-Ilib/ethdev -I../lib/ethdev -Ilib/meter -I../lib/meter -I
. -I.. -Iconfig -I../config -Ilib/eal/include -I../lib/eal/include
-Ilib/eal/linux/include -I../lib/eal/linux/include
-Ilib/eal/x86/include -I../lib/eal/x86/include -Ilib/eal/common
-I../lib/eal/common -Ilib/eal -I../lib/eal -Ilib/kvargs -I
../lib/kvargs -Ilib/metrics -I../lib/metrics -Ilib/telemetry
-I../lib/telemetry -Ilib/pci -I../lib/pci -I../drivers/bus/pci/linux
-Ilib/mbuf -I../lib/mbuf -Ilib/mempool -I../lib/mempool -Ilib/ring
-I../lib/ring -Ilib/security -I../lib/secur
ity -Ilib/cryptodev -I../lib/cryptodev -Ilib/rcu -I../lib/rcu
-fcolor-diagnostics -D_FILE_OFFSET_BITS=64 -Wall -Winvalid-pch -Wextra
-Werror -O2 -g -include rte_config.h -Wcast-qual -Wdeprecated -Wformat
-Wformat-nonliteral -Wformat-securit
y -Wmissing-declarations -Wmissing-prototypes -Wnested-externs
-Wold-style-definition -Wpointer-arith -Wsign-compare
-Wstrict-prototypes -Wundef -Wwrite-strings
-Wno-address-of-packed-member -Wno-missing-field-initializers
-D_GNU_SOURCE -fP
IC -march=native -DALLOW_EXPERIMENTAL_API -DALLOW_INTERNAL_API
-DRTE_LOG_DEFAULT_LOGTYPE=pmd.common.cnxk -MD -MQ
drivers/libtmp_rte_common_cnxk.a.p/common_cnxk_roc_npc_mcam.c.o -MF
drivers/libtmp_rte_common_cnxk.a.p/common_cnxk_roc_npc_mcam
.c.o.d -o drivers/libtmp_rte_common_cnxk.a.p/common_cnxk_roc_npc_mcam.c.o
-c ../drivers/common/cnxk/roc_npc_mcam.c
../drivers/common/cnxk/roc_npc_mcam.c:251:24: error: use of bitwise
'&' with boolean operands [-Werror,-Wbitwise-instead-of-logical]
kex_cap.bit.ip_dscp = npc_is_kex_enabled(npc, NPC_LID_LC,
NPC_LT_LC_IP, 1 * 8, 1 * 8) &
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
&&
../drivers/common/cnxk/roc_npc_mcam.c:251:24: note: cast one or both
operands to int to silence this warning
1 error generated.
[793/3405] Compiling C
>
> diff --git a/drivers/common/cnxk/roc_npc_mcam.c b/drivers/common/cnxk/roc_npc_mcam.c
> index 245eeb0107..a71f0c8b8a 100644
> --- a/drivers/common/cnxk/roc_npc_mcam.c
> +++ b/drivers/common/cnxk/roc_npc_mcam.c
> @@ -232,80 +232,64 @@ npc_get_kex_capability(struct npc *npc)
> memset(&kex_cap, 0, sizeof(kex_cap));
>
> /* Ethtype: Offset 12B, len 2B */
> - kex_cap.bit.ethtype_0 = npc_is_kex_enabled(
> - npc, NPC_LID_LA, NPC_LT_LA_ETHER, 12 * 8, 2 * 8);
> + kex_cap.bit.ethtype_0 = npc_is_kex_enabled(npc, NPC_LID_LA, NPC_LT_LA_ETHER, 12 * 8, 2 * 8);
> /* QINQ VLAN Ethtype: offset 8B, len 2B */
> - kex_cap.bit.ethtype_x = npc_is_kex_enabled(
> - npc, NPC_LID_LB, NPC_LT_LB_STAG_QINQ, 8 * 8, 2 * 8);
> + kex_cap.bit.ethtype_x =
> + npc_is_kex_enabled(npc, NPC_LID_LB, NPC_LT_LB_STAG_QINQ, 8 * 8, 2 * 8);
> /* VLAN ID0 : Outer VLAN: Offset 2B, len 2B */
> - kex_cap.bit.vlan_id_0 = npc_is_kex_enabled(
> - npc, NPC_LID_LB, NPC_LT_LB_CTAG, 2 * 8, 2 * 8);
> - /* VLAN ID0 : Inner VLAN: offset 6B, len 2B */
> - kex_cap.bit.vlan_id_x = npc_is_kex_enabled(
> - npc, NPC_LID_LB, NPC_LT_LB_STAG_QINQ, 6 * 8, 2 * 8);
> + kex_cap.bit.vlan_id_0 = npc_is_kex_enabled(npc, NPC_LID_LB, NPC_LT_LB_CTAG, 2 * 8, 2 * 8);
> + /* VLAN PCP0 : Outer VLAN: Offset 2B, len 1B */
> + kex_cap.bit.vlan_pcp_0 = npc_is_kex_enabled(npc, NPC_LID_LB, NPC_LT_LB_CTAG, 2 * 8, 2 * 1);
> + /* VLAN IDX : Inner VLAN: offset 6B, len 2B */
> + kex_cap.bit.vlan_id_x =
> + npc_is_kex_enabled(npc, NPC_LID_LB, NPC_LT_LB_STAG_QINQ, 6 * 8, 2 * 8);
> /* DMCA: offset 0B, len 6B */
> - kex_cap.bit.dmac = npc_is_kex_enabled(npc, NPC_LID_LA, NPC_LT_LA_ETHER,
> - 0 * 8, 6 * 8);
> + kex_cap.bit.dmac = npc_is_kex_enabled(npc, NPC_LID_LA, NPC_LT_LA_ETHER, 0 * 8, 6 * 8);
> /* IP proto: offset 9B, len 1B */
> - kex_cap.bit.ip_proto =
> - npc_is_kex_enabled(npc, NPC_LID_LC, NPC_LT_LC_IP, 9 * 8, 1 * 8);
> + kex_cap.bit.ip_proto = npc_is_kex_enabled(npc, NPC_LID_LC, NPC_LT_LC_IP, 9 * 8, 1 * 8);
> + /* IPv4 dscp: offset 1B, len 1B, IPv6 dscp: offset 0B, len 2B */
> + kex_cap.bit.ip_dscp = npc_is_kex_enabled(npc, NPC_LID_LC, NPC_LT_LC_IP, 1 * 8, 1 * 8) &
> + npc_is_kex_enabled(npc, NPC_LID_LC, NPC_LT_LC_IP6, 0, 2 * 8);
> /* UDP dport: offset 2B, len 2B */
> - kex_cap.bit.udp_dport = npc_is_kex_enabled(npc, NPC_LID_LD,
> - NPC_LT_LD_UDP, 2 * 8, 2 * 8);
> + kex_cap.bit.udp_dport = npc_is_kex_enabled(npc, NPC_LID_LD, NPC_LT_LD_UDP, 2 * 8, 2 * 8);
> /* UDP sport: offset 0B, len 2B */
> - kex_cap.bit.udp_sport = npc_is_kex_enabled(npc, NPC_LID_LD,
> - NPC_LT_LD_UDP, 0 * 8, 2 * 8);
> + kex_cap.bit.udp_sport = npc_is_kex_enabled(npc, NPC_LID_LD, NPC_LT_LD_UDP, 0 * 8, 2 * 8);
> /* TCP dport: offset 2B, len 2B */
> - kex_cap.bit.tcp_dport = npc_is_kex_enabled(npc, NPC_LID_LD,
> - NPC_LT_LD_TCP, 2 * 8, 2 * 8);
> + kex_cap.bit.tcp_dport = npc_is_kex_enabled(npc, NPC_LID_LD, NPC_LT_LD_TCP, 2 * 8, 2 * 8);
> /* TCP sport: offset 0B, len 2B */
> - kex_cap.bit.tcp_sport = npc_is_kex_enabled(npc, NPC_LID_LD,
> - NPC_LT_LD_TCP, 0 * 8, 2 * 8);
> + kex_cap.bit.tcp_sport = npc_is_kex_enabled(npc, NPC_LID_LD, NPC_LT_LD_TCP, 0 * 8, 2 * 8);
> /* IP SIP: offset 12B, len 4B */
> - kex_cap.bit.sip_addr = npc_is_kex_enabled(npc, NPC_LID_LC, NPC_LT_LC_IP,
> - 12 * 8, 4 * 8);
> + kex_cap.bit.sip_addr = npc_is_kex_enabled(npc, NPC_LID_LC, NPC_LT_LC_IP, 12 * 8, 4 * 8);
> /* IP DIP: offset 14B, len 4B */
> - kex_cap.bit.dip_addr = npc_is_kex_enabled(npc, NPC_LID_LC, NPC_LT_LC_IP,
> - 14 * 8, 4 * 8);
> + kex_cap.bit.dip_addr = npc_is_kex_enabled(npc, NPC_LID_LC, NPC_LT_LC_IP, 14 * 8, 4 * 8);
> /* IP6 SIP: offset 8B, len 16B */
> - kex_cap.bit.sip6_addr = npc_is_kex_enabled(npc, NPC_LID_LC, NPC_LT_LC_IP6,
> - 8 * 8, 16 * 8);
> + kex_cap.bit.sip6_addr = npc_is_kex_enabled(npc, NPC_LID_LC, NPC_LT_LC_IP6, 8 * 8, 16 * 8);
> /* IP6 DIP: offset 24B, len 16B */
> - kex_cap.bit.dip6_addr = npc_is_kex_enabled(
> - npc, NPC_LID_LC, NPC_LT_LC_IP6, 24 * 8, 16 * 8);
> + kex_cap.bit.dip6_addr = npc_is_kex_enabled(npc, NPC_LID_LC, NPC_LT_LC_IP6, 24 * 8, 16 * 8);
> /* ESP SPI: offset 0B, len 4B */
> - kex_cap.bit.ipsec_spi = npc_is_kex_enabled(npc, NPC_LID_LE,
> - NPC_LT_LE_ESP, 0 * 8, 4 * 8);
> + kex_cap.bit.ipsec_spi = npc_is_kex_enabled(npc, NPC_LID_LE, NPC_LT_LE_ESP, 0 * 8, 4 * 8);
> /* VXLAN VNI: offset 4B, len 3B */
> - kex_cap.bit.ld_vni = npc_is_kex_enabled(npc, NPC_LID_LE,
> - NPC_LT_LE_VXLAN, 0 * 8, 3 * 8);
> + kex_cap.bit.ld_vni = npc_is_kex_enabled(npc, NPC_LID_LE, NPC_LT_LE_VXLAN, 0 * 8, 3 * 8);
>
> /* Custom L3 frame: varied offset and lengths */
> - kex_cap.bit.custom_l3 =
> - npc_is_kex_enabled(npc, NPC_LID_LC, NPC_LT_LC_CUSTOM0, 0, 0);
> - kex_cap.bit.custom_l3 |= (uint64_t)npc_is_kex_enabled(npc, NPC_LID_LC,
> - NPC_LT_LC_CUSTOM1, 0, 0);
> + kex_cap.bit.custom_l3 = npc_is_kex_enabled(npc, NPC_LID_LC, NPC_LT_LC_CUSTOM0, 0, 0);
> + kex_cap.bit.custom_l3 |=
> + (uint64_t)npc_is_kex_enabled(npc, NPC_LID_LC, NPC_LT_LC_CUSTOM1, 0, 0);
> /* SCTP sport : offset 0B, len 2B */
> - kex_cap.bit.sctp_sport = npc_is_kex_enabled(
> - npc, NPC_LID_LD, NPC_LT_LD_SCTP, 0 * 8, 2 * 8);
> + kex_cap.bit.sctp_sport = npc_is_kex_enabled(npc, NPC_LID_LD, NPC_LT_LD_SCTP, 0 * 8, 2 * 8);
> /* SCTP dport : offset 2B, len 2B */
> - kex_cap.bit.sctp_dport = npc_is_kex_enabled(
> - npc, NPC_LID_LD, NPC_LT_LD_SCTP, 2 * 8, 2 * 8);
> + kex_cap.bit.sctp_dport = npc_is_kex_enabled(npc, NPC_LID_LD, NPC_LT_LD_SCTP, 2 * 8, 2 * 8);
> /* ICMP type : offset 0B, len 1B */
> - kex_cap.bit.icmp_type = npc_is_kex_enabled(
> - npc, NPC_LID_LD, NPC_LT_LD_ICMP, 0 * 8, 1 * 8);
> + kex_cap.bit.icmp_type = npc_is_kex_enabled(npc, NPC_LID_LD, NPC_LT_LD_ICMP, 0 * 8, 1 * 8);
> /* ICMP code : offset 1B, len 1B */
> - kex_cap.bit.icmp_code = npc_is_kex_enabled(
> - npc, NPC_LID_LD, NPC_LT_LD_ICMP, 1 * 8, 1 * 8);
> + kex_cap.bit.icmp_code = npc_is_kex_enabled(npc, NPC_LID_LD, NPC_LT_LD_ICMP, 1 * 8, 1 * 8);
> /* ICMP id : offset 4B, len 2B */
> - kex_cap.bit.icmp_id = npc_is_kex_enabled(npc, NPC_LID_LD,
> - NPC_LT_LD_ICMP, 4 * 8, 2 * 8);
> + kex_cap.bit.icmp_id = npc_is_kex_enabled(npc, NPC_LID_LD, NPC_LT_LD_ICMP, 4 * 8, 2 * 8);
> /* IGMP grp_addr : offset 4B, len 4B */
> - kex_cap.bit.igmp_grp_addr = npc_is_kex_enabled(
> - npc, NPC_LID_LD, NPC_LT_LD_IGMP, 4 * 8, 4 * 8);
> + kex_cap.bit.igmp_grp_addr =
> + npc_is_kex_enabled(npc, NPC_LID_LD, NPC_LT_LD_IGMP, 4 * 8, 4 * 8);
> /* GTPU teid : offset 4B, len 4B */
> - kex_cap.bit.gtpu_teid = npc_is_kex_enabled(
> - npc, NPC_LID_LE, NPC_LT_LE_GTPU, 4 * 8, 4 * 8);
> + kex_cap.bit.gtpv1_teid = npc_is_kex_enabled(npc, NPC_LID_LE, NPC_LT_LE_GTPU, 4 * 8, 4 * 8);
> return kex_cap.all_bits;
> }
>
> diff --git a/drivers/common/cnxk/roc_npc_priv.h b/drivers/common/cnxk/roc_npc_priv.h
> index b08539d8f8..1a597280d1 100644
> --- a/drivers/common/cnxk/roc_npc_priv.h
> +++ b/drivers/common/cnxk/roc_npc_priv.h
> @@ -100,6 +100,7 @@ enum npc_err_status {
> enum npc_mcam_intf { NPC_MCAM_RX, NPC_MCAM_TX };
>
> typedef union npc_kex_cap_terms_t {
> + /** Packet Matching Rule term fields */
> struct {
> /** Total length of received packet */
> uint64_t len : 1;
> @@ -111,10 +112,14 @@ typedef union npc_kex_cap_terms_t {
> uint64_t vlan_id_0 : 1;
> /** Last VLAN ID (inner) */
> uint64_t vlan_id_x : 1;
> + /** PCP in the first VLAN header */
> + uint64_t vlan_pcp_0 : 1;
> /** destination MAC address */
> uint64_t dmac : 1;
> /** IP Protocol or IPv6 Next Header */
> uint64_t ip_proto : 1;
> + /** DSCP in IP header */
> + uint64_t ip_dscp : 1;
> /** Destination UDP port, implies IPPROTO=17 */
> uint64_t udp_dport : 1;
> /** Destination TCP port implies IPPROTO=6 */
> @@ -155,12 +160,13 @@ typedef union npc_kex_cap_terms_t {
> uint64_t sctp_sport : 1;
> /** Destination SCTP port */
> uint64_t sctp_dport : 1;
> - /** GTPU Tunnel endpoint identifier */
> - uint64_t gtpu_teid : 1;
> -
> + /** GTPv1 tunnel endpoint identifier */
> + uint64_t gtpv1_teid : 1;
> } bit;
> +
> /** All bits of the bit field structure */
> uint64_t all_bits;
> +
> } npc_kex_cap_terms_t;
>
> struct npc_parse_item_info {
> --
> 2.35.3
>
next prev parent reply other threads:[~2022-08-29 5:23 UTC|newest]
Thread overview: 8+ messages / expand[flat|nested] mbox.gz Atom feed top
2022-08-01 6:16 [dpdk-dev] [PATCH 1/3] common/cnxk: fix printing disabled MKEX registers psatheesh
2022-08-01 6:16 ` [dpdk-dev] [PATCH 2/3] common/cnxk: print counters along with flow dump psatheesh
2022-08-01 6:16 ` [dpdk-dev] [PATCH 3/3] common/cnxk: update MKEX capability flags psatheesh
2022-08-29 5:22 ` Jerin Jacob [this message]
2022-08-29 6:24 ` [dpdk-dev] [PATCH v2 1/3] common/cnxk: fix printing disabled MKEX registers psatheesh
2022-08-29 6:24 ` [dpdk-dev] [PATCH v2 2/3] common/cnxk: print counters along with flow dump psatheesh
2022-08-29 6:24 ` [dpdk-dev] [PATCH v2 3/3] common/cnxk: update MKEX capability flags psatheesh
2022-09-19 13:30 ` Jerin Jacob
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=CALBAE1MzZh5G7tmAYDguS0f7P0SksOJXdK3cE3PSqQXvXXc+sA@mail.gmail.com \
--to=jerinjacobk@gmail.com \
--cc=dev@dpdk.org \
--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).