From patchwork Mon Jan 11 11:35:33 2021 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Joakim Zhang X-Patchwork-Id: 12010429 X-Patchwork-Delegate: kuba@kernel.org Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-18.7 required=3.0 tests=BAYES_00,DKIM_SIGNED, DKIM_VALID,DKIM_VALID_AU,HEADER_FROM_DIFFERENT_DOMAINS,INCLUDES_CR_TRAILER, INCLUDES_PATCH,MAILING_LIST_MULTI,MSGID_FROM_MTA_HEADER,SPF_HELO_NONE, SPF_PASS,URIBL_BLOCKED,USER_AGENT_GIT autolearn=ham autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 844CBC433DB for ; Mon, 11 Jan 2021 11:37:39 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by mail.kernel.org (Postfix) with ESMTP id 5333B224BD for ; Mon, 11 Jan 2021 11:37:39 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1729800AbhAKLhi (ORCPT ); Mon, 11 Jan 2021 06:37:38 -0500 Received: from mail-db8eur05on2050.outbound.protection.outlook.com ([40.107.20.50]:16480 "EHLO EUR05-DB8-obe.outbound.protection.outlook.com" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1729760AbhAKLhi (ORCPT ); Mon, 11 Jan 2021 06:37:38 -0500 ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=FFeG1lNQnVJHw1mm8At0ZE/Db5S94rmIJPxXtvN0yJ7FXNY3uJcqm1D5Mdq/2d9qcXV8e4vhG4hE3GlJC06686H0M0XAcWXXEio9dZyKQ1iaO36Hrlku0fjJsvxrEJeNLnU4Xkqtsvgae+cx8fbiFA4RJFHmsuLatNrbuGW3y4DsEPymnPqzTajBkI6+dvokf9b0YFXLnDOg0iP6lowJ1VfbpSD6Zi00Oj4n/qf+2dEZSi7aV4olQWkX8d+SyQyFR0jMJS/iXaWesQPhVAnaF5OhHbIFTkHOVjrWhZH3YuqxZmdMgFpE5FJHpxo2uOnuxpNSEX4ws8fmKolaICBAAQ== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector9901; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=T6Cz6xIAd5hUTGHQNYSzeNm6ProxYK7NTFlETU5NANI=; b=JsoXEDi53abibFdzTEeSlPANMwog+odsHbbrj1N3t/DmzqXQ+lddBpjTTD9/DAqPcyn42jViXh/9mybNM+IKpTT+MqaPGqXQdkVOO55dhiPH4gsRdl8eDaNkQWMh1dhSTQ9FqRORNCPMGrrLMtBCUWhaCmJIxRGvF1r0faaYza9kAFy2dCDb+imlP7G+GU3OXwZWUpTsuqXeKeWBnsYYgdsoSPEzaMZs/jnotp5cdQPwiKDSy5H0eX2P/BRWneZcOSYxQ5gIvMgXNDROUIgs4zgKp7oJ2uYDjbwSHOmvFoPIVhKOUgZONmjCIiTM2ApdO6SghyJPlK/ZW/K1/EqC1Q== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=nxp.com; dmarc=pass action=none header.from=nxp.com; dkim=pass header.d=nxp.com; arc=none DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=nxp.com; s=selector2; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=T6Cz6xIAd5hUTGHQNYSzeNm6ProxYK7NTFlETU5NANI=; b=s2txCUS19xbItyY0ft8mHBAHdQyl5pmY4fQ2SpBgb3kfULOIGDkBaRlM9QjAp8C8JPpKJxa0mVceRH/oeBSKmV9/sqH/QYANSJLuYHzECJOB4N0okbZofLSF7Ts0vaCJiJqTvZN5WXD/XXEH0yxgCTW7PggknvVnik4cHBQSA7E= Authentication-Results: st.com; dkim=none (message not signed) header.d=none;st.com; dmarc=none action=none header.from=nxp.com; Received: from DB8PR04MB6795.eurprd04.prod.outlook.com (2603:10a6:10:fa::15) by DB6PR0401MB2326.eurprd04.prod.outlook.com (2603:10a6:4:47::6) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.3742.12; Mon, 11 Jan 2021 11:36:18 +0000 Received: from DB8PR04MB6795.eurprd04.prod.outlook.com ([fe80::b83c:2edc:17e8:2666]) by DB8PR04MB6795.eurprd04.prod.outlook.com ([fe80::b83c:2edc:17e8:2666%5]) with mapi id 15.20.3742.012; Mon, 11 Jan 2021 11:36:18 +0000 From: Joakim Zhang To: peppe.cavallaro@st.com, alexandre.torgue@st.com, joabreu@synopsys.com, davem@davemloft.net, kuba@kernel.org Cc: netdev@vger.kernel.org, linux-imx@nxp.com Subject: [PATCH 1/6] ethernet: stmmac: remove redundant null check for ptp clock Date: Mon, 11 Jan 2021 19:35:33 +0800 Message-Id: <20210111113538.12077-2-qiangqing.zhang@nxp.com> X-Mailer: git-send-email 2.17.1 In-Reply-To: <20210111113538.12077-1-qiangqing.zhang@nxp.com> References: <20210111113538.12077-1-qiangqing.zhang@nxp.com> X-Originating-IP: [119.31.174.71] X-ClientProxiedBy: SG2PR0601CA0018.apcprd06.prod.outlook.com (2603:1096:3::28) To DB8PR04MB6795.eurprd04.prod.outlook.com (2603:10a6:10:fa::15) MIME-Version: 1.0 X-MS-Exchange-MessageSentRepresentingType: 1 Received: from localhost.localdomain (119.31.174.71) by SG2PR0601CA0018.apcprd06.prod.outlook.com (2603:1096:3::28) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.3742.6 via Frontend Transport; Mon, 11 Jan 2021 11:36:15 +0000 X-MS-PublicTrafficType: Email X-MS-Office365-Filtering-HT: Tenant X-MS-Office365-Filtering-Correlation-Id: 4aebac91-0457-46f1-b213-08d8b6251ca7 X-MS-TrafficTypeDiagnostic: DB6PR0401MB2326: X-MS-Exchange-Transport-Forked: True X-Microsoft-Antispam-PRVS: X-MS-Oob-TLC-OOBClassifiers: OLM:3968; X-MS-Exchange-SenderADCheck: 1 X-Microsoft-Antispam: BCL:0; X-Microsoft-Antispam-Message-Info: VN+W5clgbNLcNcuy3MnIyyVLqqbIAnMo9SKiV65mZXlmOQyWHAkexfwGQiFOMwsjZSKMEt/7ZXmZ0aHgWslGSDIzCYd2fXgTNLPcDNCca4noY5qmmbJKpx2TcFMV7OVFP4us0nz9VkH/T7DHrtzogm/ITKAkhCNVYifLKCDsQcke7xRkHPN6hUfTCOeSfuFKmfcxX2UktrrbZ1OdPXBV3ZNJ0ZRLoFY4bsg0Wvly8reytd0itX1fF/XvrOsTqkjHm2erX3/xwvp/a2HsAn+cweDV8fMYWeU5mOnSkxjjY31iKOt7NXV+DT4QT8HHFIsWpQVc6fmeTwCpsE5Nd8RHvLQCv84woKRX6YLkw0FxZDrQOiS7vNyEjYHuZyOpjjCbAcJhf6zTsl5sfvvWPXaKlfJ8aQbv7InbyCOXIB4GvntbnP6Shd6SQKcfB34O306cgiasbHoRlYzD4iQaRXCHAQ== X-Forefront-Antispam-Report: CIP:255.255.255.255;CTRY:;LANG:en;SCL:1;SRV:;IPV:NLI;SFV:NSPM;H:DB8PR04MB6795.eurprd04.prod.outlook.com;PTR:;CAT:NONE;SFS:(4636009)(376002)(39860400002)(396003)(136003)(366004)(346002)(66476007)(4326008)(66556008)(8676002)(4744005)(86362001)(316002)(1076003)(69590400011)(6512007)(6486002)(52116002)(5660300002)(36756003)(478600001)(8936002)(16526019)(956004)(186003)(2616005)(26005)(2906002)(6506007)(83380400001)(6666004)(66946007);DIR:OUT;SFP:1101; X-MS-Exchange-AntiSpam-MessageData: hAjSRb4xpLNavO+r+V8jztz4MxGHHZEOkmCDfeZDB/ShpZeWrlV1tmqr7ATCkLA6WmOsXaQ7sL+qr6b9OnGgWA1k3GUd42MFQdEUmeC+cfobS+ZyCeXTCGUfQS20jFScgu3SWbdKW9KYQbBQOwhlawDZPssL/LfBmxJOKoWKT8nn+D9k+JjST/cCfqRxnP6Z40NIy4yHNk/SekVNfvqyfbrLVC4nz9iHAwxgwUEuQv9sHdlxGUxXEaMDqauUhgZIeFex1MC+2uIhY3sb10tpTagTFsgIneMODL2/vJRs2wfYFJlaCieBuhhpALACJlYo/0G2yFRFUGkLVrXp+hIo/w5u1Z7RujWtAoWlDGTpA9KlU3WXZ352jT3OKnQ2VRrWM8IeTaTTOJlMNRigVn20G15niFZI9Vh59gNwM0dd4+mgcWyuACUh2NfjzpeaKrICrFMNRldKXMYBTzuI9Nf/Jx3R05JViMzvwXNr9wMaekJTrXko/XeRpvyGHiFwjzdpM7qC9jzigeZFD4+sI+bxXfzKUo0rAq2vVY5owW1IeY2PSoLeRSW656Qkn8NtkxEb7H9PWyTAU9Nc4rwfbySq4/UOXXqyllKjv+EnC665uvsLakuHFiUUWhSwDe2nAxaAxdVLLnkNj76QpYilW5HdgTGc+eNiTso7yQx5cjNEL1dZvLNPTGjCEUrHi8ppfXg3Hxdv3ZMzAua+T7dLpqemnlE4m1ZrdtAEkLjgKhLFAZX+ZsT0lHHKr5F+FsNa0e2wMVo0bYRoVj7mZMBCMMF++HfdnhhrcpriMtQ2fRaNcromgHrpoMoIe1VY4jmG+MYH0x/BnciXkco3JZCuekI1/2DokW94uJCt20GoXqhPfhMQogUfUvzQPqXQ5oqM5bcIToqYlJ59HAsNxgeMcQn3mM+3TjJiLmzStowemcgCbAN+v9nexB3ZxRgpsM265ZWFf1IZckCKnIHuNA79MMfI3zKYgcuoVeQm0FdkA5OA+gCcfXklDVz2u/sFgIu2Ee5i X-OriginatorOrg: nxp.com X-MS-Exchange-CrossTenant-AuthSource: DB8PR04MB6795.eurprd04.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 11 Jan 2021 11:36:18.2418 (UTC) X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted X-MS-Exchange-CrossTenant-Id: 686ea1d3-bc2b-4c6f-a92c-d99c5c301635 X-MS-Exchange-CrossTenant-Network-Message-Id: 4aebac91-0457-46f1-b213-08d8b6251ca7 X-MS-Exchange-CrossTenant-MailboxType: HOSTED X-MS-Exchange-CrossTenant-UserPrincipalName: q/xQi0qZLp40ibbXfjLm2K4zLh1I8xyhi0I+fynvRop2fN20zncHMhCkvlri3AenSDSUmNIBZrLrYWqSaiCF1g== X-MS-Exchange-Transport-CrossTenantHeadersStamped: DB6PR0401MB2326 Precedence: bulk List-ID: X-Mailing-List: netdev@vger.kernel.org X-Patchwork-Delegate: kuba@kernel.org Remove redundant null check for ptp clock. Fixes: 1c35cc9cf6a0 ("net: stmmac: remove redundant null check before clk_disable_unprepare()") Signed-off-by: Joakim Zhang Reviewed-by: Andrew Lunn --- drivers/net/ethernet/stmicro/stmmac/stmmac_main.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/drivers/net/ethernet/stmicro/stmmac/stmmac_main.c b/drivers/net/ethernet/stmicro/stmmac/stmmac_main.c index 5b1c12ff98c0..a0bd064a8421 100644 --- a/drivers/net/ethernet/stmicro/stmmac/stmmac_main.c +++ b/drivers/net/ethernet/stmicro/stmmac/stmmac_main.c @@ -5290,8 +5290,7 @@ int stmmac_resume(struct device *dev) /* enable the clk previously disabled */ clk_prepare_enable(priv->plat->stmmac_clk); clk_prepare_enable(priv->plat->pclk); - if (priv->plat->clk_ptp_ref) - clk_prepare_enable(priv->plat->clk_ptp_ref); + clk_prepare_enable(priv->plat->clk_ptp_ref); /* reset the phy so that it's ready */ if (priv->mii) stmmac_mdio_reset(priv->mii); From patchwork Mon Jan 11 11:35:34 2021 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Joakim Zhang X-Patchwork-Id: 12010427 X-Patchwork-Delegate: kuba@kernel.org Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-18.7 required=3.0 tests=BAYES_00,DKIM_SIGNED, DKIM_VALID,DKIM_VALID_AU,HEADER_FROM_DIFFERENT_DOMAINS,INCLUDES_CR_TRAILER, INCLUDES_PATCH,MAILING_LIST_MULTI,MSGID_FROM_MTA_HEADER,SPF_HELO_NONE, SPF_PASS,URIBL_BLOCKED,USER_AGENT_GIT autolearn=ham autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id D4457C433E0 for ; Mon, 11 Jan 2021 11:37:34 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by mail.kernel.org (Postfix) with ESMTP id AB14D229C4 for ; Mon, 11 Jan 2021 11:37:34 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1729779AbhAKLh2 (ORCPT ); Mon, 11 Jan 2021 06:37:28 -0500 Received: from mail-eopbgr40076.outbound.protection.outlook.com ([40.107.4.76]:22789 "EHLO EUR03-DB5-obe.outbound.protection.outlook.com" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1728731AbhAKLh1 (ORCPT ); Mon, 11 Jan 2021 06:37:27 -0500 ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=m66hrmv2DxXavgftaTWDpjHzF9AO+sEzRxKIL1pdPZ593aDC5ilyXhlnTo1dTEIw1ywZ1F2HAwJFkq9jhF1ZlojSCNoJN7ah14ZIuM9YU56VIxmiQ6SggoDwOo56fX1OFOGssBkBeql9vBmdfnIcZ+xbRcbOSBPvoDBYCK4wwzF1Wi6QL0as5/9tj50ycrA8NGQvW+rOWaJ+vDng1ze8mpP3uCAfEofAaalXGGyBABEcH+0DMWJD9ZxrZ0lFt3pjMpFq0R1qVZCDlCUwFKUIhOoQxDWszPunmBHADvQqolExBigF74glJqw1OMTDzEXng1yOlbsCyILoHFWZP48JVA== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector9901; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=VW0t89zZNGF+H+ySTgYr3rggt/tlnAucOmD85xc44Os=; b=QvA9QnaZKRouhTIZB9nGVis6GRKzTSNs9y3tAAYME/Qv+1hAtKVcBnSqPlI48JdkDCrF1qdyJEsVVvECwbNDFYxJazTC7kcNqIWmSNqL4Y5bDlW823IbhJQcvcsnMwWmn2IxD+1IjMvgZZKuS6ekuRDyAhrMu+vZxBg0bylqjqZBGqy1lWDSCauti/DvqGG+Cc95q5aAFIJWFtP21JKeuW2g2A26YFmxQmM0vxWzMzkpHmD/5FPaIaeeWyn5eLIrxpMgMGyufh+r/DOi65Bj4vFpftTOYYW/TvebdGxxbhDlTtAHnt1buwUOSwuIh+RhLTDSmh3JaxEvy0GyyUem+A== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=nxp.com; dmarc=pass action=none header.from=nxp.com; dkim=pass header.d=nxp.com; arc=none DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=nxp.com; s=selector2; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=VW0t89zZNGF+H+ySTgYr3rggt/tlnAucOmD85xc44Os=; b=R+rMk9xtxG4olfoTkqzoq4cpnISBk2wXI7R/AIWClpLM+wUe2OgPEMDiCD0JVpVfSAUDXtLLU/3BVhNFoTQCPtyUEjO2dMKk884+3zrLC3S9sTMoXUzNmz4lMdqhe8cuYjINXOVHc4MzSpunDEkfmKuZC/NSD6y7xM7fVwkXtn0= Authentication-Results: st.com; dkim=none (message not signed) header.d=none;st.com; dmarc=none action=none header.from=nxp.com; Received: from DB8PR04MB6795.eurprd04.prod.outlook.com (2603:10a6:10:fa::15) by DB6PR0401MB2326.eurprd04.prod.outlook.com (2603:10a6:4:47::6) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.3742.12; Mon, 11 Jan 2021 11:36:21 +0000 Received: from DB8PR04MB6795.eurprd04.prod.outlook.com ([fe80::b83c:2edc:17e8:2666]) by DB8PR04MB6795.eurprd04.prod.outlook.com ([fe80::b83c:2edc:17e8:2666%5]) with mapi id 15.20.3742.012; Mon, 11 Jan 2021 11:36:21 +0000 From: Joakim Zhang To: peppe.cavallaro@st.com, alexandre.torgue@st.com, joabreu@synopsys.com, davem@davemloft.net, kuba@kernel.org Cc: netdev@vger.kernel.org, linux-imx@nxp.com Subject: [PATCH 2/6] ethernet: stmmac: stop each tx channal independently Date: Mon, 11 Jan 2021 19:35:34 +0800 Message-Id: <20210111113538.12077-3-qiangqing.zhang@nxp.com> X-Mailer: git-send-email 2.17.1 In-Reply-To: <20210111113538.12077-1-qiangqing.zhang@nxp.com> References: <20210111113538.12077-1-qiangqing.zhang@nxp.com> X-Originating-IP: [119.31.174.71] X-ClientProxiedBy: SG2PR0601CA0018.apcprd06.prod.outlook.com (2603:1096:3::28) To DB8PR04MB6795.eurprd04.prod.outlook.com (2603:10a6:10:fa::15) MIME-Version: 1.0 X-MS-Exchange-MessageSentRepresentingType: 1 Received: from localhost.localdomain (119.31.174.71) by SG2PR0601CA0018.apcprd06.prod.outlook.com (2603:1096:3::28) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.3742.6 via Frontend Transport; Mon, 11 Jan 2021 11:36:18 +0000 X-MS-PublicTrafficType: Email X-MS-Office365-Filtering-HT: Tenant X-MS-Office365-Filtering-Correlation-Id: 23cc4d27-a118-4c36-c386-08d8b6251e93 X-MS-TrafficTypeDiagnostic: DB6PR0401MB2326: X-MS-Exchange-Transport-Forked: True X-Microsoft-Antispam-PRVS: X-MS-Oob-TLC-OOBClassifiers: OLM:3383; X-MS-Exchange-SenderADCheck: 1 X-Microsoft-Antispam: BCL:0; X-Microsoft-Antispam-Message-Info: nyvMtFfYkZFBZplWtPdm8q/irL+qrNMQ33Dyl7jayJa2fLMrvQfTOaHsEB7IJ/hsNVOlPY9vD+C5XdjFo0Mn8QV/9dzc5Bi/ciP6Xvb2NZ11Gl+q4W8GaboIZRebKqr5ndrAeSO+BAKLhnwfVsldmKa5nuK/s8pH8Ugfd+AZEnjadSdKBcxoFzv4IItRqR2nF2j4hndRl34hrbxjmykE1CZWpHXWpFSmlGDITGr/ZVhE62Qg/pgCkOK30bRXNGP+hT+C714L5NT3RElOAOW6qiSTlfqcCxFG9GFTGCqBfKRzD4t7oz/MeJ01T1cpWdmmhTVIDtEneqbuFsPMnz1Z8grcq6q43vlNrQsr4eVfFufJlKcjFm+KrHVeEN/C9t+13NIqdFaS6JRb4rLcDnG0rXJOCYcQH3e+n7++gkqZDCNaj7fQMkICAzilNULxqMU5E5FB8C+hqkOMqT/HLZt4Og== X-Forefront-Antispam-Report: CIP:255.255.255.255;CTRY:;LANG:en;SCL:1;SRV:;IPV:NLI;SFV:NSPM;H:DB8PR04MB6795.eurprd04.prod.outlook.com;PTR:;CAT:NONE;SFS:(4636009)(376002)(39860400002)(396003)(136003)(366004)(346002)(66476007)(4326008)(66556008)(8676002)(4744005)(86362001)(316002)(1076003)(69590400011)(6512007)(6486002)(52116002)(5660300002)(36756003)(478600001)(8936002)(16526019)(956004)(186003)(2616005)(26005)(2906002)(6506007)(83380400001)(6666004)(66946007);DIR:OUT;SFP:1101; X-MS-Exchange-AntiSpam-MessageData: sTXNgKaY9Pry8YjHAxxF/b8VmvzH8FYk8Udioc9zfsXvnFn/fjuG2UkrJCdvCUsn/KuA/9P/f8HTfjCRIcaebxsMI2jhmTZPzG5f2M2QBkKCC9E+f8YkvlPeGDGeqZNhwNxUHICFM8LM1CGsrkFsf+dYJvhiBj8qJa9QapnDXn0b17VeBQ7Ih45Fc6YH5HuaOskyLMrDRIeZEMPVqj2lTkRCxIYfm+8bffFtVdIM4pbPhcgYYujzfCz16Uf7OKfMXVjO6HrZpAz+bA2sC5wBM63CuSREVrjLU4706qPGe20TX9PCrF0Gu99fO1g7mhQGhFsvgJaz7jXCKyXuLwQStAoks/4qDhZNxzCMLPHIqfXMmLQj5KnvkFbVlXDtmGtaSPzdtlx3gpI0XN7w/PqX08XDsWM+6E7BY0bi8X0LNc8OPhtKRtZxTzfdXfoPcIf3VlathN+UYBt6Tb/EinhmLB+YH1l/ArPJnzqVSBf9M2nNTCxe3DC3KLrcUhaHC/MU21dmEDnZH7+SiYqQcpMTwSmTIMvP1pqpqBWnMOqnK6pb9AUkdE9eOYMoV35xLKg3NiCO3RIu5Kajj5U/DtxqLyXjz275y2+bIOyxc7VRJZJChhvkgdRSrT/0OEKbC2T4ClR1I7vkjKf//uvWy8vjFfydfLYvZ4FWyhCdyI+nEf3Mm1CQGtYiNjO1qUsVpr2E19dKjR+bZCu4MV75HxHqGTJBTA7/wPKks3Cd23rSMeaojvsmoUSVwsv2VatOvn7khRGy3kZcV1I3IbwQIXsDpiwm6/8uOMcWJldVyMW7cGWycjQdVnRHVXaz9YakGwnEH/JkLojyfdDOdsqqdmJiVZ+0fhtIqwKdPfE1sgB4Z8dVFjLp9zYcHoKNo4yBNkPXfxgHZ32A5G9jwPjVGVtmdClBoOp4hHuti9jVoBGXDFk8skuA1cQBBCXurwf9Kvpjat6+419MjCa79zA2T2mZb1yhhaqLvEKJNn2+URb8anfrwk6H0P2ZStf4P8FOxWB6 X-OriginatorOrg: nxp.com X-MS-Exchange-CrossTenant-AuthSource: DB8PR04MB6795.eurprd04.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 11 Jan 2021 11:36:21.4775 (UTC) X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted X-MS-Exchange-CrossTenant-Id: 686ea1d3-bc2b-4c6f-a92c-d99c5c301635 X-MS-Exchange-CrossTenant-Network-Message-Id: 23cc4d27-a118-4c36-c386-08d8b6251e93 X-MS-Exchange-CrossTenant-MailboxType: HOSTED X-MS-Exchange-CrossTenant-UserPrincipalName: f2FxpEt7BC719X+kb1QpjKu5gYrpTc1JqlbRMjxZ2Ucgvw59Z3psPZ6iN7IwPpssnnTI9PFAT7D/E599HK3zrA== X-MS-Exchange-Transport-CrossTenantHeadersStamped: DB6PR0401MB2326 Precedence: bulk List-ID: X-Mailing-List: netdev@vger.kernel.org X-Patchwork-Delegate: kuba@kernel.org If clear GMAC_CONFIG_TE bit, it would stop all tx channals, but users may only want to stop secific tx channel. Fixes: 48863ce5940f ("stmmac: add DMA support for GMAC 4.xx") Signed-off-by: Joakim Zhang --- drivers/net/ethernet/stmicro/stmmac/dwmac4_lib.c | 4 ---- 1 file changed, 4 deletions(-) diff --git a/drivers/net/ethernet/stmicro/stmmac/dwmac4_lib.c b/drivers/net/ethernet/stmicro/stmmac/dwmac4_lib.c index 0b4ee2dbb691..71e50751ef2d 100644 --- a/drivers/net/ethernet/stmicro/stmmac/dwmac4_lib.c +++ b/drivers/net/ethernet/stmicro/stmmac/dwmac4_lib.c @@ -53,10 +53,6 @@ void dwmac4_dma_stop_tx(void __iomem *ioaddr, u32 chan) value &= ~DMA_CONTROL_ST; writel(value, ioaddr + DMA_CHAN_TX_CONTROL(chan)); - - value = readl(ioaddr + GMAC_CONFIG); - value &= ~GMAC_CONFIG_TE; - writel(value, ioaddr + GMAC_CONFIG); } void dwmac4_dma_start_rx(void __iomem *ioaddr, u32 chan) From patchwork Mon Jan 11 11:35:35 2021 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Joakim Zhang X-Patchwork-Id: 12010433 X-Patchwork-Delegate: kuba@kernel.org Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-18.7 required=3.0 tests=BAYES_00,DKIM_SIGNED, DKIM_VALID,DKIM_VALID_AU,HEADER_FROM_DIFFERENT_DOMAINS,INCLUDES_CR_TRAILER, INCLUDES_PATCH,MAILING_LIST_MULTI,MSGID_FROM_MTA_HEADER,SPF_HELO_NONE, SPF_PASS,URIBL_BLOCKED,USER_AGENT_GIT autolearn=ham autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 0D27CC433DB for ; Mon, 11 Jan 2021 11:37:51 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by mail.kernel.org (Postfix) with ESMTP id BF12D224BD for ; Mon, 11 Jan 2021 11:37:50 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1729832AbhAKLhu (ORCPT ); Mon, 11 Jan 2021 06:37:50 -0500 Received: from mail-eopbgr40076.outbound.protection.outlook.com ([40.107.4.76]:22789 "EHLO EUR03-DB5-obe.outbound.protection.outlook.com" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1729760AbhAKLht (ORCPT ); Mon, 11 Jan 2021 06:37:49 -0500 ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=iBIRwXLweHrn0Wac+gmob06pS+6eUGAjLKkdRGLZF4+HZPoOAl8yNRO3Jmn31xemipmY133NGGBuqnbFKSNxYEOWgQLyU8ApK+H4FmyIm9/1QvgJFs4BM9bkPJo1sHQTpRuozTZOwJBPVZsl0tpUD4tu9YIGsM4O2+K7vdA/fsNFGVbBdYr4fjkopFa88uaEVNKXLJlvhaPaqhih5n5cETMo8+mroB2b/mv11i3+ztOrE7982kkiiDMsl8o0m0lnmng9iTGhRTOHxPj8dnmR85EZbKzga6EDbbP/PxntUlcH4sxE+FdDvVK/cXpKZzP2xYOkLzz6ZdWTIFe54N49Qg== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector9901; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=/nFpfcf/QftDjDy+VbfIxIEz0YFXCPEZzd7hx1GcbCM=; b=W+XK574q/TTN3lpN+cDejpvOcHMpcoW8/PZL7GToLNtnuD2bH4S9asa5pApJu0qvw7l+5gvnA3jqjzrrcktPKQvmfM5JnQuaayQQ3nh0WgIBzlm4b8FJog0H8o5hL01NSdm9nX8ropnmxc4ARTmoSbLQkVTPzRLvxK4GRzVtMpYPM39bRZE7C2Bh/SwGHRXJoMS6gaQHlQqc4fdL7cjeAIsCgIXE0JYi4kewWCqCt4eBzkmDoi7lfG7zXZoR+mPyK5pkzVTZeiL5doyxAu6iVRp+2zm9L8dsIPjpXMmkoM3CMj+rKvI93kXtxv87Av2WP+kWVQZV7ltO6LjaSL78VQ== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=nxp.com; dmarc=pass action=none header.from=nxp.com; dkim=pass header.d=nxp.com; arc=none DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=nxp.com; s=selector2; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=/nFpfcf/QftDjDy+VbfIxIEz0YFXCPEZzd7hx1GcbCM=; b=L1MsplRSZiJxYD4YnZe8Xw5cuVFeD3eTcMXzXu7pZMFkaO3X8wpvnAgp9Q034OAq5KLRcGVU9q/fo9TN3qpoLjXshLlVAideadVmq/5f5YWC/6zb1qybNjbiEoFLdxt4C8XsIhlA7CQItSO8kbPPYIG+Sut1Jn0zkU0i26IC3nA= Authentication-Results: st.com; dkim=none (message not signed) header.d=none;st.com; dmarc=none action=none header.from=nxp.com; Received: from DB8PR04MB6795.eurprd04.prod.outlook.com (2603:10a6:10:fa::15) by DB6PR0401MB2326.eurprd04.prod.outlook.com (2603:10a6:4:47::6) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.3742.12; Mon, 11 Jan 2021 11:36:24 +0000 Received: from DB8PR04MB6795.eurprd04.prod.outlook.com ([fe80::b83c:2edc:17e8:2666]) by DB8PR04MB6795.eurprd04.prod.outlook.com ([fe80::b83c:2edc:17e8:2666%5]) with mapi id 15.20.3742.012; Mon, 11 Jan 2021 11:36:24 +0000 From: Joakim Zhang To: peppe.cavallaro@st.com, alexandre.torgue@st.com, joabreu@synopsys.com, davem@davemloft.net, kuba@kernel.org Cc: netdev@vger.kernel.org, linux-imx@nxp.com Subject: [PATCH 3/6] ethernet: stmmac: fix watchdog timeout during suspend/resume stress test Date: Mon, 11 Jan 2021 19:35:35 +0800 Message-Id: <20210111113538.12077-4-qiangqing.zhang@nxp.com> X-Mailer: git-send-email 2.17.1 In-Reply-To: <20210111113538.12077-1-qiangqing.zhang@nxp.com> References: <20210111113538.12077-1-qiangqing.zhang@nxp.com> X-Originating-IP: [119.31.174.71] X-ClientProxiedBy: SG2PR0601CA0018.apcprd06.prod.outlook.com (2603:1096:3::28) To DB8PR04MB6795.eurprd04.prod.outlook.com (2603:10a6:10:fa::15) MIME-Version: 1.0 X-MS-Exchange-MessageSentRepresentingType: 1 Received: from localhost.localdomain (119.31.174.71) by SG2PR0601CA0018.apcprd06.prod.outlook.com (2603:1096:3::28) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.3742.6 via Frontend Transport; Mon, 11 Jan 2021 11:36:21 +0000 X-MS-PublicTrafficType: Email X-MS-Office365-Filtering-HT: Tenant X-MS-Office365-Filtering-Correlation-Id: 7f887960-92f4-4d02-672f-08d8b625202e X-MS-TrafficTypeDiagnostic: DB6PR0401MB2326: X-MS-Exchange-Transport-Forked: True X-Microsoft-Antispam-PRVS: X-MS-Oob-TLC-OOBClassifiers: OLM:7691; X-MS-Exchange-SenderADCheck: 1 X-Microsoft-Antispam: BCL:0; X-Microsoft-Antispam-Message-Info: 1FiqoZARSowvscDyBj3f7EymUwRddY0/m/QrGocCWz1PDVkZbSwYwcbUt+2b9F7k0sbw5B3/ZF070Z/CmZYns6BIxq/NhAe2vv3Ynqes2oD9mtlm7YTlhwC4z3nlGvBAExoomQgkPo4yp/NoDKoEveIVN6CnuanrcUOfJJ4Oce4DZoLxKwg52XxbxwLzWimPdRjYinLPycjVq5YwId6NvD5ZJKwD+Lw1IRfylbzroZqWRUs8ilwyHO31F7qZhlWrfilYR1kC5IG8jLbrjBvG0BhzU6fMp+8FpwizrMf6Zs6khkJDnB82MBTm+Pkxc9xldWfKXbwdxxtcwxrn+WiXxE+3dUofdIk4LB/rkzdItJBKThQXkws1BAasXvKwO8002LRnSA187gRqWGwHMJo1PUO+5Iq64KbuFVMSSWHSzFuc3QbFXGQCJ6C2xMDQf8qs5X1lsNLgE1e2Wk14xAhFnQ== X-Forefront-Antispam-Report: CIP:255.255.255.255;CTRY:;LANG:en;SCL:1;SRV:;IPV:NLI;SFV:NSPM;H:DB8PR04MB6795.eurprd04.prod.outlook.com;PTR:;CAT:NONE;SFS:(4636009)(376002)(39860400002)(396003)(136003)(366004)(346002)(66476007)(4326008)(15650500001)(66556008)(8676002)(86362001)(316002)(1076003)(69590400011)(6512007)(6486002)(52116002)(5660300002)(36756003)(478600001)(8936002)(16526019)(956004)(186003)(2616005)(26005)(2906002)(6506007)(83380400001)(6666004)(66946007);DIR:OUT;SFP:1101; X-MS-Exchange-AntiSpam-MessageData: 5nnIz4dMQaTcLUzbdjpEB+11vsqqU0kdRQykfKMMvh82OmYRPTfxKh5vPVcL+jT/m15TGh5MfyvrycvmyNGsEhnylsxsQYneL3sQP0AS21DxDMONj2fBQtqsCp96wyrC91VZxNT2eYuuARz9bY+CP4DoEDrifSmmIk2rDYnZKNmJetRfiXAGjdTSpR/AreL+yUOOUOcAwY6mucp6BlH092KxcNgT20DVN0N1kC+EoHZpPSTX4I5LP/vHw5ao3N6fvT4La87GvsAf6AuFbQrzNh3TmpdtiVzddBS68dxCCQKkcir2yFuCwZyojKop1OvPRBuZyM1KdD9twlhm5yCSi52FzNE0yc3Hk5CXZO4rhzi9FvCIkgwSVFq3UTlPICsomKJ6Rr0jgQmbfujEqaByWnzcd6iBAaRI98qe2ZPjCpBIKMDF9hXY9c8IESjU3dgMsf3WLjuVu9i6qDt1O4/7whRfOMmyAB8jBnjjmr5V7w+ySksiWeQU6V+E1KRZdhCZnyH4D7lQ06QE0DoXlPvchwdP3NNoVUBNxY2D4jfGB7g2ZSJeH3s9kBJwzL3dHZxoN+KluEzwUEmT1jl093P8u7PKfmrMO6KiqBK6MfSV4NQ9B4ve/GGYygokPFvpj3Ofc+VIlTGrDeAMgHESuayGosY8IY/rHt9rHBkAJbsxqXAuXBbK80RK0G56E2iAFt/4+QrHm2L9Az2Qs8cGloVMfo38X31KDtDOaPil0risqnkGYBm5koLDQNmcAZvJNT8VTK04TPEhTYsMZMYG7EGWKJ5dq+wxq4z8ZR8rUO/pnPAZvC0+Xu2CTYlXEnWHc+Jk/FIoGSE112KtLAOsSKBhdRhH2FnWjISpOLCXwPLkOF8sncdrQ4pzyggYaXFy91/HK4jtD8OpIOfMk4XnTimLdMmUyNvgN8MfXttmyFJEYD5CdLpP8fDLaNxZsef2uERCljCbFPDkRV+siPcjpxBGZT7MnI+bdO1nBevYEk0cv0WcJrilbaCret2boSr1Cs4w X-OriginatorOrg: nxp.com X-MS-Exchange-CrossTenant-AuthSource: DB8PR04MB6795.eurprd04.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 11 Jan 2021 11:36:24.2334 (UTC) X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted X-MS-Exchange-CrossTenant-Id: 686ea1d3-bc2b-4c6f-a92c-d99c5c301635 X-MS-Exchange-CrossTenant-Network-Message-Id: 7f887960-92f4-4d02-672f-08d8b625202e X-MS-Exchange-CrossTenant-MailboxType: HOSTED X-MS-Exchange-CrossTenant-UserPrincipalName: jnR8t0G8l/xHs0JL1ace0P73vVpTwzvypUfKyqkNZPE15PNGHtkfHsy/J35u4pqd8KhGAv2x1B9ZVaHNUBoAbA== X-MS-Exchange-Transport-CrossTenantHeadersStamped: DB6PR0401MB2326 Precedence: bulk List-ID: X-Mailing-List: netdev@vger.kernel.org X-Patchwork-Delegate: kuba@kernel.org stmmac_xmit() call stmmac_tx_timer_arm() at the end to modify tx timer to do the transmission cleanup work. Imagine such a situation, stmmac enters suspend immediately after tx timer modified, it's expire callback stmmac_tx_clean() would not be invoked. This could affect BQL, since netdev_tx_sent_queue() has been called, but netdev_tx_completed_queue() have not been involved, as a result, dql_avail(&dev_queue->dql) finally always return a negative value. __dev_queue_xmit->__dev_xmit_skb->qdisc_run->__qdisc_run->qdisc_restart->dequeue_skb: if ((q->flags & TCQ_F_ONETXQUEUE) && netif_xmit_frozen_or_stopped(txq)) // __QUEUE_STATE_STACK_XOFF is set Net core will stop transmitting any more. Finillay, net watchdong would timeout. To fix this issue, we should call netdev_tx_reset_queue() in stmmac_resume(). Fixes: 54139cf3bb33 ("net: stmmac: adding multiple buffers for rx") Signed-off-by: Joakim Zhang --- drivers/net/ethernet/stmicro/stmmac/stmmac_main.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/drivers/net/ethernet/stmicro/stmmac/stmmac_main.c b/drivers/net/ethernet/stmicro/stmmac/stmmac_main.c index a0bd064a8421..41d9a5a3cc9a 100644 --- a/drivers/net/ethernet/stmicro/stmmac/stmmac_main.c +++ b/drivers/net/ethernet/stmicro/stmmac/stmmac_main.c @@ -5256,6 +5256,8 @@ static void stmmac_reset_queues_param(struct stmmac_priv *priv) tx_q->cur_tx = 0; tx_q->dirty_tx = 0; tx_q->mss = 0; + + netdev_tx_reset_queue(netdev_get_tx_queue(priv->dev, queue)); } } From patchwork Mon Jan 11 11:35:36 2021 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Joakim Zhang X-Patchwork-Id: 12010425 X-Patchwork-Delegate: kuba@kernel.org Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-18.7 required=3.0 tests=BAYES_00,DKIM_SIGNED, DKIM_VALID,DKIM_VALID_AU,HEADER_FROM_DIFFERENT_DOMAINS,INCLUDES_CR_TRAILER, INCLUDES_PATCH,MAILING_LIST_MULTI,MSGID_FROM_MTA_HEADER,SPF_HELO_NONE, SPF_PASS,URIBL_BLOCKED,USER_AGENT_GIT autolearn=ham autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id BFECAC433E6 for ; Mon, 11 Jan 2021 11:37:34 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by mail.kernel.org (Postfix) with ESMTP id 8BB10225AB for ; Mon, 11 Jan 2021 11:37:34 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1729736AbhAKLhV (ORCPT ); Mon, 11 Jan 2021 06:37:21 -0500 Received: from mail-eopbgr10070.outbound.protection.outlook.com ([40.107.1.70]:57742 "EHLO EUR02-HE1-obe.outbound.protection.outlook.com" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1728731AbhAKLhU (ORCPT ); Mon, 11 Jan 2021 06:37:20 -0500 ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=RS6wrhBPLJ4aSB1Y88cZhDVqpOqpSB1BQjZEdJgl9qtdLmrbpGQvbjm/UQGCRb7e5jirbOzmjvx8wlIgThJQxtMfgca+urkca95YthHbASgQRAfxmv2sROtHcIEG5suOd8ll69ZOjW/SSuRA+wq5Gros/kBq+f1VJEH/fOjBWqqOKwhxfLslllmBjI6brxmC/sqInPgvvR5A7Ma2g3yZW/cVoucfOamLXIAHQwnkmMer6DdpEhjatTXnXvDKB7L0m4RgpxxkJ3NW8yB4DhbUpy5b0r1pWuSfLuz0PFiVN5ldJ0LsLV5kAvHg9INXnEm6yF0EyI/LJm9LA8tiDaQHEA== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector9901; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=9krP2sTXBBhE0sBFtZH9DGphaUAe41h/eUEi44rJhtU=; b=FF8DdOaXIV8OHN8mmox+9aAQ38KEF1QRGpnMRs55cNzgGb5c4omDq88Hyg/dI89GAgc7VYO3kK1v8MpSRIY3IteF6trc4uxSTIwyf3Lv1IQzphoyhUzv+Zr4Py8nQAcn90S7IlWZwXMB+Mp1CkmYqrK4fX2rs7oLqc9Lx8VITGYNVg34gNGeN3EILAljBywl5IdDI1lz8RWJ09YyOT9lXch8sNy8/ndah8mnA4TE4A49Qezv8+0j7KWomx2XODxSp/+Ulosx/3yQBuJupUfowXJ4hEV5RTNuo/Z6jH5VE+u1jGxDfd+KrCuLnaHUpBELkItWokh4tUN8RIthC0kUcQ== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=nxp.com; dmarc=pass action=none header.from=nxp.com; dkim=pass header.d=nxp.com; arc=none DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=nxp.com; s=selector2; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=9krP2sTXBBhE0sBFtZH9DGphaUAe41h/eUEi44rJhtU=; b=HcNAXN++tI6NbC4cWYNqta1t118MleaxNls2bRV+0eQs5cJ/ZNSWKOqqZ/m2Oisfhr1jyXkGtWjgOkahe1Dctjw3Sq4zw5GIqnBTaUJiZh93l+iYQ1Yvp11o9RGJ7leAk5aB/aUrOtsKpz/ptDeQZn52h1fezauUyYZ0UcZOtNk= Authentication-Results: st.com; dkim=none (message not signed) header.d=none;st.com; dmarc=none action=none header.from=nxp.com; Received: from DB8PR04MB6795.eurprd04.prod.outlook.com (2603:10a6:10:fa::15) by DBBPR04MB7674.eurprd04.prod.outlook.com (2603:10a6:10:1f5::10) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.3742.6; Mon, 11 Jan 2021 11:36:30 +0000 Received: from DB8PR04MB6795.eurprd04.prod.outlook.com ([fe80::b83c:2edc:17e8:2666]) by DB8PR04MB6795.eurprd04.prod.outlook.com ([fe80::b83c:2edc:17e8:2666%5]) with mapi id 15.20.3742.012; Mon, 11 Jan 2021 11:36:28 +0000 From: Joakim Zhang To: peppe.cavallaro@st.com, alexandre.torgue@st.com, joabreu@synopsys.com, davem@davemloft.net, kuba@kernel.org Cc: netdev@vger.kernel.org, linux-imx@nxp.com Subject: [PATCH 4/6] ethernet: stmmac: fix dma physical address of descriptor when display ring Date: Mon, 11 Jan 2021 19:35:36 +0800 Message-Id: <20210111113538.12077-5-qiangqing.zhang@nxp.com> X-Mailer: git-send-email 2.17.1 In-Reply-To: <20210111113538.12077-1-qiangqing.zhang@nxp.com> References: <20210111113538.12077-1-qiangqing.zhang@nxp.com> X-Originating-IP: [119.31.174.71] X-ClientProxiedBy: SG2PR0601CA0018.apcprd06.prod.outlook.com (2603:1096:3::28) To DB8PR04MB6795.eurprd04.prod.outlook.com (2603:10a6:10:fa::15) MIME-Version: 1.0 X-MS-Exchange-MessageSentRepresentingType: 1 Received: from localhost.localdomain (119.31.174.71) by SG2PR0601CA0018.apcprd06.prod.outlook.com (2603:1096:3::28) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.3742.6 via Frontend Transport; Mon, 11 Jan 2021 11:36:24 +0000 X-MS-PublicTrafficType: Email X-MS-Office365-Filtering-HT: Tenant X-MS-Office365-Filtering-Correlation-Id: 6fedb031-3a38-4fbd-da86-08d8b62522e2 X-MS-TrafficTypeDiagnostic: DBBPR04MB7674: X-MS-Exchange-Transport-Forked: True X-Microsoft-Antispam-PRVS: X-MS-Oob-TLC-OOBClassifiers: OLM:935; X-MS-Exchange-SenderADCheck: 1 X-Microsoft-Antispam: BCL:0; X-Microsoft-Antispam-Message-Info: gIs9gyWbQrY/JL/afMJ3gTmWdd02En/jY7cDc7R5M+lDvkGBRgYV2FVcxttabRwL+Yq1ZNZjJuIsM3dekZ2KWUZkhuVM/VnU5aAoHgVjfVT/HLzQ1N8d05LMjY9vhdjXw7CO6zJnlQFiX3X6DsASLcJ/dQT+emMLIymOeILBqp0eCCEtEnih2RExkURZkKVllwhoAlVnYeNyC2eknrWCXwyFCz6yH9q97FqGLjH9ZFgz8KX1eYdCwddls1iFRQQuFF79dEfOfVAUNOtWkeNKc1nzmfD9kRChTkDj8pWLi/3YjlNkZ5mG1d/ZYA+IEIQX4MWb7rB/CWggHfKtrQ78PdmD4DLNSGEwqJlhO0SCuvivBL/v7SY/H1zCWBflSxnAFwyy77KgO42s8TnW0/Bc6X8SeLRaG7Rd4wnSFTNKwIzJjq02FhjccWwk3fVgLPoP X-Forefront-Antispam-Report: CIP:255.255.255.255;CTRY:;LANG:en;SCL:1;SRV:;IPV:NLI;SFV:NSPM;H:DB8PR04MB6795.eurprd04.prod.outlook.com;PTR:;CAT:NONE;SFS:(4636009)(366004)(396003)(136003)(346002)(376002)(39860400002)(66476007)(66946007)(6512007)(8676002)(8936002)(86362001)(16526019)(6486002)(66556008)(4326008)(83380400001)(2616005)(36756003)(52116002)(69590400011)(1076003)(478600001)(956004)(5660300002)(6506007)(2906002)(186003)(316002)(6666004)(26005);DIR:OUT;SFP:1101; X-MS-Exchange-AntiSpam-MessageData: M0Vn/GnDuRUf97koGtbq5FJnnvIdmY4Ek/pdP2tKPxxx30WHhG3rCiBOMxvRu/Mi4cwl+df7ZTaXMm4dJRy96vjOcoJTgqBffCquK6KfxgoO6mUZ/G4KKF8kCpLbqf0WCpM039OsY3Rz/k2R0b+ezLQOAT8rc8IUVqYU//k53Oodg40cE4XMQX49+t3H7pe8C5pUPneUwK8gV2XbRR3CWP9yMoQTI+Di6FnVRMq9wY0L/IUFdvT7pN2ByhgH6uZ10dGlS+6qfNjHUEVTrCyWSObnjnI8548ZEwfIcFRlRM9+sTGXTv31q16TO3jicnRg9J7CfbKDc1DwYCeG/LQu23ZpGvkHWYo5qw1zIu2WebIRbSlBD20xm+wylXEOpJrJm/3MfKLyUSXXJU+IeEXxwxJ2bwv7lKRg+8Xuz8+WOE4phqomXWn5yo3m4dzQYHrYnh2lMAXLx08YEBSUUofR/cOtQZ79MtQ6N8RxZnuVFLSAxSXJugN2k1sIULnfQ2RvctZV8er6nLAMn+Zmugx0tHIe9uLKkwMbIcJQzIUFeyXiRIuGgKCLxip0DnHRb01xthxQut3KRJbZHJ/wv4RtmB+YEHguIBxd8fnrsqSWQJiX0j0eG3Qy47bCkNp9ENyOv1PQ1ofGP1LHFzxXo8H0DVlz7C1tyZ0fWE5kThIoBAYKI4BAuoQ9BBxMyM5HHZgH4RpIIdoyQxR2BP7XXEFcf15CYquuYx7zi9Ko/StsVKfQVa6GkHc/pBf924AIQzkV4dHxyCRipuSznWYlHKVPz214XaLyMe8Bl74Hw4PuCv4dyuLiWFzLD3Qau3AQLXC1f6BVUBYE6YbE1qw+D/JDpmVkdtD/t5WbQFEa5fDt/yG48b3qWKH7LpS8fYJk4tyUSMSMLPKro8DtiLCG+ZiFNlaoIzBydgDhZKiU6JZNudN5zuGU6XLbXrd0SF5g1pq8SJQXHn8nB586DZB/FnC7iM8mWthCv56KGXWHzyCy/C6AeUeS1AeCqKVT+kEiw9Ck X-OriginatorOrg: nxp.com X-MS-Exchange-CrossTenant-AuthSource: DB8PR04MB6795.eurprd04.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 11 Jan 2021 11:36:28.4638 (UTC) X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted X-MS-Exchange-CrossTenant-Id: 686ea1d3-bc2b-4c6f-a92c-d99c5c301635 X-MS-Exchange-CrossTenant-Network-Message-Id: 6fedb031-3a38-4fbd-da86-08d8b62522e2 X-MS-Exchange-CrossTenant-MailboxType: HOSTED X-MS-Exchange-CrossTenant-UserPrincipalName: oUftT8WYtlvzinfzO36uf0noUwrf4zVFUKYwa9dS2mocH4GB+vaq+nuxKGVwbiEs6yWylazqv1VNEOUiWSFEGA== X-MS-Exchange-Transport-CrossTenantHeadersStamped: DBBPR04MB7674 Precedence: bulk List-ID: X-Mailing-List: netdev@vger.kernel.org X-Patchwork-Delegate: kuba@kernel.org Driver uses dma_alloc_coherent to allocate dma memory for descriptors, dma_alloc_coherent will return both the virtual address and physical address. AFAIK, virt_to_phys could not convert virtual address to physical address, for which memory is allocated by dma_alloc_coherent. Signed-off-by: Joakim Zhang --- .../ethernet/stmicro/stmmac/dwmac4_descs.c | 5 +- .../net/ethernet/stmicro/stmmac/enh_desc.c | 5 +- drivers/net/ethernet/stmicro/stmmac/hwif.h | 3 +- .../net/ethernet/stmicro/stmmac/norm_desc.c | 5 +- .../net/ethernet/stmicro/stmmac/stmmac_main.c | 50 ++++++++++++------- 5 files changed, 44 insertions(+), 24 deletions(-) diff --git a/drivers/net/ethernet/stmicro/stmmac/dwmac4_descs.c b/drivers/net/ethernet/stmicro/stmmac/dwmac4_descs.c index c6540b003b43..8e1ee33ba1e6 100644 --- a/drivers/net/ethernet/stmicro/stmmac/dwmac4_descs.c +++ b/drivers/net/ethernet/stmicro/stmmac/dwmac4_descs.c @@ -402,7 +402,8 @@ static void dwmac4_rd_set_tx_ic(struct dma_desc *p) p->des2 |= cpu_to_le32(TDES2_INTERRUPT_ON_COMPLETION); } -static void dwmac4_display_ring(void *head, unsigned int size, bool rx) +static void dwmac4_display_ring(void *head, unsigned int size, bool rx, + unsigned int dma_rx_phy, unsigned int desc_size) { struct dma_desc *p = (struct dma_desc *)head; int i; @@ -411,7 +412,7 @@ static void dwmac4_display_ring(void *head, unsigned int size, bool rx) for (i = 0; i < size; i++) { pr_info("%03d [0x%x]: 0x%x 0x%x 0x%x 0x%x\n", - i, (unsigned int)virt_to_phys(p), + i, (unsigned int)(dma_rx_phy + i * desc_size), le32_to_cpu(p->des0), le32_to_cpu(p->des1), le32_to_cpu(p->des2), le32_to_cpu(p->des3)); p++; diff --git a/drivers/net/ethernet/stmicro/stmmac/enh_desc.c b/drivers/net/ethernet/stmicro/stmmac/enh_desc.c index d02cec296f51..a7324b9c1a02 100644 --- a/drivers/net/ethernet/stmicro/stmmac/enh_desc.c +++ b/drivers/net/ethernet/stmicro/stmmac/enh_desc.c @@ -417,7 +417,8 @@ static int enh_desc_get_rx_timestamp_status(void *desc, void *next_desc, } } -static void enh_desc_display_ring(void *head, unsigned int size, bool rx) +static void enh_desc_display_ring(void *head, unsigned int size, bool rx, + unsigned int dma_rx_phy, unsigned int desc_size) { struct dma_extended_desc *ep = (struct dma_extended_desc *)head; int i; @@ -429,7 +430,7 @@ static void enh_desc_display_ring(void *head, unsigned int size, bool rx) x = *(u64 *)ep; pr_info("%03d [0x%x]: 0x%x 0x%x 0x%x 0x%x\n", - i, (unsigned int)virt_to_phys(ep), + i, (unsigned int)(dma_rx_phy + i * desc_size), (unsigned int)x, (unsigned int)(x >> 32), ep->basic.des2, ep->basic.des3); ep++; diff --git a/drivers/net/ethernet/stmicro/stmmac/hwif.h b/drivers/net/ethernet/stmicro/stmmac/hwif.h index b40b2e0667bb..fc5260cf27ea 100644 --- a/drivers/net/ethernet/stmicro/stmmac/hwif.h +++ b/drivers/net/ethernet/stmicro/stmmac/hwif.h @@ -78,7 +78,8 @@ struct stmmac_desc_ops { /* get rx timestamp status */ int (*get_rx_timestamp_status)(void *desc, void *next_desc, u32 ats); /* Display ring */ - void (*display_ring)(void *head, unsigned int size, bool rx); + void (*display_ring)(void *head, unsigned int size, bool rx, + unsigned int dma_rx_phy, unsigned int desc_size); /* set MSS via context descriptor */ void (*set_mss)(struct dma_desc *p, unsigned int mss); /* get descriptor skbuff address */ diff --git a/drivers/net/ethernet/stmicro/stmmac/norm_desc.c b/drivers/net/ethernet/stmicro/stmmac/norm_desc.c index f083360e4ba6..c26544de0766 100644 --- a/drivers/net/ethernet/stmicro/stmmac/norm_desc.c +++ b/drivers/net/ethernet/stmicro/stmmac/norm_desc.c @@ -269,7 +269,8 @@ static int ndesc_get_rx_timestamp_status(void *desc, void *next_desc, u32 ats) return 1; } -static void ndesc_display_ring(void *head, unsigned int size, bool rx) +static void ndesc_display_ring(void *head, unsigned int size, bool rx, + unsigned int dma_rx_phy, unsigned int desc_size) { struct dma_desc *p = (struct dma_desc *)head; int i; @@ -281,7 +282,7 @@ static void ndesc_display_ring(void *head, unsigned int size, bool rx) x = *(u64 *)p; pr_info("%03d [0x%x]: 0x%x 0x%x 0x%x 0x%x", - i, (unsigned int)virt_to_phys(p), + i, (unsigned int)(dma_rx_phy + i * desc_size), (unsigned int)x, (unsigned int)(x >> 32), p->des2, p->des3); p++; diff --git a/drivers/net/ethernet/stmicro/stmmac/stmmac_main.c b/drivers/net/ethernet/stmicro/stmmac/stmmac_main.c index 41d9a5a3cc9a..ca24e268f49a 100644 --- a/drivers/net/ethernet/stmicro/stmmac/stmmac_main.c +++ b/drivers/net/ethernet/stmicro/stmmac/stmmac_main.c @@ -1133,6 +1133,7 @@ static int stmmac_phy_setup(struct stmmac_priv *priv) static void stmmac_display_rx_rings(struct stmmac_priv *priv) { u32 rx_cnt = priv->plat->rx_queues_to_use; + unsigned int desc_size; void *head_rx; u32 queue; @@ -1142,19 +1143,24 @@ static void stmmac_display_rx_rings(struct stmmac_priv *priv) pr_info("\tRX Queue %u rings\n", queue); - if (priv->extend_desc) + if (priv->extend_desc) { head_rx = (void *)rx_q->dma_erx; - else + desc_size = sizeof(struct dma_extended_desc); + } else { head_rx = (void *)rx_q->dma_rx; + desc_size = sizeof(struct dma_desc); + } /* Display RX ring */ - stmmac_display_ring(priv, head_rx, priv->dma_rx_size, true); + stmmac_display_ring(priv, head_rx, priv->dma_rx_size, true, + rx_q->dma_rx_phy, desc_size); } } static void stmmac_display_tx_rings(struct stmmac_priv *priv) { u32 tx_cnt = priv->plat->tx_queues_to_use; + unsigned int desc_size; void *head_tx; u32 queue; @@ -1164,14 +1170,19 @@ static void stmmac_display_tx_rings(struct stmmac_priv *priv) pr_info("\tTX Queue %d rings\n", queue); - if (priv->extend_desc) + if (priv->extend_desc) { head_tx = (void *)tx_q->dma_etx; - else if (tx_q->tbs & STMMAC_TBS_AVAIL) + desc_size = sizeof(struct dma_extended_desc); + } else if (tx_q->tbs & STMMAC_TBS_AVAIL) { head_tx = (void *)tx_q->dma_entx; - else + desc_size = sizeof(struct dma_edesc); + } else { head_tx = (void *)tx_q->dma_tx; + desc_size = sizeof(struct dma_desc); + } - stmmac_display_ring(priv, head_tx, priv->dma_tx_size, false); + stmmac_display_ring(priv, head_tx, priv->dma_tx_size, false, + tx_q->dma_tx_phy, desc_size); } } @@ -3736,18 +3747,23 @@ static int stmmac_rx(struct stmmac_priv *priv, int limit, u32 queue) unsigned int count = 0, error = 0, len = 0; int status = 0, coe = priv->hw->rx_csum; unsigned int next_entry = rx_q->cur_rx; + unsigned int desc_size; struct sk_buff *skb = NULL; if (netif_msg_rx_status(priv)) { void *rx_head; netdev_dbg(priv->dev, "%s: descriptor ring:\n", __func__); - if (priv->extend_desc) + if (priv->extend_desc) { rx_head = (void *)rx_q->dma_erx; - else + desc_size = sizeof(struct dma_extended_desc); + } else { rx_head = (void *)rx_q->dma_rx; + desc_size = sizeof(struct dma_desc); + } - stmmac_display_ring(priv, rx_head, priv->dma_rx_size, true); + stmmac_display_ring(priv, rx_head, priv->dma_rx_size, true, + rx_q->dma_rx_phy, desc_size); } while (count < limit) { unsigned int buf1_len = 0, buf2_len = 0; @@ -4314,7 +4330,7 @@ static int stmmac_set_mac_address(struct net_device *ndev, void *addr) static struct dentry *stmmac_fs_dir; static void sysfs_display_ring(void *head, int size, int extend_desc, - struct seq_file *seq) + struct seq_file *seq, unsigned int dma_phy_addr) { int i; struct dma_extended_desc *ep = (struct dma_extended_desc *)head; @@ -4323,7 +4339,7 @@ static void sysfs_display_ring(void *head, int size, int extend_desc, for (i = 0; i < size; i++) { if (extend_desc) { seq_printf(seq, "%d [0x%x]: 0x%x 0x%x 0x%x 0x%x\n", - i, (unsigned int)virt_to_phys(ep), + i, (unsigned int)(dma_phy_addr + i * sizeof(ep)), le32_to_cpu(ep->basic.des0), le32_to_cpu(ep->basic.des1), le32_to_cpu(ep->basic.des2), @@ -4331,7 +4347,7 @@ static void sysfs_display_ring(void *head, int size, int extend_desc, ep++; } else { seq_printf(seq, "%d [0x%x]: 0x%x 0x%x 0x%x 0x%x\n", - i, (unsigned int)virt_to_phys(p), + i, (unsigned int)(dma_phy_addr + i * sizeof(p)), le32_to_cpu(p->des0), le32_to_cpu(p->des1), le32_to_cpu(p->des2), le32_to_cpu(p->des3)); p++; @@ -4359,11 +4375,11 @@ static int stmmac_rings_status_show(struct seq_file *seq, void *v) if (priv->extend_desc) { seq_printf(seq, "Extended descriptor ring:\n"); sysfs_display_ring((void *)rx_q->dma_erx, - priv->dma_rx_size, 1, seq); + priv->dma_rx_size, 1, seq, rx_q->dma_rx_phy); } else { seq_printf(seq, "Descriptor ring:\n"); sysfs_display_ring((void *)rx_q->dma_rx, - priv->dma_rx_size, 0, seq); + priv->dma_rx_size, 0, seq, rx_q->dma_rx_phy); } } @@ -4375,11 +4391,11 @@ static int stmmac_rings_status_show(struct seq_file *seq, void *v) if (priv->extend_desc) { seq_printf(seq, "Extended descriptor ring:\n"); sysfs_display_ring((void *)tx_q->dma_etx, - priv->dma_tx_size, 1, seq); + priv->dma_tx_size, 1, seq, tx_q->dma_tx_phy); } else if (!(tx_q->tbs & STMMAC_TBS_AVAIL)) { seq_printf(seq, "Descriptor ring:\n"); sysfs_display_ring((void *)tx_q->dma_tx, - priv->dma_tx_size, 0, seq); + priv->dma_tx_size, 0, seq, tx_q->dma_tx_phy); } } From patchwork Mon Jan 11 11:35:37 2021 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Joakim Zhang X-Patchwork-Id: 12010435 X-Patchwork-Delegate: kuba@kernel.org Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-18.7 required=3.0 tests=BAYES_00,DKIM_SIGNED, DKIM_VALID,DKIM_VALID_AU,HEADER_FROM_DIFFERENT_DOMAINS,INCLUDES_CR_TRAILER, INCLUDES_PATCH,MAILING_LIST_MULTI,MSGID_FROM_MTA_HEADER,SPF_HELO_NONE, SPF_PASS,URIBL_BLOCKED,USER_AGENT_GIT autolearn=ham autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 8298AC433E0 for ; Mon, 11 Jan 2021 11:37:59 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by mail.kernel.org (Postfix) with ESMTP id 53BE0224BD for ; Mon, 11 Jan 2021 11:37:59 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1729845AbhAKLh6 (ORCPT ); Mon, 11 Jan 2021 06:37:58 -0500 Received: from mail-eopbgr150043.outbound.protection.outlook.com ([40.107.15.43]:22167 "EHLO EUR01-DB5-obe.outbound.protection.outlook.com" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1729760AbhAKLh6 (ORCPT ); Mon, 11 Jan 2021 06:37:58 -0500 ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=iXyqozjlcojVnAB70oHsDwlRAKhUUu3+QQt8Iq1vtMYBp6uf2L+lIbkdoXJRl65fDZEkvKnvNg3xDh8ia4VjsI75gyYQSR6PcLRKQlSO6myixokDmTVy6Oz1W3hD11BHVQDC8tC5NxAq50wB9zwHgobhm+nBU1wJ16aP/ecQQgdR2N4px24zcAYR8B/aNo407dGgNynPC/KKm+6R8KCTYCtczWt+j1Hdu22YUmDME4rYO1vScNbTdchUowF/22Hw5vhNqCiJyl6TSyBP98sObMM5Bs53tFsM6nZPzW3kRBfOBTWdY+xuaIdg0+/a/G5QHkLlAZsyj1iXDfS1xnhN1w== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector9901; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=1PyrxR3aJVmgVFlha4+BYYd9Mxl+++J2D7PKdmnWPfo=; b=ivPo+Fo9W8Ox7hJgTp0byRYDoLmxrBb2xdZMkIid+AUBMMeOWqulJWRGrf7AZjYnHAVvP9kUqMEq2+jCrWA+kScJSY9AkkcdIXNlLfkNWlwLR4D239MaH7hHby3J6AynrZpjXcIctsYGg1xC72+sFYoYQoMlYLsFu9IZ8GMWMRoweNgug9UJIeoEy9uC4e4DBjgh3izzQ0iOOk++IZjQw2+qySU6cDm3ugMKnGpGlGs+eK4b1sN78/HK2REmtglBUYXaOqiX+N+koVVHIBGPFikxA87GRKUrMrAlCTRadLFm34jDgoKAVnZMBT8tYECc7b378UPaw5QEIeeD4/xjEw== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=nxp.com; dmarc=pass action=none header.from=nxp.com; dkim=pass header.d=nxp.com; arc=none DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=nxp.com; s=selector2; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=1PyrxR3aJVmgVFlha4+BYYd9Mxl+++J2D7PKdmnWPfo=; b=Rx6ccf7kJF0kEe+INRLUWx2/OXs/8HN0drbkX9/GJ9FI6CXBBPRrinsgrdsoSz76AyV84hztvvXo1KhQVZ8R9nHBQsMAyTkhSO9N4aV8KlWUJ2NoqJSTc/Ythap/YdC3wo6wGVzrvcwojW2a5ehu0Sc7BPzakXGZvf4ZcOI/KD8= Authentication-Results: st.com; dkim=none (message not signed) header.d=none;st.com; dmarc=none action=none header.from=nxp.com; Received: from DB8PR04MB6795.eurprd04.prod.outlook.com (2603:10a6:10:fa::15) by DBBPR04MB7674.eurprd04.prod.outlook.com (2603:10a6:10:1f5::10) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.3742.6; Mon, 11 Jan 2021 11:36:31 +0000 Received: from DB8PR04MB6795.eurprd04.prod.outlook.com ([fe80::b83c:2edc:17e8:2666]) by DB8PR04MB6795.eurprd04.prod.outlook.com ([fe80::b83c:2edc:17e8:2666%5]) with mapi id 15.20.3742.012; Mon, 11 Jan 2021 11:36:31 +0000 From: Joakim Zhang To: peppe.cavallaro@st.com, alexandre.torgue@st.com, joabreu@synopsys.com, davem@davemloft.net, kuba@kernel.org Cc: netdev@vger.kernel.org, linux-imx@nxp.com Subject: [PATCH 5/6] ethernet: stmmac: fix wrongly set buffer2 valid when sph unsupport Date: Mon, 11 Jan 2021 19:35:37 +0800 Message-Id: <20210111113538.12077-6-qiangqing.zhang@nxp.com> X-Mailer: git-send-email 2.17.1 In-Reply-To: <20210111113538.12077-1-qiangqing.zhang@nxp.com> References: <20210111113538.12077-1-qiangqing.zhang@nxp.com> X-Originating-IP: [119.31.174.71] X-ClientProxiedBy: SG2PR0601CA0018.apcprd06.prod.outlook.com (2603:1096:3::28) To DB8PR04MB6795.eurprd04.prod.outlook.com (2603:10a6:10:fa::15) MIME-Version: 1.0 X-MS-Exchange-MessageSentRepresentingType: 1 Received: from localhost.localdomain (119.31.174.71) by SG2PR0601CA0018.apcprd06.prod.outlook.com (2603:1096:3::28) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.3742.6 via Frontend Transport; Mon, 11 Jan 2021 11:36:29 +0000 X-MS-PublicTrafficType: Email X-MS-Office365-Filtering-HT: Tenant X-MS-Office365-Filtering-Correlation-Id: 28a23278-8644-48c9-2baa-08d8b62524af X-MS-TrafficTypeDiagnostic: DBBPR04MB7674: X-MS-Exchange-Transport-Forked: True X-Microsoft-Antispam-PRVS: X-MS-Oob-TLC-OOBClassifiers: OLM:2089; X-MS-Exchange-SenderADCheck: 1 X-Microsoft-Antispam: BCL:0; X-Microsoft-Antispam-Message-Info: jrworm5LWy5hpbl/4kb94bANInBq72yuU9U1ee3pXjWuB3lr3NW6OtAHey27dbBavwK1F4xj5jkPpAmRcVHhb+PsL76riSWs0JOmUBcEi3W2BcOLZ1TEPYiKuWgvBkbPmBhYQu4FQdaBJez6wCDiapEjH3ZScYRQg/fI4g84uZiYU3mVg01JAovFpWWrG1FWZA/69soFUr/kJDzm22vrRCtmJLuqo1Ltmpm7f/pXqvI2p5pZy8m7IOUPg/vINYdeeHbVftemD97t8V3cwAOsWnr6ZEkLldoWKlZJSYbfOsf+uynBW0I2louZzcw/xCW7CaaXusVq3wxhVEchmEYOFdE2cg22U6z9zX2Zf5D4Lxl4DZQO7muG/Y3dMw+yPQNkgctlKnIgYv1/k73OWMK/F7/G/xlBGnQkyPwt4igQ26HgX88bqOQYsQfdT1LpSBXaCOJN88i/evC9E7dCvshSZQ== X-Forefront-Antispam-Report: CIP:255.255.255.255;CTRY:;LANG:en;SCL:1;SRV:;IPV:NLI;SFV:NSPM;H:DB8PR04MB6795.eurprd04.prod.outlook.com;PTR:;CAT:NONE;SFS:(4636009)(366004)(396003)(136003)(346002)(376002)(39860400002)(66476007)(66946007)(6512007)(8676002)(8936002)(86362001)(16526019)(6486002)(66556008)(4326008)(83380400001)(2616005)(36756003)(52116002)(69590400011)(1076003)(478600001)(956004)(5660300002)(6506007)(2906002)(186003)(316002)(6666004)(26005);DIR:OUT;SFP:1101; X-MS-Exchange-AntiSpam-MessageData: My6w049/+D4QKnuyhz1HBqms+xC+zNOfPfH8DahJVph08tztMt0D1nXUvQZiJfZ9O8Cf1fdkKN9zY0dyRQUvs6Jrijf/kv+MyfcjfT2LIQBZBQzPoWy8Iz977TvC+liYWZDv3fw821qMqYiz1+lXCKkIkrXZJBfPOtjajF0/Ch61lp4ulveKtsUcx5a4O3Gu0OKsN2oxMLojWB/V19B3STldY0D457xbyPaWSCxAw9PIMh1g9Vbz/glmvsPxDcAZJYCWsSzbnjazAjdNhuLVd7GTdcgBDOSnBbGZWBg3GBcs42Bz4bpJn0hLetqKGT5eDpnq+Jhaw+DXgpha80WWvYNdKLTWRECQUOBHQBRKI/ZfOEFiWnKNo/gBgD/zNtYeE1HjFen6YxUQZcf3Z9XrKqs5EfvVBOhVDua4gSDX6IOQUD0/a2SOicMCLUaMYXYxny9lQ6OA7BojPPmTbIcaWl9LGytzvn/3NKBtFqN338WzhitX/+Xxeq8RUOI3yQEGfdK1iROckjwq1DZmozeywesKwNIb+btUBAYdS4L/kTOWaXSjDf0jozGxJivqLUe6NnIpvy4Z2ICsUTGF6bKI3BDtc2YSUlcnU9ngOP6ra0RtoHvVeC1avJvM+3hkrYUU4WVROvCyNVhB3CNtrYN2YAvgeDelUy6K9iuN8dKlE/Ly5GY1d2AOjcuy2r35xFa2DD+fOBnn7CGVLmQJNr9aypz67XrzDFzag8/TCHs6HdDMLnHFM5oTk91FlFhlrLBB5UjzXmOhUgzCKhs1o4RkVCWzjR9BkZiHAcvmNNQLn8fyEn9na0zP3rNiugzqcw3190ymGatbbjoJeAeiW+XFdc8kxqbjrGZfBP0pmMfHGrWUwvz1IO8Mp3+hWXV3jj/z5P7nWr6QIHnq7YlOuxRCAvyuBapjmxPEiaFmpQng6LcttibtX9bd9ODRx6YztajPL5kPMCIYxFLZQLcy+lkCRIeqBKWSiZOXYsEM3GnuRMvklAJnpVuGVvhuOymSt7il X-OriginatorOrg: nxp.com X-MS-Exchange-CrossTenant-AuthSource: DB8PR04MB6795.eurprd04.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 11 Jan 2021 11:36:31.7195 (UTC) X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted X-MS-Exchange-CrossTenant-Id: 686ea1d3-bc2b-4c6f-a92c-d99c5c301635 X-MS-Exchange-CrossTenant-Network-Message-Id: 28a23278-8644-48c9-2baa-08d8b62524af X-MS-Exchange-CrossTenant-MailboxType: HOSTED X-MS-Exchange-CrossTenant-UserPrincipalName: DVXcxjo8rn9dyMEdi1AiWKmJ7M9yL5yeHZUmn/5+1ugxNMjcs+U0SqbKfEYpmPK4o8VRaTv0k5NvZNVFPUvqOw== X-MS-Exchange-Transport-CrossTenantHeadersStamped: DBBPR04MB7674 Precedence: bulk List-ID: X-Mailing-List: netdev@vger.kernel.org X-Patchwork-Delegate: kuba@kernel.org In current driver, buffer2 available only when hardware supports split header. Wrongly set buffer2 valid in stmmac_rx_refill when refill buffer address. You can see that desc3 is 0x81000000 after initialization, but turn out to be 0x83000000 after refill. Fixes: 67afd6d1cfdf ("net: stmmac: Add Split Header support and enable it in XGMAC cores") Signed-off-by: Joakim Zhang --- drivers/net/ethernet/stmicro/stmmac/dwmac4_descs.c | 9 +++++++-- drivers/net/ethernet/stmicro/stmmac/dwxgmac2_descs.c | 2 +- drivers/net/ethernet/stmicro/stmmac/hwif.h | 2 +- drivers/net/ethernet/stmicro/stmmac/stmmac_main.c | 8 ++++++-- 4 files changed, 15 insertions(+), 6 deletions(-) diff --git a/drivers/net/ethernet/stmicro/stmmac/dwmac4_descs.c b/drivers/net/ethernet/stmicro/stmmac/dwmac4_descs.c index 8e1ee33ba1e6..aecba8b1302f 100644 --- a/drivers/net/ethernet/stmicro/stmmac/dwmac4_descs.c +++ b/drivers/net/ethernet/stmicro/stmmac/dwmac4_descs.c @@ -500,10 +500,15 @@ static void dwmac4_get_rx_header_len(struct dma_desc *p, unsigned int *len) *len = le32_to_cpu(p->des2) & RDES2_HL; } -static void dwmac4_set_sec_addr(struct dma_desc *p, dma_addr_t addr) +static void dwmac4_set_sec_addr(struct dma_desc *p, dma_addr_t addr, bool buf2_valid) { p->des2 = cpu_to_le32(lower_32_bits(addr)); - p->des3 = cpu_to_le32(upper_32_bits(addr) | RDES3_BUFFER2_VALID_ADDR); + p->des3 = cpu_to_le32(upper_32_bits(addr)); + + if (buf2_valid) + p->des3 |= RDES3_BUFFER2_VALID_ADDR; + else + p->des3 &= ~RDES3_BUFFER2_VALID_ADDR; } static void dwmac4_set_tbs(struct dma_edesc *p, u32 sec, u32 nsec) diff --git a/drivers/net/ethernet/stmicro/stmmac/dwxgmac2_descs.c b/drivers/net/ethernet/stmicro/stmmac/dwxgmac2_descs.c index 0aaf19ab5672..ccfb0102dde4 100644 --- a/drivers/net/ethernet/stmicro/stmmac/dwxgmac2_descs.c +++ b/drivers/net/ethernet/stmicro/stmmac/dwxgmac2_descs.c @@ -292,7 +292,7 @@ static void dwxgmac2_get_rx_header_len(struct dma_desc *p, unsigned int *len) *len = le32_to_cpu(p->des2) & XGMAC_RDES2_HL; } -static void dwxgmac2_set_sec_addr(struct dma_desc *p, dma_addr_t addr) +static void dwxgmac2_set_sec_addr(struct dma_desc *p, dma_addr_t addr, bool is_valid) { p->des2 = cpu_to_le32(lower_32_bits(addr)); p->des3 = cpu_to_le32(upper_32_bits(addr)); diff --git a/drivers/net/ethernet/stmicro/stmmac/hwif.h b/drivers/net/ethernet/stmicro/stmmac/hwif.h index fc5260cf27ea..6423dd5ef45a 100644 --- a/drivers/net/ethernet/stmicro/stmmac/hwif.h +++ b/drivers/net/ethernet/stmicro/stmmac/hwif.h @@ -92,7 +92,7 @@ struct stmmac_desc_ops { int (*get_rx_hash)(struct dma_desc *p, u32 *hash, enum pkt_hash_types *type); void (*get_rx_header_len)(struct dma_desc *p, unsigned int *len); - void (*set_sec_addr)(struct dma_desc *p, dma_addr_t addr); + void (*set_sec_addr)(struct dma_desc *p, dma_addr_t addr, bool buf2_valid); void (*set_sarc)(struct dma_desc *p, u32 sarc_type); void (*set_vlan_tag)(struct dma_desc *p, u16 tag, u16 inner_tag, u32 inner_type); diff --git a/drivers/net/ethernet/stmicro/stmmac/stmmac_main.c b/drivers/net/ethernet/stmicro/stmmac/stmmac_main.c index ca24e268f49a..e30529b8f40a 100644 --- a/drivers/net/ethernet/stmicro/stmmac/stmmac_main.c +++ b/drivers/net/ethernet/stmicro/stmmac/stmmac_main.c @@ -1314,9 +1314,10 @@ static int stmmac_init_rx_buffers(struct stmmac_priv *priv, struct dma_desc *p, return -ENOMEM; buf->sec_addr = page_pool_get_dma_addr(buf->sec_page); - stmmac_set_desc_sec_addr(priv, p, buf->sec_addr); + stmmac_set_desc_sec_addr(priv, p, buf->sec_addr, true); } else { buf->sec_page = NULL; + stmmac_set_desc_sec_addr(priv, p, buf->sec_addr, false); } buf->addr = page_pool_get_dma_addr(buf->page); @@ -3659,7 +3660,10 @@ static inline void stmmac_rx_refill(struct stmmac_priv *priv, u32 queue) DMA_FROM_DEVICE); stmmac_set_desc_addr(priv, p, buf->addr); - stmmac_set_desc_sec_addr(priv, p, buf->sec_addr); + if (priv->sph) + stmmac_set_desc_sec_addr(priv, p, buf->sec_addr, true); + else + stmmac_set_desc_sec_addr(priv, p, buf->sec_addr, false); stmmac_refill_desc3(priv, rx_q, p); rx_q->rx_count_frames++; From patchwork Mon Jan 11 11:35:38 2021 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Joakim Zhang X-Patchwork-Id: 12010431 X-Patchwork-Delegate: kuba@kernel.org Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-18.7 required=3.0 tests=BAYES_00,DKIM_SIGNED, DKIM_VALID,DKIM_VALID_AU,HEADER_FROM_DIFFERENT_DOMAINS,INCLUDES_CR_TRAILER, INCLUDES_PATCH,MAILING_LIST_MULTI,MSGID_FROM_MTA_HEADER,SPF_HELO_NONE, SPF_PASS,URIBL_BLOCKED,USER_AGENT_GIT autolearn=ham autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 45F71C433E6 for ; Mon, 11 Jan 2021 11:37:44 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by mail.kernel.org (Postfix) with ESMTP id 06071224BD for ; Mon, 11 Jan 2021 11:37:43 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1729821AbhAKLhn (ORCPT ); Mon, 11 Jan 2021 06:37:43 -0500 Received: from mail-eopbgr10070.outbound.protection.outlook.com ([40.107.1.70]:57742 "EHLO EUR02-HE1-obe.outbound.protection.outlook.com" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1729760AbhAKLhm (ORCPT ); Mon, 11 Jan 2021 06:37:42 -0500 ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=hxAK+RuDplIWIiQUVjuV7nxuCTaMV8ckCgC8YoG6ToABjClvqdKmygAIL8Wb5ke6PHJp3ITQHRUeWwdDbDrtdKR6wDLsQNlZz9OoEECvh5vPX2zK448mSMHn9aN1gCoGHc1f1jGeXDnc6l658jylrAPKI6g988BzqqLEQATF2uPk1qnUY0F5ThY6LS+T4+Op6Tm0Ely7hOKoXzZFwnTY072tN9lbvcUtmOohy6MRHzmYjP5RZdQDBV7+7VTp+JA0/+YyA3kLyHjgT9uG1JgtIr/GKU8NJbmZEm+RCIQY6acgjJ1m52ltS98o5zYM2qEZDm2kjZx3ozINOjf5oxV8xw== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector9901; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=MQHH37juAr+2j8jNeRVJt0TEgDe/mPuItrnLw6fochw=; b=IheAMznTy33W/nhiLHm86gbrXjbhvsc9ntV08XXjcLjOnhTEdP73GV+ki6z5rKjcmw+dctGxu9qcyqPa6NeOAf0j9Xy8MVKrO8qUWrVFj1tHsfo2oFb3Oh1tWkk8BnkTOdxRQGSaNQwbs4WeB8SW04h4oOg2CMUbqsU3JwESZURkQYMRPF5cVu8f3k/O1Mbhs27gOXnhTj66RrC4e4fwlF7i0JkMnXaXNkK5N4Mleqm0eqa6hj3fEh/pha2C7qgHG1/iYOcmVFS8Vl/R494KBSF9ebdGr5LnkX2E1gKR/62bDHYM+R0VJf/HQ1um5kRxGbfb2Guufu4Pobng3vx1SQ== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=nxp.com; dmarc=pass action=none header.from=nxp.com; dkim=pass header.d=nxp.com; arc=none DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=nxp.com; s=selector2; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=MQHH37juAr+2j8jNeRVJt0TEgDe/mPuItrnLw6fochw=; b=sYE/nX8FBMp9G7zOl84GafvsmMfW1PRRW9te8yTG84/ypbPsUZTS9q1V6cu+U2214lA07zH2sFPzpFn3cz6+r1I5em2OaEoU9+E6QpkOA/NTdjm6iINMkhujHULN4Uza7k+Ahs7/pDc409sXwk7zliNw6SqRnpsWj2QJY1CmJto= Authentication-Results: st.com; dkim=none (message not signed) header.d=none;st.com; dmarc=none action=none header.from=nxp.com; Received: from DB8PR04MB6795.eurprd04.prod.outlook.com (2603:10a6:10:fa::15) by DBBPR04MB7674.eurprd04.prod.outlook.com (2603:10a6:10:1f5::10) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.3742.6; Mon, 11 Jan 2021 11:36:34 +0000 Received: from DB8PR04MB6795.eurprd04.prod.outlook.com ([fe80::b83c:2edc:17e8:2666]) by DB8PR04MB6795.eurprd04.prod.outlook.com ([fe80::b83c:2edc:17e8:2666%5]) with mapi id 15.20.3742.012; Mon, 11 Jan 2021 11:36:34 +0000 From: Joakim Zhang To: peppe.cavallaro@st.com, alexandre.torgue@st.com, joabreu@synopsys.com, davem@davemloft.net, kuba@kernel.org Cc: netdev@vger.kernel.org, linux-imx@nxp.com Subject: [PATCH 6/6] ethernet: stmmac: re-init rx buffers when mac resume back Date: Mon, 11 Jan 2021 19:35:38 +0800 Message-Id: <20210111113538.12077-7-qiangqing.zhang@nxp.com> X-Mailer: git-send-email 2.17.1 In-Reply-To: <20210111113538.12077-1-qiangqing.zhang@nxp.com> References: <20210111113538.12077-1-qiangqing.zhang@nxp.com> X-Originating-IP: [119.31.174.71] X-ClientProxiedBy: SG2PR0601CA0018.apcprd06.prod.outlook.com (2603:1096:3::28) To DB8PR04MB6795.eurprd04.prod.outlook.com (2603:10a6:10:fa::15) MIME-Version: 1.0 X-MS-Exchange-MessageSentRepresentingType: 1 Received: from localhost.localdomain (119.31.174.71) by SG2PR0601CA0018.apcprd06.prod.outlook.com (2603:1096:3::28) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.3742.6 via Frontend Transport; Mon, 11 Jan 2021 11:36:32 +0000 X-MS-PublicTrafficType: Email X-MS-Office365-Filtering-HT: Tenant X-MS-Office365-Filtering-Correlation-Id: 879d18bc-5696-4734-bf7f-08d8b6252654 X-MS-TrafficTypeDiagnostic: DBBPR04MB7674: X-MS-Exchange-Transport-Forked: True X-Microsoft-Antispam-PRVS: X-MS-Oob-TLC-OOBClassifiers: OLM:7219; X-MS-Exchange-SenderADCheck: 1 X-Microsoft-Antispam: BCL:0; X-Microsoft-Antispam-Message-Info: HU6CFUdiZNbNk0RfJRzPtuLOcOifDjklnH9ogwascdqsLbILt6biO4XQtQKjR9dCjMWMdSu9lSkoMOGauAkUHGr3tnQX++YFSmiiOycKRlcHUVMJoYrilodLWpsvuIQS89sP+f8o2JAgr07SKyf3zcsGBNKUXQYHMCRI6koRmjkykgVV8RRFkW7zikD2yqI6rwxbeUBFmoQLMKRsb5xyjXoTLrFT6NaflkaMyQZrz909t8K5GtmFB+b9Y6xdEaSJBlnith/wdnHSXb2yRDdScrsyzfoz9B92+lRcrBBgKU+iB1Ok8hjbQWx6W4xczZVeMVbOa07HhYrUpJPrnfS0Q6oROkJKJs5Ohj8uLQC0RMdDdcNNOlVMgXnyVdBv2HcJp1qIJVPQWrfJm+hNTGiRUITz14Jy3xeZWcLyD1lnE0nCE2RSPUYxJExr2Mxs6h8Q X-Forefront-Antispam-Report: CIP:255.255.255.255;CTRY:;LANG:en;SCL:1;SRV:;IPV:NLI;SFV:NSPM;H:DB8PR04MB6795.eurprd04.prod.outlook.com;PTR:;CAT:NONE;SFS:(4636009)(366004)(396003)(136003)(346002)(376002)(39860400002)(66476007)(66946007)(6512007)(8676002)(8936002)(86362001)(16526019)(6486002)(66556008)(4326008)(83380400001)(2616005)(36756003)(52116002)(69590400011)(1076003)(478600001)(956004)(5660300002)(6506007)(2906002)(186003)(316002)(26005);DIR:OUT;SFP:1101; X-MS-Exchange-AntiSpam-MessageData: 1lVZxsAiS2KAmN/+5VLKf8J3NJfWcKNI4J9Mn4EQobdE5Wfp76eX/XQg/34CiTuFddC+jOX5P8IYtj8Pdx77WPiI+Am7aYD6r2yRI97Bv9vfYaMs14uytQq7F1GjNba16CJbJ+IuaTNHZqNUwHwdSHzCGGgoOwZy3o60X6RTvr3NrvM2GIPJe+OIFVtW/NraDH4F6bmop4fSLV3kr6MOkfzXWtLBy9TzG7byLIG6gFR174pofSRnHdYFeYRJve1R3z44C7QC2jrYt3xbsCim5OTG2fkTyeuUfTfd2paF893WT7e+oNsfS2h4GrIabZJTXuzgTbhU/rVA++t5ImH0Autx0pkWkCnHZg5K2gbML6+1XQdSlLeV283IDuV9b8A4CZr+A4I8oHB5PKbgOXcyxuJOZQg8I7TYSSNUQR4Eq0K/u1gNCuhgYlJJjSzMY+Gmm9bmCd2CgIISasAc/ZyGldAqE1vAjL8WVDY0PeebROSd0K4E//iTSkTwEKASiW+qcE5oowCUqfbg/VTUa89APPnfBKh8tD55v26f26hyW+WoVCvp4Nq25Yzqduawf1VJe3/Udvy5pAQk22zvrAfClQRbhfHoqRSH4IocTXeeLM1hgOaJu9uZS8sLYYo2lIJl7wDSyu+FDRGjrO5qna03bg9OQ6hw2pg5UDET9lpTTNISAdxkmsyUzfS2+o1DOuA4FZiImm8DKGzW04GC8Lbh7eeWPTEqb+lgVDuJKIUmvAgsJ0ROUrNc+/V46ijomZ7J8WUbgjkMcaZJHRcuw8N/L1b4gfyzgQXAF4W8guyswCa+6fJNFr8ySlGQGIgEBFrFKxlHa0FkymHd3SAhmCNU8LZ1R76nmKJ0x9S6Tz8FfowX1AVKRjn14hSId3b1uQa5FsPoWpHuSmB3Yi6QqHTV1hj0rDj9faxo+BUu/ycIi7ZJlJh2w1BL19ZuRdZUX6gpMIieLwmAESOf9rk8+5N2jJ/+yGfZGny4W2VwHffL6AShKHr09o6P2KRQG6jRR2cB X-OriginatorOrg: nxp.com X-MS-Exchange-CrossTenant-AuthSource: DB8PR04MB6795.eurprd04.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 11 Jan 2021 11:36:34.4555 (UTC) X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted X-MS-Exchange-CrossTenant-Id: 686ea1d3-bc2b-4c6f-a92c-d99c5c301635 X-MS-Exchange-CrossTenant-Network-Message-Id: 879d18bc-5696-4734-bf7f-08d8b6252654 X-MS-Exchange-CrossTenant-MailboxType: HOSTED X-MS-Exchange-CrossTenant-UserPrincipalName: etRnSGW+PHAGpqMnoN86Vw5VEivN1BRd2GTaCv8pCUvjLV8bKzLEdM8YjBNpIjPr4xcaVp7zUUYB90tIkM+aEw== X-MS-Exchange-Transport-CrossTenantHeadersStamped: DBBPR04MB7674 Precedence: bulk List-ID: X-Mailing-List: netdev@vger.kernel.org X-Patchwork-Delegate: kuba@kernel.org During suspend/resume stress test, we found descriptor write back by DMA could exhibit unusual behavior, e.g.: 003 [0xc4310030]: 0x0 0x40 0x0 0xb5010040 We can see that desc3 write back is 0xb5010040, it is still ownd by DMA, so application would not recycle this buffer. It will trigger fatal bus error when DMA try to use this descriptor again. To fix this issue, we should re-init all rx buffers when mac resume back. Signed-off-by: Joakim Zhang --- .../net/ethernet/stmicro/stmmac/stmmac_main.c | 73 ++++++++++++++++++- 1 file changed, 72 insertions(+), 1 deletion(-) diff --git a/drivers/net/ethernet/stmicro/stmmac/stmmac_main.c b/drivers/net/ethernet/stmicro/stmmac/stmmac_main.c index e30529b8f40a..e02d798e605b 100644 --- a/drivers/net/ethernet/stmicro/stmmac/stmmac_main.c +++ b/drivers/net/ethernet/stmicro/stmmac/stmmac_main.c @@ -1328,6 +1328,77 @@ static int stmmac_init_rx_buffers(struct stmmac_priv *priv, struct dma_desc *p, return 0; } +/** + * stmmac_reinit_rx_buffers - reinit the RX descriptor buffer. + * @priv: driver private structure + * Description: this function is called to re-allocate a receive buffer, perform + * the DMA mapping and init the descriptor. + */ +static int stmmac_reinit_rx_buffers(struct stmmac_priv *priv) +{ + u32 rx_count = priv->plat->rx_queues_to_use; + u32 queue; + int i; + + for (queue = 0; queue < rx_count; queue++) { + struct stmmac_rx_queue *rx_q = &priv->rx_queue[queue]; + + for (i = 0; i < priv->dma_rx_size; i++) { + struct stmmac_rx_buffer *buf = &rx_q->buf_pool[i]; + + if (buf->page) { + page_pool_recycle_direct(rx_q->page_pool, buf->page); + buf->page = NULL; + } + + if (priv->sph && buf->sec_page) { + page_pool_recycle_direct(rx_q->page_pool, buf->sec_page); + buf->sec_page = NULL; + } + } + } + + for (queue = 0; queue < rx_count; queue++) { + struct stmmac_rx_queue *rx_q = &priv->rx_queue[queue]; + + for (i = 0; i < priv->dma_rx_size; i++) { + struct stmmac_rx_buffer *buf = &rx_q->buf_pool[i]; + struct dma_desc *p; + + if (priv->extend_desc) + p = &((rx_q->dma_erx + i)->basic); + else + p = rx_q->dma_rx + i; + + if (!buf->page) { + buf->page = page_pool_dev_alloc_pages(rx_q->page_pool); + if (!buf->page) + break; + + buf->addr = page_pool_get_dma_addr(buf->page); + } + + if (priv->sph && !buf->sec_page) { + buf->sec_page = page_pool_dev_alloc_pages(rx_q->page_pool); + if (!buf->sec_page) + break; + + buf->sec_addr = page_pool_get_dma_addr(buf->sec_page); + } + + stmmac_set_desc_addr(priv, p, buf->addr); + if (priv->sph) + stmmac_set_desc_sec_addr(priv, p, buf->sec_addr, true); + else + stmmac_set_desc_sec_addr(priv, p, buf->sec_addr, false); + if (priv->dma_buf_sz == BUF_SIZE_16KiB) + stmmac_init_desc3(priv, p); + } + } + + return 0; +} + /** * stmmac_free_rx_buffer - free RX dma buffers * @priv: private structure @@ -5338,7 +5409,7 @@ int stmmac_resume(struct device *dev) mutex_lock(&priv->lock); stmmac_reset_queues_param(priv); - + stmmac_reinit_rx_buffers(priv); stmmac_free_tx_skbufs(priv); stmmac_clear_descriptors(priv);