Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1755418AbdCJP35 (ORCPT ); Fri, 10 Mar 2017 10:29:57 -0500 Received: from mail-bn3nam01on0117.outbound.protection.outlook.com ([104.47.33.117]:23177 "EHLO NAM01-BN3-obe.outbound.protection.outlook.com" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1754980AbdCJP3y (ORCPT ); Fri, 10 Mar 2017 10:29:54 -0500 From: KY Srinivasan To: Greg Kroah-Hartman , Ben Hutchings CC: "linux-kernel@vger.kernel.org" , "stable@vger.kernel.org" , Long Li , "Martin K. Petersen" Subject: RE: [PATCH 4.4 34/91] scsi: storvsc: use tagged SRB requests if supported by the device Thread-Topic: [PATCH 4.4 34/91] scsi: storvsc: use tagged SRB requests if supported by the device Thread-Index: AQHSmX50B7mdoDI2O0KSg0QwBrhf/aGOKiWAgAAG24CAAAFoIA== Date: Fri, 10 Mar 2017 15:29:49 +0000 Message-ID: References: <20170310083900.730556986@linuxfoundation.org> <20170310083902.482819326@linuxfoundation.org> <1489157819.2593.9.camel@decadent.org.uk> <20170310152131.GA20522@kroah.com> In-Reply-To: <20170310152131.GA20522@kroah.com> Accept-Language: en-US Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: authentication-results: linuxfoundation.org; dkim=none (message not signed) header.d=none;linuxfoundation.org; dmarc=none action=none header.from=microsoft.com; x-originating-ip: [2601:600:8c00:1040:f0b9:ea0d:fa7d:131f] x-ms-office365-filtering-correlation-id: d9ca80c2-be7e-404c-ca12-08d467ca4a64 x-ms-office365-filtering-ht: Tenant x-microsoft-antispam: UriScan:;BCL:0;PCL:0;RULEID:(22001)(48565401081);SRVR:CY1PR03MB2235; x-microsoft-exchange-diagnostics: 1;CY1PR03MB2235;7:2XbP1LBpUFB6EYKZWgVEOqGJ9OGo5gGn0iiYrA2/o1LDTCsWP8XV7EbfzZ1Xr4ATERnNPgBmpsNZsPcWKpnaYbtyE6YwjqMpkz1Ak+c0EqiMn3XlsDvbkKizCp4zJXI40w9fZAnTffM6vjPHXa99LCZaxBmF5y3ZGrPajDRdIVDwWlnjJY8FaQk7jljXef6Xf2Cz4sI04XQ7lbgkFvBul5mm/7KTz/2IjjAkpe8eaHerYvdEp2RUzZOWgzMNBAiKT6XREepgBH4xO7asdfQCy5ZVOb2Iw2wML49W/5yMSX2gnfy+yvKg6+FZvGcLl0g8svJuVRYHBONcGlkpP/D5Bc/Si61Y2mLPOiQOyjL17f4= x-microsoft-antispam-prvs: x-exchange-antispam-report-test: UriScan:(9452136761055)(146099531331640); x-exchange-antispam-report-cfa-test: BCL:0;PCL:0;RULEID:(61425038)(6040375)(601004)(2401047)(8121501046)(5005006)(3002001)(10201501046)(6055026)(61426038)(61427038)(6041248)(20161123564025)(20161123562025)(20161123558025)(20161123560025)(20161123555025)(6072148);SRVR:CY1PR03MB2235;BCL:0;PCL:0;RULEID:;SRVR:CY1PR03MB2235; x-forefront-prvs: 02426D11FE x-forefront-antispam-report: SFV:NSPM;SFS:(10019020)(6009001)(39860400002)(39450400003)(39840400002)(39850400002)(39410400002)(13464003)(24454002)(377454003)(377424004)(2950100002)(93886004)(229853002)(2900100001)(74316002)(7736002)(2906002)(53546006)(3280700002)(9686003)(53936002)(5005710100001)(33656002)(3660700001)(7696004)(6436002)(54906002)(77096006)(305945005)(122556002)(6506006)(38730400002)(106116001)(76176999)(86362001)(8936002)(54356999)(4326008)(86612001)(50986999)(5660300001)(55016002)(10290500002)(8676002)(6246003)(81166006)(10090500001)(6116002)(102836003)(99286003)(25786008)(189998001);DIR:OUT;SFP:1102;SCL:1;SRVR:CY1PR03MB2235;H:DM5PR03MB2490.namprd03.prod.outlook.com;FPR:;SPF:None;MLV:sfv;LANG:en; spamdiagnosticoutput: 1:99 spamdiagnosticmetadata: NSPM Content-Type: text/plain; charset="iso-8859-1" MIME-Version: 1.0 X-OriginatorOrg: microsoft.com X-MS-Exchange-CrossTenant-originalarrivaltime: 10 Mar 2017 15:29:49.3537 (UTC) X-MS-Exchange-CrossTenant-fromentityheader: Hosted X-MS-Exchange-CrossTenant-id: 72f988bf-86f1-41af-91ab-2d7cd011db47 X-MS-Exchange-Transport-CrossTenantHeadersStamped: CY1PR03MB2235 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Transfer-Encoding: 8bit X-MIME-Autoconverted: from quoted-printable to 8bit by mail.home.local id v2AFU1Bw003170 Content-Length: 1388 Lines: 42 > -----Original Message----- > From: Greg Kroah-Hartman [mailto:gregkh@linuxfoundation.org] > Sent: Friday, March 10, 2017 7:22 AM > To: Ben Hutchings > Cc: linux-kernel@vger.kernel.org; stable@vger.kernel.org; Long Li > ; KY Srinivasan ; Martin K. > Petersen > Subject: Re: [PATCH 4.4 34/91] scsi: storvsc: use tagged SRB requests if > supported by the device > > On Fri, Mar 10, 2017 at 02:56:59PM +0000, Ben Hutchings wrote: > > On Fri, 2017-03-10 at 10:08 +0100, Greg Kroah-Hartman wrote: > > > 4.4-stable review patch.??If anyone has any objections, please let me > know. > > > > > > ------------------ > > > > > > From: Long Li > > > > > > commit 3cd6d3d9b1abab8dcdf0800224ce26daac24eea2 upstream. > > > > > > Properly set SRB flags when hosting device supports tagged queuing. > > > This patch improves the performance on Fiber Channel disks. > > [...] > > > > This looks more like a feature/optimisation than a bug fix. Is this > > really within the stable rules? > > Making things work properly (i.e. i/o does not suck anymore), is ok for > stable stuff. KY, that's the case here, right? If I recall correctly, this was needed for Fibre Channel SANs where without this fix the performance was so bad to be almost unusable. K. Y > > thanks, > > greg k-h