Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752985AbcLIUso (ORCPT ); Fri, 9 Dec 2016 15:48:44 -0500 Received: from mail-sn1nam02on0050.outbound.protection.outlook.com ([104.47.36.50]:41183 "EHLO NAM02-SN1-obe.outbound.protection.outlook.com" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1751214AbcLIUsn (ORCPT ); Fri, 9 Dec 2016 15:48:43 -0500 From: "Madhani, Himanshu" To: "Michael S. Tsirkin" , "qla2xxx-upstream@qlogic.com" , "jejb@linux.vnet.ibm.com" , "martin.petersen@oracle.com" , "linux-scsi@vger.kernel.org" , "linux-kernel@vger.kernel.org" , "Bart.VanAssche@sandisk.com" Subject: Re: [PATCH] scsi/qla2xxx: label endian-ness for many fields Thread-Topic: [PATCH] scsi/qla2xxx: label endian-ness for many fields Thread-Index: AQHSUl0hZs+qwq6olkWGQEPmj9nOFKD/kGcA Date: Fri, 9 Dec 2016 20:48:40 +0000 Message-ID: <0B61D7E9-8659-4FE6-ACD9-8F05B5608999@cavium.com> References: <20161209224216-mutt-send-email-mst@kernel.org> In-Reply-To: <20161209224216-mutt-send-email-mst@kernel.org> Accept-Language: en-US Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: authentication-results: spf=none (sender IP is ) smtp.mailfrom=Himanshu.Madhani@cavium.com; x-ms-exchange-messagesentrepresentingtype: 1 x-originating-ip: [173.186.134.106] x-ms-office365-filtering-correlation-id: fea234b0-814e-4c3c-9e7a-08d42074c204 x-microsoft-antispam: UriScan:;BCL:0;PCL:0;RULEID:(22001);SRVR:SN1PR0701MB1840; x-microsoft-exchange-diagnostics: 1;SN1PR0701MB1840;7:h8u4F/9DHs2cUVB5Oj3FxFjZLFq5SKosKRiWQ44EscMJBFTn2wjgvTBaDK0t+plCpAV+sxL4R0dlk1/DyqiOyKyfGW3JOtw1TfIRRgqjYcpZRK55AJ7MKZORJZGqpW7HM/YUrIvztUYDJ3WnY6+AJMQSp/peF6Tw4gL8MqS6cETcrkihyq8qWq/9o3k9wz7cVX0FaHsRHPWlWZaDNUGmF5jfsqUyKJZeNq64kAFMCg95gl4pHbKtfxN3BWb/7GH9jOxUNqCN7Y4Yk9Bo3MOdag5AM+MMCvMgjlti1hCtEcHODyf+c36JjZ9U61nhfwA7Er9AUtm1g1jWuiGAHLV+xITzPlaeFgpIUCTh9EY9euh15nyXkzynWTTC1XM82Xs7UssUwfXOSjXPBc/8OZSaq8DSzE4Hcz5ZMAsQmeCbZswfckytJcAsEciGoCgEirEcXa2VWeI38m0kL3pG0hJMNQ== 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)(3002001)(10201501046)(6041248)(20161123560025)(20161123555025)(20161123562025)(20161123558021)(20161123564025)(6072148);SRVR:SN1PR0701MB1840;BCL:0;PCL:0;RULEID:;SRVR:SN1PR0701MB1840; x-forefront-prvs: 015114592F x-forefront-antispam-report: SFV:NSPM;SFS:(10009020)(6009001)(7916002)(39410400002)(39850400002)(39450400003)(39840400002)(24454002)(199003)(189002)(377454003)(33656002)(107886002)(83716003)(3846002)(2906002)(3660700001)(8666005)(68736007)(3280700002)(102836003)(305945005)(189998001)(99286002)(77096006)(122556002)(38730400001)(6116002)(2950100002)(105586002)(106116001)(7736002)(106356001)(1691005)(92566002)(6512006)(101416001)(5001770100001)(6486002)(97736004)(76176999)(54356999)(2900100001)(5660300001)(6436002)(50986999)(36756003)(8676002)(6506006)(66066001)(81156014)(86362001)(82746002)(2201001)(229853002)(8936002)(81166006)(2501003)(7059030)(104396002);DIR:OUT;SFP:1101;SCL:1;SRVR:SN1PR0701MB1840;H:SN1PR0701MB1837.namprd07.prod.outlook.com;FPR:;SPF:None;PTR:InfoNoRecords;A:1;MX:1;LANG:en; spamdiagnosticoutput: 1:99 spamdiagnosticmetadata: NSPM Content-Type: text/plain; charset="utf-8" Content-ID: MIME-Version: 1.0 X-OriginatorOrg: cavium.com X-MS-Exchange-CrossTenant-originalarrivaltime: 09 Dec 2016 20:48:40.6687 (UTC) X-MS-Exchange-CrossTenant-fromentityheader: Hosted X-MS-Exchange-CrossTenant-id: 711e4ccf-2e9b-4bcf-a551-4094005b6194 X-MS-Exchange-Transport-CrossTenantHeadersStamped: SN1PR0701MB1840 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 base64 to 8bit by mail.home.local id uB9KmttU015664 Content-Length: 432 Lines: 18 On 12/9/16, 12:45 PM, "Michael S. Tsirkin" wrote: >This adds endian-ness labels for lots of qla structs. >Doing this cuts down number of sparse warnings from ~1700 to ~1400. >Will help find and resolve some of real issues down the road. > >Signed-off-by: Michael S. Tsirkin > >--- > >Compile-tested only. We’ll review and run regression test with this change and update. Thanks, - Himanshu