Message ID | 20170112010902.GA9597@felix.cavium.com |
---|---|
State | Accepted, archived |
Delegated to: | David Miller |
Headers | show
Return-Path: <netdev-owner@vger.kernel.org> X-Original-To: patchwork-incoming@ozlabs.org Delivered-To: patchwork-incoming@ozlabs.org Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by ozlabs.org (Postfix) with ESMTP id 3tzSNv40Qrz9t2g for <patchwork-incoming@ozlabs.org>; Thu, 12 Jan 2017 12:10:23 +1100 (AEDT) Authentication-Results: ozlabs.org; dkim=pass (1024-bit key; unprotected) header.d=CAVIUMNETWORKS.onmicrosoft.com header.i=@CAVIUMNETWORKS.onmicrosoft.com header.b="NNA9nvlz"; dkim-atps=neutral Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1761523AbdALBKH (ORCPT <rfc822;patchwork-incoming@ozlabs.org>); Wed, 11 Jan 2017 20:10:07 -0500 Received: from mail-sn1nam01on0070.outbound.protection.outlook.com ([104.47.32.70]:62795 "EHLO NAM01-SN1-obe.outbound.protection.outlook.com" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1761280AbdALBKD (ORCPT <rfc822;netdev@vger.kernel.org>); Wed, 11 Jan 2017 20:10:03 -0500 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=CAVIUMNETWORKS.onmicrosoft.com; s=selector1-cavium-com; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version; bh=2rHjhY3DA7h9C5SYf+9VtFW0LaI346DGWsxV7LKP2Gk=; b=NNA9nvlzI7xLPX547wAQZgGvG2dEFU/SNqLj2rEHL7JVOKidj8iKR+Kyjw+sTlMbUykeOP3UIAQuFeMo4OTaBpdDjI1FjdSSij0TJnU7O1BXj4ZZAssGDi715Mdv1dKCSQy62o035Vc5XoE7sUjRjYBs1T5h/Se6vLx9dKef8bw= Authentication-Results: spf=none (sender IP is ) smtp.mailfrom=Felix.Manlunas@cavium.com; Received: from localhost (73.71.186.126) by MWHPR07MB3182.namprd07.prod.outlook.com (10.172.96.140) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_CBC_SHA384_P384) id 15.1.829.7; Thu, 12 Jan 2017 01:10:01 +0000 Date: Wed, 11 Jan 2017 17:09:02 -0800 From: Felix Manlunas <felix.manlunas@cavium.com> To: <davem@davemloft.net> CC: <netdev@vger.kernel.org>, <raghu.vatsavayi@cavium.com>, <derek.chickles@cavium.com>, <satananda.burla@cavium.com> Subject: [PATCH net-next] liquidio: remove unnecessary code Message-ID: <20170112010902.GA9597@felix.cavium.com> MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Disposition: inline User-Agent: Mutt/1.6.1 (2016-04-27) X-Originating-IP: [73.71.186.126] X-ClientProxiedBy: CY1PR08CA0032.namprd08.prod.outlook.com (10.163.94.170) To MWHPR07MB3182.namprd07.prod.outlook.com (10.172.96.140) X-MS-Office365-Filtering-Correlation-Id: d517eca2-efb0-4032-a4a2-08d43a87bc36 X-Microsoft-Antispam: UriScan:; BCL:0; PCL:0; RULEID:(22001); SRVR:MWHPR07MB3182; X-Microsoft-Exchange-Diagnostics: 1; MWHPR07MB3182; 3:o9G5KZ4/CvmF3mw6lFmPitR4p8VZKQXzGsXeudRz+FcdCp6xk44U8t8FrhMSyTzbvXpzC1MK3wL9yC9mHof2tAYEBRrQkMbUt/SI6BLfVcSjwniehIodP8JP84XkZZOKRaPcv9irQSU32Qi5YvRK4ZjunTkI4iv+lL7pd1u2w5Wllz+udecpbL4nR1UCNMOG8KKYFX/JXbwviaK9zfNs2I+bun6jwbJ3QDVDHa1MR0rgEde4jsX1X1H9o7ExxRGQj/xfEZL2I9zKXQQ97YAe2g== X-Microsoft-Exchange-Diagnostics: 1; MWHPR07MB3182; 25:aUiP7DEwlPHGi56iAVFCabZO5ZVs8DXCjTqTmXqWv9/2hjeW3cRz7qDot/OIzE6HdCg2cyTIwM26XUU3NAamwhvEggHENrat2fiR+Ejea2iqBG9w5MGSNcF4rfvSGuvQ1rI3nW0RzsFKwyi/iKmTnAa0UN8rEA9eor6f/HA+w/L7lNsDywbQNvjpUSZMS7iUti4rlCPk44PZnuGcmbOmBhES5bPSiDmQnx9ioEiFd5FH4jqVEwE8bWaPB5CBEKA+c19jS/oXUaI64HWFROCwKdAEiQKeX1f21r8bDBCQ9+c/hejebdVzEOQMqugs4S7Xp0Qx/u3ZUZxg7JIrVE97DzRDtRnfrEqGvSd3pM9cYFoYOmzIY9fM2vv4FJ3zdi4Mr98cFZgK4zn/Nnes3F29jc9KRfVsWqXmB8hsPPp5DOXz7RO6F0fdX8I/pRYdOc4C9LhBSP/VGfVOB1zouHF5RhIk97fe+y1qo5/pG2t+ynM1DmDBMTK48+qjLjH648T1zCOrQoO7VPxZNcMN23wfC9hmIsmt2rwiYK3lEGHC80g2rBtOayxel+Jeul6L+vKF2Isv+9u+E1hrU+2TLXWkpFtgKPDKMtcpUKSozKgqauW29F6/YCq2iInuJYKRxWkuNeJQFr1haIjMxPAgVEk37c16EKsalITyJDgnMnI026aehEHIfyhs8V45EimFzqEV8VkpwytQvZIVbKUvSnCMg+AMrHDQ1n5OgHn/DScPQk1gdJY3QxL/+waB/GBsDooI7dY04E2SPG2WyNxiGkOhdC3yUJtAFhYSIobjyBs0vv8= X-Microsoft-Exchange-Diagnostics: 1; MWHPR07MB3182; 31:wRrhAb0sFpoHbIn8V0DDjgzKUypnQBkPOQYuzNCkwVTU/2v4aCKAsgLM5JzVG0LOstJAWw++2SFWfUKk+kh3ocjZYym+N91VZQvJaFJ27vPLkf9nNftiuHjhqCJ1orA2Rj8Y5V0KZ6gGOcTOdoNyBab2+gdxyFY5mn/ll8II0aldxqfze0vGDzyvCoEiluf6As5HfwzKdPvWh8h2t0NTA/l+euKekUXIEHAtIC9ISyu8GXlR6ViUfisvnzvUm970; 20:OkU7orZkjOL6s6prLYRXms2RX+ChUpkgacXYOqXXxXJg00JISjsk3d0KTYsLunQKSt02c3MyrGenP0imTDrNhK3ppQ+TJyEaSzYtPYhxQLnTJ8AXTDRHztEWqgPNzNuBPdPUruVAMbwhfbrefiGELeGsFK7n9n5Lbz3c0Dt2vE5swZQPxFKhdMD6ApDUfvU7+M7h9uYWTOOpcwZOxXGV24fDP+oCU0J14snR5LQG4L/fXv6fagFyiOi++aXmovZczJl8bRVeOBiN1kD2IhWmINIiDxIHjmbHv45ifd8h/lo8EPW9NH+RA2dBrlYZdOlfbrrD3TRz/1u3VPBJ16Pnxx4llgY7MjrEUQ0eMeqgvi+KGrtq8smSnGle+mMoEytNsVFuqqmit0Hnt02xyyrpky4gpqzvtMTqOaF46oMvq65GqsM+gCcy4rWb7pFs4GrGivBoCpdGZ49WlWnNwY9DNQhs2cnkc2aJ5KsA4NR9AbNrQyKzH8UO1gI4jgKTMQfs X-Microsoft-Antispam-PRVS: <MWHPR07MB318252CDFC800F47EF216520E7790@MWHPR07MB3182.namprd07.prod.outlook.com> X-Exchange-Antispam-Report-Test: UriScan:; X-Exchange-Antispam-Report-CFA-Test: BCL:0; PCL:0; RULEID:(6040375)(601004)(2401047)(5005006)(8121501046)(10201501046)(3002001)(6041248)(20161123555025)(20161123560025)(20161123564025)(20161123562025)(6072148); SRVR:MWHPR07MB3182; BCL:0; PCL:0; RULEID:; SRVR:MWHPR07MB3182; X-Microsoft-Exchange-Diagnostics: 1; MWHPR07MB3182; 4:59fFicZKkpULMukfMlc/VqN0Bn11Hl6iPcHAhRlVaiBK4Z0UyTrXTXF0Zo8AUcd6i89X6Qw2w2xYazT4lx6kAeww1EOe7Saes9VlnEs6ve3IV4cqeCjowCMqrT82jgR4lGwKfad67yVquDlewzLcR0H1WVCR7pmCxrAj73uyLvxw8p2Yq6ZbU4wx/IPfcizshGyEAIulJTOhs2c1SnsI4s+KOAWeW7nctcJnIO8zqlHdst40kzDA5ZjiaRujFa30nSx3oq4FkCnr6PfCE9QQ88+5sfs+iT8FyVHem1SXTnT43RLmI3sibLKxtwlmBSS0b8DUDEXGqWbAV0pwdv5WXO+fr912wlIBeuEkcm6HQ5GBwb5Zxo6wDLKdEa3tBeMPOLbI871ouOWc1Ewy4et55flyJ+Qdc/ctKToqMhTID+SH5FVuw3Jg/K3AyJoGPzAjHMDoti6q8lBHvfQ/FWcomc0zRtND7u+SpI5/Ybj3hgQb7zLX64OCsuvd4XvUQ9JQR20XCpBVEUnD6cih3fB/xhY2vzaU6dyWczMKaJAjgtWNijuyFj7e6Q3Zatvq9A2cZoEOMWpZZA70IJLYTO+IpQ== X-Forefront-PRVS: 018577E36E X-Forefront-Antispam-Report: SFV:NSPM; SFS:(10009020)(4630300001)(6009001)(6069001)(7916002)(39450400003)(199003)(189002)(6486002)(6496003)(33656002)(97756001)(38730400001)(97736004)(46406003)(5660300001)(81166006)(23726003)(1076002)(6116002)(83506001)(3846002)(4001430100002)(2906002)(66066001)(54906002)(68736007)(25786008)(4326007)(47776003)(86362001)(305945005)(101416001)(7736002)(92566002)(107886002)(189998001)(4001350100001)(106356001)(81156014)(2351001)(50466002)(8676002)(76506005)(110136003)(50986999)(54356999)(42186005)(6916009)(105586002)(18370500001); DIR:OUT; SFP:1101; SCL:1; SRVR:MWHPR07MB3182; H:localhost; FPR:; SPF:None; PTR:InfoNoRecords; A:1; MX:1; LANG:en; Received-SPF: None (protection.outlook.com: cavium.com does not designate permitted sender hosts) X-Microsoft-Exchange-Diagnostics: =?us-ascii?Q?1; MWHPR07MB3182; 23:hgErsW7h9efGoGv09Lej+hAAQvwEEWmpM/RbziMK3?= =?us-ascii?Q?6I8xDwDACTdFG/7pUd0V8BiQsb1brt9eRFJ0qNU3/jx2ZjICP9dDlWkLRtoW?= =?us-ascii?Q?/DUhzK+PmqcHMIR+Y0eY8ljJlyv4INgxAKcfMRhjLfmA7akxH/bjg9WgGH5x?= =?us-ascii?Q?huY9lj6+YOKJhDH0VmsMBqbf/ZfP/EMc2O4YlHI8Jn+cjb253+cht4c3cVMz?= =?us-ascii?Q?FCtkiZXpjhE/7EM5HcFTlxeb60LGTJP4Ki2Eq6kLR8ApsZKAbZWfLEodiuVj?= =?us-ascii?Q?pY6fOOmB/KJyODszWhfoSxSlLnMf3ywLUfna9zrLng/qiCiKXaNKj7Ny15YJ?= =?us-ascii?Q?+fS8FCgWVWQKtg/SkbTgEI04JVCQMxdXCrRvsFn/BPeDTXzjIUUJG9eI2zB1?= =?us-ascii?Q?FG8RgxwmCylt4aUt3L+VekJHkk3lbJ7e1MRd/hTh3s8WyVGVbyJnomY//sEt?= =?us-ascii?Q?3/1gtQAMynP3kvpiFaLQUyMwBOfEjlrAiLnMjwJc8xgh5Ojxh3QTx5w4XCLi?= =?us-ascii?Q?UK1WQ4VHQc3wiBfb48cqIs/glFa2NjfDBSjczPagbpeLtr4GxOWHGpDXZ+LW?= =?us-ascii?Q?+jyElFR1iAGugrFliEOk+QmaNx41qWv3wJdqFnxpb1Qj7Y9u43iWg9WYgbyQ?= =?us-ascii?Q?vztp6fDpzsO9/H24XO9BTPop640/sLaW0NuzM6qqGb68FuztjtnbxivBQAln?= =?us-ascii?Q?+LUvyCmLn8OHKRsF/+LlqfSOPGH5l4a9ZmfOfCwHUJ44FV+YdO3L4K0EmOwU?= =?us-ascii?Q?dGkBH4zHqkaOw0u8EwxOiWlbfqGw0iWQLzaYwQGLeZjNWtP/DB7Br2cHqmf1?= =?us-ascii?Q?v/7NAw+9WtEu3sIBkauT+gEXKxrJ+lyf5PBkydeCw/aNXWHxEaZzRj+oDgpE?= =?us-ascii?Q?/sxAzNvTiqcOhzJMW6sFyhZPzTtn52PXTFuqmdciGPMnuEnKXywsTay+dM2v?= =?us-ascii?Q?jf1zJqMB90R1IitA1nZofyPgCKs9wASP034kN3IqFInUS1W3zRtXkhlnlOI+?= =?us-ascii?Q?CtxyZ0uNlNaT0rVUIJcOM+RfZrNZHu48rVOSczqSoJlGgbGqW4KnCv377S6v?= =?us-ascii?Q?ZlWUQNPrmef+63X4wf8fWUg0xzwYbV77XYg3yIVGu2cNYT7RtjIfiRzQqn1Y?= =?us-ascii?Q?gp1qfIxbWtsr2nJMY54QlydDAFT4Hql2beA5dXMa5MrAik6gdNm+sWYtH1bn?= =?us-ascii?Q?XGoUSddlGEoY0df0ZodHUH4z1XjWtULmmrb?= X-Microsoft-Exchange-Diagnostics: 1; MWHPR07MB3182; 6:bBxIBVnWQ3U7BMMOUHlVG2TRixVG8F0vMebrSD6DWSKrmfQgmE2/rGQkMpv0l5+23uXAeXwuOX/c2Xkyv0A1zq6PTVynKqS0W4RhbERQZ0xgU89EfTEUVBkgTQVacdwVTZYMs0t/+l7FwMriYFKl88rbO1Y2hOnWYYUL7UravES5k4TVUjC8MWUR3be28goUH0nWsuCUmEUL0IcSc5hg6tc2hoTbId8XUFjLINTrIGI27PrWf+TexmMN1Ah0xC524BfPObC5sZfGgUm5HFPC8fCeIMgqW33O6B+b37ZErzaVgoDFeX4gYTzRxyGbi7487DpOc5cLWx2ua831/35MvN9XKGLFo1IfwtCuug0vhQw+o3ptCUSQ+CC/GrKH3oxL0UJ3U6LouAGYpq5bhDSV9wra0ZuQmjajDxMEmeW2Fgc=; 5:uZeWLwgU8QHor3S878xxKfj42M1IxjZ++eM1HrjxntL/6ujK8s4/95oi8e6b9aKRtF1NCPZPHkU+NQQxuyvORhSO3i09quIfBiDKAlUCMw1Q4miw/CZu1rSBJGSQ43yjGLNdRS9pkCTZoT/Q/uhN+g==; 24:AoWB2PIYjhe2N+GACdqWBCTjESicFLyNdn6r8k6w2Te1SxasncotSE6iDR8habsRHhvVK2LotdZ8qRRy9rUC4oJWve6A/ZiNeBcwtvo1Q/8= SpamDiagnosticOutput: 1:99 SpamDiagnosticMetadata: NSPM X-Microsoft-Exchange-Diagnostics: 1; MWHPR07MB3182; 7:YR8Evk7fxGnojxFPmPVMjpMnkWq20laEZccxr0/SMX5rJM0A/MLintuMOu7H1+E0AMrcnkuyYA9iEsahtR2xXC+EYl7ZOJDy8HJydqXSvGOYZNALWol99Yq6gpLrggFC95k9lSGbRczCM9YMsp/AttZpWlEnNRNFjO6ZM0rmQB//gX7QDEdrft7O/qa8ZXi/FPkpfc6u/nOckbcgaM5tUY+XkjsWjCiqZcJTJLoaK/W/XbLoPYntxvaB8vLOTF8zAFfYXUol2Un+ZlchEbpjvaIphXl4Mf3rd/4lAXy2X4ZtWjCqutJErzd4quIsc0M3ZTpiuqJk+Xl2AFWFxy8Z+VN/3j6ef+MXqxhDgHOTuaIimyERf2Z+eWRGHKkBUgBLYtdSb8TDsFqhXhTrejugkXx+en3GIGgKXmw6/kebrjwe+ypHFQdbpwBm894SwlWxo87YdrvLG2SoKFkTZNa9vQ== X-OriginatorOrg: cavium.com X-MS-Exchange-CrossTenant-OriginalArrivalTime: 12 Jan 2017 01:10:01.6894 (UTC) X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted X-MS-Exchange-Transport-CrossTenantHeadersStamped: MWHPR07MB3182 Sender: netdev-owner@vger.kernel.org Precedence: bulk List-ID: <netdev.vger.kernel.org> X-Mailing-List: netdev@vger.kernel.org |
From: Felix Manlunas <felix.manlunas@cavium.com> Date: Wed, 11 Jan 2017 17:09:02 -0800 > Remove code that's no longer needed. It used to serve a purpose, which was > to fix a link-related bug. For a while now, the NIC firmware has had a > more elegant fix for that bug. > > Signed-off-by: Felix Manlunas <felix.manlunas@cavium.com> > Signed-off-by: Derek Chickles <derek.chickles@cavium.com> > Signed-off-by: Satanand Burla <satananda.burla@cavium.com> Applied.
diff --git a/drivers/net/ethernet/cavium/liquidio/lio_main.c b/drivers/net/ethernet/cavium/liquidio/lio_main.c index b8b579d..cc825d5 100644 --- a/drivers/net/ethernet/cavium/liquidio/lio_main.c +++ b/drivers/net/ethernet/cavium/liquidio/lio_main.c @@ -2693,13 +2693,7 @@ static int liquidio_stop(struct net_device *netdev) lio->linfo.link.s.link_up = 0; lio->link_changes++; - /* Pause for a moment and wait for Octeon to flush out (to the wire) any - * egress packets that are in-flight. - */ - set_current_state(TASK_INTERRUPTIBLE); - schedule_timeout(msecs_to_jiffies(100)); - - /* Now it should be safe to tell Octeon that nic interface is down. */ + /* Tell Octeon that nic interface is down. */ send_rx_ctrl_cmd(lio, 0); if (OCTEON_CN23XX_PF(oct)) {