From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mails.dpdk.org (mails.dpdk.org [217.70.189.124]) by inbox.dpdk.org (Postfix) with ESMTP id AAF584625C for ; Tue, 18 Feb 2025 13:43:17 +0100 (CET) Received: from mails.dpdk.org (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id A57C6402A0; Tue, 18 Feb 2025 13:43:17 +0100 (CET) Received: from NAM12-BN8-obe.outbound.protection.outlook.com (mail-bn8nam12on2040.outbound.protection.outlook.com [40.107.237.40]) by mails.dpdk.org (Postfix) with ESMTP id D568A402A0 for ; Tue, 18 Feb 2025 13:43:15 +0100 (CET) ARC-Seal: i=1; a=rsa-sha256; s=arcselector10001; d=microsoft.com; cv=none; b=v59uhvGCL/yXktX1U3VMnAyPdY5SuwE+bilC+0HXEm2hbOaU9zG+fS1ykyM6Pl68vMQwtxzWAfzy73BnaDyVOe8UC7Jke+kZajsFB2qCVgkpNxPbx3nKPdniF0RXI8AvGNevUsz7vgmNbb7Fg1IV+31D19m7dyCAAao/UGzdVNo2y6bfFRGcT/+/o7q6/VD9yWUjg+8h7bbp37F+MtM9EiLXOyGk+gAbXe8vG97ukJRGiQDZ/Ahz+rve8IkovUDALV7UK/D8NvhK1DNGuKk4ch71nLrn9fISQ4H76ypF9pAt8hTA8PONdaafEO2qeEY0QM8kyyyRutUTKmkxC7vVWA== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector10001; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-AntiSpam-MessageData-ChunkCount:X-MS-Exchange-AntiSpam-MessageData-0:X-MS-Exchange-AntiSpam-MessageData-1; bh=KdJkmwlBYvVWOaJttfDgGexBnyars9f8JtVZvsQN6hg=; b=a4wyGkCehHtjH/xFH/p3nBjyl6q57JhCAvGaGM8w4QmncB0nTvgq+Ly0TslvbyNua+moHRdZ9aUh7YSlmImKjgDKLX/dm38Q+4MtmqYtOSy3I9soyxH77x3MsfzlgRGcuCTPwWopMnTH6L5EfLc54AlCtt+2xqcxGgmcNGzcoU4jHPQOaZy1MxFAlz5cEibkZDDlBq0eHUuSxkTLKX1M3zQpXHJmBrR3Sk8SNYmjQt0tQCs6DuFp3wRJhnQtY++Twd/qqgG/yhpM4ExqAr54wkJOWO+jt1P719d6NYG+J227tEFPr80b2QBAiQctyR02oX0LqHl84bqu6aBnqbrxzQ== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass (sender ip is 216.228.117.161) smtp.rcpttodomain=huawei.com smtp.mailfrom=nvidia.com; dmarc=pass (p=reject sp=reject pct=100) action=none header.from=nvidia.com; dkim=none (message not signed); arc=none (0) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=Nvidia.com; s=selector2; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=KdJkmwlBYvVWOaJttfDgGexBnyars9f8JtVZvsQN6hg=; b=kHSVqBAk0NMCToWLVguHoAw0lQIgjvIMU/Fv4soAgbzXlxJDxyN9lCFXfcOZH0jR7Qm+ekeP2d1QjI6ssF57EJCtvmIR9KvIhBy3evaxr8XazcrhA9FruhzSk77JwdrOZ7IBTk0nZK/0V/SrjiP4Gdk6hkjArKdmg8yJGHhP0dZW3OeU5c6LRsUeucDbg8WJX5MEari0wSSVbhFZPpvB9lL1g19hUUnjRFX+f4KfQL4d0WR2YbzuG2cDBMhuWPZJxLV09tTgHJLT6EpHeEXcSZMz2sAfWZHmnTgIoRlBgxAnHpaTu2fXmYcOrNvSE4VDcFuG2I2eBe6yKmUth55AXQ== Received: from BY5PR17CA0067.namprd17.prod.outlook.com (2603:10b6:a03:167::44) by CY8PR12MB7683.namprd12.prod.outlook.com (2603:10b6:930:86::5) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.8445.19; Tue, 18 Feb 2025 12:43:10 +0000 Received: from SA2PEPF000015CA.namprd03.prod.outlook.com (2603:10b6:a03:167:cafe::8b) by BY5PR17CA0067.outlook.office365.com (2603:10b6:a03:167::44) with Microsoft SMTP Server (version=TLS1_3, cipher=TLS_AES_256_GCM_SHA384) id 15.20.8466.14 via Frontend Transport; Tue, 18 Feb 2025 12:43:09 +0000 X-MS-Exchange-Authentication-Results: spf=pass (sender IP is 216.228.117.161) smtp.mailfrom=nvidia.com; dkim=none (message not signed) header.d=none;dmarc=pass action=none header.from=nvidia.com; Received-SPF: Pass (protection.outlook.com: domain of nvidia.com designates 216.228.117.161 as permitted sender) receiver=protection.outlook.com; client-ip=216.228.117.161; helo=mail.nvidia.com; pr=C Received: from mail.nvidia.com (216.228.117.161) by SA2PEPF000015CA.mail.protection.outlook.com (10.167.241.200) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.8466.11 via Frontend Transport; Tue, 18 Feb 2025 12:43:08 +0000 Received: from rnnvmail201.nvidia.com (10.129.68.8) by mail.nvidia.com (10.129.200.67) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.2.1544.4; Tue, 18 Feb 2025 04:42:56 -0800 Received: from nvidia.com (10.126.230.35) by rnnvmail201.nvidia.com (10.129.68.8) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.2.1544.14; Tue, 18 Feb 2025 04:42:54 -0800 From: Xueming Li To: Dengdui Huang CC: , dpdk stable Subject: patch 'net/hns3: fix reset timeout' has been queued to stable release 23.11.4 Date: Tue, 18 Feb 2025 20:35:08 +0800 Message-ID: <20250218123523.36836-74-xuemingl@nvidia.com> X-Mailer: git-send-email 2.34.1 In-Reply-To: <20250218123523.36836-1-xuemingl@nvidia.com> References: <20250218123523.36836-1-xuemingl@nvidia.com> MIME-Version: 1.0 Content-Transfer-Encoding: 8bit Content-Type: text/plain X-Originating-IP: [10.126.230.35] X-ClientProxiedBy: rnnvmail201.nvidia.com (10.129.68.8) To rnnvmail201.nvidia.com (10.129.68.8) X-EOPAttributedMessage: 0 X-MS-PublicTrafficType: Email X-MS-TrafficTypeDiagnostic: SA2PEPF000015CA:EE_|CY8PR12MB7683:EE_ X-MS-Office365-Filtering-Correlation-Id: 36cca130-e250-4669-f2d0-08dd5019cc1c X-MS-Exchange-SenderADCheck: 1 X-MS-Exchange-AntiSpam-Relay: 0 X-Microsoft-Antispam: BCL:0; ARA:13230040|36860700013|82310400026|1800799024|376014|13003099007|7053199007; X-Microsoft-Antispam-Message-Info: =?us-ascii?Q?nstLLrD51hmCC+Lh/USl2gmrEkybvor6jaRHTJtGx3o6ijO1fsF9M3MDIOn0?= =?us-ascii?Q?ptt2B3fkibfb1aVBEGz9sQNNSBFGXfe+kPmLLNEDT0km+SIB6DAH1Mps1DH9?= =?us-ascii?Q?Cp5LhZbJAlcbJi5YMWbb0PJNauxQ/bWdji5Hz2x3FhUhSog1K+r6r3x6WuYD?= =?us-ascii?Q?DWp2UZOI+Bc2MnAYEO5x1jJNzneSpFE0RqLucpBHzsOoUafsC4Fb7aQQEHcV?= =?us-ascii?Q?P0HU+JqJqF9EItskZbfX8jqVFL0h7qBZKKw2NKKw9CwZ1+mAXCzMkLlh6TsX?= =?us-ascii?Q?3ciy9FZjX/SVTO0GtpgoadwpoR1K18EhQrR6yfU94TDof47XkF9fGjudskWd?= =?us-ascii?Q?sjrZzQjGA2gfTdeafQT0rqv8P9xQ/SypJK85T5CSPMKgl9VvzfjXG2uJ677S?= =?us-ascii?Q?GS7jYkNkcMZVIdiEPgpIGhWjfn5UtVFAFrxOTpMndu32X8amTE6Z9CJThoqk?= =?us-ascii?Q?xIUs0ElESB8YR5CZbThen+XiSHMU5cbmWmxJkqfnkM4badOtlsYg022sqzO9?= =?us-ascii?Q?dap0YBeFFYWGD/q65ilL2x/5uFph6ztPYiLwYLEaSXCorjQrhPsjvZ8iLn11?= =?us-ascii?Q?SmSeVKcfieAO91wUlfXDN7vCM1113IfPFDfd7rGfbyfVqpjJD73hA2gib7qn?= =?us-ascii?Q?tj4uh7FCf0VhO9lQ779liRi6cHnKZUbD1MTNvovn0CL+HzCr2QyoPafK0UAh?= =?us-ascii?Q?Vr8bYdf6bW7vMnhm5mCLnx1IZCYrtQDMsHTdWqKw56OIB085+Ta636BX+sYk?= =?us-ascii?Q?5ftNXzQPUCpq1suAk1J8TBaQMtnIGcAl5HTZV4DaOEBZB/gOAxR5aMDcCtVV?= =?us-ascii?Q?INnyYV/GrAQNg3nok/wk1SN9404044wyabN4xSsR2ylQ/E0dPgLa2rwBBJ8a?= =?us-ascii?Q?4LUbdjzR+w6gc8xTK3l2dJuhigG+Xd6hMH2SGcDM+R23hI5JVogazhA8ikI1?= =?us-ascii?Q?30CrlLHbgL4GivxD1hfZSg/ZAXqyAOjwvIf0QVwnwEeKdIfadr5/a7cA3gTS?= =?us-ascii?Q?m/e5hWpiOuyLqmGAmBfFW8Sr1j8P4xeXzjv8dEQ7XRnEFp6GGKN/1CZiXgRD?= =?us-ascii?Q?aTotPZISWwqEy48AcBxz33iIImf1RVrWHFZ5i+2nGGSutcgwmBpgKsoY2dKY?= =?us-ascii?Q?44XdOl6nXRsmQ2i5UV6Lu/EuIgZB5rGRFO7l2SmZIAiM5sxuCbN2aXJVmh4M?= =?us-ascii?Q?UQYBb/Fb/Xa7MDu+1X0NHo6RTkUxfBgOKrqgQ7E62HtPEDYJbEOJ0xpbGhUW?= =?us-ascii?Q?W32vIleTS4AAV1xz+7xT6jGwPbo4jjjqpZhocOplLVrVH1y/pfAiY56Xp2ER?= =?us-ascii?Q?iLQKWhARB35gaK229eChUtUhJ3fgd+GsgOob54KlZS5YRsUxZOkm0uHeYG81?= =?us-ascii?Q?ko5SnqZcBzFlYQEqXCGXNJQuPuovKgdrXIwn0lmOkaUsvXf77vvU7pQj71PT?= =?us-ascii?Q?hrR44inzkPI9hBo2susHxuXvZXn0l4itwdIZpKWPGoX1dxUDbH1uOHzrMjzu?= =?us-ascii?Q?GZMHL4reEV89sT8=3D?= X-Forefront-Antispam-Report: CIP:216.228.117.161; CTRY:US; LANG:en; SCL:1; SRV:; IPV:NLI; SFV:NSPM; H:mail.nvidia.com; PTR:dc6edge2.nvidia.com; CAT:NONE; SFS:(13230040)(36860700013)(82310400026)(1800799024)(376014)(13003099007)(7053199007); DIR:OUT; SFP:1101; X-OriginatorOrg: Nvidia.com X-MS-Exchange-CrossTenant-OriginalArrivalTime: 18 Feb 2025 12:43:08.4238 (UTC) X-MS-Exchange-CrossTenant-Network-Message-Id: 36cca130-e250-4669-f2d0-08dd5019cc1c X-MS-Exchange-CrossTenant-Id: 43083d15-7273-40c1-b7db-39efd9ccc17a X-MS-Exchange-CrossTenant-OriginalAttributedTenantConnectingIp: TenantId=43083d15-7273-40c1-b7db-39efd9ccc17a; Ip=[216.228.117.161]; Helo=[mail.nvidia.com] X-MS-Exchange-CrossTenant-AuthSource: SA2PEPF000015CA.namprd03.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Anonymous X-MS-Exchange-CrossTenant-FromEntityHeader: HybridOnPrem X-MS-Exchange-Transport-CrossTenantHeadersStamped: CY8PR12MB7683 X-BeenThere: stable@dpdk.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: patches for DPDK stable branches List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: stable-bounces@dpdk.org Hi, FYI, your patch has been queued to stable release 23.11.4 Note it hasn't been pushed to http://dpdk.org/browse/dpdk-stable yet. Please shout if anyone has objections. Also note that after the patch there's a diff of the upstream commit vs the patch applied to the branch. This will indicate if there was any rebasing needed to apply to the stable branch. If there were code changes for rebasing (ie: not only metadata diffs), please double check that the rebase was correctly done. Queued patches are on a temporary branch at: https://git.dpdk.org/dpdk-stable/log/?h=23.11-staging This queued commit can be viewed at: https://git.dpdk.org/dpdk-stable/commit/?h=23.11-staging&id=d6d17fcb1e2175c0a6e798c8749b06aa97948e87 Thanks. Xueming Li --- >From d6d17fcb1e2175c0a6e798c8749b06aa97948e87 Mon Sep 17 00:00:00 2001 From: Dengdui Huang Date: Mon, 10 Feb 2025 11:01:13 +0800 Subject: [PATCH] net/hns3: fix reset timeout Cc: Xueming Li [ upstream commit 9f7c28c5e98062576dfbf555cd5ede7e33d6624b ] There is low probability that the driver reset timeout, the root cause is that the firmware processing take a litter long than normal when process reset command. This patch fix it by changing the timeout of the reset command to 100 ms. Fixes: 737f30e1c3ab ("net/hns3: support command interface with firmware") Signed-off-by: Dengdui Huang --- drivers/net/hns3/hns3_cmd.c | 18 ++++++++++++------ drivers/net/hns3/hns3_cmd.h | 4 ++-- 2 files changed, 14 insertions(+), 8 deletions(-) diff --git a/drivers/net/hns3/hns3_cmd.c b/drivers/net/hns3/hns3_cmd.c index 2c1664485b..62da6dd160 100644 --- a/drivers/net/hns3/hns3_cmd.c +++ b/drivers/net/hns3/hns3_cmd.c @@ -304,8 +304,17 @@ hns3_cmd_get_hardware_reply(struct hns3_hw *hw, return hns3_cmd_convert_err_code(desc_ret); } -static int hns3_cmd_poll_reply(struct hns3_hw *hw) +static uint32_t hns3_get_cmd_tx_timeout(uint16_t opcode) { + if (opcode == HNS3_OPC_CFG_RST_TRIGGER) + return HNS3_COMQ_CFG_RST_TIMEOUT; + + return HNS3_CMDQ_TX_TIMEOUT_DEFAULT; +} + +static int hns3_cmd_poll_reply(struct hns3_hw *hw, uint16_t opcode) +{ + uint32_t cmdq_tx_timeout = hns3_get_cmd_tx_timeout(opcode); struct hns3_adapter *hns = HNS3_DEV_HW_TO_ADAPTER(hw); uint32_t timeout = 0; @@ -326,7 +335,7 @@ static int hns3_cmd_poll_reply(struct hns3_hw *hw) rte_delay_us(1); timeout++; - } while (timeout < hw->cmq.tx_timeout); + } while (timeout < cmdq_tx_timeout); hns3_err(hw, "Wait for reply timeout"); return -ETIME; } @@ -400,7 +409,7 @@ hns3_cmd_send(struct hns3_hw *hw, struct hns3_cmd_desc *desc, int num) * if multi descriptors to be sent, use the first one to check. */ if (HNS3_CMD_SEND_SYNC(rte_le_to_cpu_16(desc->flag))) { - retval = hns3_cmd_poll_reply(hw); + retval = hns3_cmd_poll_reply(hw, desc->opcode); if (!retval) retval = hns3_cmd_get_hardware_reply(hw, desc, num, ntc); @@ -611,9 +620,6 @@ hns3_cmd_init_queue(struct hns3_hw *hw) hw->cmq.csq.desc_num = HNS3_NIC_CMQ_DESC_NUM; hw->cmq.crq.desc_num = HNS3_NIC_CMQ_DESC_NUM; - /* Setup Tx write back timeout */ - hw->cmq.tx_timeout = HNS3_CMDQ_TX_TIMEOUT; - /* Setup queue rings */ ret = hns3_alloc_cmd_queue(hw, HNS3_TYPE_CSQ); if (ret) { diff --git a/drivers/net/hns3/hns3_cmd.h b/drivers/net/hns3/hns3_cmd.h index 79a8c1edad..4d707c13b2 100644 --- a/drivers/net/hns3/hns3_cmd.h +++ b/drivers/net/hns3/hns3_cmd.h @@ -10,7 +10,8 @@ #include #include -#define HNS3_CMDQ_TX_TIMEOUT 30000 +#define HNS3_CMDQ_TX_TIMEOUT_DEFAULT 30000 +#define HNS3_COMQ_CFG_RST_TIMEOUT 100000 #define HNS3_CMDQ_CLEAR_WAIT_TIME 200 #define HNS3_CMDQ_RX_INVLD_B 0 #define HNS3_CMDQ_RX_OUTVLD_B 1 @@ -62,7 +63,6 @@ enum hns3_cmd_return_status { struct hns3_cmq { struct hns3_cmq_ring csq; struct hns3_cmq_ring crq; - uint16_t tx_timeout; enum hns3_cmd_return_status last_status; }; -- 2.34.1 --- Diff of the applied patch vs upstream commit (please double-check if non-empty: --- --- - 2025-02-18 19:39:03.254276995 +0800 +++ 0073-net-hns3-fix-reset-timeout.patch 2025-02-18 19:39:00.698244036 +0800 @@ -1 +1 @@ -From 9f7c28c5e98062576dfbf555cd5ede7e33d6624b Mon Sep 17 00:00:00 2001 +From d6d17fcb1e2175c0a6e798c8749b06aa97948e87 Mon Sep 17 00:00:00 2001 @@ -4,0 +5,3 @@ +Cc: Xueming Li + +[ upstream commit 9f7c28c5e98062576dfbf555cd5ede7e33d6624b ] @@ -12 +14,0 @@ -Cc: stable@dpdk.org @@ -21 +23 @@ -index 146444e2fa..398b75384e 100644 +index 2c1664485b..62da6dd160 100644