From patchwork Thu Mar 8 06:36:46 2018 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: "Manlunas, Felix" X-Patchwork-Id: 882973 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; dmarc=none (p=none dis=none) header.from=cavium.com Authentication-Results: ozlabs.org; dkim=pass (1024-bit key; unprotected) header.d=CAVIUMNETWORKS.onmicrosoft.com header.i=@CAVIUMNETWORKS.onmicrosoft.com header.b="MP/prB2D"; dkim-atps=neutral Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by ozlabs.org (Postfix) with ESMTP id 3zxglt5m6tz9sd5 for ; Thu, 8 Mar 2018 17:36:58 +1100 (AEDT) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1755544AbeCHGgy (ORCPT ); Thu, 8 Mar 2018 01:36:54 -0500 Received: from mail-cys01nam02on0077.outbound.protection.outlook.com ([104.47.37.77]:44960 "EHLO NAM02-CY1-obe.outbound.protection.outlook.com" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1755511AbeCHGgu (ORCPT ); Thu, 8 Mar 2018 01:36:50 -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=nxksb94Pk3x0kFqkm9VjINxz/sAVQemw0Zy2GIQJ+tY=; b=MP/prB2DhrwQMUHfz5C3daOaCCqsmGbOlFn+NkD+0zT4rqdviHolAREhSDwyDYDJ/TG76sJkw5kiUn5IBiz5zAQBomXpp+7hFFgoyHyCEPejkZ/97/xPJ3MjBrfLaGbYZhRs1REu9wGOCpj+Su3SypBHiUDWgPEEtaIsAYteBlw= Received: from localhost (50.233.148.156) by BYAPR07MB4213.namprd07.prod.outlook.com (2603:10b6:a02:be::15) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_CBC_SHA384_P256) id 15.20.548.13; Thu, 8 Mar 2018 06:36:48 +0000 Date: Wed, 7 Mar 2018 22:36:46 -0800 From: Felix Manlunas To: davem@davemloft.net Cc: netdev@vger.kernel.org, raghu.vatsavayi@cavium.com, derek.chickles@cavium.com, satananda.burla@cavium.com, felix.manlunas@cavium.com, veerasenareddy.burru@cavium.com Subject: [PATCH net-next] liquidio: fix ndo_change_mtu to always return correct status to the caller Message-ID: <20180308063646.GA5384@felix-thinkpad.cavium.com> MIME-Version: 1.0 Content-Disposition: inline User-Agent: Mutt/1.6.1 (2016-04-27) X-Originating-IP: [50.233.148.156] X-ClientProxiedBy: BYAPR01CA0009.prod.exchangelabs.com (2603:10b6:a02:80::22) To BYAPR07MB4213.namprd07.prod.outlook.com (2603:10b6:a02:be::15) X-MS-PublicTrafficType: Email X-MS-Office365-Filtering-Correlation-Id: 398627cb-d7e3-4021-f35e-08d584bef7f7 X-Microsoft-Antispam: UriScan:; BCL:0; PCL:0; RULEID:(7020095)(4652020)(5600026)(4604075)(4534165)(4627221)(201703031133081)(201702281549075)(2017052603328)(7153060)(7193020); SRVR:BYAPR07MB4213; X-Microsoft-Exchange-Diagnostics: 1; BYAPR07MB4213; 3:F3L4Kh7dcz8e9NFIUFYQgJFn+jl68PCpYu10vccfs2C8tKbSLrNFFfXYnbiyTTojNyR5RRToxTN3CZId/TRI/5NjBVWrgqHoBDZRPE7syRmsDxu/ZdFFSP/zkZQFJBcRTJbvGD+m4xi+DlVUrWF1WjACzXc5OJA40zz7K2sKYHE7dYiG73Nir0vqZqpR3NZnDE782oEXNRrCBnP4uhWMawCcRRi6GcXe+khb5emYmxbNPStoMCns/oN6PrSYtWHL; 25:njFHIdvLIIwAeiIDLhvt26KDgA9TnRfbY0e9m9Rq+uMH/pKfif6Xn4nGOXqr1iTEcA909wnjpxbhirZvNarrR7v9+WabngddeUsvCDxWD4T0jjuGclhwztYNh2PrQp/ShKqPpWe6ZuztdItKZJUGKFGqMxCczT/PHdOQoNfHd/PxKgv5BCVAwD6YLJUBekdSAasKuAosqeOawUqLhxdtHh6jpyOyGTO14frixtgtnh7UU/alsvbdTSDpR3ZPa5CySIGYIKmQZm5E7cqEYk784nTvKrUFKrUFA+FBTNzw6nGD0P2jryE3vEyarDlCjP3c0CZJGxm56hfvAkAm3DwdmQ==; 31:85ByKX6UChVU9Hqa2+XHuVW4U7xDcUjcB0jA9oh6JJ0ckkNQ4kkXA63nsUWJB8qGoctgJIJNlHRVUJjoSQtKjaMJSn344/tkAREPDlslBaDGFdnXO1tz/0yl0nk2MQusYwLCswEliA+b6Hq0US5Qi0Nqc5e199NWiiRZkDkfqa0o0OuI5+Bl39+iSmG4vLoPik+etni7zJCYizxT5j3kLk2FrI3OCQXZc4QmWfKau3I= X-MS-TrafficTypeDiagnostic: BYAPR07MB4213: Authentication-Results: spf=none (sender IP is ) smtp.mailfrom=Felix.Manlunas@cavium.com; X-Microsoft-Exchange-Diagnostics: 1; BYAPR07MB4213; 20:XDd5efLPsxZxc95t56a/07IiHPmdmtm4A8X6tPLbg7WMmiKMSzZh1vhd9jF57nxbIMofx+KnyrH0olWlXErx60X2bRB2ggOjSLhBd1zc9VhBaW3Q71zzbI+bstEDHUUWPZ1S/jmmkP+gaxlQt+R4Y3pZg2j3T5NXSPLZvpIx9nhfudR81t93JfedrbHYHrEEj5ikbNk+KlVe0p2+I6sxNkWYmwVTJhNhPvg1iR6vCXgK6iiRtx5Mo3sYS+7ZQQ/moqCILg61AGcW7HwUSrYh2LUt7uVXLZ4kwhTdlEs4gL7gKu5AghctV3nTH/DA1WOVBcEMJQ+7oyw3PmnWWcoEonwbovAmCGDeJZhAtEmwsViStmY/bapPgW94DcLV2PVuBI9JZEAg44XL9QhRC/YnYN7KGwmC/IcuJ5sXkPX4fPY1I9DdULed5G9TrJeJFivi492UUxXMP7M6G7SgATfSbR10musEdYFbhrmHly+r+wEmY7FEGrgVW6q0NT3PYnnL; 4:AXnnSfBPWaj5J/ZQ5r2wzQl5lc1kMgwkJ3K24xNRpcTPjCXSuJZoAD4idYqGJoa73Iy2OB2eqJwGXIa0ysANXY5tklI27WtoGLmevbfgZ9GzREhLm3PYbxjIC4hHRtfmz0oKBpLcZZCPtIhYdRChApWxCZOMQKljFZ446fFGuPms+ZVzzirnL7b3bxNDepqFlEQJm66iUSTzIRSZT6UImC82WT/jNvOoZiOEMGymXIfmvmrgE2608aEDYlEP9DuHvCfUbOurLs3WfBQg77MeAQ== X-Microsoft-Antispam-PRVS: X-Exchange-Antispam-Report-Test: UriScan:; X-Exchange-Antispam-Report-CFA-Test: BCL:0; PCL:0; RULEID:(8211001083)(6040501)(2401047)(8121501046)(5005006)(3231220)(944501244)(52105095)(3002001)(10201501046)(93006095)(93001095)(6041288)(20161123562045)(20161123564045)(201703131423095)(201702281528075)(20161123555045)(201703061421075)(201703061406153)(20161123560045)(20161123558120)(6072148)(201708071742011); SRVR:BYAPR07MB4213; BCL:0; PCL:0; RULEID:; SRVR:BYAPR07MB4213; X-Forefront-PRVS: 060503E79B X-Forefront-Antispam-Report: SFV:NSPM; SFS:(10009020)(6069001)(39380400002)(39860400002)(376002)(346002)(366004)(396003)(199004)(189003)(6916009)(76506005)(72206003)(6486002)(105586002)(2906002)(58126008)(305945005)(316002)(81156014)(81166006)(16586007)(50466002)(97736004)(7736002)(33656002)(106356001)(53936002)(2351001)(2361001)(5660300001)(107886003)(16526019)(186003)(478600001)(8676002)(6116002)(1076002)(3846002)(26005)(386003)(59450400001)(68736007)(52116002)(23726003)(66066001)(8936002)(6496006)(25786009)(4326008)(47776003)(86362001)(18370500001); DIR:OUT; SFP:1101; SCL:1; SRVR:BYAPR07MB4213; 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: 1; BYAPR07MB4213; 23:EHqbLhW8eSg4Qx3kEZ/LXbzR8arzDPinNaguy4DrZT3dwmAeavPMyA9a/aczndhR705cTWyhkYs6cB6tgP6DjvHWxj4BX15r0elHWJ7NP/qMlB9YGjMk5eh81PPIH/LRwy8UO6AW0vkD/jvMnd2n3DN6XPykMDOPvUh9cJA4Y4IxXoMA5miSm0MgB6l59A8t3IYG/yvC4he8FBNBFen9ndRWjr5J4rMXtYMJFAIVErMJwiXHfgy0+3f7Xf9OD+eHk0cncMXnuEuOm/WrMQD2O51eIYjh3nu1727hbD6/4XyAl26JaNp0YWfT8CDIbYqJDLGC3M1ifdVLPxKp1IhJHjXldYFcCGHy1kIOzFlKEtywSwARtfLfe/qz0rZ0ew+hENoiV8WTVv1cuyUfpHRrHLhSUrW4zaeH6Ywvdc3MoVhXbQxLuczVwJG6WNh+ktUNu3EKRmXs3edKmqn9KGUcApVuj3TCW7qVuPqf+Bm1oAhZpd3dDRnNaAOMDRjcJNUOOreBpAoh4CFPY54ag2iF6WtE2r1uCvCg+gLm9plYnkDBiihgOppMoMSguTWf+m1hqQ4IORdqE+wVV6RZh2f633nGP3CleDbK79HJOvg6/Qt2TWO4khP/Y77uJkJJoVMCXjGIAvlxeTICvncRNMwrksJOqX6DRugPF1hyf1NLM2Q+dL51mjPqT98zfn0RFj0nLGIJWCshAzLjbKrTV9p9ljgCtx7eYS4DqEv1uLpeOBKTWn3a5VBSNwPuESr8sQmnMaVpsl6SGGgsuhv7hbTZahEPEpRl/zHALT9HdtbWjZMmOtdXfT5SspAZAJjun3Aa+X+xNurni45Z8A64D3bkIt6qimjCQclURc/Ekk7+oWVee7N1GBYiO7GmzvrY2YprxDieC3QMUeXrMXk9gKl8RsteL2n+y3WGrKA63lA4mI9rIktvsRp8BSy3HW50v35UP0O7rBkrymOUwV8XPQdJx9qQjk5uGdCGG6H8W1x4HUr1966vhlSbekxwJfX6WH5TyK35IaLb4tqwqK/yMY3RgCvJJvVWJRwLCYefwV4HYhJTyAQuqg/foNEZCyU/J5LIeyEIIWxeE4gkQp9gUBbqeWrRehD8I+qSYMpjyxm0DI2hu3YQkbcRRUdQuhK1tTbuoFu0AlG8qyTtBcbjE519VLnFAwJS8HauNHrIYwsK3ic= X-Microsoft-Antispam-Message-Info: zImC2v3fgXMtMyas0OrxTApoWpF+EHbdxYv0AIT1yCHpSP9djMV5DUoRVaBCbMTvaUTQg8d+4BqX+G9usz3F3qDFaEs6vlrTMGUzM2bDnzl9v6D5N4UawQUIgilgareBSUpMDiEA42aTThOTLixEIZy5hhdSkyrlewppR4XcEXSdPbmN+DIT/B/8V9B8lIL1 X-Microsoft-Exchange-Diagnostics: 1; BYAPR07MB4213; 6:NxstO9hXZeSPKUvWID87xaKJvgcE+S48kua6CtAsE+Jeq0qaG0kv4A0EVyoU6LqVYS90VU0ZHLVc+yFUCTXejg9PGqLCGWICJXQIsLTbyJcn1UQ3OwkIPDo1jQO2Ki1kAzRa2tttohpcqw3mIlBVLpyJwS48RCVDK5NgyLP3n2s6G7+rBCfcraP8LqsLWFz5Y44Nbu28TBjLOcXlSO0+zjAoWUZ6PVo74LJ4i7LJPuyHxXJbA+oUz6n5ffP7LLhV10g4cqb1AO5q0+KQQyImZy44Hu3HUR0Zi35JU7JCS0yoEsHi0/Z3V3ed9wLAWmaiXOd0tUH3PPcsp1bnA7vCiWdwHj8wBhX+aYd3jxxeIHo=; 5:ALl8NYKo1pNk0RdYHBCRAElQprRiU6Msd4vjOWebvz2/HftqlT804N6aVRWEodvDuOCEl6lFI2vMj5j+EEq1DmWWthlkXbqc2moyqeztOeFTZgYgC1ZiwDL8v+W9S8h9O6ODB+DsN/LwF12kO5NYdrp+gOtRHl8GVJUZZh6SHCc=; 24:CoCb8y4StAFiS6/25P2FLcS7vSIn2H3oMr04RUQbqWkxvv5ZL2T/hyYpWcCBPgmSae5EoVTi/4NeOR4rP+/T8gHpGqtG00X8aOvzHnoezFs=; 7:zPZV6+19Abnw3VEVfBbw7P2Oa32q7baV7UK41u+v6PM9SzQ+4P5hiSLv5wTBhs/gdq9LisPphfI2swtWFgL/D3M5nk4rcSgoaFA/d8Dfy2a2bqdBDrHxI6E3HxKh215udfd9mBMBAICujZ/+ZDbnRBcHD8E/WejOAnf/G2pooL4rUvFjHTSTmgDBdSjXkZqV9a668M09F/m285qmC2mAsb0G7bKCxyDH5RTbBMo53Iq5bTj9Bx8dWZSbs80qauJB SpamDiagnosticOutput: 1:99 SpamDiagnosticMetadata: NSPM X-OriginatorOrg: cavium.com X-MS-Exchange-CrossTenant-OriginalArrivalTime: 08 Mar 2018 06:36:48.1199 (UTC) X-MS-Exchange-CrossTenant-Network-Message-Id: 398627cb-d7e3-4021-f35e-08d584bef7f7 X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted X-MS-Exchange-CrossTenant-Id: 711e4ccf-2e9b-4bcf-a551-4094005b6194 X-MS-Exchange-Transport-CrossTenantHeadersStamped: BYAPR07MB4213 Sender: netdev-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: netdev@vger.kernel.org From: Veerasenareddy Burru In a scenario where the command queued to firmware get dropped or times out, MTU change from host will not propagate to firmware. So, it is required for host driver to wait for response from firmware or timeout and then return correct status to caller of ndo_change_mtu. Also moved the common code for MTU change from PF and VF driver files to common file lio_core.c Signed-off-by: Veerasenareddy Burru Signed-off-by: Felix Manlunas --- drivers/net/ethernet/cavium/liquidio/lio_core.c | 96 ++++++++++++++++++++-- drivers/net/ethernet/cavium/liquidio/lio_main.c | 70 ++++++---------- drivers/net/ethernet/cavium/liquidio/lio_vf_main.c | 67 +++------------ .../net/ethernet/cavium/liquidio/liquidio_common.h | 3 +- .../net/ethernet/cavium/liquidio/octeon_network.h | 20 +++++ 5 files changed, 143 insertions(+), 113 deletions(-) diff --git a/drivers/net/ethernet/cavium/liquidio/lio_core.c b/drivers/net/ethernet/cavium/liquidio/lio_core.c index 8b1ee83..ff4bfb28 100644 --- a/drivers/net/ethernet/cavium/liquidio/lio_core.c +++ b/drivers/net/ethernet/cavium/liquidio/lio_core.c @@ -164,15 +164,6 @@ void liquidio_link_ctrl_cmd_completion(void *nctrl_ptr) } break; - case OCTNET_CMD_CHANGE_MTU: - /* If command is successful, change the MTU. */ - netif_info(lio, probe, lio->netdev, "MTU Changed from %d to %d\n", - netdev->mtu, nctrl->ncmd.s.param1); - netdev->mtu = nctrl->ncmd.s.param1; - queue_delayed_work(lio->link_status_wq.wq, - &lio->link_status_wq.wk.work, 0); - break; - case OCTNET_CMD_GPIO_ACCESS: netif_info(lio, probe, lio->netdev, "LED Flashing visual identification\n"); @@ -1081,3 +1072,90 @@ int octeon_setup_interrupt(struct octeon_device *oct, u32 num_ioqs) } return 0; } + +static void liquidio_change_mtu_completion(struct octeon_device *oct, + u32 status, void *buf) +{ + struct octeon_soft_command *sc = (struct octeon_soft_command *)buf; + struct liquidio_if_cfg_context *ctx; + + ctx = (struct liquidio_if_cfg_context *)sc->ctxptr; + + if (status) { + dev_err(&oct->pci_dev->dev, "MTU change failed. Status: %llx\n", + CVM_CAST64(status)); + WRITE_ONCE(ctx->cond, LIO_CHANGE_MTU_FAIL); + } else { + WRITE_ONCE(ctx->cond, LIO_CHANGE_MTU_SUCCESS); + } + + /* This barrier is required to be sure that the response has been + * written fully before waking up the handler + */ + wmb(); + + wake_up_interruptible(&ctx->wc); +} + +/** + * \brief Net device change_mtu + * @param netdev network device + */ +int liquidio_change_mtu(struct net_device *netdev, int new_mtu) +{ + struct lio *lio = GET_LIO(netdev); + struct octeon_device *oct = lio->oct_dev; + struct liquidio_if_cfg_context *ctx; + struct octeon_soft_command *sc; + union octnet_cmd *ncmd; + int ctx_size; + int ret = 0; + + ctx_size = sizeof(struct liquidio_if_cfg_context); + sc = (struct octeon_soft_command *) + octeon_alloc_soft_command(oct, OCTNET_CMD_SIZE, 16, ctx_size); + + ncmd = (union octnet_cmd *)sc->virtdptr; + ctx = (struct liquidio_if_cfg_context *)sc->ctxptr; + + WRITE_ONCE(ctx->cond, 0); + ctx->octeon_id = lio_get_device_id(oct); + init_waitqueue_head(&ctx->wc); + + ncmd->u64 = 0; + ncmd->s.cmd = OCTNET_CMD_CHANGE_MTU; + ncmd->s.param1 = new_mtu; + + octeon_swap_8B_data((u64 *)ncmd, (OCTNET_CMD_SIZE >> 3)); + + sc->iq_no = lio->linfo.txpciq[0].s.q_no; + + octeon_prepare_soft_command(oct, sc, OPCODE_NIC, + OPCODE_NIC_CMD, 0, 0, 0); + + sc->callback = liquidio_change_mtu_completion; + sc->callback_arg = sc; + sc->wait_time = 100; + + ret = octeon_send_soft_command(oct, sc); + if (ret == IQ_SEND_FAILED) { + netif_info(lio, rx_err, lio->netdev, "Failed to change MTU\n"); + return -EINVAL; + } + /* Sleep on a wait queue till the cond flag indicates that the + * response arrived or timed-out. + */ + if (sleep_cond(&ctx->wc, &ctx->cond) == -EINTR || + ctx->cond == LIO_CHANGE_MTU_FAIL) { + octeon_free_soft_command(oct, sc); + return -EINVAL; + } + /* command is successful, change the MTU. */ + netif_info(lio, probe, lio->netdev, "MTU changed from %d to %d\n", + netdev->mtu, new_mtu); + netdev->mtu = new_mtu; + lio->mtu = new_mtu; + + octeon_free_soft_command(oct, sc); + return 0; +} diff --git a/drivers/net/ethernet/cavium/liquidio/lio_main.c b/drivers/net/ethernet/cavium/liquidio/lio_main.c index a5eecd8..140085b 100644 --- a/drivers/net/ethernet/cavium/liquidio/lio_main.c +++ b/drivers/net/ethernet/cavium/liquidio/lio_main.c @@ -91,20 +91,6 @@ static int octeon_console_debug_enabled(u32 console) */ #define LIO_SYNC_OCTEON_TIME_INTERVAL_MS 60000 -struct liquidio_if_cfg_context { - int octeon_id; - - wait_queue_head_t wc; - - int cond; -}; - -struct liquidio_if_cfg_resp { - u64 rh; - struct liquidio_if_cfg_info cfg_info; - u64 status; -}; - struct liquidio_rx_ctl_context { int octeon_id; @@ -841,8 +827,12 @@ static void octnet_link_status_change(struct work_struct *work) struct cavium_wk *wk = (struct cavium_wk *)work; struct lio *lio = (struct lio *)wk->ctxptr; + /* lio->linfo.link.s.mtu always contains max MTU of the lio interface. + * this API is invoked only when new max-MTU of the interface is + * less than current MTU. + */ rtnl_lock(); - call_netdevice_notifiers(NETDEV_CHANGEMTU, lio->netdev); + dev_set_mtu(lio->netdev, lio->linfo.link.s.mtu); rtnl_unlock(); } @@ -891,7 +881,11 @@ static inline void update_link_status(struct net_device *netdev, { struct lio *lio = GET_LIO(netdev); int changed = (lio->linfo.link.u64 != ls->u64); + int current_max_mtu = lio->linfo.link.s.mtu; + struct octeon_device *oct = lio->oct_dev; + dev_dbg(&oct->pci_dev->dev, "%s: lio->linfo.link.u64=%llx, ls->u64=%llx\n", + __func__, lio->linfo.link.u64, ls->u64); lio->linfo.link.u64 = ls->u64; if ((lio->intf_open) && (changed)) { @@ -899,12 +893,26 @@ static inline void update_link_status(struct net_device *netdev, lio->link_changes++; if (lio->linfo.link.s.link_up) { + dev_dbg(&oct->pci_dev->dev, "%s: link_up", __func__); netif_carrier_on(netdev); txqs_wake(netdev); } else { + dev_dbg(&oct->pci_dev->dev, "%s: link_off", __func__); netif_carrier_off(netdev); stop_txq(netdev); } + if (lio->linfo.link.s.mtu != current_max_mtu) { + netif_info(lio, probe, lio->netdev, "Max MTU changed from %d to %d\n", + current_max_mtu, lio->linfo.link.s.mtu); + netdev->max_mtu = lio->linfo.link.s.mtu; + } + if (lio->linfo.link.s.mtu < netdev->mtu) { + dev_warn(&oct->pci_dev->dev, + "Current MTU is higher than new max MTU; Reducing the current mtu from %d to %d\n", + netdev->mtu, lio->linfo.link.s.mtu); + queue_delayed_work(lio->link_status_wq.wq, + &lio->link_status_wq.wk.work, 0); + } } } @@ -2449,38 +2457,6 @@ static struct net_device_stats *liquidio_get_stats(struct net_device *netdev) } /** - * \brief Net device change_mtu - * @param netdev network device - */ -static int liquidio_change_mtu(struct net_device *netdev, int new_mtu) -{ - struct lio *lio = GET_LIO(netdev); - struct octeon_device *oct = lio->oct_dev; - struct octnic_ctrl_pkt nctrl; - int ret = 0; - - memset(&nctrl, 0, sizeof(struct octnic_ctrl_pkt)); - - nctrl.ncmd.u64 = 0; - nctrl.ncmd.s.cmd = OCTNET_CMD_CHANGE_MTU; - nctrl.ncmd.s.param1 = new_mtu; - nctrl.iq_no = lio->linfo.txpciq[0].s.q_no; - nctrl.wait_time = 100; - nctrl.netpndev = (u64)netdev; - nctrl.cb_fn = liquidio_link_ctrl_cmd_completion; - - ret = octnet_send_nic_ctrl_pkt(lio->oct_dev, &nctrl); - if (ret < 0) { - dev_err(&oct->pci_dev->dev, "Failed to set MTU\n"); - return -1; - } - - lio->mtu = new_mtu; - - return 0; -} - -/** * \brief Handler for SIOCSHWTSTAMP ioctl * @param netdev network device * @param ifr interface request diff --git a/drivers/net/ethernet/cavium/liquidio/lio_vf_main.c b/drivers/net/ethernet/cavium/liquidio/lio_vf_main.c index fd70a48..3342d64 100644 --- a/drivers/net/ethernet/cavium/liquidio/lio_vf_main.c +++ b/drivers/net/ethernet/cavium/liquidio/lio_vf_main.c @@ -40,20 +40,6 @@ #define DEFAULT_MSG_ENABLE (NETIF_MSG_DRV | NETIF_MSG_PROBE | NETIF_MSG_LINK) -struct liquidio_if_cfg_context { - int octeon_id; - - wait_queue_head_t wc; - - int cond; -}; - -struct liquidio_if_cfg_resp { - u64 rh; - struct liquidio_if_cfg_info cfg_info; - u64 status; -}; - struct liquidio_rx_ctl_context { int octeon_id; @@ -564,8 +550,12 @@ static void octnet_link_status_change(struct work_struct *work) struct cavium_wk *wk = (struct cavium_wk *)work; struct lio *lio = (struct lio *)wk->ctxptr; + /* lio->linfo.link.s.mtu always contains max MTU of the lio interface. + * this API is invoked only when new max-MTU of the interface is + * less than current MTU. + */ rtnl_lock(); - call_netdevice_notifiers(NETDEV_CHANGEMTU, lio->netdev); + dev_set_mtu(lio->netdev, lio->linfo.link.s.mtu); rtnl_unlock(); } @@ -613,6 +603,7 @@ static void update_link_status(struct net_device *netdev, union oct_link_status *ls) { struct lio *lio = GET_LIO(netdev); + int current_max_mtu = lio->linfo.link.s.mtu; struct octeon_device *oct = lio->oct_dev; if ((lio->intf_open) && (lio->linfo.link.u64 != ls->u64)) { @@ -629,18 +620,17 @@ static void update_link_status(struct net_device *netdev, txqs_stop(netdev); } - if (lio->linfo.link.s.mtu != netdev->max_mtu) { - dev_info(&oct->pci_dev->dev, "Max MTU Changed from %d to %d\n", - netdev->max_mtu, lio->linfo.link.s.mtu); + if (lio->linfo.link.s.mtu != current_max_mtu) { + dev_info(&oct->pci_dev->dev, + "Max MTU Changed from %d to %d\n", + current_max_mtu, lio->linfo.link.s.mtu); netdev->max_mtu = lio->linfo.link.s.mtu; } if (lio->linfo.link.s.mtu < netdev->mtu) { dev_warn(&oct->pci_dev->dev, - "PF has changed the MTU for gmx port. Reducing the mtu from %d to %d\n", + "Current MTU is higher than new max MTU; Reducing the current mtu from %d to %d\n", netdev->mtu, lio->linfo.link.s.mtu); - lio->mtu = lio->linfo.link.s.mtu; - netdev->mtu = lio->linfo.link.s.mtu; queue_delayed_work(lio->link_status_wq.wq, &lio->link_status_wq.wk.work, 0); } @@ -1538,41 +1528,6 @@ static struct net_device_stats *liquidio_get_stats(struct net_device *netdev) } /** - * \brief Net device change_mtu - * @param netdev network device - */ -static int liquidio_change_mtu(struct net_device *netdev, int new_mtu) -{ - struct octnic_ctrl_pkt nctrl; - struct octeon_device *oct; - struct lio *lio; - int ret = 0; - - lio = GET_LIO(netdev); - oct = lio->oct_dev; - - memset(&nctrl, 0, sizeof(struct octnic_ctrl_pkt)); - - nctrl.ncmd.u64 = 0; - nctrl.ncmd.s.cmd = OCTNET_CMD_CHANGE_MTU; - nctrl.ncmd.s.param1 = new_mtu; - nctrl.iq_no = lio->linfo.txpciq[0].s.q_no; - nctrl.wait_time = LIO_CMD_WAIT_TM; - nctrl.netpndev = (u64)netdev; - nctrl.cb_fn = liquidio_link_ctrl_cmd_completion; - - ret = octnet_send_nic_ctrl_pkt(lio->oct_dev, &nctrl); - if (ret < 0) { - dev_err(&oct->pci_dev->dev, "Failed to set MTU\n"); - return -EIO; - } - - lio->mtu = new_mtu; - - return 0; -} - -/** * \brief Handler for SIOCSHWTSTAMP ioctl * @param netdev network device * @param ifr interface request diff --git a/drivers/net/ethernet/cavium/liquidio/liquidio_common.h b/drivers/net/ethernet/cavium/liquidio/liquidio_common.h index 522dcc4..4f72ed1 100644 --- a/drivers/net/ethernet/cavium/liquidio/liquidio_common.h +++ b/drivers/net/ethernet/cavium/liquidio/liquidio_common.h @@ -192,7 +192,8 @@ static inline void add_sg_size(struct octeon_sg_entry *sg_entry, #define OCTNET_MAX_FRM_SIZE (16000 + OCTNET_FRM_HEADER_SIZE) -#define OCTNET_DEFAULT_FRM_SIZE (1500 + OCTNET_FRM_HEADER_SIZE) +#define OCTNET_DEFAULT_MTU (1500) +#define OCTNET_DEFAULT_FRM_SIZE (OCTNET_DEFAULT_MTU + OCTNET_FRM_HEADER_SIZE) /** NIC Commands are sent using this Octeon Input Queue */ #define OCTNET_CMD_Q 0 diff --git a/drivers/net/ethernet/cavium/liquidio/octeon_network.h b/drivers/net/ethernet/cavium/liquidio/octeon_network.h index f2d1a07..76803a5 100644 --- a/drivers/net/ethernet/cavium/liquidio/octeon_network.h +++ b/drivers/net/ethernet/cavium/liquidio/octeon_network.h @@ -35,6 +35,18 @@ #define LIO_IFSTATE_RX_TIMESTAMP_ENABLED 0x08 #define LIO_IFSTATE_RESETTING 0x10 +struct liquidio_if_cfg_context { + u32 octeon_id; + wait_queue_head_t wc; + int cond; +}; + +struct liquidio_if_cfg_resp { + u64 rh; + struct liquidio_if_cfg_info cfg_info; + u64 status; +}; + struct oct_nic_stats_resp { u64 rh; struct oct_link_stats stats; @@ -184,6 +196,14 @@ irqreturn_t liquidio_msix_intr_handler(int irq __attribute__((unused)), */ void liquidio_set_ethtool_ops(struct net_device *netdev); +/** + * \brief Net device change_mtu + * @param netdev network device + */ +int liquidio_change_mtu(struct net_device *netdev, int new_mtu); +#define LIO_CHANGE_MTU_SUCCESS 1 +#define LIO_CHANGE_MTU_FAIL 2 + #define SKB_ADJ_MASK 0x3F #define SKB_ADJ (SKB_ADJ_MASK + 1)