From patchwork Sun May 29 14:28:50 2022 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Muhammad Jawad Hussain X-Patchwork-Id: 112021 X-Patchwork-Delegate: thomas@monjalon.net Return-Path: X-Original-To: patchwork@inbox.dpdk.org Delivered-To: patchwork@inbox.dpdk.org Received: from mails.dpdk.org (mails.dpdk.org [217.70.189.124]) by inbox.dpdk.org (Postfix) with ESMTP id 1C4F5A0544; Sun, 29 May 2022 16:29:25 +0200 (CEST) Received: from [217.70.189.124] (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id 04DDF40A82; Sun, 29 May 2022 16:29:25 +0200 (CEST) Received: from mail-wr1-f50.google.com (mail-wr1-f50.google.com [209.85.221.50]) by mails.dpdk.org (Postfix) with ESMTP id 3B28D40041 for ; Sun, 29 May 2022 16:29:24 +0200 (CEST) Received: by mail-wr1-f50.google.com with SMTP id p10so11568784wrg.12 for ; Sun, 29 May 2022 07:29:24 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=emumba-com.20210112.gappssmtp.com; s=20210112; h=from:to:cc:subject:date:message-id:mime-version :content-transfer-encoding; bh=TT0xOPJp90zR8oJegQcRS4Nue5nd48T2w/GNxZS31Y4=; b=4VZj6NyjcwvUcoGhaMVyIYjUdd1dK1dlcfAF2EN514YFxXBXBziDitL3Qmbcptaw8n IqvmZOcTDxQrw65pboHWmE/dJ3wgvwP4fZpBCOI0zAwOrt7tn4Db4Rtymt8oBlhKiEc4 ZjKbh/q1SQ6eWUFvz910ZuFLXoInPttMhO3XmrV+cjKumY1MilpKp63mXdx0AsOx4M7U foDGajgzR11Zl2QNsDlQDsZ296AdHsGP1Hm1Jjf4LbAOr2fQo7JxpVLmX5G/HYbZKFmr jqzjs7WQHnOgVjrLJ2cTLY3ZF8BDPx29EuGGzwc2A+j1rZsYzu6C/1LbDjEF++unc5lR S0XQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=x-gm-message-state:from:to:cc:subject:date:message-id:mime-version :content-transfer-encoding; bh=TT0xOPJp90zR8oJegQcRS4Nue5nd48T2w/GNxZS31Y4=; b=HRTQXWGgLHC79dRYQLh2+uuBy43nY48ryuimtGecZHFrirysBtyCYYEA1QT2opb2b3 FtcncAgj08gC5/lu/NCf5kpLlxfAVsjlIykrmpiTqnrV0F/UrJ/3AZ0ETlUmj+g8oM1L sxfSwU46JFZYkxmesDIl4DfaY+xuP5Q5y6CTMNEa4L8C/c/LpYE49WfB1EiJ5Bu90wxE rinD4d1whPbICwLNoA7irpuJBDCmX6lNOwL2f/cDEX9xlR92zXKEV8gU6OicGhK5l/KX GoFd8n71X4GFrO0MvMX+PM/RW9dj+XmHB8anLRRTzKxk3CsAfDr22HjcxgSjcOJ0TqHs Ry3A== X-Gm-Message-State: AOAM531ZmRC1PWo48tKmz/O6EV7yLzLvfjxygbb4oDDEECB3kSRPZ6Ti JSfX88LsclatapfIpHIQSeENdg/X/y25xQ== X-Google-Smtp-Source: ABdhPJxqURi7Y6TLNmfwqfYAA4mZTJr4MDxToL1k2YJYBys/keU7k0ltLtUD+pAamE1+yIsvQnEVAQ== X-Received: by 2002:a5d:64ab:0:b0:20f:ef37:a9d1 with SMTP id m11-20020a5d64ab000000b0020fef37a9d1mr22008163wrp.197.1653834563659; Sun, 29 May 2022 07:29:23 -0700 (PDT) Received: from localhost.localdomain ([39.32.39.45]) by smtp.gmail.com with ESMTPSA id p2-20020a5d59a2000000b0020c7ec0fdf4sm7872862wrr.117.2022.05.29.07.29.22 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Sun, 29 May 2022 07:29:23 -0700 (PDT) From: Muhammad Jawad Hussain To: dev@dpdk.org Cc: jawad.hussain@emumba.com Subject: [PATCH v2] examples/ethtool: update rxmode to increase functionality Date: Sun, 29 May 2022 19:28:50 +0500 Message-Id: <20220529142850.76501-1-jawad.hussain@emumba.com> X-Mailer: git-send-email 2.25.1 MIME-Version: 1.0 X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org previously rxmode functionality did not allow the user to choose between different vfs nor did it allow the user to choose rxmode settings by default it was set to vf = 0, rxmode = AUPE and the on/off state toggled without letting the user know what state it is in also there were no error messages Changes: - added flag for vf id - added flags for AUPE|ROPE|BAM|MPE rxmodes - added flag for on/off - added error messages - added info messages Signed-off-by: Muhammad Jawad Hussain --- doc/guides/sample_app_ug/ethtool.rst | 2 +- examples/ethtool/ethtool-app/ethapp.c | 134 ++++++++++++++++++++------ examples/ethtool/lib/rte_ethtool.c | 38 -------- examples/ethtool/meson.build | 4 +- 4 files changed, 110 insertions(+), 68 deletions(-) diff --git a/doc/guides/sample_app_ug/ethtool.rst b/doc/guides/sample_app_ug/ethtool.rst index 159e9e0639..2e53084cc3 100644 --- a/doc/guides/sample_app_ug/ethtool.rst +++ b/doc/guides/sample_app_ug/ethtool.rst @@ -53,7 +53,7 @@ they do as follows: * ``portstats``: Print port statistics * ``regs``: Dump port register(s) to file * ``ringparam``: Get/set ring parameters -* ``rxmode``: Toggle port Rx mode +* ``rxmode``: Set rxmode of vfs on ports * ``stop``: Stop port * ``validate``: Check that given MAC address is valid unicast address * ``vlan``: Add/remove VLAN id diff --git a/examples/ethtool/ethtool-app/ethapp.c b/examples/ethtool/ethtool-app/ethapp.c index 78e86534e8..214433a98b 100644 --- a/examples/ethtool/ethtool-app/ethapp.c +++ b/examples/ethtool/ethtool-app/ethapp.c @@ -8,12 +8,30 @@ #include #include #include - +#ifdef RTE_NET_IXGBE +#include +#endif +#ifdef RTE_NET_BNXT +#include +#endif #include "rte_ethtool.h" #include "ethapp.h" #define EEPROM_DUMP_CHUNKSIZE 1024 +typedef uint16_t portid_t; + +/* *** CONFIGURE VF RECEIVE MODE *** */ +struct cmd_set_vf_rxmode { + cmdline_fixed_string_t set; + cmdline_fixed_string_t port; + portid_t port_id; + cmdline_fixed_string_t vf; + uint8_t vf_id; + cmdline_fixed_string_t what; + cmdline_fixed_string_t mode; + cmdline_fixed_string_t on; +}; struct pcmd_get_params { cmdline_fixed_string_t cmd; @@ -65,8 +83,6 @@ cmdline_parse_token_string_t pcmd_open_token_cmd = TOKEN_STRING_INITIALIZER(struct pcmd_int_params, cmd, "open"); cmdline_parse_token_string_t pcmd_stop_token_cmd = TOKEN_STRING_INITIALIZER(struct pcmd_int_params, cmd, "stop"); -cmdline_parse_token_string_t pcmd_rxmode_token_cmd = - TOKEN_STRING_INITIALIZER(struct pcmd_int_params, cmd, "rxmode"); cmdline_parse_token_string_t pcmd_portstats_token_cmd = TOKEN_STRING_INITIALIZER(struct pcmd_int_params, cmd, "portstats"); cmdline_parse_token_num_t pcmd_int_token_port = @@ -133,6 +149,31 @@ cmdline_parse_token_string_t pcmd_vlan_token_mode = cmdline_parse_token_num_t pcmd_vlan_token_vid = TOKEN_NUM_INITIALIZER(struct pcmd_vlan_params, vid, RTE_UINT16); +/* rxmode */ +cmdline_parse_token_string_t cmd_set_vf_rxmode_set = + TOKEN_STRING_INITIALIZER(struct cmd_set_vf_rxmode, + set, "set"); +cmdline_parse_token_string_t cmd_set_vf_rxmode_port = + TOKEN_STRING_INITIALIZER(struct cmd_set_vf_rxmode, + port, "port"); +cmdline_parse_token_num_t cmd_set_vf_rxmode_portid = + TOKEN_NUM_INITIALIZER(struct cmd_set_vf_rxmode, + port_id, RTE_UINT16); +cmdline_parse_token_string_t cmd_set_vf_rxmode_vf = + TOKEN_STRING_INITIALIZER(struct cmd_set_vf_rxmode, + vf, "vf"); +cmdline_parse_token_num_t cmd_set_vf_rxmode_vfid = + TOKEN_NUM_INITIALIZER(struct cmd_set_vf_rxmode, + vf_id, RTE_UINT8); +cmdline_parse_token_string_t cmd_set_vf_rxmode_what = + TOKEN_STRING_INITIALIZER(struct cmd_set_vf_rxmode, + what, "rxmode"); +cmdline_parse_token_string_t cmd_set_vf_rxmode_mode = + TOKEN_STRING_INITIALIZER(struct cmd_set_vf_rxmode, + mode, "AUPE#ROPE#BAM#MPE"); +cmdline_parse_token_string_t cmd_set_vf_rxmode_on = + TOKEN_STRING_INITIALIZER(struct cmd_set_vf_rxmode, + on, "on#off"); static void pcmd_quit_callback(__rte_unused void *ptr_params, @@ -142,7 +183,6 @@ pcmd_quit_callback(__rte_unused void *ptr_params, cmdline_quit(ctx); } - static void pcmd_drvinfo_callback(__rte_unused void *ptr_params, __rte_unused struct cmdline *ctx, @@ -150,7 +190,6 @@ pcmd_drvinfo_callback(__rte_unused void *ptr_params, { struct ethtool_drvinfo info; uint16_t id_port; - RTE_ETH_FOREACH_DEV(id_port) { memset(&info, 0, sizeof(info)); if (rte_ethtool_get_drvinfo(id_port, &info)) { @@ -447,26 +486,57 @@ pcmd_stop_callback(__rte_unused void *ptr_params, printf("Port %i: Error stopping device\n", params->port); } - static void -pcmd_rxmode_callback(void *ptr_params, - __rte_unused struct cmdline *ctx, - __rte_unused void *ptr_data) +pcmd_rxmode_callback(void *parsed_result, + __rte_unused struct cmdline *cl, + __rte_unused void *data) { - struct pcmd_intstr_params *params = ptr_params; - int stat; + int ret = -ENOTSUP; + uint16_t vf_rxmode = 0; + struct cmd_set_vf_rxmode *res = parsed_result; + + int is_on = (strcmp(res->on, "on") == 0) ? 1 : 0; + if (!strcmp(res->what, "rxmode")) { + if (!strcmp(res->mode, "AUPE")) + vf_rxmode |= RTE_ETH_VMDQ_ACCEPT_UNTAG; + else if (!strcmp(res->mode, "ROPE")) + vf_rxmode |= RTE_ETH_VMDQ_ACCEPT_HASH_UC; + else if (!strcmp(res->mode, "BAM")) + vf_rxmode |= RTE_ETH_VMDQ_ACCEPT_BROADCAST; + else if (!strncmp(res->mode, "MPE", 3)) + vf_rxmode |= RTE_ETH_VMDQ_ACCEPT_MULTICAST; + } - if (!rte_eth_dev_is_valid_port(params->port)) { - printf("Error: Invalid port number %i\n", params->port); - return; + RTE_SET_USED(is_on); + RTE_SET_USED(vf_rxmode); + +#ifdef RTE_NET_IXGBE + if (ret == -ENOTSUP) { + ret = rte_pmd_ixgbe_set_vf_rxmode(res->port_id, res->vf_id, + vf_rxmode, (uint8_t)is_on); + if (ret == -ENOTSUP) + printf("ixgbe not supported\n"); } - stat = rte_ethtool_net_set_rx_mode(params->port); - if (stat == 0) - return; - else if (stat == -ENOTSUP) - printf("Port %i: Operation not supported\n", params->port); - else - printf("Port %i: Error setting rx mode\n", params->port); +#endif +#ifdef RTE_NET_BNXT + if (ret == -ENOTSUP) { + ret = rte_pmd_bnxt_set_vf_rxmode(res->port_id, res->vf_id, + vf_rxmode, (uint8_t)is_on); + if (ret == -ENOTSUP) + printf("bnxt not supported\n"); + } +#endif + int ret_offload = -ENOTSUP; + if (!strcmp(res->mode, "AUPE")) { + ret_offload = rte_eth_dev_set_vlan_offload(res->port_id, RTE_ETH_VLAN_FILTER_MASK); + if (ret_offload == -ENOTSUP) + printf("Port %i: VLAN Filter setting mask not supported\n", res->port_id); + } + if (ret < 0) + fprintf(stderr, + "bad VF receive mode parameter, return code = %d\n", ret); + else if (ret == 0) + printf("Successful\n"); } @@ -763,16 +833,25 @@ cmdline_parse_inst_t pcmd_stop = { NULL }, }; -cmdline_parse_inst_t pcmd_rxmode = { + +cmdline_parse_inst_t cmd_set_vf_rxmode = { .f = pcmd_rxmode_callback, .data = NULL, - .help_str = "rxmode \n Toggle port Rx mode", + .help_str = "set port vf rxmode " + "AUPE|ROPE|BAM|MPE on|off", .tokens = { - (void *)&pcmd_rxmode_token_cmd, - (void *)&pcmd_int_token_port, - NULL + (void *)&cmd_set_vf_rxmode_set, + (void *)&cmd_set_vf_rxmode_port, + (void *)&cmd_set_vf_rxmode_portid, + (void *)&cmd_set_vf_rxmode_vf, + (void *)&cmd_set_vf_rxmode_vfid, + (void *)&cmd_set_vf_rxmode_what, + (void *)&cmd_set_vf_rxmode_mode, + (void *)&cmd_set_vf_rxmode_on, + NULL, }, }; + cmdline_parse_inst_t pcmd_macaddr_get = { .f = pcmd_macaddr_callback, .data = NULL, @@ -869,7 +948,6 @@ cmdline_parse_inst_t pcmd_vlan = { }, }; - cmdline_parse_ctx_t list_prompt_commands[] = { (cmdline_parse_inst_t *)&pcmd_drvinfo, (cmdline_parse_inst_t *)&pcmd_eeprom, @@ -885,7 +963,7 @@ cmdline_parse_ctx_t list_prompt_commands[] = { (cmdline_parse_inst_t *)&pcmd_regs, (cmdline_parse_inst_t *)&pcmd_ringparam, (cmdline_parse_inst_t *)&pcmd_ringparam_set, - (cmdline_parse_inst_t *)&pcmd_rxmode, + (cmdline_parse_inst_t *)&cmd_set_vf_rxmode, (cmdline_parse_inst_t *)&pcmd_stop, (cmdline_parse_inst_t *)&pcmd_validate, (cmdline_parse_inst_t *)&pcmd_vlan, diff --git a/examples/ethtool/lib/rte_ethtool.c b/examples/ethtool/lib/rte_ethtool.c index ffaad96498..6951f0c2a8 100644 --- a/examples/ethtool/lib/rte_ethtool.c +++ b/examples/ethtool/lib/rte_ethtool.c @@ -373,44 +373,6 @@ rte_ethtool_net_vlan_rx_kill_vid(uint16_t port_id, uint16_t vid) return rte_eth_dev_vlan_filter(port_id, vid, 0); } -/* - * The set_rx_mode provides driver-specific rx mode setting. - * This implementation implements rx mode setting based upon - * ixgbe/igb drivers. Further improvement is to provide a - * callback op field over struct rte_eth_dev::dev_ops so each - * driver can register device-specific implementation - */ -int -rte_ethtool_net_set_rx_mode(uint16_t port_id) -{ - uint16_t num_vfs; - struct rte_eth_dev_info dev_info; - uint16_t vf; - int ret; - - ret = rte_eth_dev_info_get(port_id, &dev_info); - if (ret != 0) - return ret; - - num_vfs = dev_info.max_vfs; - - /* Set VF vf_rx_mode, VF unsupport status is discard */ - for (vf = 0; vf < num_vfs; vf++) { -#ifdef RTE_NET_IXGBE - rte_pmd_ixgbe_set_vf_rxmode(port_id, vf, - RTE_ETH_VMDQ_ACCEPT_UNTAG, 0); -#endif - } - - /* Enable Rx vlan filter, VF unsupported status is discard */ - ret = rte_eth_dev_set_vlan_offload(port_id, RTE_ETH_VLAN_FILTER_MASK); - if (ret != 0) - return ret; - - return 0; -} - - int rte_ethtool_get_ringparam(uint16_t port_id, struct ethtool_ringparam *ring_param) diff --git a/examples/ethtool/meson.build b/examples/ethtool/meson.build index d7f63d48af..175a5ddd7a 100644 --- a/examples/ethtool/meson.build +++ b/examples/ethtool/meson.build @@ -22,5 +22,7 @@ deps += 'bus_pci' if dpdk_conf.has('RTE_NET_IXGBE') deps += 'net_ixgbe' endif - +if dpdk_conf.has('RTE_NET_BNXT') + deps += 'net_bnxt' +endif allow_experimental_apis = true