Patch Detail
get:
Show a patch.
patch:
Update a patch.
put:
Update a patch.
GET /api/patches/139841/?format=api
{ "id": 139841, "url": "https://patches.dpdk.org/api/patches/139841/?format=api", "web_url": "https://patches.dpdk.org/project/dpdk/patch/4f505a9aa0a6fcfa6ce5ecc94632ae7b1b6eb437.1714744628.git.anatoly.burakov@intel.com/", "project": { "id": 1, "url": "https://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": "<4f505a9aa0a6fcfa6ce5ecc94632ae7b1b6eb437.1714744628.git.anatoly.burakov@intel.com>", "list_archive_url": "https://inbox.dpdk.org/dev/4f505a9aa0a6fcfa6ce5ecc94632ae7b1b6eb437.1714744628.git.anatoly.burakov@intel.com", "date": "2024-05-03T13:57:44", "name": "[v2,13/27] net/ixgbe/base: handle -Wimplicit-fallthrough", "commit_ref": null, "pull_url": null, "state": "new", "archived": false, "hash": "00aec8549964422083f44a5faebc7ce8fe602b96", "submitter": { "id": 4, "url": "https://patches.dpdk.org/api/people/4/?format=api", "name": "Burakov, Anatoly", "email": "anatoly.burakov@intel.com" }, "delegate": null, "mbox": "https://patches.dpdk.org/project/dpdk/patch/4f505a9aa0a6fcfa6ce5ecc94632ae7b1b6eb437.1714744628.git.anatoly.burakov@intel.com/mbox/", "series": [ { "id": 31875, "url": "https://patches.dpdk.org/api/series/31875/?format=api", "web_url": "https://patches.dpdk.org/project/dpdk/list/?series=31875", "date": "2024-05-03T13:57:31", "name": "Update IXGBE base driver", "version": 2, "mbox": "https://patches.dpdk.org/series/31875/mbox/" } ], "comments": "https://patches.dpdk.org/api/patches/139841/comments/", "check": "warning", "checks": "https://patches.dpdk.org/api/patches/139841/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 159A043F76;\n\tFri, 3 May 2024 15:59:51 +0200 (CEST)", "from mails.dpdk.org (localhost [127.0.0.1])\n\tby mails.dpdk.org (Postfix) with ESMTP id A047040DD8;\n\tFri, 3 May 2024 15:58:50 +0200 (CEST)", "from mgamail.intel.com (mgamail.intel.com [192.198.163.15])\n by mails.dpdk.org (Postfix) with ESMTP id 42BB940DCB\n for <dev@dpdk.org>; Fri, 3 May 2024 15:58:48 +0200 (CEST)", "from fmviesa002.fm.intel.com ([10.60.135.142])\n by fmvoesa109.fm.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384;\n 03 May 2024 06:58:47 -0700", "from silpixa00401119.ir.intel.com ([10.55.129.167])\n by fmviesa002.fm.intel.com with ESMTP; 03 May 2024 06:58:46 -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=1714744728; x=1746280728;\n h=from:to:cc:subject:date:message-id:in-reply-to:\n references:mime-version:content-transfer-encoding;\n bh=dePQDzWCecIc5qVMytR5k7/yBC+DFM9dedoM3jAXfY8=;\n b=YXTRnJjoZ8Nqyrkd6YFKIrkhw6ZploAy1xmJRn71C57yIOJx9sWIW3Dg\n B6bqxJB1/7RKfATE9NFsMS0G+ZXg7MnOZgeB+YolKgGA3V57nyZddlvHE\n LYaXcTO4GaTQtaiZq9QsfmQSqCSzfrW+nVzfYrLoFvqAnNYxqhbritNDF\n N7x2zPZoNpw5P+VxQtLRGgzWY9cHh8aKwtPHRE7Kda0abGkiCaz2FHYk5\n uzJKentJVK65lAT+Rycfq0G91u/sJLOhtAw2XWY7LYMlfhOPr2ZwrExCK\n 6BFPJqOAiRSQchZo6lt/2BoduVXHvbfDZB7F+VEEQq4PIggsMNpLJVrqW A==;", "X-CSE-ConnectionGUID": [ "C92xvGcUTDuH4/lyIbM7Yg==", "TsVhidirSWGmkrobYY7hlQ==" ], "X-CSE-MsgGUID": [ "cyLJ3Y8KR46qW1rr+9PQrg==", "ccvfQohOSKKvZ47OMkoJAw==" ], "X-IronPort-AV": [ "E=McAfee;i=\"6600,9927,11063\"; a=\"10714948\"", "E=Sophos;i=\"6.07,251,1708416000\"; d=\"scan'208\";a=\"10714948\"", "E=Sophos;i=\"6.07,251,1708416000\"; d=\"scan'208\";a=\"50642013\"" ], "X-ExtLoop1": "1", "From": "Anatoly Burakov <anatoly.burakov@intel.com>", "To": "dev@dpdk.org", "Cc": "Piotr Skajewski <piotrx.skajewski@intel.com>,\n\tbruce.richardson@intel.com, vladimir.medvedkin@intel.com,\n\tMichael@dpdk.org, Alice <alice.michael@intel.com>", "Subject": "[PATCH v2 13/27] net/ixgbe/base: handle -Wimplicit-fallthrough", "Date": "Fri, 3 May 2024 14:57:44 +0100", "Message-ID": "\n <4f505a9aa0a6fcfa6ce5ecc94632ae7b1b6eb437.1714744628.git.anatoly.burakov@intel.com>", "X-Mailer": "git-send-email 2.43.0", "In-Reply-To": "<cover.1714744628.git.anatoly.burakov@intel.com>", "References": "<cover.1713964707.git.anatoly.burakov@intel.com>\n <cover.1714744628.git.anatoly.burakov@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": "From: Piotr Skajewski <piotrx.skajewski@intel.com>\n\nAs there have been a long list of flaws due to missing \"break\" statement\nkernel no longer allow implict fall-through and reports warning during\ncompilation. Convert all \"fall-through\" comments to actual code.\n\nSigned-off-by: Piotr Skajewski <piotrx.skajewski@intel.com>\nReviewed-by: Michael, Alice <alice.michael@intel.com>\n---\n drivers/net/ixgbe/base/ixgbe_82599.c | 7 +++++--\n drivers/net/ixgbe/base/ixgbe_common.c | 8 ++++++--\n drivers/net/ixgbe/base/ixgbe_x550.c | 11 +++++++----\n 3 files changed, 18 insertions(+), 8 deletions(-)", "diff": "diff --git a/drivers/net/ixgbe/base/ixgbe_82599.c b/drivers/net/ixgbe/base/ixgbe_82599.c\nindex 73c7766ecf..562034b242 100644\n--- a/drivers/net/ixgbe/base/ixgbe_82599.c\n+++ b/drivers/net/ixgbe/base/ixgbe_82599.c\n@@ -1719,7 +1719,8 @@ s32 ixgbe_fdir_set_input_mask_82599(struct ixgbe_hw *hw,\n \tcase 0x0000:\n \t\t/* mask VLAN ID */\n \t\tfdirm |= IXGBE_FDIRM_VLANID;\n-\t\t/* fall through */\n+\t\tfdirm |= IXGBE_FDIRM_VLANP;\n+\t\tbreak;\n \tcase 0x0FFF:\n \t\t/* mask VLAN priority */\n \t\tfdirm |= IXGBE_FDIRM_VLANP;\n@@ -2009,7 +2010,9 @@ s32 ixgbe_fdir_add_perfect_filter_82599(struct ixgbe_hw *hw,\n \t\t\tDEBUGOUT(\" Error on src/dst port\\n\");\n \t\t\treturn IXGBE_ERR_CONFIG;\n \t\t}\n-\t\t/* fall through */\n+\t\tinput_mask->formatted.flow_type = IXGBE_ATR_L4TYPE_IPV6_MASK |\n+\t\t\t\t\t\t IXGBE_ATR_L4TYPE_MASK;\n+\t\tbreak;\n \tcase IXGBE_ATR_FLOW_TYPE_TCPV4:\n \tcase IXGBE_ATR_FLOW_TYPE_TUNNELED_TCPV4:\n \tcase IXGBE_ATR_FLOW_TYPE_UDPV4:\ndiff --git a/drivers/net/ixgbe/base/ixgbe_common.c b/drivers/net/ixgbe/base/ixgbe_common.c\nindex dd8310748b..27f633bc2f 100644\n--- a/drivers/net/ixgbe/base/ixgbe_common.c\n+++ b/drivers/net/ixgbe/base/ixgbe_common.c\n@@ -231,7 +231,8 @@ s32 ixgbe_setup_fc_generic(struct ixgbe_hw *hw)\n \t\tif (ret_val != IXGBE_SUCCESS)\n \t\t\tgoto out;\n \n-\t\t/* fall through - only backplane uses autoc */\n+\t\treg = IXGBE_READ_REG(hw, IXGBE_PCS1GANA);\n+\t\tbreak;\n \tcase ixgbe_media_type_fiber_qsfp:\n \tcase ixgbe_media_type_fiber:\n \t\treg = IXGBE_READ_REG(hw, IXGBE_PCS1GANA);\n@@ -4725,7 +4726,10 @@ void ixgbe_set_rxpba_generic(struct ixgbe_hw *hw, int num_pb, u32 headroom,\n \t\trxpktsize <<= IXGBE_RXPBSIZE_SHIFT;\n \t\tfor (; i < (num_pb / 2); i++)\n \t\t\tIXGBE_WRITE_REG(hw, IXGBE_RXPBSIZE(i), rxpktsize);\n-\t\t/* fall through - configure remaining packet buffers */\n+\t\trxpktsize = (pbsize / (num_pb - i)) << IXGBE_RXPBSIZE_SHIFT;\n+\t\tfor (; i < num_pb; i++)\n+\t\t\tIXGBE_WRITE_REG(hw, IXGBE_RXPBSIZE(i), rxpktsize);\n+\t\tbreak;\n \tcase PBA_STRATEGY_EQUAL:\n \t\trxpktsize = (pbsize / (num_pb - i)) << IXGBE_RXPBSIZE_SHIFT;\n \t\tfor (; i < num_pb; i++)\ndiff --git a/drivers/net/ixgbe/base/ixgbe_x550.c b/drivers/net/ixgbe/base/ixgbe_x550.c\nindex ebfc78235e..29055a818a 100644\n--- a/drivers/net/ixgbe/base/ixgbe_x550.c\n+++ b/drivers/net/ixgbe/base/ixgbe_x550.c\n@@ -326,8 +326,7 @@ STATIC s32 ixgbe_identify_phy_x550em(struct ixgbe_hw *hw)\n \t\t/* set up for CS4227 usage */\n \t\tixgbe_setup_mux_ctl(hw);\n \t\tixgbe_check_cs4227(hw);\n-\t\t/* Fallthrough */\n-\n+\t\treturn ixgbe_identify_sfp_module_X550em(hw);\n \tcase IXGBE_DEV_ID_X550EM_A_SFP_N:\n \t\treturn ixgbe_identify_sfp_module_X550em(hw);\n \t\tbreak;\n@@ -1838,7 +1837,9 @@ s32 ixgbe_get_link_capabilities_X550em(struct ixgbe_hw *hw,\n \t\t\t\t\tbreak;\n \t\t\t\t}\n \t\t\t}\n-\t\t\t/* fall through */\n+\t\t\t*speed = IXGBE_LINK_SPEED_10GB_FULL |\n+\t\t\t\t IXGBE_LINK_SPEED_1GB_FULL;\n+\t\t\tbreak;\n \t\tdefault:\n \t\t\t*speed = IXGBE_LINK_SPEED_10GB_FULL |\n \t\t\t\t IXGBE_LINK_SPEED_1GB_FULL;\n@@ -3558,7 +3559,9 @@ u64 ixgbe_get_supported_physical_layer_X550em(struct ixgbe_hw *hw)\n \t\t\t\tbreak;\n \t\t\t}\n \t\t}\n-\t\t/* fall through */\n+\t\tphysical_layer = IXGBE_PHYSICAL_LAYER_10GBASE_KR |\n+\t\t\t\t IXGBE_PHYSICAL_LAYER_1000BASE_KX;\n+\t\tbreak;\n \tcase ixgbe_phy_x550em_xfi:\n \t\tphysical_layer = IXGBE_PHYSICAL_LAYER_10GBASE_KR |\n \t\t\t\t IXGBE_PHYSICAL_LAYER_1000BASE_KX;\n", "prefixes": [ "v2", "13/27" ] }