Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752158AbdFLODy (ORCPT ); Mon, 12 Jun 2017 10:03:54 -0400 Received: from mail-sn1nam01on0084.outbound.protection.outlook.com ([104.47.32.84]:52484 "EHLO NAM01-SN1-obe.outbound.protection.outlook.com" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1752009AbdFLODu (ORCPT ); Mon, 12 Jun 2017 10:03:50 -0400 Authentication-Results: wanadoo.fr; dkim=none (message not signed) header.d=none;wanadoo.fr; dmarc=none action=none header.from=cavium.com; Date: Mon, 12 Jun 2017 10:03:30 -0400 (EDT) From: Chad Dupuis X-X-Sender: cdupuis@n5127mncfb42q1.qlogic.org To: Christophe JAILLET cc: QLogic-Storage-Upstream@cavium.com, jejb@linux.vnet.ibm.com, martin.petersen@oracle.com, linux-scsi@vger.kernel.org, linux-kernel@vger.kernel.org, kernel-janitors@vger.kernel.org Subject: Re: [PATCH 0/3] scsi: qedf: Fix a return value + some cleanups In-Reply-To: Message-ID: References: User-Agent: Alpine 2.00 (OSX 1167 2008-08-23) MIME-Version: 1.0 Content-Type: TEXT/PLAIN; charset=US-ASCII X-Originating-IP: [4.15.251.130] X-ClientProxiedBy: DM5PR2201CA0018.namprd22.prod.outlook.com (10.172.42.28) To BN6PR07MB2770.namprd07.prod.outlook.com (10.173.27.144) X-MS-PublicTrafficType: Email X-MS-TrafficTypeDiagnostic: BN6PR07MB2770: X-MS-Office365-Filtering-Correlation-Id: 19fda456-72e2-463a-92e0-08d4b19bd95d X-Microsoft-Antispam: UriScan:;BCL:0;PCL:0;RULEID:(22001)(201703131423075)(201703031133081);SRVR:BN6PR07MB2770; X-Microsoft-Exchange-Diagnostics: 1;BN6PR07MB2770;3:OdrIPKj8WlqdbsPht/BcoThCJwI9rTp4jF7WSNDL2RXB6lR5Z86sR/i7tuE0ifiYcP6CjxEPMW9uVgckBlKgUQNYBB34WHg/xK6glgYnrGMDUFyHjZ/OKzpRDExm0t27L/E/94vl5KscgDzyvXxRsFTKzppFchgu0ct76fF4wNWE5s6+xHgB3yr+7K5MBhgCTM/YQbPuj8swki+mCHkuuhlEYCpLGc00ysQCwpICAB16ubAG6RC1enU/2+B3gmVuokBUSUuZJBhYlb0AlI4FXIGa4URY0BJ5I8VA3oWrqpSHch10snAdHYXM0cXAYv37tNoF+sZ+RjAgNES8Q3hpOQ==;25:YfZHfisKtQnaLuDzp4BsQJaQ5ADB0H3P/Ej4H5z5/E6y1c9BF1dvpTL2JotCOIL/VGFR62EoHw4khBCuye7cbgSQixBtL+xAN4nFWW6rp6wefjpdqFhO1vczaHYsaAYBUIaDhH/dz5A+57+gPzT5FIyUQA04JNU47q9kXu+TOQufZG+0EyK1dFrV0hc5Nc3owupQGM+/zDgUDT9iOlxSAePXTV/MyF4dasppW9RkejlnvX9rltv0jhn2dBtNN/eqf+TR6OIOLuY7NazD4wKdU1ULOpDnKa5W/1VE2Pi9+ctnIq4A18S3M8NVzAItoRZQZWCAK6nGXQXHAcGeLOZ8xcWnLpVoeXaMhEdsKFzlEtFd21Y3qfZMpkaan2REpSfKOhZbGdMgMaJO3x5mo17Ym6QWWirsmkiXPMSEgyDGd9eTPlwMa05/eJh4s/CNnc6SXFThXVyJxhNn+IcXACg4WneARXPjN7P4R2VmS505cyo= X-Microsoft-Exchange-Diagnostics: 1;BN6PR07MB2770;31:W8Pp3rQ9UOtz1gpz9888M7UYjO6HTpYz376RlRt7jNMe1NZvXulbuuo3hvg5TJS8y1cNBxabmXKvz1K928vQmEAmV1XQSdWITzqIUNiF5TzvEJLVPohZgZ8jH6Fv7pyA6OQm4q1nNJgL8oXXrvBMXF0VZZYOe1d5HB+f6PveT2q+8t4fOnzjoach+i6fAE2oGLeG3F7ahwoe55nAgZ5fJMz+/5KsDXbg3XK/scwpN1I=;20:QhsEj2BWIvHueUULBmh1BvYLU9aj7XuxczGST+0NOy3LtgHa8P/3Da9dH68gNBjmC3Od0+ahaOYTbla2sPlnAu7rVNg4WZUGZRs5O0qjH3Au44OMDOk9AW+pfrE6DmkB6qn6Wq++qIqYDu3cr+g9ZDuVXAP8HRaZNOX1+arK+61AoVyzH6RvDy+eZ5auVvi/+4qR75nRA42kT7yiTBcJfVym3iJ58B0mGR9paTJcInXlvytDnQmFiuNFMWkAJgZhohz19VI81kZ1d5j9VrwIeaAku6pYB4hCW9N5N1usEL0AebFN1yqJU0xGOkXRyYbyXqzmf3UkwrC7BgRxprj4JRSwYekz9t1fIOYrhUT3eHYCHDRgXVJno+uSsqYcNxz7AxxXXT0lfbuAzremfkQ68Yy/YaXV/GV2pL5CIYwn7IY7qL3p7w5lb08VtiOA8wICuytG57bqdpzaImlmU8Uojl7XM/IVj70rk6wa/JIaU6Fr8/oxEsmyJd7NwWkHKr5n X-Microsoft-Antispam-PRVS: X-Exchange-Antispam-Report-Test: UriScan:; X-Exchange-Antispam-Report-CFA-Test: BCL:0;PCL:0;RULEID:(100000700101)(100105000095)(100000701101)(100105300095)(100000702101)(100105100095)(6040450)(601004)(2401047)(5005006)(8121501046)(93006095)(93001095)(10201501046)(100000703101)(100105400095)(3002001)(6041248)(201703131423075)(201702281528075)(201703061421075)(201703061406153)(20161123558100)(20161123564025)(20161123555025)(20161123560025)(20161123562025)(6072148)(100000704101)(100105200095)(100000705101)(100105500095);SRVR:BN6PR07MB2770;BCL:0;PCL:0;RULEID:(100000800101)(100110000095)(100000801101)(100110300095)(100000802101)(100110100095)(100000803101)(100110400095)(100000804101)(100110200095)(100000805101)(100110500095);SRVR:BN6PR07MB2770; X-Microsoft-Exchange-Diagnostics: =?us-ascii?Q?1;BN6PR07MB2770;4:13m+kgiBj8ehpE4OBU41/065WFT5b8B+PtZ/lKBsdJ?= =?us-ascii?Q?T4ygljeNOEssquNUXB/v8YN/ePejrjzcqVOByw+jcDw9WRj0UQTYUbwR46W2?= =?us-ascii?Q?Rfw0nt4bK0STGL6a3xgPAxwR0UtG6A9FpXS5jriaFwAyQexrfoiLutbcJI7u?= =?us-ascii?Q?ZOj0NSEiwffy7aDcFiOcRavK7sgJKNq3fka5kWMnRAjzhYze75cflvXlwO2K?= =?us-ascii?Q?RQxepJm/lf21+aDcxls3XyPRX1huSV+XPilXpB/m0DydsM2RkyO4g7jsO1im?= =?us-ascii?Q?pr81vTeyZzENtZGb3c96G9+uwgUMM1htma9n6lBBup8TRCw9b0c8uA4EfJu1?= =?us-ascii?Q?RzTThlItWNP/J6fi+tSaBUNxHSpPNKC3BApWL36EyhZpy8GM2nX+9ijzVfQP?= =?us-ascii?Q?5cW4J7LtoXUHLOsmeGPKyuFuKmsE2u6GIQ088zzEDVgAs/JKVe5gddORPZS1?= =?us-ascii?Q?yz/nlXc1qe3/7sKiIXBNubGfEa9+mXyZUaZL1a2jMvikwc/yK+qmQAOzzBcV?= =?us-ascii?Q?uGU8YQZCfgzRwadXZAc/U9jPyuOG/pMantbCx8e+qyrSsWfUO7w5TIDIqfyl?= =?us-ascii?Q?DGVRHnSY99qVySYNSkIg27A+e3J3ET2zwkkgTsHcyYp4opAAo4buViR4Z5wV?= =?us-ascii?Q?LTl3elzBkciTOUHHTzM2N4lQBPOuaUjAQR65az8jA2/DqLqFNLZjEhjzcWlc?= =?us-ascii?Q?0rLdHYiwweQKtofSFxjQ3fOX31cINzYTYLxYEca+muQnhHr6i3YxfIhoy71N?= =?us-ascii?Q?0KJjz+PWTNTXGnVK0jLsz6d5fs/OkqI1VKrjBOxQlDPBEH0084B51N4tbaao?= =?us-ascii?Q?aVHg3L1k6EdGwFVuhMWt3/GcV/thr7eMiSrZNt5tvcJAriUz3ZCmnABezrWf?= =?us-ascii?Q?SUaCqfL4HPayk1ylCF8mMvA/HiN31k20CsNRnj5504e8vZ/UMkR7wKvF0RjK?= =?us-ascii?Q?wg/Td9+STYjkjKhbllkD9w8x+VNLpKAP1DUMFwL0OYaOK7gDKJQ7sVUAX+9s?= =?us-ascii?Q?TbI5ytXUgduNo513qCtMQ57JBT/PkiYNGiUP1dsKim/pwK8i09aEw/B6rOBy?= =?us-ascii?Q?nhmXpQUCOI5cl8ecUBZGidC2zFilEb4/BIsowu3k9gpUhqky+l45fyMxK34B?= =?us-ascii?Q?8Z0h16LztTq1yOprrx6+HRJmzqudbC?= X-Forefront-PRVS: 03361FCC43 X-Forefront-Antispam-Report: SFV:NSPM;SFS:(10009020)(4630300001)(6009001)(39840400002)(39450400003)(39400400002)(39850400002)(39410400002)(24454002)(53936002)(7736002)(86362001)(81166006)(8676002)(6506006)(3846002)(5005980100005)(25786009)(38730400002)(189998001)(305945005)(6246003)(110136004)(6916009)(42186005)(2950100002)(50986999)(76176999)(6666003)(54356999)(478600001)(9686003)(37156001)(72206003)(53416004)(47776003)(229853002)(4326008)(23726003)(5660300001)(2906002)(55016002)(66066001);DIR:OUT;SFP:1101;SCL:1;SRVR:BN6PR07MB2770;H:n5127mncfb42q1.qlogic.org;FPR:;SPF:None;MLV:sfv;LANG:en; X-Microsoft-Exchange-Diagnostics: =?us-ascii?Q?1;BN6PR07MB2770;23:bo5UStFjpGoAUX89/e4K4/Xv9TsBLrKURSx1bMGBr?= =?us-ascii?Q?wUSUEYK7uPrUfx1O5pbSLcjucBcelScbGdcNwbKGBf8aO7seLKcFPFsT7BaT?= =?us-ascii?Q?qPYw44OUYOEzJ/AyeEWQMI/Xogu7PluoS+6MpFr3l1WPYQWwl3iY+9PsPPaI?= =?us-ascii?Q?nRqdH/JGPd0enFHKGnnV/k7uzpv6hIBq5Lw7WmS5Tm/taXC8Moq28FSbJHNd?= =?us-ascii?Q?fmFCr9lu/9EWKxttuwE9lyf8yoUChXMi0F4vxpG0RhExqaSaLp88wMFX/LfB?= =?us-ascii?Q?fCjUzZLCaY/f8D3DBU6xKDWDMmA1ihu3LK1LwFaAF6w8IZwNXaH7/KmbRtM+?= =?us-ascii?Q?chaOo2SuXYi6irru2pnWwYWIIj0G0o66lDrvtBt2v0Gjpxu92Z84J04wX68j?= =?us-ascii?Q?zMqEfNMy3az+70S7V9I3SEZyBtTGaBlWQvIJxvn3MowND0qMlfqhtpHklcAC?= =?us-ascii?Q?LwpIve+aAWoJ2hlPNvkLjptY7mFqwkotnTBuKSUheS6eFdJSPbqPULsCD1J9?= =?us-ascii?Q?9/icT41EY0nRcMAsH/EK8Z4OYtNmeb/Be/4Znp6w6nGwp6pvAWzd4idrLVmc?= =?us-ascii?Q?dkohU8+mCy6cnol6vhDxQGe0N/U1SscoMimS6z8PFE6Slm4qbEDLpfnIowu9?= =?us-ascii?Q?BaS5e6vvNPVx7svVXv7jL062SsU/a4tUilv9lhrQfq5wnROjR60CD5AJU3v9?= =?us-ascii?Q?yONv0dkIUngqbX0C14NY25Xt1W8dq02xmFEEMYo2h/refwB5kZP5i+DKaOeE?= =?us-ascii?Q?Fk9vqdt4SNRjrgtq1CkU07A0ctWxpHM/MBpgO3JXWCT2loSayscuE5QjaWVE?= =?us-ascii?Q?/5fWrFVIx4hJtMNDk4jcygLxlg9c6+AlpB0rb6BCCKCono1NioZ2eCTzu5d3?= =?us-ascii?Q?bCZPRdETsJ/Oj+MdzQnUDLZRnK2C4mlZzwPeeF9rYT1vbjvc530uW1QConoB?= =?us-ascii?Q?CGjzpW/k0QIcHgFH2+RSRlXtLPq55X49B8lBkYBgJZq4M6PvILTSc316g166?= =?us-ascii?Q?u0NtlgzyePmkbA8RG4Y2l5ncmCrNUp5i7WtDSCsixnLnKqaOPVNPoTgvJ4Q6?= =?us-ascii?Q?WK/R8Fj+AczUTYFsf/iHGPWhOKE?= X-Microsoft-Exchange-Diagnostics: 1;BN6PR07MB2770;6:3OwNYGnSDoeX6WP2WMOOnvefWdLDB+ZiCoHZmp/43TdI24YiaXTQRvya9QPU1hJILdGfwBcyxev/dlhKFNQp0ykHOhU3wPS/NnhT4NGEQEIX+vlcHl75+QSZ4QesaBsWs7CrkhA914uyirrDzwzK6WGQZqmRlGufcYZXK0UuJxhCVGPNmZYvFkC5kPJ6KB8/U92a7xH6b15yJIwP/kZ8rvWc9/Wy6ZUO71TLjvy22m2djv+4y3E28/AFVFUb3vK+DZ/+xzrrotsQGOBfZF1SH87ItirpcnscoI40SMMXGPxPWTsReYxfbx1NsmasKJQGHK3/W9AwJnmg16oYMVXw5SDV9jWN1Xwkqb5PrPM6y22EZGnThSP8Yzkw67g4VxX40/qWk8JtaokWlbWfVP/ndvbo/ZCZL40vxb96SjMbimliXZDNhW+JyFnf6777i6vCcVSGZMzCHKZBqwhza6R6oen9hrGiSPMRb+8byGF5s3bnx9u45+01xWSLybxbrESAS1XCqtI4f81Rk3oZubnQLw== X-Microsoft-Exchange-Diagnostics: 1;BN6PR07MB2770;5:fCy1F8dMm6AwwijvePjH5n/eQfDHrSdPLxJbbcx7bwjRRIHQK+7J7Z1RELd5p/mP62CjNA9cH3U+HXdRb3rDlEO5YlGEdujatkMJffwM30X95uYPHzybbygd8LNaIcM8qZoyhNbIAvpOtvuUIbqi/XNpIU0XYk7V923Rt9UIUMzwAUpYJXHXuoQa6keg1khaZjz1YgpclfvIIsY6L1AjH7A1CdQSknSDLD2nkrf+1CbBSmAMa/6G35j8DWIUxEjCWn3RrlRKlRf8jEnnWRX70UVO/rah04lF/dDOFTuiImu2SwjpMow+YIxwOm6WYDnre9I/YcljFx0lk2iHmF4cD2SF3ZTezfZzL/XIs7rSmqe+SZtMTiAT9sMSTrxsSuByTXFnaRlMKuUu26QltNp11RvUysOdMcPQeSEyw4KSzqh+MGGPvc1wGpCr8lWhiK1htb6Zhq3Ab7LNrUmE9/BUDDFQ8JqaMBbFwtEpSfpXz9gl/wn07c7h0CCHkDNtAlW7;24:Uep1qnHk6GOQiIk1yyqw2cDQ0wZXUVBSq2Zvu0rCMnbFz7DFCCmInKDUe5MoSQxLfGMtw90WJQR0nrcFeJDKTWeMYigBexRkQ6YdHjOYKis= SpamDiagnosticOutput: 1:99 SpamDiagnosticMetadata: NSPM X-Microsoft-Exchange-Diagnostics: 1;BN6PR07MB2770;7:6BDNxE4LGN3+k52Ss010BxbTlY689i10FR+lwZfWUqYQ8wFq9phj//dZ5RAg6KooQUGSBWjApzG7uRvgwEI50o/kKyLBv/N7pvemuAZ+Jh5/9+bstUVSLJh15FQRPNjEOG3c55d4K3nYNhY0M8pDCjtsbXAh/M4KLnPvrbrBsR99mti5c0biJPJQ/bqcxwUo2KofTBoVT+LBlCh+7IuStQHjS3JGniQHeBi28oGC9E/t04DknhvT+ZsUnDxrlFuAQbxtKBq0Cj9aSKWcsMDH0sN7oSPgzN0ILxzpdsxVR2Qllo1rbvWPU8OQjfCx9AOQYZhWhW1VKrjlM8S5ZI2xwg== X-OriginatorOrg: cavium.com X-MS-Exchange-CrossTenant-OriginalArrivalTime: 12 Jun 2017 14:03:47.8317 (UTC) X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted X-MS-Exchange-Transport-CrossTenantHeadersStamped: BN6PR07MB2770 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 682 Lines: 20 On Sun, 11 Jun 2017, 2:16am, Christophe JAILLET wrote: > This patch serie first fixes a case where an error code was missing. > The 2 other patches are just cleanups in the same area. > > Christophe JAILLET (3): > scsi: qedf: Fix a return value in case of error in > 'qedf_alloc_global_queues' > scsi: qedf: Use 'dma_zalloc_coherent' to reduce code verbosity. > scsi: qedf: Merge a few quoted strings split across lines > > drivers/scsi/qedf/qedf_main.c | 38 +++++++++++++------------------------- > 1 file changed, 13 insertions(+), 25 deletions(-) > > All are sensible small improvements. Ack the series. series-acked-by: Chad Dupuis