From patchwork Thu Jan 12 01:40:27 2017 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: "Manlunas, Felix" X-Patchwork-Id: 714055 X-Patchwork-Delegate: davem@davemloft.net Return-Path: 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 3tzT4y0JPkz9t15 for ; Thu, 12 Jan 2017 12:41:37 +1100 (AEDT) Authentication-Results: ozlabs.org; dkim=pass (1024-bit key; unprotected) header.d=CAVIUMNETWORKS.onmicrosoft.com header.i=@CAVIUMNETWORKS.onmicrosoft.com header.b="LQHIUIAE"; dkim-atps=neutral Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1762786AbdALBlb (ORCPT ); Wed, 11 Jan 2017 20:41:31 -0500 Received: from mail-cys01nam02on0069.outbound.protection.outlook.com ([104.47.37.69]:58507 "EHLO NAM02-CY1-obe.outbound.protection.outlook.com" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1752912AbdALBla (ORCPT ); Wed, 11 Jan 2017 20:41:30 -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=ifYNgZfFjpTYuI6EkxzXV4SgG0axRC2J5NDvb4EFa7I=; b=LQHIUIAEYZbYN8qxYXgLJ4yb39T6rG8wrdEbs//k0YgigLzbCQKn7KISuvKUjzixrAwc0XzfPHqFgqx8DAIyu3gAQya3maYlKVb9Hzy2vRYUXM1v5TPgYU75J+oe0dr3fmOqFo30T0/nkErcukjbVtmeviWko8FIGeeTa9VM5Mo= Authentication-Results: spf=none (sender IP is ) smtp.mailfrom=Felix.Manlunas@cavium.com; Received: from localhost (73.71.186.126) by DM5PR07MB3178.namprd07.prod.outlook.com (10.172.85.140) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_CBC_SHA384_P384) id 15.1.845.12; Thu, 12 Jan 2017 01:41:27 +0000 Date: Wed, 11 Jan 2017 17:40:27 -0800 From: Felix Manlunas To: CC: , , , Subject: [PATCH net-next] liquidio VF: reduce load time of module Message-ID: <20170112014027.GA9704@felix.cavium.com> MIME-Version: 1.0 Content-Disposition: inline User-Agent: Mutt/1.6.1 (2016-04-27) X-Originating-IP: [73.71.186.126] X-ClientProxiedBy: BY2PR1001CA0066.namprd10.prod.outlook.com (10.164.163.34) To DM5PR07MB3178.namprd07.prod.outlook.com (10.172.85.140) X-MS-Office365-Filtering-Correlation-Id: 85dce58a-6b30-47ba-1272-08d43a8c2016 X-Microsoft-Antispam: UriScan:; BCL:0; PCL:0; RULEID:(22001); SRVR:DM5PR07MB3178; X-Microsoft-Exchange-Diagnostics: 1; DM5PR07MB3178; 3:HLvAWIYM1xcupTqiZyz/83hFZpXQVKjxfyut1xgjc75jvXGsnNl3+zC7abo4YICeq0453UQCyA6O0k5ja4f4dRymmdetNk/qo6cMxI/ylE4rAZvSAp16O2qv2abXR1tbG+R41hT5uGOhnB3gL8o6cLihPAH+AdfjSpS2bGZpDT1y+xbuBaHHlYU8Akxjk8APXp8yFuJrc4pJ6wsDXd95I3PWb+30OhJC2juspzyXh57ZwDCLac8YI8uw+jhtm6dE76P1iSAOha/qhYizLFyV4w==; 25:Hpd1n2mk6AvanRR7UaumaOBIjZX6/mJzN4I170bi9tBi3DF9cczxP9sKFRzZITYvCr7SY4w9mglXogmZzPeA7D975tArAhorYfHJwth8v7jXWSg8MM37JynUr21ODKkofcGA9qnmwzh8N3J5cvhRFuw52IN9M9y3ibgC4vKzCtlY5p1o71fHvalv3quig6Nzn2t5dho0eypv5rL3nwR8ITlyDrMedMSmU8P14BYXKnXFC2JYDlIlah07HsfzAEKcNFQNhr0AznuQsoH/Kbh/w6csvzVzqzeiuPeFtEtOwdi/ynccNvYuultibwqLImDH0YbXJzucYKyQooxxI3KI4jHLnLfgAg9hT+Xe/+8dZcLeZUzHq4/uRGF03BDocj3Ro2ovNq+F60uLf355d4wBpfybkK0fcdmkXllm0i/uzQ4EEHUpnKz3cs+Eo7eM8jS94Q0QvsPSBf1HSfligJk2NA== X-Microsoft-Exchange-Diagnostics: 1; DM5PR07MB3178; 31:O0dpLlbuHWoHGQri18blWkP2r2splVvcUFOnVzTDlht5ONNOj3L+o86BJXEcFBWjj7/nVtgJzgz26Cx1RYfubBQ9Vf0WW027ujeMeKUZ+A29JHYvYl85RqvYMdKNNiPtqH7xpyUXMGY7igFrJt7yCZWCUNZgm/6bwBa/wrnlV8Ggu3xp1WpVojy/SalhiZNk3ICMZZIfXOBIs6qcxKb0gX5QZzpMhMFw9eTNnJbKMu19y7DEhAmG7EHV7VwsfWP+eAlFjwcFDvImUAKlQUAHCQ==; 20:wPHJnSoUIB4sNrIS509E7aSmlbZaeXB4BSaPs9ayEMZyWL9/smco0sccHCaz+BQbe9ro9i5OL+jK7ZLb4MBK7H+hdmzyfUZ0UfDAT8gs1gEpYU1/TCgp3DfmxV2WPgdRIZldkBkUK39OXYyNmBFd8X3XfRzM1W7J3aDuRYKVT+OB+BtgC+bDGVu11v9CcRc6YBJTe1qGtra0E2a82MjZS6LVyGMZmdzBeMFAorSUvPctOa/DYSXnq2J/KDSJeBJ1LtYmuDA9qz2Ls9/W67A4vW5X4aIalGck5+2XAyD1VDqEUgMJUb9QheQ1y+pFuIZJkk91N6ES5Ld7NDUYq53I2dg1n3AORdtJIc9gVEW5ZKrs2c+P5znyx+mbOxs0zj0BataoK5C71x6XN2Is3D1DZPIgc/AeSgYJhZ5BTSpLjM6xm2xYFnUMN1/u8fXICO9exfH+7kyS1k7q7PpOT40n85n0IJnVP5/yS2aHI1Zxet2zmiOYixakCWvHeuO5JSQN X-Microsoft-Antispam-PRVS: 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)(20161123564025)(20161123562025)(20161123555025)(20161123560025)(6072148); SRVR:DM5PR07MB3178; BCL:0; PCL:0; RULEID:; SRVR:DM5PR07MB3178; X-Microsoft-Exchange-Diagnostics: 1; DM5PR07MB3178; 4:DOGGHwM5DlLiThCgsVnITsSrcOvOFGYf7uqInf4xC9aOPbwc/RZfyJpyegRKWCpidaIDRM2ZRGW97hsdD412kG8vmQ7Fzi0C6jOAp5kHFlV/f6ctbCEI6D/uz1zqpYR7R3v155IF4j9q5vba6lmwsnzfZYzzhWUhlNMjltEhSrZOIYkuk0R5uPk4XqysVAqcqAtcwRisUZfFrFyOMgnqIoM7L0aYb/WihRyutYNb+baGfIiIXIQPq4V/7b7s8EPfhvMUZat3Ne9qZv/F/Ct6qgpdI7dkStQ5BASsJWKUm4UxQiNSBhkllhuxHzl7TO6ftWQeU6sbZYwzMcme1tz7o3HH728Xs/5cjgePT2KHak17QyA/G1PYWHk+EMsgZqkL4JKhkMjwXvCG//z5D1iCP+4e9RUc/B56jqrviF0NxYaJXgBMuW0OQMiDC/quYna0P+dV051KdsIldyD/J0XipBwTtn8mZEMc5Y6Y9KU8D/Y/Ge4GzhEqtBMoFckH3rP5p+Uo7fqmQJEOTLbjfVf7BeaJL9Ct9ZRuzGW6Ui6bkRNwWN4KCcponSd3NFoxlAYAyfb47ZU1ZpRaGINq19hNtw== X-Forefront-PRVS: 018577E36E X-Forefront-Antispam-Report: SFV:NSPM; SFS:(10009020)(4630300001)(6069001)(6009001)(7916002)(39450400003)(189002)(199003)(23726003)(6916009)(92566002)(1076002)(97756001)(110136003)(33656002)(46406003)(47776003)(5660300001)(86362001)(4326007)(81166006)(189998001)(107886002)(81156014)(8676002)(6116002)(3846002)(7736002)(4001430100002)(305945005)(66066001)(54356999)(101416001)(68736007)(97736004)(6486002)(25786008)(6496003)(4001350100001)(106356001)(50466002)(2906002)(76506005)(42186005)(2351001)(105586002)(54906002)(83506001)(38730400001)(50986999)(18370500001); DIR:OUT; SFP:1101; SCL:1; SRVR:DM5PR07MB3178; H:localhost; FPR:; SPF:None; PTR:InfoNoRecords; MX:1; A: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; DM5PR07MB3178; 23:Pno8VeAJpn9MgZXVphepyHdPBI+ImST5aS878Uui2?= =?us-ascii?Q?bKW8NBS37id3vhcFdYbE/421AYjIlylvzJ7PwWY7ONMUagAg/0+hIkOqnTyd?= =?us-ascii?Q?1F0ygJdOb69NBAcn4l7sL4eHLSVEDWOW/zFBgfLzt/C1fn3UkUSCnkp7lg4A?= =?us-ascii?Q?5tVYnv3723R9v5DDqUYuQgABsaB9s6Ymw5UsAWYTqfAhD41U6ahOb0XNW8b0?= =?us-ascii?Q?Cpj6GAna3IB2jzFtVehooaCBZ/6KojsK6kkRXRHN8stgF9mdlJihRPYLqfnM?= =?us-ascii?Q?DbhGyxrKpUQZUk4WR1WHaCT6KI9cZ1wjxJpyuSpGAofuFg6JqmTTlQ6j6vo2?= =?us-ascii?Q?IZnljHi8M4ZlkOKxp6LYh3IvCuZKyS7agL85rA43AaTGyOo+mgu4IF1CLnjE?= =?us-ascii?Q?0iJrLZExicIOifICdPZq9fe/O70q1rYZ45v/lG5zw8rz8HluC9MCobxnGFcq?= =?us-ascii?Q?Hyxqio+jwf75jRZNMb0njCcyXPLRnTSdeE8xSOjLvNwcvNcK3cVfGC67mDgc?= =?us-ascii?Q?Xp0Z44IxbROP1rz1Cn31VXR5S422WxbxFfCAH01WxHa9HUlTYpM3O2WAfX9T?= =?us-ascii?Q?tOHa+6x/EpZUFpo7SGNEe/8TKlXzCveAgqmR/5AubQ/4Nee6dDYmReQzbjdG?= =?us-ascii?Q?Ob1jsMUvx6DtpgN6m6A/Kym7dPMs3hKKsGt0t1Lp+unmPqX4eFEaXSlrqpra?= =?us-ascii?Q?9InIa5b5nnaa1xVtTC/BVCSo9l0qcIEt24fPS2glQqjhzVWh5YvlHEsMg+lE?= =?us-ascii?Q?C9IRVLgYwEwH52FZALzoaRc6o1kOeGPx2A3PgpRXvSmr7c2GXFD7BBgjeHyO?= =?us-ascii?Q?If2NamO20ngt/h7cjLfU481IPwL5/pWmfqNtOikytCpz1esKrSF095zsIEEr?= =?us-ascii?Q?A2Fo+jscT+szDDjLXWY1yFQQY2IwmAsJPGq4bY9RG2jSElTdb3ZWk8bb7mZR?= =?us-ascii?Q?gonMkhV/4lgRaMqIwmwF6qIKqCDru0hR/S1f85ueMqYMoIwsiBcy8WYQsW3f?= =?us-ascii?Q?Ydwfbcbk/suyjRBlcpXbdUq0idTtoIpgrseEd1VffIo00644pm2heGq585Dd?= =?us-ascii?Q?+v36+Q47LIa9DyiNPaqGNSxGQ9jNTimXS8jZ8a0oPDrnVMhYQLBHtz78EVKn?= =?us-ascii?Q?C1V6AOkayElSy24nzMCzo84DwbGHrC8LQljK1NSbUysrxeNbFB/gWQhLb5Mq?= =?us-ascii?Q?4/XFFdSKlByTRwS/oSzFofBTmm0SluWmbN8?= X-Microsoft-Exchange-Diagnostics: 1; DM5PR07MB3178; 6:fbgqc3rdSOMpPF/APXBhAqwFm9Fgtf6wgX6tGWQfGTKFNH15k+9joKmsd491F/D4rxWDGuDZ4TRwow66evr8N9rlceCrpkYLNyn2rTmdG9yRtk/sNPCPZ/y8MRi8mzIQCL83EN5+EMtnD5EQMAJO2ViYt9g7ZwwGORXfxhvngNC4TagLXIkFodzTpfNs4Dy4HydlVYuKPTpuqWgODIwHslknIszN+QKK0T8TM9JlJcn1wye3tTS6vQXUIJx2LcxoxKR14GqbQWM37O0Jl+TgP8Mguwmih7HEAonQn8QNAvxleEpy/cr75y1zRHeouM9gsn6X82TpFdDQmxHVfV/XAdimClDlzhjRQPwEkiFSR+LLw2cHIBNEHW+36uF4FLmIQbSItHfqDReZxofP1uW1YiD1GdpuUZ9R6TVtLJzWU6Y=; 5:Jxq6Ciz+QyNvw7MUJYRTTlQjn77Je5vmOpaGwjWBy+tFr/5KoBhueTbAijAum6hhXZtqhdt3CGrxO+tOEcnY8SoW/1y/JSIUz8FqoXnG/tQ/hCyu21xLSrqJ7saRCx2yhg0JHH9SIkkSk/KwQlo+2M0jGzpGRBs3QvHbqiX/Sbo=; 24:OgCdVz3qv9mG86dChI+eUQFL8FudiVynUec96F76d+mUUw7a9+FPNCoZXhi7BoLLhzxglQfNPraVfjmbweVMlvLyVasTJxoMZsDyW8yMDVE= SpamDiagnosticOutput: 1:99 SpamDiagnosticMetadata: NSPM X-Microsoft-Exchange-Diagnostics: 1; DM5PR07MB3178; 7:kL3S9ASrvlbOcQVeT819MZTng2Yrta2OQEjjEAZNbO6+MwOZCrES/XxrFk138l1RcaPbVzkdsXpHdfmGc4aqFWA+CHiAcyrv9wNrATOrfZV7xkgcrFm2amwg+/qZbQ9cAWCdF4LvA2jyfRB/+6/9fpAdXxR7jvQm+oftPhSnFovXlq+gD5td8svOIPlodKsInV3rM8qtMgM0hByP53GJVgCZq3YpoP8TVVAT3SbW3C0ZDwUbaho291HPePkofiHlOW2arrrB8XRgOmSPdNNQaBoxtQMGAb0EvFMDDKp3hKX/XTBGb1R39SDLQYKOY4dkHkWX1OmlDx0ocWjWQmRKVB0ex3hmjjEM7brs84NUKL5HTtjxHXyFPQgGfR8MbCnV1kRZrn0AltmoF/P0f/1wTFpRPzDJMStHdvoiROZYRKiObC2ljelKLm/QK9gBXeeS1cGu4q6W1oroG2nciMnOrA== X-OriginatorOrg: cavium.com X-MS-Exchange-CrossTenant-OriginalArrivalTime: 12 Jan 2017 01:41:27.2641 (UTC) X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted X-MS-Exchange-Transport-CrossTenantHeadersStamped: DM5PR07MB3178 Sender: netdev-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: netdev@vger.kernel.org From: Prasad Kanneganti Reduce the load time of the VF driver by decreasing the wait time between iterations of the loop that polls for a mailbox response from the PF. Also change the wait time units from jiffies to milliseconds. Signed-off-by: Prasad Kanneganti Signed-off-by: Felix Manlunas Signed-off-by: Raghu Vatsavayi Signed-off-by: Derek Chickles Signed-off-by: Satanand Burla --- drivers/net/ethernet/cavium/liquidio/octeon_mailbox.c | 5 +++-- drivers/net/ethernet/cavium/liquidio/octeon_mailbox.h | 4 ++-- 2 files changed, 5 insertions(+), 4 deletions(-) diff --git a/drivers/net/ethernet/cavium/liquidio/octeon_mailbox.c b/drivers/net/ethernet/cavium/liquidio/octeon_mailbox.c index 73696b42..201b987 100644 --- a/drivers/net/ethernet/cavium/liquidio/octeon_mailbox.c +++ b/drivers/net/ethernet/cavium/liquidio/octeon_mailbox.c @@ -131,6 +131,7 @@ int octeon_mbox_write(struct octeon_device *oct, { struct octeon_mbox *mbox = oct->mbox[mbox_cmd->q_no]; u32 count, i, ret = OCTEON_MBOX_STATUS_SUCCESS; + long timeout = LIO_MBOX_WRITE_WAIT_TIME; unsigned long flags; spin_lock_irqsave(&mbox->lock, flags); @@ -158,7 +159,7 @@ int octeon_mbox_write(struct octeon_device *oct, count = 0; while (readq(mbox->mbox_write_reg) != OCTEON_PFVFSIG) { - schedule_timeout_uninterruptible(LIO_MBOX_WRITE_WAIT_TIME); + schedule_timeout_uninterruptible(timeout); if (count++ == LIO_MBOX_WRITE_WAIT_CNT) { ret = OCTEON_MBOX_STATUS_FAILED; break; @@ -171,7 +172,7 @@ int octeon_mbox_write(struct octeon_device *oct, count = 0; while (readq(mbox->mbox_write_reg) != OCTEON_PFVFACK) { - schedule_timeout_uninterruptible(10); + schedule_timeout_uninterruptible(timeout); if (count++ == LIO_MBOX_WRITE_WAIT_CNT) { ret = OCTEON_MBOX_STATUS_FAILED; break; diff --git a/drivers/net/ethernet/cavium/liquidio/octeon_mailbox.h b/drivers/net/ethernet/cavium/liquidio/octeon_mailbox.h index fe60a3e..c9376fe 100644 --- a/drivers/net/ethernet/cavium/liquidio/octeon_mailbox.h +++ b/drivers/net/ethernet/cavium/liquidio/octeon_mailbox.h @@ -31,8 +31,8 @@ #define OCTEON_PFVFSIG 0x1122334455667788 #define OCTEON_PFVFERR 0xDEADDEADDEADDEAD -#define LIO_MBOX_WRITE_WAIT_CNT 1000 -#define LIO_MBOX_WRITE_WAIT_TIME 10 +#define LIO_MBOX_WRITE_WAIT_CNT 1000 +#define LIO_MBOX_WRITE_WAIT_TIME msecs_to_jiffies(1) enum octeon_mbox_cmd_status { OCTEON_MBOX_STATUS_SUCCESS = 0,