From: Shahaf Shuler <shahafs@mellanox.com>
To: Slava Ovsiienko <viacheslavo@mellanox.com>,
"shahaf@mellanox.com" <shahaf@mellanox.com>,
"ferruh.yigit@intel.com" <ferruh.yigit@intel.com>,
"wenzhuo.lu@intel.com" <wenzhuo.lu@intel.com>,
"jingjing.wu@intel.com" <jingjing.wu@intel.com>,
"bernard.iremonger@intel.com" <bernard.iremonger@intel.com>
Cc: "dev@dpdk.org" <dev@dpdk.org>
Subject: Re: [dpdk-dev] [PATCH v2] app/testpmd: add tos and ttl field to vxlan encapsulation
Date: Mon, 14 Jan 2019 06:04:02 +0000 [thread overview]
Message-ID: <DB7PR05MB442601050F917FA1E0D73690C3800@DB7PR05MB4426.eurprd05.prod.outlook.com> (raw)
In-Reply-To: <1547390409-3530-1-git-send-email-viacheslavo@mellanox.com>
Adding some relevant folks.
Any comments or concerns on this patch?
Sunday, January 13, 2019 4:40 PM, Viacheslav Ovsiienko:
> Subject: [dpdk-dev] [PATCH v2] app/testpmd: add tos and ttl field to vxlan
> encapsulation
>
> The new testpmd set vxlan-tos-ttl command is added. It allows to specify tos
> and tll fields for encapsulation IP header.
>
> IPv4 VXLAN outer header:
>
> testpmd> set vxlan-tos-ttl ip-version ipv4 vni 4 udp-src 4
> udp-dst 4 ip-tos 0 ip-ttl 255 ip-src 127.0.0.1
> ip-dst 128.0.0.1 eth-src 11:11:11:11:11:11
> eth-dst 22:22:22:22:22:22
>
> IPv6 VXLAN outer header:
> testpmd> set vxlan-tos-ttl ip-version ipv6 vni 4 udp-src 4
> udp-dst 4 ip-tos 0 ip-ttl 255 ::1 ip-dst ::2222
> eth-src 11:11:11:11:11:11 eth-dst
> 22:22:22:22:22:22
>
> Note: ip-ttl parameter corresponds the nop_limits field for IPv6.
>
> Signed-off-by: Viacheslav Ovsiienko <viacheslavo@mellanox.com>
> ---
>
> v2:
> The patchset is splitted into two parts, one is related to mlx5 PMD, and
> another is update for testpmd, these parts are going to be merged into
> different next branches.
>
> v1:
> Original patchset:
> https://emea01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fpat
> ches.dpdk.org%2Fproject%2Fdpdk%2Flist%2F%3Fseries%3D2974&data
> =02%7C01%7Cshahafs%40mellanox.com%7C8d911e89ae384e23a1f908d6796
> 50b0f%7Ca652971c7d2e4d9ba6a4d149256f461b%7C0%7C0%7C6368298722325
> 87143&sdata=EMLbTbGYyxdYdEdZ4IFsvaKoDRD4oQrmWThJD9Xqsr8%3
> D&reserved=0
>
> app/test-pmd/cmdline.c | 63
> +++++++++++++++++++++++++++++
> app/test-pmd/cmdline_flow.c | 32 +++++++++++++++
> app/test-pmd/testpmd.c | 3 ++
> app/test-pmd/testpmd.h | 3 ++
> doc/guides/testpmd_app_ug/testpmd_funcs.rst | 16 ++++++++
> 5 files changed, 117 insertions(+)
>
> diff --git a/app/test-pmd/cmdline.c b/app/test-pmd/cmdline.c index
> 3ddc3e0..9e9e898 100644
> --- a/app/test-pmd/cmdline.c
> +++ b/app/test-pmd/cmdline.c
> @@ -794,6 +794,12 @@ static void cmd_help_long_parsed(void
> *parsed_result,
> " eth-dst (eth-dst)\n"
> " Configure the VXLAN encapsulation for
> flows.\n\n"
>
> + "vxlan-tos-ttl ip-version (ipv4|ipv6) vni (vni) udp-src"
> + " (udp-src) udp-dst (udp-dst) ip-tos (ip-tos) ip-ttl (ip-
> ttl)"
> + " ip-src (ip-src) ip-dst (ip-dst) eth-src (eth-src)"
> + " eth-dst (eth-dst)\n"
> + " Configure the VXLAN encapsulation for
> flows.\n\n"
> +
> "nvgre ip-version (ipv4|ipv6) tni (tni) ip-src"
> " (ip-src) ip-dst (ip-dst) eth-src (eth-src) eth-dst"
> " (eth-dst)\n"
> @@ -15034,6 +15040,8 @@ struct cmd_set_vxlan_result {
> cmdline_ipaddr_t ip_src;
> cmdline_ipaddr_t ip_dst;
> uint16_t tci;
> + uint8_t tos;
> + uint8_t ttl;
> struct ether_addr eth_src;
> struct ether_addr eth_dst;
> };
> @@ -15042,6 +15050,9 @@ struct cmd_set_vxlan_result {
> TOKEN_STRING_INITIALIZER(struct cmd_set_vxlan_result, set,
> "set"); cmdline_parse_token_string_t cmd_set_vxlan_vxlan =
> TOKEN_STRING_INITIALIZER(struct cmd_set_vxlan_result, vxlan,
> "vxlan");
> +cmdline_parse_token_string_t cmd_set_vxlan_vxlan_tos_ttl =
> + TOKEN_STRING_INITIALIZER(struct cmd_set_vxlan_result, vxlan,
> + "vxlan-tos-ttl");
> cmdline_parse_token_string_t cmd_set_vxlan_vxlan_with_vlan =
> TOKEN_STRING_INITIALIZER(struct cmd_set_vxlan_result, vxlan,
> "vxlan-with-vlan");
> @@ -15066,6 +15077,16 @@ struct cmd_set_vxlan_result {
> "udp-dst");
> cmdline_parse_token_num_t cmd_set_vxlan_udp_dst_value =
> TOKEN_NUM_INITIALIZER(struct cmd_set_vxlan_result, udp_dst,
> UINT16);
> +cmdline_parse_token_string_t cmd_set_vxlan_ip_tos =
> + TOKEN_STRING_INITIALIZER(struct cmd_set_vxlan_result,
> pos_token,
> + "ip-tos");
> +cmdline_parse_token_num_t cmd_set_vxlan_ip_tos_value =
> + TOKEN_NUM_INITIALIZER(struct cmd_set_vxlan_result, tos, UINT8);
> +cmdline_parse_token_string_t cmd_set_vxlan_ip_ttl =
> + TOKEN_STRING_INITIALIZER(struct cmd_set_vxlan_result,
> pos_token,
> + "ip-ttl");
> +cmdline_parse_token_num_t cmd_set_vxlan_ip_ttl_value =
> + TOKEN_NUM_INITIALIZER(struct cmd_set_vxlan_result, ttl, UINT8);
> cmdline_parse_token_string_t cmd_set_vxlan_ip_src =
> TOKEN_STRING_INITIALIZER(struct cmd_set_vxlan_result,
> pos_token,
> "ip-src");
> @@ -15104,10 +15125,15 @@ static void cmd_set_vxlan_parsed(void
> *parsed_result,
> .vxlan_id = rte_cpu_to_be_32(res->vni) &
> RTE_BE32(0x00ffffff),
> };
>
> + vxlan_encap_conf.select_tos = 0;
> if (strcmp(res->vxlan, "vxlan") == 0)
> vxlan_encap_conf.select_vlan = 0;
> else if (strcmp(res->vxlan, "vxlan-with-vlan") == 0)
> vxlan_encap_conf.select_vlan = 1;
> + else if (strcmp(res->vxlan, "vxlan-tos-ttl") == 0) {
> + vxlan_encap_conf.select_vlan = 0;
> + vxlan_encap_conf.select_tos = 1;
> + }
> if (strcmp(res->ip_version, "ipv4") == 0)
> vxlan_encap_conf.select_ipv4 = 1;
> else if (strcmp(res->ip_version, "ipv6") == 0) @@ -15117,6 +15143,8
> @@ static void cmd_set_vxlan_parsed(void *parsed_result,
> rte_memcpy(vxlan_encap_conf.vni, &id.vni[1], 3);
> vxlan_encap_conf.udp_src = rte_cpu_to_be_16(res->udp_src);
> vxlan_encap_conf.udp_dst = rte_cpu_to_be_16(res->udp_dst);
> + vxlan_encap_conf.ip_tos = res->tos;
> + vxlan_encap_conf.ip_ttl = res->ttl;
> if (vxlan_encap_conf.select_ipv4) {
> IPV4_ADDR_TO_UINT(res->ip_src,
> vxlan_encap_conf.ipv4_src);
> IPV4_ADDR_TO_UINT(res->ip_dst,
> vxlan_encap_conf.ipv4_dst); @@ -15161,6 +15189,40 @@ static void
> cmd_set_vxlan_parsed(void *parsed_result,
> },
> };
>
> +cmdline_parse_inst_t cmd_set_vxlan_tos_ttl = {
> + .f = cmd_set_vxlan_parsed,
> + .data = NULL,
> + .help_str = "set vxlan-tos-ttl ip-version ipv4|ipv6 vni <vni> udp-src"
> + " <udp-src> udp-dst <udp-dst> ip-tos <ip-tos> ip-ttl <ip-ttl>"
> + " ip-src <ip-src> ip-dst <ip-dst> eth-src <eth-src>"
> + " eth-dst <eth-dst>",
> + .tokens = {
> + (void *)&cmd_set_vxlan_set,
> + (void *)&cmd_set_vxlan_vxlan_tos_ttl,
> + (void *)&cmd_set_vxlan_ip_version,
> + (void *)&cmd_set_vxlan_ip_version_value,
> + (void *)&cmd_set_vxlan_vni,
> + (void *)&cmd_set_vxlan_vni_value,
> + (void *)&cmd_set_vxlan_udp_src,
> + (void *)&cmd_set_vxlan_udp_src_value,
> + (void *)&cmd_set_vxlan_udp_dst,
> + (void *)&cmd_set_vxlan_udp_dst_value,
> + (void *)&cmd_set_vxlan_ip_tos,
> + (void *)&cmd_set_vxlan_ip_tos_value,
> + (void *)&cmd_set_vxlan_ip_ttl,
> + (void *)&cmd_set_vxlan_ip_ttl_value,
> + (void *)&cmd_set_vxlan_ip_src,
> + (void *)&cmd_set_vxlan_ip_src_value,
> + (void *)&cmd_set_vxlan_ip_dst,
> + (void *)&cmd_set_vxlan_ip_dst_value,
> + (void *)&cmd_set_vxlan_eth_src,
> + (void *)&cmd_set_vxlan_eth_src_value,
> + (void *)&cmd_set_vxlan_eth_dst,
> + (void *)&cmd_set_vxlan_eth_dst_value,
> + NULL,
> + },
> +};
> +
> cmdline_parse_inst_t cmd_set_vxlan_with_vlan = {
> .f = cmd_set_vxlan_parsed,
> .data = NULL,
> @@ -18696,6 +18758,7 @@ struct cmd_show_tx_metadata_result {
> (cmdline_parse_inst_t *)&cmd_set_port_tm_hierarchy_default,
> #endif
> (cmdline_parse_inst_t *)&cmd_set_vxlan,
> + (cmdline_parse_inst_t *)&cmd_set_vxlan_tos_ttl,
> (cmdline_parse_inst_t *)&cmd_set_vxlan_with_vlan,
> (cmdline_parse_inst_t *)&cmd_set_nvgre,
> (cmdline_parse_inst_t *)&cmd_set_nvgre_with_vlan, diff --git
> a/app/test-pmd/cmdline_flow.c b/app/test-pmd/cmdline_flow.c index
> 659908f..97ecc8a 100644
> --- a/app/test-pmd/cmdline_flow.c
> +++ b/app/test-pmd/cmdline_flow.c
> @@ -3501,6 +3501,38 @@ static int comp_vc_action_rss_queue(struct
> context *, const struct token *,
> if (!vxlan_encap_conf.select_vlan)
> action_vxlan_encap_data->items[1].type =
> RTE_FLOW_ITEM_TYPE_VOID;
> + if (vxlan_encap_conf.select_tos) {
> + if (vxlan_encap_conf.select_ipv4) {
> + static struct rte_flow_item_ipv4 ipv4_mask_tos;
> +
> + memcpy(&ipv4_mask_tos,
> &rte_flow_item_ipv4_mask,
> + sizeof(ipv4_mask_tos));
> + ipv4_mask_tos.hdr.type_of_service = 0xff;
> + ipv4_mask_tos.hdr.time_to_live = 0xff;
> + action_vxlan_encap_data-
> >item_ipv4.hdr.type_of_service =
> + vxlan_encap_conf.ip_tos;
> + action_vxlan_encap_data-
> >item_ipv4.hdr.time_to_live =
> + vxlan_encap_conf.ip_ttl;
> + action_vxlan_encap_data->items[2].mask =
> + &ipv4_mask_tos;
> + } else {
> + static struct rte_flow_item_ipv6 ipv6_mask_tos;
> +
> + memcpy(&ipv6_mask_tos,
> &rte_flow_item_ipv6_mask,
> + sizeof(ipv6_mask_tos));
> + ipv6_mask_tos.hdr.vtc_flow |=
> + RTE_BE32(0xfful << IPV6_HDR_TC_SHIFT);
> + ipv6_mask_tos.hdr.hop_limits = 0xff;
> + action_vxlan_encap_data->item_ipv6.hdr.vtc_flow
> |=
> + rte_cpu_to_be_32
> + ((uint32_t)vxlan_encap_conf.ip_tos
> <<
> + IPV6_HDR_TC_SHIFT);
> + action_vxlan_encap_data->item_ipv6.hdr.hop_limits
> =
> + vxlan_encap_conf.ip_ttl;
> + action_vxlan_encap_data->items[2].mask =
> + &ipv6_mask_tos;
> + }
> + }
> memcpy(action_vxlan_encap_data->item_vxlan.vni,
> vxlan_encap_conf.vni,
> RTE_DIM(vxlan_encap_conf.vni));
> action->conf = &action_vxlan_encap_data->conf; diff --git
> a/app/test-pmd/testpmd.c b/app/test-pmd/testpmd.c index
> 15a9488..2638d06 100644
> --- a/app/test-pmd/testpmd.c
> +++ b/app/test-pmd/testpmd.c
> @@ -469,6 +469,7 @@ struct rte_fdir_conf fdir_conf = { struct
> vxlan_encap_conf vxlan_encap_conf = {
> .select_ipv4 = 1,
> .select_vlan = 0,
> + .select_tos = 0,
> .vni = "\x00\x00\x00",
> .udp_src = 0,
> .udp_dst = RTE_BE16(4789),
> @@ -479,6 +480,8 @@ struct vxlan_encap_conf vxlan_encap_conf = {
> .ipv6_dst = "\x00\x00\x00\x00\x00\x00\x00\x00"
> "\x00\x00\x00\x00\x00\x00\x11\x11",
> .vlan_tci = 0,
> + .ip_tos = 0,
> + .ip_ttl = 255,
> .eth_src = "\x00\x00\x00\x00\x00\x00",
> .eth_dst = "\xff\xff\xff\xff\xff\xff", }; diff --git a/app/test-
> pmd/testpmd.h b/app/test-pmd/testpmd.h index 3ff11e6..559f2aa 100644
> --- a/app/test-pmd/testpmd.h
> +++ b/app/test-pmd/testpmd.h
> @@ -488,6 +488,7 @@ struct gso_status {
> struct vxlan_encap_conf {
> uint32_t select_ipv4:1;
> uint32_t select_vlan:1;
> + uint32_t select_tos:1;
> uint8_t vni[3];
> rte_be16_t udp_src;
> rte_be16_t udp_dst;
> @@ -496,6 +497,8 @@ struct vxlan_encap_conf {
> uint8_t ipv6_src[16];
> uint8_t ipv6_dst[16];
> rte_be16_t vlan_tci;
> + uint8_t ip_tos;
> + uint8_t ip_ttl;
> uint8_t eth_src[ETHER_ADDR_LEN];
> uint8_t eth_dst[ETHER_ADDR_LEN];
> };
> diff --git a/doc/guides/testpmd_app_ug/testpmd_funcs.rst
> b/doc/guides/testpmd_app_ug/testpmd_funcs.rst
> index 854af2d..0291480 100644
> --- a/doc/guides/testpmd_app_ug/testpmd_funcs.rst
> +++ b/doc/guides/testpmd_app_ug/testpmd_funcs.rst
> @@ -1583,6 +1583,10 @@ Configure the outer layer to encapsulate a packet
> inside a VXLAN tunnel::
> udp-dst (udp-dst) ip-src (ip-src) ip-dst (ip-dst) vlan-tci (vlan-tci) \
> eth-src (eth-src) eth-dst (eth-dst)
>
> + set vxlan-tos-ttl ip-version (ipv4|ipv6) vni (vni) udp-src (udp-src) \
> + udp-dst (udp-dst) ip-tos (ip-tos) ip-ttl (ip-ttl) ip-src (ip-src) \
> + ip-dst (ip-dst) eth-src (eth-src) eth-dst (eth-dst)
> +
> Those command will set an internal configuration inside testpmd, any
> following flow rule using the action vxlan_encap will use the last
> configuration set.
> To have a different encapsulation header, one of those commands must be
> called @@ -4241,6 +4245,12 @@ IPv4 VXLAN outer header::
> testpmd> flow create 0 ingress pattern end actions vxlan_encap /
> queue index 0 / end
>
> + testpmd> set vxlan-tos-ttl ip-version ipv4 vni 4 udp-src 4 udp-dst 4
> + testpmd> ip-tos 0
> + ip-ttl 255 ip-src 127.0.0.1 ip-dst 128.0.0.1 eth-src 11:11:11:11:11:11
> + eth-dst 22:22:22:22:22:22
> + testpmd> flow create 0 ingress pattern end actions vxlan_encap /
> + queue index 0 / end
> +
> IPv6 VXLAN outer header::
>
> testpmd> set vxlan ip-version ipv6 vni 4 udp-src 4 udp-dst 4 ip-src ::1 @@ -
> 4254,6 +4264,12 @@ IPv6 VXLAN outer header::
> testpmd> flow create 0 ingress pattern end actions vxlan_encap /
> queue index 0 / end
>
> + testpmd> set vxlan-tos-ttl ip-version ipv6 vni 4 udp-src 4 udp-dst 4
> + ip-tos 0 ip-ttl 255 ::1 ip-dst ::2222 eth-src 11:11:11:11:11:11
> + eth-dst 22:22:22:22:22:22
> + testpmd> flow create 0 ingress pattern end actions vxlan_encap /
> + queue index 0 / end
> +
> Sample NVGRE encapsulation rule
> ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
>
> --
> 1.8.3.1
next prev parent reply other threads:[~2019-01-14 6:04 UTC|newest]
Thread overview: 17+ messages / expand[flat|nested] mbox.gz Atom feed top
2018-12-29 18:51 [dpdk-dev] [PATCH 0/4] net/mlx5: add tos and ttl flower match and tunnel keys Viacheslav Ovsiienko
2018-12-29 18:51 ` [dpdk-dev] [PATCH 1/4] " Viacheslav Ovsiienko
2018-12-29 18:51 ` [dpdk-dev] [PATCH 2/4] net/mlx5: add tos and ttl fields support on E-Switch Viacheslav Ovsiienko
2018-12-29 18:51 ` [dpdk-dev] [PATCH 3/4] net/mlx5: add tos and ttl validation " Viacheslav Ovsiienko
2018-12-29 18:51 ` [dpdk-dev] [PATCH 4/4] app/testpmd: add tos and ttl field to vxlan encapsulation Viacheslav Ovsiienko
2019-01-13 14:15 ` [dpdk-dev] [PATCH v2 0/3] net/mlx5: add tos and ttl flower match and tunnel keys Viacheslav Ovsiienko
2019-01-13 14:15 ` [dpdk-dev] [PATCH v2 1/3] " Viacheslav Ovsiienko
2019-01-13 14:15 ` [dpdk-dev] [PATCH v2 2/3] net/mlx5: add tos and ttl fields support on E-Switch Viacheslav Ovsiienko
2019-01-13 14:15 ` [dpdk-dev] [PATCH v2 3/3] net/mlx5: add tos and ttl validation " Viacheslav Ovsiienko
2019-01-14 6:14 ` [dpdk-dev] [PATCH v2 0/3] net/mlx5: add tos and ttl flower match and tunnel keys Shahaf Shuler
2019-01-13 14:40 ` [dpdk-dev] [PATCH v2] app/testpmd: add tos and ttl field to vxlan encapsulation Viacheslav Ovsiienko
2019-01-14 6:04 ` Shahaf Shuler [this message]
2019-01-14 10:08 ` Iremonger, Bernard
2019-01-16 6:59 ` Ori Kam
2019-01-22 10:57 ` [dpdk-dev] [PATCH v3] " Viacheslav Ovsiienko
2019-01-23 19:45 ` Ori Kam
2019-02-07 12:08 ` 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=DB7PR05MB442601050F917FA1E0D73690C3800@DB7PR05MB4426.eurprd05.prod.outlook.com \
--to=shahafs@mellanox.com \
--cc=bernard.iremonger@intel.com \
--cc=dev@dpdk.org \
--cc=ferruh.yigit@intel.com \
--cc=jingjing.wu@intel.com \
--cc=shahaf@mellanox.com \
--cc=viacheslavo@mellanox.com \
--cc=wenzhuo.lu@intel.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).