From patchwork Thu Aug 11 12:58:40 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: 114841 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 9ED57A0548; Thu, 11 Aug 2022 14:58:55 +0200 (CEST) Received: from [217.70.189.124] (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id 451BD427F2; Thu, 11 Aug 2022 14:58:55 +0200 (CEST) Received: from mail-wm1-f52.google.com (mail-wm1-f52.google.com [209.85.128.52]) by mails.dpdk.org (Postfix) with ESMTP id 2DCB4410FC for ; Thu, 11 Aug 2022 14:58:53 +0200 (CEST) Received: by mail-wm1-f52.google.com with SMTP id az6-20020a05600c600600b003a530cebbe3so2642955wmb.0 for ; Thu, 11 Aug 2022 05:58:53 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=emumba-com.20210112.gappssmtp.com; s=20210112; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:from:to:cc; bh=TT0xOPJp90zR8oJegQcRS4Nue5nd48T2w/GNxZS31Y4=; b=X4INzG90BRmoAHnhbaJtgxX6uDxBO8Fz+lRBUmq0/tFP9YP8p/THb78Z53k1qBmOK3 0ZY8SnjRtfzrgFO9XbyRoc7zJL59Q+c3PLOdo+o/Tae8KBVPsJL69jQCUpngBM6P20My Y8EzysDRadc3DJvFPDYa2+KUVBMASKZnXc37dZpWFeKyomgP7jMtwscQc9A63xq+WO/2 cUTbwtmdi2SgZ2/KCnEb3bMVtpDEgmrnOPwJ7OdfTBhR1GCpcQbTAKyyLXrQ7qSEeDod JMQS0Duynw68fPo0wj7F+pT/Ohze38U2MHB4rNNtOkZN6IwHJ2cQHqnOuz4FsXH0OZ2U FkVA== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:x-gm-message-state:from:to:cc; bh=TT0xOPJp90zR8oJegQcRS4Nue5nd48T2w/GNxZS31Y4=; b=KcrPqwbmZknZlY6gy9VWU4ut4nHEoDCdxCeqyMnvxtY2T31RKG0/QTHLKPEPfy2MqM GRmTxG1G/Oxf3AQnBCWn54cOMDXsm1ukRO3nZlfQHHSskYfgN44ea+YCQKD7CtkABKNw fZ5smfHW+aWJrNVUQGu5EWqHwgx6ky/fv98rzAtzZfkGLUBAc05avP0oVK1Qoju4FlJa eYhbQY54d1xKk6ZG3cm3GgRHae+fq7nHSxL+QmAdun288H26t756YOfBhtAr6z5f4tVo cdYz6Qu5p0heGNJqOXVKPS3BLPJkwt4Toqx731myjI0XCFzBlqgyD1jOPKqrC/0lISec G6qQ== X-Gm-Message-State: ACgBeo1yFD7y1gfF343fUgbs0qFtURY5j3R8B28oDMQ0L2ws0uW/3Yfa V9144EI2o1uK1pZ584UAjOufuzpXK9emNg== X-Google-Smtp-Source: AA6agR7QkDF48MpKgHUbLC/5fAqLgUbRw3KtNOan+o09Y937iH6bzzzBxdDhJ/ctwtdGEpyHi6j+3Q== X-Received: by 2002:a05:600c:35d5:b0:3a3:2490:c984 with SMTP id r21-20020a05600c35d500b003a32490c984mr5823833wmq.162.1660222732525; Thu, 11 Aug 2022 05:58:52 -0700 (PDT) Received: from localhost.localdomain ([39.32.44.65]) by smtp.gmail.com with ESMTPSA id f68-20020a1c3847000000b003a17ab4e7c8sm5862829wma.39.2022.08.11.05.58.50 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Thu, 11 Aug 2022 05:58:52 -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: Thu, 11 Aug 2022 17:58:40 +0500 Message-Id: <20220811125840.108887-1-jawad.hussain@emumba.com> X-Mailer: git-send-email 2.25.1 In-Reply-To: <20220525061230.123956-1-jawad.hussain@emumba.com> References: <20220525061230.123956-1-jawad.hussain@emumba.com> 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