From patchwork Fri Apr 5 12:04:02 2019 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Nithin Dabilpuram X-Patchwork-Id: 52330 X-Patchwork-Delegate: ferruh.yigit@amd.com Return-Path: X-Original-To: patchwork@dpdk.org Delivered-To: patchwork@dpdk.org Received: from [92.243.14.124] (localhost [127.0.0.1]) by dpdk.org (Postfix) with ESMTP id 6208A1B4C1; Fri, 5 Apr 2019 14:04:16 +0200 (CEST) Received: from mail-pg1-f176.google.com (mail-pg1-f176.google.com [209.85.215.176]) by dpdk.org (Postfix) with ESMTP id DBFD81B4C0 for ; Fri, 5 Apr 2019 14:04:14 +0200 (CEST) Received: by mail-pg1-f176.google.com with SMTP id y3so2959825pgk.12 for ; Fri, 05 Apr 2019 05:04:14 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025; h=from:to:cc:subject:date:message-id:in-reply-to:references; bh=65PTcDOdVrZIcIX1O98NQobb5V8SL1pSTJyI2KBJcgo=; b=un41f30/hFq9ioYria6ofGM+2ZDlq49Zw5bsWtGK8/w2nK8nMZLaCwB3T5nTxDy1zK 5EAEUMcAbT/SQm9zj4wd0nF9LJS1xLuVXojYzGXeAJtVMPfNnB1Ar4KzI/iOnqEUxIP+ K5LHo9Okol75BNYI4W+WBq4QOm8VJwb9qhfygRWsyAtb63G+d0B6dTv4GOW/0BwGeXV6 PlU8/rO6GqDKv/B33vrtLoOpY6mEyAeKDRJcPHtPfOnDFSJ+86GFf6qM8iUToIDlulsT A29ejtE+tpp6t2gTcOuFvUVO10WIOW/7BsPnMuslyYSJ8FUOVUfZskw5FrqUkSAuELXu MaGw== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:from:to:cc:subject:date:message-id:in-reply-to :references; bh=65PTcDOdVrZIcIX1O98NQobb5V8SL1pSTJyI2KBJcgo=; b=BjXDq0UbiaO+Kxa5ZcJrIOGlBdg4TtIgwZ042dyIoQhEMil0A2EyIPY7UBYKUMO9o3 pphCcYOsxnP6mdi3aMyKeWfHxDms1qWPqjWmdom83HA7TSo8QglYxxYXA6Kpl0n8nUO+ iR3rYf7vvmbuMhCgePZ9Hp215WiC03AVDw0ULPJ6DWMl6rogd6hLtg0dkIn1aj/UCUAI yDrGxk6nkM4VZgXIF6ORUA1eesdZHJHKcj1HsJLjRXFJD7I0cU5lJwB5q2NDqsylcl6w 4QUW6bWaN6NALNdG7Xpfjma3CtUYSYCW7JauOEeRxR7DdPMARZGrdAald7ZG/LEptI2H vbng== X-Gm-Message-State: APjAAAWjgFmmZyTUinZCx0oRiwUi78nM6JI7s2YEVGjz1kmyMD2vjgi2 tj3XZfer3c9GAL60JJ/aAic= X-Google-Smtp-Source: APXvYqwjho5/vwe5Zh9fjc6CpewpUviT60fcE0PwbcTZwMgxoUsFiveQW29PsRQQ7Fypm2yy/ShIgg== X-Received: by 2002:a65:65c5:: with SMTP id y5mr11380474pgv.192.1554465854012; Fri, 05 Apr 2019 05:04:14 -0700 (PDT) Received: from hyd1434t5810.caveonetworks.com ([115.113.156.2]) by smtp.gmail.com with ESMTPSA id r87sm33786816pfa.71.2019.04.05.05.04.11 (version=TLS1_2 cipher=ECDHE-RSA-AES128-SHA bits=128/128); Fri, 05 Apr 2019 05:04:13 -0700 (PDT) From: Nithin Dabilpuram To: Wenzhuo Lu , Jingjing Wu , Bernard Iremonger Cc: Ferruh Yigit , dev@dpdk.org, Wang Xiao W , Nithin Dabilpuram Date: Fri, 5 Apr 2019 17:34:02 +0530 Message-Id: <20190405120403.5446-1-nithind1988@gmail.com> X-Mailer: git-send-email 2.8.4 In-Reply-To: <20190219064840.12046-1-ndabilpuram@marvell.com> References: <20190219064840.12046-1-ndabilpuram@marvell.com> Subject: [dpdk-dev] [PATCH v4 1/2] app/testpmd: fix Tx VLAN and QinQ dependency X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org Sender: "dev" From: Nithin Dabilpuram Tx VLAN & QinQ insert enable need not depend on Rx VLAN offload ETH_VLAN_EXTEND_OFFLOAD. For Tx VLAN insert enable, error check is now to see if QinQ was enabled but only single VLAN id is set. Fixes: 6a34f91690d0 ("app/testpmd: fix error message when setting Tx VLAN") Cc: xiao.w.wang@intel.com Signed-off-by: Nithin Dabilpuram Acked-by: Bernard Iremonger --- v4: * Resend v3 from different mailserver to avoid CRLF v3: * Add back error check in tx_vlan_set() to check if QinQ is already enabled. Also fix headline. v2: * Split change into two seperate patches as suggested. app/test-pmd/config.c | 11 ++--------- 1 file changed, 2 insertions(+), 9 deletions(-) diff --git a/app/test-pmd/config.c b/app/test-pmd/config.c index cadcb51..010e26d 100644 --- a/app/test-pmd/config.c +++ b/app/test-pmd/config.c @@ -2962,7 +2962,6 @@ vlan_tpid_set(portid_t port_id, enum rte_vlan_type vlan_type, uint16_t tp_id) void tx_vlan_set(portid_t port_id, uint16_t vlan_id) { - int vlan_offload; struct rte_eth_dev_info dev_info; if (port_id_is_invalid(port_id, ENABLED_WARN)) @@ -2970,8 +2969,8 @@ tx_vlan_set(portid_t port_id, uint16_t vlan_id) if (vlan_id_is_invalid(vlan_id)) return; - vlan_offload = rte_eth_dev_get_vlan_offload(port_id); - if (vlan_offload & ETH_VLAN_EXTEND_OFFLOAD) { + if (ports[port_id].dev_conf.txmode.offloads & + DEV_TX_OFFLOAD_QINQ_INSERT) { printf("Error, as QinQ has been enabled.\n"); return; } @@ -2990,7 +2989,6 @@ tx_vlan_set(portid_t port_id, uint16_t vlan_id) void tx_qinq_set(portid_t port_id, uint16_t vlan_id, uint16_t vlan_id_outer) { - int vlan_offload; struct rte_eth_dev_info dev_info; if (port_id_is_invalid(port_id, ENABLED_WARN)) @@ -3000,11 +2998,6 @@ tx_qinq_set(portid_t port_id, uint16_t vlan_id, uint16_t vlan_id_outer) if (vlan_id_is_invalid(vlan_id_outer)) return; - vlan_offload = rte_eth_dev_get_vlan_offload(port_id); - if (!(vlan_offload & ETH_VLAN_EXTEND_OFFLOAD)) { - printf("Error, as QinQ hasn't been enabled.\n"); - return; - } rte_eth_dev_info_get(port_id, &dev_info); if ((dev_info.tx_offload_capa & DEV_TX_OFFLOAD_QINQ_INSERT) == 0) { printf("Error: qinq insert not supported by port %d\n", From patchwork Fri Apr 5 12:04:03 2019 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Nithin Dabilpuram X-Patchwork-Id: 52331 X-Patchwork-Delegate: ferruh.yigit@amd.com Return-Path: X-Original-To: patchwork@dpdk.org Delivered-To: patchwork@dpdk.org Received: from [92.243.14.124] (localhost [127.0.0.1]) by dpdk.org (Postfix) with ESMTP id C094D1B4D1; Fri, 5 Apr 2019 14:04:20 +0200 (CEST) Received: from mail-pf1-f179.google.com (mail-pf1-f179.google.com [209.85.210.179]) by dpdk.org (Postfix) with ESMTP id 51D6B1B4CB for ; Fri, 5 Apr 2019 14:04:18 +0200 (CEST) Received: by mail-pf1-f179.google.com with SMTP id z5so3183770pfn.3 for ; Fri, 05 Apr 2019 05:04:18 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025; h=from:to:cc:subject:date:message-id:in-reply-to:references; bh=tDoIIFqtTcPi7/xdARNLjE6HNG83KM7EDxtkqIB+9Z8=; b=aAk3ILVGS4mmV52ESyRoRm8Wi60DxORNdcPVBv9qRktadRR3zRQUVwrxtO36DNsYWT /jBuFKD00pe4DMXXIu4Z+7kSEZX8EuXGbIih/b3iCqU4F0wa7LwIW/XFR7SYaWQe8/ER jbyTscbvkIal34lxsdPJReHjeP3QA91hp1Ukp+vP5EF+U2/oEH54RZZenx6PbT6SSeY6 ADWOzgdGhMiuCLbibgYgXyX6iAScF5WP4HRJnxgaGxJb+A0hI+rI2ov1FVUGQGvC+g8w F2NCmJ1V/RXYgtY3j1PcEjznF3+n3HKvfsrToh0FlM+L8TvY+sQRMPGt8zP4d7OhUkZt kuNA== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:from:to:cc:subject:date:message-id:in-reply-to :references; bh=tDoIIFqtTcPi7/xdARNLjE6HNG83KM7EDxtkqIB+9Z8=; b=fF7VfoVvuCeijNs0ecTvVEPRBa6RoWXtpCpuw4F5HPXCs7qtOnAfPaST1q+MxdggjJ 9DrktJme08riFJA9HdPu7cvajgcOGBiEk/uWtBxffw5iM3y+ZKNqCYC60/XNV736vZIs aNBHhi8ruHds4vI29J3x6oD8fCX9NvPfPPs7N2ICvcgKjIxlcnBu2Pgj6TaKGZgBSeRp u4szmlryAIZoRpuRBOhKHlaJ2jAtZxKrHfqGoXlUIbL2Dy6ofb4IZbMbwlTgqPSv9o5/ /OSx7aGFG9puvElipLNdkPpweQjpanZjeH0tl/j3L8w7Xovtp/RBInSxSfNy1lx9ITP7 p6eg== X-Gm-Message-State: APjAAAXqLFIFi9nyzc4Mdj1Xbn3+QymeBwMhHQcCslQEiB/dZCNB02+I G03mjKlcSnhgsINhFiEpIh4= X-Google-Smtp-Source: APXvYqzXr5CGOXbxNUal8WwBVshXcnHyOwtBTtdWJSyK60/vQZpTJrYN9xc8vGGv83BeU5W02/VhmQ== X-Received: by 2002:a63:720c:: with SMTP id n12mr11702277pgc.348.1554465857219; Fri, 05 Apr 2019 05:04:17 -0700 (PDT) Received: from hyd1434t5810.caveonetworks.com ([115.113.156.2]) by smtp.gmail.com with ESMTPSA id r87sm33786816pfa.71.2019.04.05.05.04.14 (version=TLS1_2 cipher=ECDHE-RSA-AES128-SHA bits=128/128); Fri, 05 Apr 2019 05:04:16 -0700 (PDT) From: Nithin Dabilpuram To: Wenzhuo Lu , Jingjing Wu , Bernard Iremonger Cc: Ferruh Yigit , dev@dpdk.org, Wang Xiao W , Nithin Dabilpuram , shahafs@mellanox.com Date: Fri, 5 Apr 2019 17:34:03 +0530 Message-Id: <20190405120403.5446-2-nithind1988@gmail.com> X-Mailer: git-send-email 2.8.4 In-Reply-To: <20190405120403.5446-1-nithind1988@gmail.com> References: <20190219064840.12046-1-ndabilpuram@marvell.com> <20190405120403.5446-1-nithind1988@gmail.com> Subject: [dpdk-dev] [PATCH v4 2/2] app/testpmd: fix Tx QinQ set X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org Sender: "dev" From: Nithin Dabilpuram Enable DEV_TX_OFFLOAD_VLAN_INSERT also along with DEV_TX_OFFLOAD_VLAN_QINQ in tx_qinq_set() as it takes both vlan id's as arguments. Fixes: 597f9fafe13b ("app/testpmd: convert to new Tx offloads API") Cc: shahafs@mellanox.com Signed-off-by: Nithin Dabilpuram Acked-by: Bernard Iremonger --- v4: * Resend v3 from different mailserver to avoid CRLF v3: * Rename headline v2: * Split change into two seperate patches as suggested. app/test-pmd/config.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/app/test-pmd/config.c b/app/test-pmd/config.c index 010e26d..f9cb129 100644 --- a/app/test-pmd/config.c +++ b/app/test-pmd/config.c @@ -3006,7 +3006,8 @@ tx_qinq_set(portid_t port_id, uint16_t vlan_id, uint16_t vlan_id_outer) } tx_vlan_reset(port_id); - ports[port_id].dev_conf.txmode.offloads |= DEV_TX_OFFLOAD_QINQ_INSERT; + ports[port_id].dev_conf.txmode.offloads |= (DEV_TX_OFFLOAD_VLAN_INSERT | + DEV_TX_OFFLOAD_QINQ_INSERT); ports[port_id].tx_vlan_id = vlan_id; ports[port_id].tx_vlan_id_outer = vlan_id_outer; }