[dpdk-dev,v2,1/2] app/testpmd: add commands to test new Rx offload API
Checks
Commit Message
Add following testpmd run-time commands to support test of
new Rx offload API:
rx_offload get capability <port_id>
rx_offload get configuration <port_id>
rx_offload enable|disable per_port <offload> <port_id>
rx_offload enable|disable per_queue <offload> <port_id> <queue_id>
Above last 2 commands should be run when the port is stopped.
And <offload> can be one of "vlan_strip", "ipv4_cksum", ...
Signed-off-by: Wei Dai <wei.dai@intel.com>
---
app/test-pmd/cmdline.c | 424 +++++++++++++++++++++++++++++++++++++++++++++++++
app/test-pmd/testpmd.c | 15 +-
app/test-pmd/testpmd.h | 1 +
3 files changed, 438 insertions(+), 2 deletions(-)
Comments
On 03/13/2018 09:42 AM, Wei Dai wrote:
> Add following testpmd run-time commands to support test of
> new Rx offload API:
> rx_offload get capability <port_id>
> rx_offload get configuration <port_id>
> rx_offload enable|disable per_port <offload> <port_id>
> rx_offload enable|disable per_queue <offload> <port_id> <queue_id>
>
> Above last 2 commands should be run when the port is stopped.
> And <offload> can be one of "vlan_strip", "ipv4_cksum", ...
>
> Signed-off-by: Wei Dai <wei.dai@intel.com>
> ---
> app/test-pmd/cmdline.c | 424 +++++++++++++++++++++++++++++++++++++++++++++++++
> app/test-pmd/testpmd.c | 15 +-
> app/test-pmd/testpmd.h | 1 +
> 3 files changed, 438 insertions(+), 2 deletions(-)
>
> diff --git a/app/test-pmd/cmdline.c b/app/test-pmd/cmdline.c
> index d1dc1de..dfd0ca6 100644
> --- a/app/test-pmd/cmdline.c
> +++ b/app/test-pmd/cmdline.c
> @@ -15996,6 +15996,426 @@ cmdline_parse_inst_t cmd_ptype_mapping_update = {
> },
> };
>
> +/* Get Rx offloads capability */
> +struct cmd_rx_offload_get_capa_result {
> + cmdline_fixed_string_t rx_offload;
> + cmdline_fixed_string_t get;
> + cmdline_fixed_string_t capability;
> + portid_t port_id;
> +};
> +
> +cmdline_parse_token_string_t cmd_rx_offload_get_capa_rx_offload =
> + TOKEN_STRING_INITIALIZER
> + (struct cmd_rx_offload_get_capa_result,
> + rx_offload, "rx_offload");
> +cmdline_parse_token_string_t cmd_rx_offload_get_capa_get =
> + TOKEN_STRING_INITIALIZER
> + (struct cmd_rx_offload_get_capa_result,
> + get, "get");
> +cmdline_parse_token_string_t cmd_rx_offload_get_capa_capability =
> + TOKEN_STRING_INITIALIZER
> + (struct cmd_rx_offload_get_capa_result,
> + capability, "capability");
> +cmdline_parse_token_num_t cmd_rx_offload_get_capa_port_id =
> + TOKEN_NUM_INITIALIZER
> + (struct cmd_rx_offload_get_capa_result,
> + port_id, UINT16);
> +
> +static void
> +print_rx_offloads(uint64_t offloads)
> +{
> + uint64_t single_offload;
> +
> + for (single_offload = DEV_RX_OFFLOAD_VLAN_STRIP;
> + single_offload <= DEV_RX_OFFLOAD_SECURITY;
It requires attention when a new offload is added.
Please, consider to use something like __builtin_ffsll(), print name of
the found bit, clear it and retry. It allows to avoid boundaries
specification.
<snip>
> +static int
> +config_rx_offload(const char *name, uint64_t *offload, int on)
> +{
> + uint64_t local = *offload;
> +
> + if (!strcmp(name, "vlan_strip")) {
> + if (on)
> + local |= DEV_RX_OFFLOAD_VLAN_STRIP;
> + else
> + local &= ~DEV_RX_OFFLOAD_VLAN_STRIP;
> + } else if (!strcmp(name, "ipv4_cksum")) {
> + if (on)
> + local |= DEV_RX_OFFLOAD_IPV4_CKSUM;
> + else
> + local &= ~DEV_RX_OFFLOAD_IPV4_CKSUM;
> + } else if (!strcmp(name, "udp_cksum")) {
> + if (on)
> + local |= DEV_RX_OFFLOAD_UDP_CKSUM;
> + else
> + local &= ~DEV_RX_OFFLOAD_UDP_CKSUM;
> + } else if (!strcmp(name, "tcp_cksum")) {
> + if (on)
> + local |= DEV_RX_OFFLOAD_TCP_CKSUM;
> + else
> + local &= ~DEV_RX_OFFLOAD_TCP_CKSUM;
> + } else if (!strcmp(name, "tcp_lro")) {
> + if (on)
> + local |= DEV_RX_OFFLOAD_TCP_LRO;
> + else
> + local &= ~DEV_RX_OFFLOAD_TCP_LRO;
> + } else if (!strcmp(name, "qinq_strip")) {
> + if (on)
> + local |= DEV_RX_OFFLOAD_QINQ_STRIP;
> + else
> + local &= ~DEV_RX_OFFLOAD_QINQ_STRIP;
> + } else if (!strcmp(name, "outer_ipv4_cksum")) {
> + if (on)
> + local |= DEV_RX_OFFLOAD_OUTER_IPV4_CKSUM;
> + else
> + local &= ~DEV_RX_OFFLOAD_OUTER_IPV4_CKSUM;
> + } else if (!strcmp(name, "macsec_strip")) {
> + if (on)
> + local |= DEV_RX_OFFLOAD_MACSEC_STRIP;
> + else
> + local &= ~DEV_RX_OFFLOAD_MACSEC_STRIP;
> + } else if (!strcmp(name, "header_split")) {
> + if (on)
> + local |= DEV_RX_OFFLOAD_HEADER_SPLIT;
> + else
> + local &= ~DEV_RX_OFFLOAD_HEADER_SPLIT;
> + } else if (!strcmp(name, "vlan_filter")) {
> + if (on)
> + local |= DEV_RX_OFFLOAD_VLAN_FILTER;
> + else
> + local &= ~DEV_RX_OFFLOAD_VLAN_FILTER;
> + } else if (!strcmp(name, "vlan_extend")) {
> + if (on)
> + local |= DEV_RX_OFFLOAD_VLAN_EXTEND;
> + else
> + local &= ~DEV_RX_OFFLOAD_VLAN_EXTEND;
> + } else if (!strcmp(name, "jumbo_frame")) {
> + if (on)
> + local |= DEV_RX_OFFLOAD_JUMBO_FRAME;
> + else
> + local &= ~DEV_RX_OFFLOAD_JUMBO_FRAME;
> + } else if (!strcmp(name, "crc_strip")) {
> + if (on)
> + local |= DEV_RX_OFFLOAD_CRC_STRIP;
> + else
> + local &= ~DEV_RX_OFFLOAD_CRC_STRIP;
> + } else if (!strcmp(name, "scatter")) {
> + if (on)
> + local |= DEV_RX_OFFLOAD_SCATTER;
> + else
> + local &= ~DEV_RX_OFFLOAD_SCATTER;
> + } else if (!strcmp(name, "timestamp")) {
> + if (on)
> + local |= DEV_RX_OFFLOAD_TIMESTAMP;
> + else
> + local &= ~DEV_RX_OFFLOAD_TIMESTAMP;
> + } else if (!strcmp(name, "security")) {
> + if (on)
> + local |= DEV_RX_OFFLOAD_SECURITY;
> + else
> + local &= ~DEV_RX_OFFLOAD_SECURITY;
> + } else
> + return -1;
I was going to suggest rte_eth_dev_rx_offload_name(), strcasecmp() and loop.
It is possible to iterate on all U64 bits or add a way to get all
offload bits from rte_ethdev.
However it still requires list of offloads in the help below. So, it is
not easy to get rig of
offloads list completely. Just an idea. Up to you.
<snip>
> +cmdline_parse_inst_t cmd_config_per_port_rx_offload = {
> + .f = cmd_config_per_port_rx_offload_parsed,
> + .data = NULL,
> + .help_str = "rx_offload enable|disable per_port vlan_strip|ipv4_cksum|"
> + "udp_cksum|tcp_cksum|tcp_lro|qinq_strip|outer_ipv4_cksum|"
> + "macsec_strip|header_split|vlan_filter|vlan_extend|"
> + "jumbo_frame|crc_strip|scatter|timestamp|security "
> + "<port_id>",
>
<snip>
> -----Original Message-----
> From: dev [mailto:dev-bounces@dpdk.org] On Behalf Of Andrew Rybchenko
> Sent: Tuesday, March 13, 2018 7:21 AM
> To: Dai, Wei <wei.dai@intel.com>; Lu, Wenzhuo <wenzhuo.lu@intel.com>; Wu, Jingjing <jingjing.wu@intel.com>
> Cc: dev@dpdk.org
> Subject: Re: [dpdk-dev] [PATCH v2 1/2] app/testpmd: add commands to test new Rx offload API
>
> On 03/13/2018 09:42 AM, Wei Dai wrote:
> > Add following testpmd run-time commands to support test of
> > new Rx offload API:
> > rx_offload get capability <port_id>
> > rx_offload get configuration <port_id>
> > rx_offload enable|disable per_port <offload> <port_id>
> > rx_offload enable|disable per_queue <offload> <port_id> <queue_id>
> >
> > Above last 2 commands should be run when the port is stopped.
> > And <offload> can be one of "vlan_strip", "ipv4_cksum", ...
> >
> > Signed-off-by: Wei Dai <wei.dai@intel.com>
> > ---
> > app/test-pmd/cmdline.c | 424 +++++++++++++++++++++++++++++++++++++++++++++++++
> > app/test-pmd/testpmd.c | 15 +-
> > app/test-pmd/testpmd.h | 1 +
> > 3 files changed, 438 insertions(+), 2 deletions(-)
> >
> > diff --git a/app/test-pmd/cmdline.c b/app/test-pmd/cmdline.c
> > index d1dc1de..dfd0ca6 100644
> > --- a/app/test-pmd/cmdline.c
> > +++ b/app/test-pmd/cmdline.c
> > @@ -15996,6 +15996,426 @@ cmdline_parse_inst_t cmd_ptype_mapping_update = {
> > },
> > };
> >
> > +/* Get Rx offloads capability */
> > +struct cmd_rx_offload_get_capa_result {
> > + cmdline_fixed_string_t rx_offload;
> > + cmdline_fixed_string_t get;
> > + cmdline_fixed_string_t capability;
> > + portid_t port_id;
> > +};
> > +
> > +cmdline_parse_token_string_t cmd_rx_offload_get_capa_rx_offload =
> > + TOKEN_STRING_INITIALIZER
> > + (struct cmd_rx_offload_get_capa_result,
> > + rx_offload, "rx_offload");
> > +cmdline_parse_token_string_t cmd_rx_offload_get_capa_get =
> > + TOKEN_STRING_INITIALIZER
> > + (struct cmd_rx_offload_get_capa_result,
> > + get, "get");
> > +cmdline_parse_token_string_t cmd_rx_offload_get_capa_capability =
> > + TOKEN_STRING_INITIALIZER
> > + (struct cmd_rx_offload_get_capa_result,
> > + capability, "capability");
> > +cmdline_parse_token_num_t cmd_rx_offload_get_capa_port_id =
> > + TOKEN_NUM_INITIALIZER
> > + (struct cmd_rx_offload_get_capa_result,
> > + port_id, UINT16);
> > +
> > +static void
> > +print_rx_offloads(uint64_t offloads)
> > +{
> > + uint64_t single_offload;
> > +
> > + for (single_offload = DEV_RX_OFFLOAD_VLAN_STRIP;
> > + single_offload <= DEV_RX_OFFLOAD_SECURITY;
>
> It requires attention when a new offload is added.
> Please, consider to use something like __builtin_ffsll(), print name of
> the found bit, clear it and retry. It allows to avoid boundaries
> specification.
>
> <snip>
>
>
> > +static int
> > +config_rx_offload(const char *name, uint64_t *offload, int on)
> > +{
> > + uint64_t local = *offload;
> > +
> > + if (!strcmp(name, "vlan_strip")) {
> > + if (on)
> > + local |= DEV_RX_OFFLOAD_VLAN_STRIP;
> > + else
> > + local &= ~DEV_RX_OFFLOAD_VLAN_STRIP;
> > + } else if (!strcmp(name, "ipv4_cksum")) {
> > + if (on)
> > + local |= DEV_RX_OFFLOAD_IPV4_CKSUM;
> > + else
> > + local &= ~DEV_RX_OFFLOAD_IPV4_CKSUM;
> > + } else if (!strcmp(name, "udp_cksum")) {
> > + if (on)
> > + local |= DEV_RX_OFFLOAD_UDP_CKSUM;
> > + else
> > + local &= ~DEV_RX_OFFLOAD_UDP_CKSUM;
> > + } else if (!strcmp(name, "tcp_cksum")) {
> > + if (on)
> > + local |= DEV_RX_OFFLOAD_TCP_CKSUM;
> > + else
> > + local &= ~DEV_RX_OFFLOAD_TCP_CKSUM;
> > + } else if (!strcmp(name, "tcp_lro")) {
> > + if (on)
> > + local |= DEV_RX_OFFLOAD_TCP_LRO;
> > + else
> > + local &= ~DEV_RX_OFFLOAD_TCP_LRO;
> > + } else if (!strcmp(name, "qinq_strip")) {
> > + if (on)
> > + local |= DEV_RX_OFFLOAD_QINQ_STRIP;
> > + else
> > + local &= ~DEV_RX_OFFLOAD_QINQ_STRIP;
> > + } else if (!strcmp(name, "outer_ipv4_cksum")) {
> > + if (on)
> > + local |= DEV_RX_OFFLOAD_OUTER_IPV4_CKSUM;
> > + else
> > + local &= ~DEV_RX_OFFLOAD_OUTER_IPV4_CKSUM;
> > + } else if (!strcmp(name, "macsec_strip")) {
> > + if (on)
> > + local |= DEV_RX_OFFLOAD_MACSEC_STRIP;
> > + else
> > + local &= ~DEV_RX_OFFLOAD_MACSEC_STRIP;
> > + } else if (!strcmp(name, "header_split")) {
> > + if (on)
> > + local |= DEV_RX_OFFLOAD_HEADER_SPLIT;
> > + else
> > + local &= ~DEV_RX_OFFLOAD_HEADER_SPLIT;
> > + } else if (!strcmp(name, "vlan_filter")) {
> > + if (on)
> > + local |= DEV_RX_OFFLOAD_VLAN_FILTER;
> > + else
> > + local &= ~DEV_RX_OFFLOAD_VLAN_FILTER;
> > + } else if (!strcmp(name, "vlan_extend")) {
> > + if (on)
> > + local |= DEV_RX_OFFLOAD_VLAN_EXTEND;
> > + else
> > + local &= ~DEV_RX_OFFLOAD_VLAN_EXTEND;
> > + } else if (!strcmp(name, "jumbo_frame")) {
> > + if (on)
> > + local |= DEV_RX_OFFLOAD_JUMBO_FRAME;
> > + else
> > + local &= ~DEV_RX_OFFLOAD_JUMBO_FRAME;
> > + } else if (!strcmp(name, "crc_strip")) {
> > + if (on)
> > + local |= DEV_RX_OFFLOAD_CRC_STRIP;
> > + else
> > + local &= ~DEV_RX_OFFLOAD_CRC_STRIP;
> > + } else if (!strcmp(name, "scatter")) {
> > + if (on)
> > + local |= DEV_RX_OFFLOAD_SCATTER;
> > + else
> > + local &= ~DEV_RX_OFFLOAD_SCATTER;
> > + } else if (!strcmp(name, "timestamp")) {
> > + if (on)
> > + local |= DEV_RX_OFFLOAD_TIMESTAMP;
> > + else
> > + local &= ~DEV_RX_OFFLOAD_TIMESTAMP;
> > + } else if (!strcmp(name, "security")) {
> > + if (on)
> > + local |= DEV_RX_OFFLOAD_SECURITY;
> > + else
> > + local &= ~DEV_RX_OFFLOAD_SECURITY;
> > + } else
> > + return -1;
>
> I was going to suggest rte_eth_dev_rx_offload_name(), strcasecmp() and loop.
+1
> It is possible to iterate on all U64 bits or add a way to get all
> offload bits from rte_ethdev.
> However it still requires list of offloads in the help below. So, it is
> not easy to get rig of
> offloads list completely. Just an idea. Up to you.
>
> <snip>
>
> > +cmdline_parse_inst_t cmd_config_per_port_rx_offload = {
> > + .f = cmd_config_per_port_rx_offload_parsed,
> > + .data = NULL,
> > + .help_str = "rx_offload enable|disable per_port vlan_strip|ipv4_cksum|"
> > + "udp_cksum|tcp_cksum|tcp_lro|qinq_strip|outer_ipv4_cksum|"
> > + "macsec_strip|header_split|vlan_filter|vlan_extend|"
> > + "jumbo_frame|crc_strip|scatter|timestamp|security "
> > + "<port_id>",
> >
>
> <snip>
<...>
> +static int
> +config_rx_offload(const char *name, uint64_t *offload, int on)
> +{
> + uint64_t local = *offload;
> +
> + if (!strcmp(name, "vlan_strip")) {
> + if (on)
> + local |= DEV_RX_OFFLOAD_VLAN_STRIP;
> + else
> + local &= ~DEV_RX_OFFLOAD_VLAN_STRIP;
Would it decrease the lines if move the "on" checking to the end of this function, just set the
BIT which you want to set or mask here?
<...>
> +static void
> +cmd_config_per_port_rx_offload_parsed(void *parsed_result,
> + __attribute__((unused)) struct cmdline *cl,
> + __attribute__((unused)) void *data)
> +{
> + struct cmd_config_per_port_rx_offload_result *res = parsed_result;
> + portid_t port_id = res->port_id;
> + struct rte_port *port = &ports[port_id];
> + int on;
> +
> + if (port->port_status != RTE_PORT_STOPPED) {
> + printf("Error: Can't config offload when Port %d "
> + "is not stopped\n", port_id);
> + return;
> + }
> +
> + if (!strcmp(res->en_dis, "enable"))
> + on = 1;
> + else if (!strcmp(res->en_dis, "disable"))
> + on = 0;
> + else {
> + printf("Unknown parameter: %s\n", res->en_dis);
> + return;
The en_dis is already defined as "enable#disable", so the else is useless here.
<...>
> +static void
> +cmd_config_per_queue_rx_offload_parsed(void *parsed_result,
> + __attribute__((unused)) struct cmdline *cl,
> + __attribute__((unused)) void *data)
> +{
> + struct cmd_config_per_queue_rx_offload_result *res = parsed_result;
> + struct rte_eth_dev_info dev_info;
> + portid_t port_id = res->port_id;
> + uint16_t queue_id = res->queue_id;
> + struct rte_port *port = &ports[port_id];
> + int on;
> +
> + if (port->port_status != RTE_PORT_STOPPED) {
> + printf("Error: Can't config offload when Port %d "
> + "is not stopped\n", port_id);
> + return;
> + }
> +
> + if (!strcmp(res->en_dis, "enable"))
> + on = 1;
> + else if (!strcmp(res->en_dis, "disable"))
> + on = 0;
> + else {
> + printf("Unknown parameter: %s\n", res->en_dis);
> + return;
Same comments as above.
<...>
> @@ -707,6 +708,17 @@ init_config(void)
> }
> }
>
> + port->rx_offloads = rte_zmalloc("testpmd: port->rx_offloads",
> + sizeof(port->rx_offloads[0]) *
> + port->dev_info.max_rx_queues,
> + sizeof(port->rx_offloads[0]));
When will you free it?
> + if (port->rx_offloads == NULL)
> + rte_exit(EXIT_FAILURE,
> + "rte_zmalloc(%d port->rx_offload[0]) "
> + "failed\n", port->dev_info.max_rx_queues);
> + for (k = 0; k < port->dev_info.max_rx_queues; k++)
> + port->rx_offloads[k] = port->dev_conf.rxmode.offloads;
> +
In the get configure command, the port->rx_offloads is printed.
Here you init it to be port configuration, when will you update it?
Thanks to Konstantin and Andrew for your feedback.
I will use __builtin_ctzll() and __builtin_clzll() in my v3 patch set.
I also will use strcasecmp and loop to do config_rx_offload() and
config_tx_offload() following your guidance.
> -----Original Message-----
> From: Ananyev, Konstantin
> Sent: Tuesday, March 13, 2018 5:31 PM
> To: Andrew Rybchenko <arybchenko@solarflare.com>; Dai, Wei
> <wei.dai@intel.com>; Lu, Wenzhuo <wenzhuo.lu@intel.com>; Wu, Jingjing
> <jingjing.wu@intel.com>
> Cc: dev@dpdk.org
> Subject: RE: [dpdk-dev] [PATCH v2 1/2] app/testpmd: add commands to test
> new Rx offload API
>
>
>
> > -----Original Message-----
> > From: dev [mailto:dev-bounces@dpdk.org] On Behalf Of Andrew
> Rybchenko
> > Sent: Tuesday, March 13, 2018 7:21 AM
> > To: Dai, Wei <wei.dai@intel.com>; Lu, Wenzhuo
> <wenzhuo.lu@intel.com>;
> > Wu, Jingjing <jingjing.wu@intel.com>
> > Cc: dev@dpdk.org
> > Subject: Re: [dpdk-dev] [PATCH v2 1/2] app/testpmd: add commands to
> > test new Rx offload API
> >
> > On 03/13/2018 09:42 AM, Wei Dai wrote:
> > > Add following testpmd run-time commands to support test of new Rx
> > > offload API:
> > > rx_offload get capability <port_id>
> > > rx_offload get configuration <port_id> rx_offload enable|disable
> > > per_port <offload> <port_id> rx_offload enable|disable per_queue
> > > <offload> <port_id> <queue_id>
> > >
> > > Above last 2 commands should be run when the port is stopped.
> > > And <offload> can be one of "vlan_strip", "ipv4_cksum", ...
> > >
> > > Signed-off-by: Wei Dai <wei.dai@intel.com>
> > > ---
> > > + for (single_offload = DEV_RX_OFFLOAD_VLAN_STRIP;
> > > + single_offload <= DEV_RX_OFFLOAD_SECURITY;
> >
> > It requires attention when a new offload is added.
> > Please, consider to use something like __builtin_ffsll(), print name
> > of the found bit, clear it and retry. It allows to avoid boundaries
> > specification.
> >
> > <snip>
> >
> >
> > I was going to suggest rte_eth_dev_rx_offload_name(), strcasecmp() and
> loop.
>
> +1
>
> > It is possible to iterate on all U64 bits or add a way to get all
> > offload bits from rte_ethdev.
> > However it still requires list of offloads in the help below. So, it
> > is not easy to get rig of offloads list completely. Just an idea. Up
> > to you.
> >
> > <snip>
> >
> -----Original Message-----
> From: Wu, Jingjing
> Sent: Thursday, March 15, 2018 3:41 AM
> To: Dai, Wei <wei.dai@intel.com>; Lu, Wenzhuo <wenzhuo.lu@intel.com>
> Cc: dev@dpdk.org
> Subject: RE: [PATCH v2 1/2] app/testpmd: add commands to test new Rx
> offload API
>
> <...>
>
> > +static int
> > +config_rx_offload(const char *name, uint64_t *offload, int on) {
> > + uint64_t local = *offload;
> > +
> > + if (!strcmp(name, "vlan_strip")) {
> > + if (on)
> > + local |= DEV_RX_OFFLOAD_VLAN_STRIP;
> > + else
> > + local &= ~DEV_RX_OFFLOAD_VLAN_STRIP;
> Would it decrease the lines if move the "on" checking to the end of this
> function, just set the BIT which you want to set or mask here?
Thanks, will follow your suggestion in v3 patch set.
> <...>
>
> > +static void
> > +cmd_config_per_port_rx_offload_parsed(void *parsed_result,
> > + __attribute__((unused)) struct cmdline *cl,
> > + __attribute__((unused)) void *data) {
> > + struct cmd_config_per_port_rx_offload_result *res = parsed_result;
> > + portid_t port_id = res->port_id;
> > + struct rte_port *port = &ports[port_id];
> > + int on;
> > +
> > + if (port->port_status != RTE_PORT_STOPPED) {
> > + printf("Error: Can't config offload when Port %d "
> > + "is not stopped\n", port_id);
> > + return;
> > + }
> > +
> > + if (!strcmp(res->en_dis, "enable"))
> > + on = 1;
> > + else if (!strcmp(res->en_dis, "disable"))
> > + on = 0;
> > + else {
> > + printf("Unknown parameter: %s\n", res->en_dis);
> > + return;
> The en_dis is already defined as "enable#disable", so the else is useless here.
Thanks, the second if and second else can be removed and will change in v3 patch set.
>
> <...>
>
> > +static void
> > +cmd_config_per_queue_rx_offload_parsed(void *parsed_result,
> > + __attribute__((unused)) struct cmdline *cl,
> > + __attribute__((unused)) void *data) {
> > + struct cmd_config_per_queue_rx_offload_result *res = parsed_result;
> > + struct rte_eth_dev_info dev_info;
> > + portid_t port_id = res->port_id;
> > + uint16_t queue_id = res->queue_id;
> > + struct rte_port *port = &ports[port_id];
> > + int on;
> > +
> > + if (port->port_status != RTE_PORT_STOPPED) {
> > + printf("Error: Can't config offload when Port %d "
> > + "is not stopped\n", port_id);
> > + return;
> > + }
> > +
> > + if (!strcmp(res->en_dis, "enable"))
> > + on = 1;
> > + else if (!strcmp(res->en_dis, "disable"))
> > + on = 0;
> > + else {
> > + printf("Unknown parameter: %s\n", res->en_dis);
> > + return;
> Same comments as above.
> <...>
>
> > @@ -707,6 +708,17 @@ init_config(void)
> > }
> > }
> >
> > + port->rx_offloads = rte_zmalloc("testpmd: port->rx_offloads",
> > + sizeof(port->rx_offloads[0]) *
> > + port->dev_info.max_rx_queues,
> > + sizeof(port->rx_offloads[0]));
>
> When will you free it?
I will free port->rx_offloads and port->tx_offloads when testpmd is existed.
>
> > + if (port->rx_offloads == NULL)
> > + rte_exit(EXIT_FAILURE,
> > + "rte_zmalloc(%d port->rx_offload[0]) "
> > + "failed\n", port->dev_info.max_rx_queues);
> > + for (k = 0; k < port->dev_info.max_rx_queues; k++)
> > + port->rx_offloads[k] = port->dev_conf.rxmode.offloads;
> > +
> In the get configure command, the port->rx_offloads is printed.
> Here you init it to be port configuration, when will you update it?
This init_config() is only run once in the initialization stage in the main().
Port->rx_offloads[] can be changed using new testpmd command in this patch set.
Testpmd> rx_offload enable/disable per_queue <offload> <port_id> <queue_id>
And will be applied when the command port start <port_id> is run.
@@ -15996,6 +15996,426 @@ cmdline_parse_inst_t cmd_ptype_mapping_update = {
},
};
+/* Get Rx offloads capability */
+struct cmd_rx_offload_get_capa_result {
+ cmdline_fixed_string_t rx_offload;
+ cmdline_fixed_string_t get;
+ cmdline_fixed_string_t capability;
+ portid_t port_id;
+};
+
+cmdline_parse_token_string_t cmd_rx_offload_get_capa_rx_offload =
+ TOKEN_STRING_INITIALIZER
+ (struct cmd_rx_offload_get_capa_result,
+ rx_offload, "rx_offload");
+cmdline_parse_token_string_t cmd_rx_offload_get_capa_get =
+ TOKEN_STRING_INITIALIZER
+ (struct cmd_rx_offload_get_capa_result,
+ get, "get");
+cmdline_parse_token_string_t cmd_rx_offload_get_capa_capability =
+ TOKEN_STRING_INITIALIZER
+ (struct cmd_rx_offload_get_capa_result,
+ capability, "capability");
+cmdline_parse_token_num_t cmd_rx_offload_get_capa_port_id =
+ TOKEN_NUM_INITIALIZER
+ (struct cmd_rx_offload_get_capa_result,
+ port_id, UINT16);
+
+static void
+print_rx_offloads(uint64_t offloads)
+{
+ uint64_t single_offload;
+
+ for (single_offload = DEV_RX_OFFLOAD_VLAN_STRIP;
+ single_offload <= DEV_RX_OFFLOAD_SECURITY;
+ single_offload <<= 1)
+ if (offloads & single_offload)
+ printf(" %s",
+ rte_eth_dev_rx_offload_name(single_offload));
+}
+
+static void
+cmd_rx_offload_get_capa_parsed(
+ void *parsed_result,
+ __attribute__((unused)) struct cmdline *cl,
+ __attribute__((unused)) void *data)
+{
+ struct cmd_rx_offload_get_capa_result *res = parsed_result;
+ struct rte_eth_dev_info dev_info;
+ portid_t port_id = res->port_id;
+ uint64_t queue_offloads;
+ uint64_t port_offloads;
+
+ rte_eth_dev_info_get(port_id, &dev_info);
+ queue_offloads = dev_info.rx_queue_offload_capa;
+ port_offloads = dev_info.rx_offload_capa ^ queue_offloads;
+
+ printf("Rx Offloading Capabilities of port %d :\n", port_id);
+ printf(" Per Queue :");
+ print_rx_offloads(queue_offloads);
+
+ printf("\n");
+ printf(" Per Port :");
+ print_rx_offloads(port_offloads);
+ printf("\n\n");
+}
+
+cmdline_parse_inst_t cmd_rx_offload_get_capa = {
+ .f = cmd_rx_offload_get_capa_parsed,
+ .data = NULL,
+ .help_str = "rx_offload get capability <port_id>",
+ .tokens = {
+ (void *)&cmd_rx_offload_get_capa_rx_offload,
+ (void *)&cmd_rx_offload_get_capa_get,
+ (void *)&cmd_rx_offload_get_capa_capability,
+ (void *)&cmd_rx_offload_get_capa_port_id,
+ NULL,
+ }
+};
+
+/* Get Rx offloads configuration */
+struct cmd_rx_offload_get_configuration_result {
+ cmdline_fixed_string_t rx_offload;
+ cmdline_fixed_string_t get;
+ cmdline_fixed_string_t configuration;
+ portid_t port_id;
+};
+
+cmdline_parse_token_string_t cmd_rx_offload_get_configuration_rx_offload =
+ TOKEN_STRING_INITIALIZER
+ (struct cmd_rx_offload_get_configuration_result,
+ rx_offload, "rx_offload");
+cmdline_parse_token_string_t cmd_rx_offload_get_configuration_get =
+ TOKEN_STRING_INITIALIZER
+ (struct cmd_rx_offload_get_configuration_result,
+ get, "get");
+cmdline_parse_token_string_t cmd_rx_offload_get_configuration_configuration =
+ TOKEN_STRING_INITIALIZER
+ (struct cmd_rx_offload_get_configuration_result,
+ configuration, "configuration");
+cmdline_parse_token_num_t cmd_rx_offload_get_configuration_port_id =
+ TOKEN_NUM_INITIALIZER
+ (struct cmd_rx_offload_get_configuration_result,
+ port_id, UINT16);
+
+static void
+cmd_rx_offload_get_configuration_parsed(
+ void *parsed_result,
+ __attribute__((unused)) struct cmdline *cl,
+ __attribute__((unused)) void *data)
+{
+ struct cmd_rx_offload_get_configuration_result *res = parsed_result;
+ struct rte_eth_dev_info dev_info;
+ portid_t port_id = res->port_id;
+ struct rte_port *port = &ports[port_id];
+ uint64_t port_offloads;
+ uint64_t queue_offloads;
+ uint16_t nb_rx_queues;
+ int q;
+
+ printf("Rx Offloading Configuration of port %d :\n", port_id);
+
+ port_offloads = port->dev_conf.rxmode.offloads;
+ printf(" Port :");
+ print_rx_offloads(port_offloads);
+ printf("\n");
+
+ rte_eth_dev_info_get(port_id, &dev_info);
+ nb_rx_queues = dev_info.nb_rx_queues;
+ for (q = 0; q < nb_rx_queues; q++) {
+ queue_offloads = port->rx_offloads[q];
+ printf(" Queue[%2d] :", q);
+ print_rx_offloads(queue_offloads);
+ printf("\n");
+ }
+ printf("\n");
+}
+
+cmdline_parse_inst_t cmd_rx_offload_get_configuration = {
+ .f = cmd_rx_offload_get_configuration_parsed,
+ .data = NULL,
+ .help_str = "rx_offload get configuration <port_id>",
+ .tokens = {
+ (void *)&cmd_rx_offload_get_configuration_rx_offload,
+ (void *)&cmd_rx_offload_get_configuration_get,
+ (void *)&cmd_rx_offload_get_configuration_configuration,
+ (void *)&cmd_rx_offload_get_configuration_port_id,
+ NULL,
+ }
+};
+
+/* Enable/Disable a per port offloading */
+struct cmd_config_per_port_rx_offload_result {
+ cmdline_fixed_string_t rx_offload;
+ cmdline_fixed_string_t en_dis;
+ cmdline_fixed_string_t per_port;
+ cmdline_fixed_string_t offload;
+ portid_t port_id;
+};
+
+cmdline_parse_token_string_t cmd_config_per_port_rx_offload_result_rx_offload =
+ TOKEN_STRING_INITIALIZER
+ (struct cmd_config_per_port_rx_offload_result,
+ rx_offload, "rx_offload");
+cmdline_parse_token_string_t cmd_config_per_port_rx_offload_result_en_dis =
+ TOKEN_STRING_INITIALIZER
+ (struct cmd_config_per_port_rx_offload_result,
+ en_dis, "enable#disable");
+cmdline_parse_token_string_t cmd_config_per_port_rx_offload_result_per_port =
+ TOKEN_STRING_INITIALIZER
+ (struct cmd_config_per_port_rx_offload_result,
+ per_port, "per_port");
+cmdline_parse_token_string_t cmd_config_per_port_rx_offload_result_offload =
+ TOKEN_STRING_INITIALIZER
+ (struct cmd_config_per_port_rx_offload_result,
+ offload, "vlan_strip#ipv4_cksum#udp_cksum#tcp_cksum#tcp_lro#"
+ "qinq_strip#outer_ipv4_cksum#macsec_strip#"
+ "header_split#vlan_filter#vlan_extend#jumbo_frame#"
+ "crc_strip#scatter#timestamp#security");
+cmdline_parse_token_num_t cmd_config_per_port_rx_offload_result_port_id =
+ TOKEN_NUM_INITIALIZER
+ (struct cmd_config_per_port_rx_offload_result,
+ port_id, UINT16);
+
+static int
+config_rx_offload(const char *name, uint64_t *offload, int on)
+{
+ uint64_t local = *offload;
+
+ if (!strcmp(name, "vlan_strip")) {
+ if (on)
+ local |= DEV_RX_OFFLOAD_VLAN_STRIP;
+ else
+ local &= ~DEV_RX_OFFLOAD_VLAN_STRIP;
+ } else if (!strcmp(name, "ipv4_cksum")) {
+ if (on)
+ local |= DEV_RX_OFFLOAD_IPV4_CKSUM;
+ else
+ local &= ~DEV_RX_OFFLOAD_IPV4_CKSUM;
+ } else if (!strcmp(name, "udp_cksum")) {
+ if (on)
+ local |= DEV_RX_OFFLOAD_UDP_CKSUM;
+ else
+ local &= ~DEV_RX_OFFLOAD_UDP_CKSUM;
+ } else if (!strcmp(name, "tcp_cksum")) {
+ if (on)
+ local |= DEV_RX_OFFLOAD_TCP_CKSUM;
+ else
+ local &= ~DEV_RX_OFFLOAD_TCP_CKSUM;
+ } else if (!strcmp(name, "tcp_lro")) {
+ if (on)
+ local |= DEV_RX_OFFLOAD_TCP_LRO;
+ else
+ local &= ~DEV_RX_OFFLOAD_TCP_LRO;
+ } else if (!strcmp(name, "qinq_strip")) {
+ if (on)
+ local |= DEV_RX_OFFLOAD_QINQ_STRIP;
+ else
+ local &= ~DEV_RX_OFFLOAD_QINQ_STRIP;
+ } else if (!strcmp(name, "outer_ipv4_cksum")) {
+ if (on)
+ local |= DEV_RX_OFFLOAD_OUTER_IPV4_CKSUM;
+ else
+ local &= ~DEV_RX_OFFLOAD_OUTER_IPV4_CKSUM;
+ } else if (!strcmp(name, "macsec_strip")) {
+ if (on)
+ local |= DEV_RX_OFFLOAD_MACSEC_STRIP;
+ else
+ local &= ~DEV_RX_OFFLOAD_MACSEC_STRIP;
+ } else if (!strcmp(name, "header_split")) {
+ if (on)
+ local |= DEV_RX_OFFLOAD_HEADER_SPLIT;
+ else
+ local &= ~DEV_RX_OFFLOAD_HEADER_SPLIT;
+ } else if (!strcmp(name, "vlan_filter")) {
+ if (on)
+ local |= DEV_RX_OFFLOAD_VLAN_FILTER;
+ else
+ local &= ~DEV_RX_OFFLOAD_VLAN_FILTER;
+ } else if (!strcmp(name, "vlan_extend")) {
+ if (on)
+ local |= DEV_RX_OFFLOAD_VLAN_EXTEND;
+ else
+ local &= ~DEV_RX_OFFLOAD_VLAN_EXTEND;
+ } else if (!strcmp(name, "jumbo_frame")) {
+ if (on)
+ local |= DEV_RX_OFFLOAD_JUMBO_FRAME;
+ else
+ local &= ~DEV_RX_OFFLOAD_JUMBO_FRAME;
+ } else if (!strcmp(name, "crc_strip")) {
+ if (on)
+ local |= DEV_RX_OFFLOAD_CRC_STRIP;
+ else
+ local &= ~DEV_RX_OFFLOAD_CRC_STRIP;
+ } else if (!strcmp(name, "scatter")) {
+ if (on)
+ local |= DEV_RX_OFFLOAD_SCATTER;
+ else
+ local &= ~DEV_RX_OFFLOAD_SCATTER;
+ } else if (!strcmp(name, "timestamp")) {
+ if (on)
+ local |= DEV_RX_OFFLOAD_TIMESTAMP;
+ else
+ local &= ~DEV_RX_OFFLOAD_TIMESTAMP;
+ } else if (!strcmp(name, "security")) {
+ if (on)
+ local |= DEV_RX_OFFLOAD_SECURITY;
+ else
+ local &= ~DEV_RX_OFFLOAD_SECURITY;
+ } else
+ return -1;
+
+ *offload = local;
+ return 0;
+}
+
+static void
+cmd_config_per_port_rx_offload_parsed(void *parsed_result,
+ __attribute__((unused)) struct cmdline *cl,
+ __attribute__((unused)) void *data)
+{
+ struct cmd_config_per_port_rx_offload_result *res = parsed_result;
+ portid_t port_id = res->port_id;
+ struct rte_port *port = &ports[port_id];
+ int on;
+
+ if (port->port_status != RTE_PORT_STOPPED) {
+ printf("Error: Can't config offload when Port %d "
+ "is not stopped\n", port_id);
+ return;
+ }
+
+ if (!strcmp(res->en_dis, "enable"))
+ on = 1;
+ else if (!strcmp(res->en_dis, "disable"))
+ on = 0;
+ else {
+ printf("Unknown parameter: %s\n", res->en_dis);
+ return;
+ }
+
+ if (config_rx_offload(res->offload,
+ &port->dev_conf.rxmode.offloads, on)) {
+ printf("Unknown offload name: %s", res->offload);
+ }
+}
+
+cmdline_parse_inst_t cmd_config_per_port_rx_offload = {
+ .f = cmd_config_per_port_rx_offload_parsed,
+ .data = NULL,
+ .help_str = "rx_offload enable|disable per_port vlan_strip|ipv4_cksum|"
+ "udp_cksum|tcp_cksum|tcp_lro|qinq_strip|outer_ipv4_cksum|"
+ "macsec_strip|header_split|vlan_filter|vlan_extend|"
+ "jumbo_frame|crc_strip|scatter|timestamp|security "
+ "<port_id>",
+ .tokens = {
+ (void *)&cmd_config_per_port_rx_offload_result_rx_offload,
+ (void *)&cmd_config_per_port_rx_offload_result_en_dis,
+ (void *)&cmd_config_per_port_rx_offload_result_per_port,
+ (void *)&cmd_config_per_port_rx_offload_result_offload,
+ (void *)&cmd_config_per_port_rx_offload_result_port_id,
+ NULL,
+ }
+};
+
+/* Enable/Disable a per queue offloading */
+struct cmd_config_per_queue_rx_offload_result {
+ cmdline_fixed_string_t rx_offload;
+ cmdline_fixed_string_t en_dis;
+ cmdline_fixed_string_t per_queue;
+ cmdline_fixed_string_t offload;
+ portid_t port_id;
+ uint16_t queue_id;
+};
+
+cmdline_parse_token_string_t cmd_config_per_queue_rx_offload_result_rx_offload =
+ TOKEN_STRING_INITIALIZER
+ (struct cmd_config_per_queue_rx_offload_result,
+ rx_offload, "rx_offload");
+cmdline_parse_token_string_t cmd_config_per_queue_rx_offload_result_en_dis =
+ TOKEN_STRING_INITIALIZER
+ (struct cmd_config_per_queue_rx_offload_result,
+ en_dis, "enable#disable");
+cmdline_parse_token_string_t cmd_config_per_queue_rx_offload_result_per_queue =
+ TOKEN_STRING_INITIALIZER
+ (struct cmd_config_per_queue_rx_offload_result,
+ per_queue, "per_queue");
+cmdline_parse_token_string_t cmd_config_per_queue_rx_offload_result_offload =
+ TOKEN_STRING_INITIALIZER
+ (struct cmd_config_per_queue_rx_offload_result,
+ offload, "vlan_strip#ipv4_cksum#udp_cksum#tcp_cksum#tcp_lro#"
+ "qinq_strip#outer_ipv4_cksum#macsec_strip#"
+ "header_split#vlan_filter#vlan_extend#jumbo_frame#"
+ "crc_strip#scatter#timestamp#security");
+cmdline_parse_token_num_t cmd_config_per_queue_rx_offload_result_port_id =
+ TOKEN_NUM_INITIALIZER
+ (struct cmd_config_per_queue_rx_offload_result,
+ port_id, UINT16);
+cmdline_parse_token_num_t cmd_config_per_queue_rx_offload_result_queue_id =
+ TOKEN_NUM_INITIALIZER
+ (struct cmd_config_per_queue_rx_offload_result,
+ queue_id, UINT16);
+
+
+static void
+cmd_config_per_queue_rx_offload_parsed(void *parsed_result,
+ __attribute__((unused)) struct cmdline *cl,
+ __attribute__((unused)) void *data)
+{
+ struct cmd_config_per_queue_rx_offload_result *res = parsed_result;
+ struct rte_eth_dev_info dev_info;
+ portid_t port_id = res->port_id;
+ uint16_t queue_id = res->queue_id;
+ struct rte_port *port = &ports[port_id];
+ int on;
+
+ if (port->port_status != RTE_PORT_STOPPED) {
+ printf("Error: Can't config offload when Port %d "
+ "is not stopped\n", port_id);
+ return;
+ }
+
+ if (!strcmp(res->en_dis, "enable"))
+ on = 1;
+ else if (!strcmp(res->en_dis, "disable"))
+ on = 0;
+ else {
+ printf("Unknown parameter: %s\n", res->en_dis);
+ return;
+ }
+
+ rte_eth_dev_info_get(port_id, &dev_info);
+ if (queue_id >= dev_info.nb_rx_queues) {
+ printf("Error: input queue_id should be 0 ... "
+ "%d", dev_info.nb_rx_queues - 1);
+ return;
+ }
+
+ if (config_rx_offload(res->offload,
+ &port->rx_offloads[queue_id], on)) {
+ printf("Unknown offload name: %s", res->offload);
+ }
+}
+
+cmdline_parse_inst_t cmd_config_per_queue_rx_offload = {
+ .f = cmd_config_per_queue_rx_offload_parsed,
+ .data = NULL,
+ .help_str = "rx_offload enable|disable per_queue vlan_strip|ipv4_cksum|"
+ "udp_cksum|tcp_cksum|tcp_lro|qinq_strip|outer_ipv4_cksum|"
+ "macsec_strip|header_split|vlan_filter|vlan_extend|"
+ "jumbo_frame|crc_strip|scatter|timestamp|security "
+ "<port_id> <queue_id>",
+ .tokens = {
+ (void *)&cmd_config_per_queue_rx_offload_result_rx_offload,
+ (void *)&cmd_config_per_queue_rx_offload_result_en_dis,
+ (void *)&cmd_config_per_queue_rx_offload_result_per_queue,
+ (void *)&cmd_config_per_queue_rx_offload_result_offload,
+ (void *)&cmd_config_per_queue_rx_offload_result_port_id,
+ (void *)&cmd_config_per_queue_rx_offload_result_queue_id,
+ NULL,
+ }
+};
+
/* Common result structure for file commands */
struct cmd_cmdfile_result {
cmdline_fixed_string_t load;
@@ -16272,6 +16692,10 @@ cmdline_parse_ctx_t main_ctx[] = {
(cmdline_parse_inst_t *)&cmd_del_port_tm_node,
(cmdline_parse_inst_t *)&cmd_set_port_tm_node_parent,
(cmdline_parse_inst_t *)&cmd_port_tm_hierarchy_commit,
+ (cmdline_parse_inst_t *)&cmd_rx_offload_get_capa,
+ (cmdline_parse_inst_t *)&cmd_rx_offload_get_configuration,
+ (cmdline_parse_inst_t *)&cmd_config_per_port_rx_offload,
+ (cmdline_parse_inst_t *)&cmd_config_per_queue_rx_offload,
NULL,
};
@@ -656,6 +656,7 @@ init_config(void)
uint8_t port_per_socket[RTE_MAX_NUMA_NODES];
struct rte_gro_param gro_param;
uint32_t gso_types;
+ int k;
memset(port_per_socket,0,RTE_MAX_NUMA_NODES);
@@ -707,6 +708,17 @@ init_config(void)
}
}
+ port->rx_offloads = rte_zmalloc("testpmd: port->rx_offloads",
+ sizeof(port->rx_offloads[0]) *
+ port->dev_info.max_rx_queues,
+ sizeof(port->rx_offloads[0]));
+ if (port->rx_offloads == NULL)
+ rte_exit(EXIT_FAILURE,
+ "rte_zmalloc(%d port->rx_offload[0]) "
+ "failed\n", port->dev_info.max_rx_queues);
+ for (k = 0; k < port->dev_info.max_rx_queues; k++)
+ port->rx_offloads[k] = port->dev_conf.rxmode.offloads;
+
/* set flag to initialize port/queue */
port->need_reconfig = 1;
port->need_reconfig_queues = 1;
@@ -1615,10 +1627,9 @@ start_port(portid_t pid)
port->need_reconfig_queues = 1;
return -1;
}
- /* Apply Rx offloads configuration */
- port->rx_conf.offloads = port->dev_conf.rxmode.offloads;
/* setup rx queues */
for (qi = 0; qi < nb_rxq; qi++) {
+ port->rx_conf.offloads = port->rx_offloads[qi];
if ((numa_support) &&
(rxring_numa[pi] != NUMA_NO_CONFIG)) {
struct rte_mempool * mp =
@@ -183,6 +183,7 @@ struct rte_port {
uint8_t dcb_flag; /**< enable dcb */
struct rte_eth_rxconf rx_conf; /**< rx configuration */
struct rte_eth_txconf tx_conf; /**< tx configuration */
+ uint64_t *rx_offloads; /**< offloads of each Rx queue */
struct ether_addr *mc_addr_pool; /**< pool of multicast addrs */
uint32_t mc_addr_nb; /**< nb. of addr. in mc_addr_pool */
uint8_t slave_flag; /**< bonding slave port */