From patchwork Fri Aug 18 00:02:30 2017 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Tom Lendacky X-Patchwork-Id: 802977 X-Patchwork-Delegate: davem@davemloft.net Return-Path: X-Original-To: patchwork-incoming@ozlabs.org Delivered-To: patchwork-incoming@ozlabs.org Authentication-Results: ozlabs.org; spf=none (mailfrom) smtp.mailfrom=vger.kernel.org (client-ip=209.132.180.67; helo=vger.kernel.org; envelope-from=netdev-owner@vger.kernel.org; receiver=) Authentication-Results: ozlabs.org; dkim=pass (1024-bit key; unprotected) header.d=amdcloud.onmicrosoft.com header.i=@amdcloud.onmicrosoft.com header.b="Wa08BX5F"; dkim-atps=neutral Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by ozlabs.org (Postfix) with ESMTP id 3xYNZB5Zlyz9t41 for ; Fri, 18 Aug 2017 10:02:42 +1000 (AEST) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753856AbdHRACi (ORCPT ); Thu, 17 Aug 2017 20:02:38 -0400 Received: from mail-sn1nam02on0072.outbound.protection.outlook.com ([104.47.36.72]:13088 "EHLO NAM02-SN1-obe.outbound.protection.outlook.com" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1753836AbdHRACg (ORCPT ); Thu, 17 Aug 2017 20:02:36 -0400 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=amdcloud.onmicrosoft.com; s=selector1-amd-com; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version; bh=WW0OLNhTDugmu3SF6uFPF3nxS/9NWF/VonzxE7eQXeY=; b=Wa08BX5F0eU+J/gnnUO5yuUpVYUqY8yrtbVJwhigANLuJzf+5SUKiA5Rrgp/SjPgJ2yHOe2JHVKvh0ds5hYXue749NmeoOILV0RHcxmP4YwDSiOMEq+bG2GvAAPjgHssAguT9i+NXLZfKQSyOw0iOjF2n2ztHtnZYul4PvsAXTo= Authentication-Results: spf=none (sender IP is ) smtp.mailfrom=Thomas.Lendacky@amd.com; Received: from tlendack-t1.amdoffice.net (165.204.77.1) by DM5PR12MB1147.namprd12.prod.outlook.com (10.168.236.142) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_CBC_SHA384_P256) id 15.1.1362.18; Fri, 18 Aug 2017 00:02:34 +0000 From: Tom Lendacky Subject: [PATCH net-next v1 03/14] amd-xgbe: Be sure driver shuts down cleanly on module removal To: netdev@vger.kernel.org Cc: David Miller Date: Thu, 17 Aug 2017 19:02:30 -0500 Message-ID: <20170818000230.10005.19944.stgit@tlendack-t1.amdoffice.net> In-Reply-To: <20170818000201.10005.36182.stgit@tlendack-t1.amdoffice.net> References: <20170818000201.10005.36182.stgit@tlendack-t1.amdoffice.net> User-Agent: StGit/0.17.1-dirty MIME-Version: 1.0 X-Originating-IP: [165.204.77.1] X-ClientProxiedBy: CY4PR13CA0026.namprd13.prod.outlook.com (10.173.156.140) To DM5PR12MB1147.namprd12.prod.outlook.com (10.168.236.142) X-MS-PublicTrafficType: Email X-MS-Office365-Filtering-Correlation-Id: c12b271b-05be-4cce-a65c-08d4e5cc6e45 X-MS-Office365-Filtering-HT: Tenant X-Microsoft-Antispam: UriScan:; BCL:0; PCL:0; RULEID:(300000500095)(300135000095)(300000501095)(300135300095)(22001)(300000502095)(300135100095)(2017030254152)(300000503095)(300135400095)(48565401081)(201703131423075)(201703031133081)(201702281549075)(300000504095)(300135200095)(300000505095)(300135600095)(300000506095)(300135500095); SRVR:DM5PR12MB1147; X-Microsoft-Exchange-Diagnostics: 1; DM5PR12MB1147; 3:L4UVyq4FdWe4gldLL3eV6bb3GSJQ0/tNj6IeWk58qQSMisgENdavmadYZktyFmL7YC/c+oDCsz6a1cAcFAV/vBMoVvOWEu3a70GH8Pxk2xPaax7Jn2ovNifJaMwPreRViZOHzFT38DiWsTZI9stFMY4XqYiIkpiUtFvFjBPYJRqeqbwvZdC4TObp4Vkvy2I5BRg5Y/BEWX69QaxjItIa6Jf4OXfxbbuHEQHd1uc6fA/Rr28t6r3g55DbC4KSg3W1; 25:pwqJ64qRkoPJ4Q5S4xgaHm8t7JPFBqe77BGNczAHbQW0Gfx6/TtAfJAqBibjUnQo13X0EV9VT5yONsWhDc2z1tztyV47533FVX9e8TczrxpL1s6gp1SYcSmHw6DRIf3CAhU1q+cJ7EpKN7CxJYkDIx9itrqFYLeRpCK2RM45jDSpIjdKBH2lx3eW+usMCzKwK8zAe+jA4dnEW5f2/cz4sAUwd91Ceqkx6SZSu5xFOQdyjZipEzUzFVmyKXcNzcClLxIihDkE+9HUoZZjotQItaRThX3bsdLt8A2qXtD7bvcl6vp5Hb5YbHPuDsFsyr/Zz/GAS4V/Ayg5ELUtDa4zDA==; 31:TpempSuRpd339hyuRVd8JjXkiLKmKk0MVrx0DOP2mXZLON1nRqgfLSAjnFlqb6i+e6gWxq0ETryDGAKmpC0/jt4NuX0wIDzwuWRuktP1/oEPX9+KriQ2/ALb68lcBX5lcsWZk5owHPuu9CkBYIZo3ALPvGq5T5WDLjJ5l7e+S3g53P9U648aiX+mZCxvJGDNXPLDiqvJpbnzZsqcN1rrVOOBiXR7TdXd6ibRMIh+SIA= X-MS-TrafficTypeDiagnostic: DM5PR12MB1147: X-Microsoft-Exchange-Diagnostics: 1; DM5PR12MB1147; 20:Iwcqwp3Fa5+x555x32rNO1ylp9OLZkRGP72yDpU0yRlkb0rRA6RTCsfcikecO39N1YEnTMmBy4KwaLyp3zyhRGQOIXVl+tsdCkJx4vx4TtfOPMtYklC8cqLy9vWdxe0zie4VPCoJ5QFPDFcJtu0ibgRG+8UlPjZHKrdK0DaETExgxdrm+4ydvP2KbnkR24hrhoNTdzmwguAQu7Ts01n5VgpgSNkeb7aU+2Fugrt30XqZ7bXTvPHS5KJJP4FQyC1Ucb31jd91AQTr/xaMpnNMKKroCJqHGSssNCrERlNl6FDIDUxd2sRxmU2Yz08uECQ4DYEvgmVsmdF94fVRfQjQi2AZmhbdS14n3aGhPqW860XDHGFqZuhR0qGKBn4Til+bnqoS1WLNbXsJCXNPC6NCRYztjKP7ocnvBvaTJ0G3IyPlILaKeCuBzDOHO+Vb/8gvN7byODXSlVcCB0nxdW9PV0/t25IQ0vvwWiQ/DPCo68zZQZ6tz633jBH2nwAb3FDN; 4:MsXqtFm8laE0u1RgUawaEdEAukvKnSSttw79rTm/VOuaR7BfxUzHTYpvmPHhRLciW3Nbwl7maf772Kpdy1Ka0M4Af8zmb/XeoGOGUHvW6QaZex22Tzihal+UTTzMYy7JtUBa2sqkqEPFoy3YQVuHx6iytL7f5FP4lFBmOQq8cOGf6xYZ+XpFX8rt5MegvD2Y0MEd/vL+zbVrd78nmU58OLObQGhxD8jmWUANPrjylR5vGVi5g9UcxS1cQ7T7Mfwja1pLexiDxm8X3k3W9b7w7UYEuQTD8L/1BqKQ+6LVn3Q= X-Exchange-Antispam-Report-Test: UriScan:(767451399110); X-Microsoft-Antispam-PRVS: X-Exchange-Antispam-Report-CFA-Test: BCL:0; PCL:0; RULEID:(100000700101)(100105000095)(100000701101)(100105300095)(100000702101)(100105100095)(6040450)(601004)(2401047)(5005006)(8121501046)(10201501046)(100000703101)(100105400095)(93006095)(93001095)(3002001)(6055026)(6041248)(20161123558100)(20161123562025)(20161123564025)(201703131423075)(201702281528075)(201703061421075)(201703061406153)(20161123560025)(20161123555025)(6072148)(201708071742011)(100000704101)(100105200095)(100000705101)(100105500095); SRVR:DM5PR12MB1147; BCL:0; PCL:0; RULEID:(100000800101)(100110000095)(100000801101)(100110300095)(100000802101)(100110100095)(100000803101)(100110400095)(100000804101)(100110200095)(100000805101)(100110500095); SRVR:DM5PR12MB1147; X-Forefront-PRVS: 040359335D X-Forefront-Antispam-Report: SFV:NSPM; SFS:(10009020)(4630300001)(7370300001)(6009001)(39860400002)(199003)(189002)(2361001)(8676002)(97736004)(105586002)(106356001)(2351001)(4326008)(50466002)(3846002)(76176999)(9686003)(53416004)(50986999)(66066001)(2906002)(54356999)(1076002)(47776003)(42186005)(305945005)(101416001)(55016002)(6916009)(6666003)(2950100002)(6116002)(23676002)(81156014)(81166006)(68736007)(5660300001)(7350300001)(4001350100001)(7736002)(72206003)(6506006)(103116003)(33646002)(189998001)(86362001)(53936002)(230700001)(83506001)(110136004)(25786009)(97746001)(478600001)(69596002); DIR:OUT; SFP:1101; SCL:1; SRVR:DM5PR12MB1147; H:tlendack-t1.amdoffice.net; FPR:; SPF:None; PTR:InfoNoRecords; A:1; MX:1; LANG:en; Received-SPF: None (protection.outlook.com: amd.com does not designate permitted sender hosts) X-Microsoft-Exchange-Diagnostics: =?utf-8?B?MTtETTVQUjEyTUIxMTQ3OzIzOjRPbkVONlIyYVZLUytldTdhM3I4MUd2L2Rz?= =?utf-8?B?cDR1d21RMmFzZkhSWldHRFJUaFV6YmtDN2tVc0ZBeUZwZWpSOFJCVnBveUZG?= =?utf-8?B?bDUvdWIvbkQvT1JsQ3BMeExiQ2FtZk1mYXFaUnNDQmgvSHk2OGRmWlVmdEhM?= =?utf-8?B?VFQvYmRJdktDdjNmWlozS1lRS1pNSWM4V1lKTnVQTGViRGIrdGM3ZTQxYjM5?= =?utf-8?B?ajdMZk9rUTYvZGdmYWpmd3pYTS9ZbmNVTTM2QVR0bzR1NVJJYkMvVUczVXVy?= =?utf-8?B?ZTNZWHZPVWNscityMGpNcElLbERjbDJva1B0VDhWNlBGYmR1MlhidzI1dHla?= =?utf-8?B?clBSYW93bjVobFVjaGFYQnRmQmpUYmFRY1dSd1RtczRSelZZcS9GUy8zWmsz?= =?utf-8?B?UHkzMnhtVE1wMVIvUU96czQ1Tk03MkFWdkgxdU1WSVRaeklyU0N6YVhzQnZZ?= =?utf-8?B?QlFvRnl5eldjcS9sTHhxZ3RIRGRGS25yYThIRElodWljTVkyZ1JaR3JubFFP?= =?utf-8?B?YXd5VjFQaUd3cHRKRGxTalBFMnZFa003eFlOMGN2dytjNXQ1UEJoL0JBWTN2?= =?utf-8?B?NnFMZnRnUTNkcWdOaSswZkh5VXJEYklPRHhxMmswaGFFMnAvbFBlSU9tYytq?= =?utf-8?B?TWVuV0pSK04ybzFmQm91b1JOWUJ2WkNXaHpld3FrY280N3I5ekVOUGxaU1N6?= =?utf-8?B?UlZwZFVzQjA3YnVqeWozMmp1d2o5Vmx6dWNtTTZEYllNdEw4YVVtVkg2QURq?= =?utf-8?B?MXVRUU5wVmtrV2hLRC9jUUZ4cHRBcEdjQ0RqWkVyd0JJOVkyOWtPaTg2UHZK?= =?utf-8?B?c215S2luTi9CdnM0V2trc2l6dEQ1bHB4d2M0bFdrdWVEQ2lwNElYK2dMeFFs?= =?utf-8?B?Y0FhQWNlVk53YzVMYUVFd0dBVGVNNmttb01GdXBTaEJiWEVIdDJVZXFkNExP?= =?utf-8?B?RUp4aG13QWFuekpuYS8wSzNWNy9XTEpPa2xwbm1wb2hCdWZsb2xoRXkwcXVI?= =?utf-8?B?d0pwZy9WUHRHN1B3OTV6SktwcDh0elh2Zy9tTElRdUQzZ0RPT3U4ekFCYW5z?= =?utf-8?B?cG9WL29ibnZDRDIvRXVFbmVSZjdvY3I1eU9UMjJwYkh4QjRBMnlFdUx0bTR4?= =?utf-8?B?YkdWWXYwTXp0aXhhT3JjUHhEb1luOUpoMDZCTndUbEhNNitnQjRuOFE1RllZ?= =?utf-8?B?QTZoRnlDTTdNYXIxcHNacDFZd21UTEVwN2hSS3pnclI5eXBxdFdtay9ENUtJ?= =?utf-8?B?Q3hqL2hzYTJlUmgxL2ZSRlVqU3psZHB1T2taQ0Y3NVhyMDNsZGNub0pzQkJR?= =?utf-8?B?L0c0YU9nTlByM29ITkVqY2JtV2JaMGt6ZVdWMElTL0tuK3pqeDZHZ2J5YlNu?= =?utf-8?B?Q3BqK3ZNRzd3VzhQUmtBVzM4bnRxczZUVVVuV2J3SkUxZUpJYWh0Szdtd2NP?= =?utf-8?B?blVFUDFzakhEVkRRY3pHdWp2aWI5M0xrV1BsWDBzcmY0VUprdnN5WFYzK2FY?= =?utf-8?B?YzVuMVN5UW1Dd2pKNzhZNnppWExiZDhLNFVUY3BpWkxLSjJDNHVsTTMvNmpS?= =?utf-8?B?dDhua3BkdDEwNFpUYTdrT2VoUG5JL2x1Q0RnOWM4SXFsMEwwYnhlc2ZHb2x1?= =?utf-8?B?ZEtJZlVITUxwbnFzSGgyMC9FOUNGM2FWMWdnRktsOHVIU20vUG5HSDdJQ05W?= =?utf-8?B?TnRiSkJtWFJIWmhHYnpIUUV3eHF3cWFheXhmZWYxOVhhOU9kck9qaFo0cjJr?= =?utf-8?B?b3ZqNUVjNFRDdVI5THY5QT09?= X-Microsoft-Exchange-Diagnostics: 1; DM5PR12MB1147; 6:9jJVQqJzDCuxnBDgSFs2lMi2t6brvPeiTIm2xtY6B0rQJXXmnZdhVx5MQ0ftrEBcvc/5Y+9VhH3P9r5cHVXfWW19DJiMxvetGDUCjSKhMztji5gJP1taVTCoLC03TBtdG1i89fkp7NTib55xXhQ47Nqh7c5LNqAIA6M12gg33MZL3srGjbEFkNM49un10fk0dZeQmhNqhgNNPRO1UaijHYYWgfrIVCueIorSRM5GYKQLSYc49VYVJ9pg4nAJu2/D3CguzL7KDCV8oMcaMuyGmkS8dTy1kiCa3ZXRLlPGlPHrvshJIHtj3zv5jnEioCkqiSkj7LiOUt2cHH2DMsGQIA==; 5:UFsXxN1iPjc1UobPDtJg+9UuZjHk3WSDwf5qZXTtNy3Cic1WSuK9+dv6n4sIKaL0If84g9k9QQgyFRkoCHhtb7kTH7PV2tHb1AiaGyXLoSswD6hmDoW+f5T9zfn4SHdVp6+bV47pAyDVpQKrTlK1FQ==; 24:wUicm/bz5sqxHEqNAVqGrnTT5moyTd0v+vGdTRZ25714Yoxf2qqftSv73GdIBWCzZaNhj+7gm3QWzepJA8cXg7gbRkcGc9SJWjS6FGv0RJo=; 7:MHSt+xMDTr9y8z5FV41VvkrRT5b7KJr49xWvX6ycbtJO3bdq6Dp3qhu+mXgj0BtTgyYoCoQShwvKQMPVYE9CVphZapgpEjAVopOSITrRePrB0VZ8cPkCm16nQoA6dkP2CAfCfA8q9f/fqnm3vCXADZc6vnGb9M86oPEe72o/fn8T2p4mo05BC0FxqPP5d2Ui7kvpeyeS5qlIfga7prKc3fzABEFRBArk9o4XKYrJEF0= SpamDiagnosticOutput: 1:99 SpamDiagnosticMetadata: NSPM X-Microsoft-Exchange-Diagnostics: 1; DM5PR12MB1147; 20:W5ortKzCCLjsBk1b7gy6k6Ip0RJsMlBHJ8Ke3ftbIaxNJFOI50Css/ttnAPwmNUuPv3z5HzvWImN2otgr5xN5MCFOUZ12q0uhfRgO5uSpdo9q4Tf7m3GUbADmJXcreU57GyZPd4tprq7b9PuMRpawrz+CAVwsalAIjVR4vpAZUYfdcNQhzVtHNLR9an0x3+FcH8ce/ar2/I6BtFgL4UfuflS56zNC2O2G8gWoGP5SbobWJL9JEb0EC6lCOW0sS3a X-OriginatorOrg: amd.com X-MS-Exchange-CrossTenant-OriginalArrivalTime: 18 Aug 2017 00:02:34.9968 (UTC) X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted X-MS-Exchange-Transport-CrossTenantHeadersStamped: DM5PR12MB1147 Sender: netdev-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: netdev@vger.kernel.org Sometimes when the driver is being unloaded while the devices are still up the driver can issue errors. This is based on timing and the double invocation of some routines. The phy_exit() call needs to be run after the network device has been closed and unregistered from the system. Also, the phy_exit() does not need to invoke phy_stop() since that will be called as part of the device closing, so remove that call. Signed-off-by: Tom Lendacky --- drivers/net/ethernet/amd/xgbe/xgbe-main.c | 4 ++-- drivers/net/ethernet/amd/xgbe/xgbe-mdio.c | 2 -- 2 files changed, 2 insertions(+), 4 deletions(-) diff --git a/drivers/net/ethernet/amd/xgbe/xgbe-main.c b/drivers/net/ethernet/amd/xgbe/xgbe-main.c index 500147d..53a425c 100644 --- a/drivers/net/ethernet/amd/xgbe/xgbe-main.c +++ b/drivers/net/ethernet/amd/xgbe/xgbe-main.c @@ -458,6 +458,8 @@ void xgbe_deconfig_netdev(struct xgbe_prv_data *pdata) if (IS_REACHABLE(CONFIG_PTP_1588_CLOCK)) xgbe_ptp_unregister(pdata); + unregister_netdev(netdev); + pdata->phy_if.phy_exit(pdata); flush_workqueue(pdata->an_workqueue); @@ -465,8 +467,6 @@ void xgbe_deconfig_netdev(struct xgbe_prv_data *pdata) flush_workqueue(pdata->dev_workqueue); destroy_workqueue(pdata->dev_workqueue); - - unregister_netdev(netdev); } static int __init xgbe_mod_init(void) diff --git a/drivers/net/ethernet/amd/xgbe/xgbe-mdio.c b/drivers/net/ethernet/amd/xgbe/xgbe-mdio.c index 2222bbf8..2409202 100644 --- a/drivers/net/ethernet/amd/xgbe/xgbe-mdio.c +++ b/drivers/net/ethernet/amd/xgbe/xgbe-mdio.c @@ -1533,8 +1533,6 @@ static int xgbe_phy_best_advertised_speed(struct xgbe_prv_data *pdata) static void xgbe_phy_exit(struct xgbe_prv_data *pdata) { - xgbe_phy_stop(pdata); - pdata->phy_if.phy_impl.exit(pdata); }