Patch Detail
get:
Show a patch.
patch:
Update a patch.
put:
Update a patch.
GET /api/patches/113780/?format=api
{ "id": 113780, "url": "http://patches.dpdk.org/api/patches/113780/?format=api", "web_url": "http://patches.dpdk.org/project/dpdk/patch/20220707170434.2159759-1-kevinx.liu@intel.com/", "project": { "id": 1, "url": "http://patches.dpdk.org/api/projects/1/?format=api", "name": "DPDK", "link_name": "dpdk", "list_id": "dev.dpdk.org", "list_email": "dev@dpdk.org", "web_url": "http://core.dpdk.org", "scm_url": "git://dpdk.org/dpdk", "webscm_url": "http://git.dpdk.org/dpdk", "list_archive_url": "https://inbox.dpdk.org/dev", "list_archive_url_format": "https://inbox.dpdk.org/dev/{}", "commit_url_format": "" }, "msgid": "<20220707170434.2159759-1-kevinx.liu@intel.com>", "list_archive_url": "https://inbox.dpdk.org/dev/20220707170434.2159759-1-kevinx.liu@intel.com", "date": "2022-07-07T17:04:34", "name": "[v4] net/i40e: restore disable double VLAN by default", "commit_ref": null, "pull_url": null, "state": "accepted", "archived": true, "hash": "c734ad1f46a42ab237273e866a523a72c819bd47", "submitter": { "id": 2440, "url": "http://patches.dpdk.org/api/people/2440/?format=api", "name": "Kevin Liu", "email": "kevinx.liu@intel.com" }, "delegate": { "id": 1540, "url": "http://patches.dpdk.org/api/users/1540/?format=api", "username": "qzhan15", "first_name": "Qi", "last_name": "Zhang", "email": "qi.z.zhang@intel.com" }, "mbox": "http://patches.dpdk.org/project/dpdk/patch/20220707170434.2159759-1-kevinx.liu@intel.com/mbox/", "series": [ { "id": 23920, "url": "http://patches.dpdk.org/api/series/23920/?format=api", "web_url": "http://patches.dpdk.org/project/dpdk/list/?series=23920", "date": "2022-07-07T17:04:34", "name": "[v4] net/i40e: restore disable double VLAN by default", "version": 4, "mbox": "http://patches.dpdk.org/series/23920/mbox/" } ], "comments": "http://patches.dpdk.org/api/patches/113780/comments/", "check": "success", "checks": "http://patches.dpdk.org/api/patches/113780/checks/", "tags": {}, "related": [], "headers": { "Return-Path": "<dev-bounces@dpdk.org>", "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])\n\tby inbox.dpdk.org (Postfix) with ESMTP id F066FA0540;\n\tThu, 7 Jul 2022 11:06:50 +0200 (CEST)", "from [217.70.189.124] (localhost [127.0.0.1])\n\tby mails.dpdk.org (Postfix) with ESMTP id DDA0940A7B;\n\tThu, 7 Jul 2022 11:06:50 +0200 (CEST)", "from mga07.intel.com (mga07.intel.com [134.134.136.100])\n by mails.dpdk.org (Postfix) with ESMTP id 970CD406B4\n for <dev@dpdk.org>; Thu, 7 Jul 2022 11:06:49 +0200 (CEST)", "from fmsmga008.fm.intel.com ([10.253.24.58])\n by orsmga105.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384;\n 07 Jul 2022 02:06:33 -0700", "from intel-cd-odc-kevin.cd.intel.com ([10.240.178.191])\n by fmsmga008-auth.fm.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384;\n 07 Jul 2022 02:06:31 -0700" ], "DKIM-Signature": "v=1; a=rsa-sha256; c=relaxed/simple;\n d=intel.com; i=@intel.com; q=dns/txt; s=Intel;\n t=1657184809; x=1688720809;\n h=from:to:cc:subject:date:message-id:in-reply-to:\n references:mime-version:content-transfer-encoding;\n bh=VsTV8pH3ZUn48D4FtM0sk0afEWrPhLF0rdhqE8z3Gfk=;\n b=TQpTsiq8tOTYUGEAa0prydSdlzHYqw2ouwMJuBwvjSKLTUekS4/9kFF5\n 4cU+3tPcI3/c1I0ApihVnJnbOprE/IeK+u69N8cRwkygwS8tjC1NApGP8\n YkLI+g4lQfHFq1+SK5dcAbLxPw/EJSXjznt4NS0JQ0k8YO2jC7eRHdWM9\n Vo2E1fhSv0WWvHPeS182RitJIu1M7ny+KUhbkxYaV8XfpCKjF4Vi5Nqmm\n MPdMt7xtEktgC+mLvnyu8ClBawvIWnUT5OKr9Hcp8NCKWf7+2nmGFD3Na\n QmEkvwzoe8B4ep9Ws9zV+Z4076nepLP0LYs9AR0cbVaW4H90Ly9d9RLOC g==;", "X-IronPort-AV": [ "E=McAfee;i=\"6400,9594,10400\"; a=\"347958319\"", "E=Sophos;i=\"5.92,252,1650956400\"; d=\"scan'208\";a=\"347958319\"", "E=Sophos;i=\"5.92,252,1650956400\"; d=\"scan'208\";a=\"651054106\"" ], "From": "Kevin Liu <kevinx.liu@intel.com>", "To": "dev@dpdk.org", "Cc": "beilei.xing@intel.com, Yuying.Zhang@intel.com, stevex.yang@intel.com,\n Kevin Liu <kevinx.liu@intel.com>", "Subject": "[PATCH v4] net/i40e: restore disable double VLAN by default", "Date": "Thu, 7 Jul 2022 17:04:34 +0000", "Message-Id": "<20220707170434.2159759-1-kevinx.liu@intel.com>", "X-Mailer": "git-send-email 2.34.1", "In-Reply-To": "<20220707104732.1816933-1-kevinx.liu@intel.com>", "References": "<20220707104732.1816933-1-kevinx.liu@intel.com>", "MIME-Version": "1.0", "Content-Transfer-Encoding": "8bit", "X-BeenThere": "dev@dpdk.org", "X-Mailman-Version": "2.1.29", "Precedence": "list", "List-Id": "DPDK patches and discussions <dev.dpdk.org>", "List-Unsubscribe": "<https://mails.dpdk.org/options/dev>,\n <mailto:dev-request@dpdk.org?subject=unsubscribe>", "List-Archive": "<http://mails.dpdk.org/archives/dev/>", "List-Post": "<mailto:dev@dpdk.org>", "List-Help": "<mailto:dev-request@dpdk.org?subject=help>", "List-Subscribe": "<https://mails.dpdk.org/listinfo/dev>,\n <mailto:dev-request@dpdk.org?subject=subscribe>", "Errors-To": "dev-bounces@dpdk.org" }, "content": "Previously, QinQ is enabled by default and can't be disabled,\nbut there'll be performance drop if QinQ is enabled.\n\nSo, disable QinQ by default.\n\nFixes: ae97b8b89826 (\"net/i40e: fix error disable double VLAN\")\nSigned-off-by: Kevin Liu <kevinx.liu@intel.com>\n\n---\nv2: update doc and refine commit log\n---\nv3: refine commit log\n---\nv4: update doc\n---\n doc/guides/nics/i40e.rst | 13 ++++++++-----\n drivers/net/i40e/i40e_ethdev.c | 12 ------------\n 2 files changed, 8 insertions(+), 17 deletions(-)", "diff": "diff --git a/doc/guides/nics/i40e.rst b/doc/guides/nics/i40e.rst\nindex 85fdc4944d..d5938fa8e4 100644\n--- a/doc/guides/nics/i40e.rst\n+++ b/doc/guides/nics/i40e.rst\n@@ -969,11 +969,14 @@ it will fail and return the info \"Conflict with the first rule's input set\",\n which means the current rule's input set conflicts with the first rule's.\n Remove the first rule if want to change the input set of the PCTYPE.\n \n-Disable QinQ is not supported when FW >= 8.4\n-~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~\n-\n-If upgrade FW to version 8.4 and higher, enable QinQ by default and disable QinQ is not supported.\n-\n+Vlan related Features miss when FW >= 8.4\n+~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~\n+\n+If FW version >= 8.4, there'll be some Vlan related issues:\n+1. TCI input set for QinQ is invalid.\n+2. Fail to configure TPID for QinQ.\n+3. Need to enable QinQ before enabling Vlan filter.\n+4. Fail to strip outer Vlan.\n \n Example of getting best performance with l3fwd example\n ------------------------------------------------------\ndiff --git a/drivers/net/i40e/i40e_ethdev.c b/drivers/net/i40e/i40e_ethdev.c\nindex 684e095026..117dd85c11 100644\n--- a/drivers/net/i40e/i40e_ethdev.c\n+++ b/drivers/net/i40e/i40e_ethdev.c\n@@ -4027,12 +4027,6 @@ i40e_vlan_offload_set(struct rte_eth_dev *dev, int mask)\n \t}\n \n \tif (mask & RTE_ETH_VLAN_EXTEND_MASK) {\n-\t\t/* Double VLAN not allowed to be disabled.*/\n-\t\tif (pf->fw8_3gt && !(rxmode->offloads & RTE_ETH_RX_OFFLOAD_VLAN_EXTEND)) {\n-\t\t\tPMD_DRV_LOG(WARNING,\n-\t\t\t\t\"Disable double VLAN is not allowed after firmwarev8.3!\");\n-\t\t\treturn 0;\n-\t\t}\n \t\ti = 0;\n \t\tnum = vsi->mac_num;\n \t\tmac_filter = rte_zmalloc(\"mac_filter_info_data\",\n@@ -6296,7 +6290,6 @@ int i40e_vsi_cfg_inner_vlan_stripping(struct i40e_vsi *vsi, bool on)\n static int\n i40e_dev_init_vlan(struct rte_eth_dev *dev)\n {\n-\tstruct i40e_pf *pf = I40E_DEV_PRIVATE_TO_PF(dev->data->dev_private);\n \tstruct rte_eth_dev_data *data = dev->data;\n \tint ret;\n \tint mask = 0;\n@@ -6307,11 +6300,6 @@ i40e_dev_init_vlan(struct rte_eth_dev *dev)\n \t RTE_ETH_VLAN_FILTER_MASK |\n \t RTE_ETH_VLAN_EXTEND_MASK;\n \n-\t/* Double VLAN be enabled by default.*/\n-\tif (pf->fw8_3gt) {\n-\t\tstruct rte_eth_rxmode *rxmode = &dev->data->dev_conf.rxmode;\n-\t\trxmode->offloads |= RTE_ETH_RX_OFFLOAD_VLAN_EXTEND;\n-\t}\n \tret = i40e_vlan_offload_set(dev, mask);\n \tif (ret) {\n \t\tPMD_DRV_LOG(INFO, \"Failed to update vlan offload\");\n", "prefixes": [ "v4" ] }