Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753019AbcLLOwI (ORCPT ); Mon, 12 Dec 2016 09:52:08 -0500 Received: from esa1.cray.iphmx.com ([68.232.142.33]:11567 "EHLO esa1.cray.iphmx.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752389AbcLLOwG (ORCPT ); Mon, 12 Dec 2016 09:52:06 -0500 X-Greylist: delayed 572 seconds by postgrey-1.27 at vger.kernel.org; Mon, 12 Dec 2016 09:52:06 EST X-IronPort-AV: E=Sophos;i="5.33,336,1477958400"; d="scan'208";a="10350299" X-Cray-OBMMKR: 1433258124 10350299 From: Ben Evans To: Greg Kroah-Hartman , James Simmons CC: "devel@driverdev.osuosl.org" , Andreas Dilger , Oleg Drokin , Ben Evans , Linux Kernel Mailing List , Lustre Development List Subject: Re: [PATCH v2 3/5] staging: lustre: headers: sort headers affected by obdo move Thread-Topic: [PATCH v2 3/5] staging: lustre: headers: sort headers affected by obdo move Thread-Index: AQHSUxAYsvZabz3ew0qX2fCwoXLUJ6EBfGOAgAKVlYA= Date: Mon, 12 Dec 2016 14:42:29 +0000 Message-ID: References: <1481393161-22623-1-git-send-email-jsimmons@infradead.org> <1481393161-22623-4-git-send-email-jsimmons@infradead.org> <20161210181433.GA8477@kroah.com> In-Reply-To: <20161210181433.GA8477@kroah.com> 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=bevans@cray.com; x-ms-exchange-messagesentrepresentingtype: 1 x-originating-ip: [73.167.225.12] x-microsoft-exchange-diagnostics: 1;MWHPR11MB1774;7:oGufhbA0sDn0Lthipa+iDVdozBqxEkDQmP8f2lJjXaSY7fxb/BVvP5lLky86BTuUh8KpC1D23hkrwHhA097tEPqxYE9dUz4nvzxDym1yWR61PQQaWjDbg3BjyIXab8//PMfsFF18iSKn3yISiUjPQXyKUzdNLG2+aKvgDPmlVdwLu7FMPP5scto4DANntjCRk4Xdv5uREpLziV0OLxH/Uiu3I1NULZLTFQKUI3uU/ZydMmheRKXFTJvpYcwVyzJVaUEYusuuFsnYlhYTGgIkUvk72Ea9G1kkebJ8NgSmkZ0GOYjgOYVhIJ8ReAIJsGPF3wbSZyzfDQBHu0nC2mxt57QeaYQtFTv3/0m9V3lrP79SizDz7UG7+J2sZFUBYROqwK+eym5OM4uinshbAU2He1HBn2tn36Mh0pMKQTWPpUXEOum7Jtym2NGXkZbplwmmMVgwes8WOpFKHdEdh6qyoQ== x-forefront-antispam-report: SFV:SKI;SCL:-1SFV:NSPM;SFS:(10019020)(6009001)(7916002)(39450400003)(377454003)(24454002)(199003)(189002)(229853002)(5660300001)(86362001)(101416001)(3846002)(122556002)(6436002)(50986999)(102836003)(5001770100001)(38730400001)(77096006)(76176999)(189998001)(6116002)(6486002)(66066001)(54356999)(68736007)(2900100001)(2906002)(6506006)(6512006)(305945005)(99286002)(2950100002)(97736004)(7736002)(92566002)(106116001)(106356001)(3660700001)(8936002)(3280700002)(8676002)(4326007)(105586002)(81166006)(81156014)(36756003)(13693001)(94096001);DIR:OUT;SFP:1102;SCL:1;SRVR:MWHPR11MB1774;H:MWHPR11MB1774.namprd11.prod.outlook.com;FPR:;SPF:None;PTR:InfoNoRecords;A:1;MX:1;LANG:en; x-ms-office365-filtering-correlation-id: 204ca21d-e470-44b7-4805-08d4229d19cd x-microsoft-antispam: UriScan:;BCL:0;PCL:0;RULEID:(22001);SRVR:MWHPR11MB1774; x-microsoft-antispam-prvs: x-exchange-antispam-report-test: UriScan:; x-exchange-antispam-report-cfa-test: BCL:0;PCL:0;RULEID:(6040375)(601004)(2401047)(8121501046)(5005006)(10201501046)(3002001)(6041248)(20161123564025)(20161123555025)(20161123558021)(20161123560025)(20161123562025)(6072148);SRVR:MWHPR11MB1774;BCL:0;PCL:0;RULEID:;SRVR:MWHPR11MB1774; x-forefront-prvs: 0154C61618 spamdiagnosticoutput: 1:99 spamdiagnosticmetadata: NSPM Content-Type: text/plain; charset="us-ascii" Content-ID: <63CBFD026A11EF4CA77D1558DA9038B6@namprd11.prod.outlook.com> MIME-Version: 1.0 X-MS-Exchange-CrossTenant-originalarrivaltime: 12 Dec 2016 14:42:29.8940 (UTC) X-MS-Exchange-CrossTenant-fromentityheader: Hosted X-MS-Exchange-CrossTenant-id: e7b8488a-c0cd-4614-aae1-996bfabec247 X-MS-Exchange-Transport-CrossTenantHeadersStamped: MWHPR11MB1774 X-OriginatorOrg: cray.com 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 uBCEqDPY020458 Content-Length: 739 Lines: 24 This was done to conform to the Lustre Coding Guidelines. -Ben On 12/10/16, 1:14 PM, "Greg Kroah-Hartman" wrote: >On Sat, Dec 10, 2016 at 01:05:59PM -0500, James Simmons wrote: >> From: Ben Evans >> >> It was found if you sort the headers alphabetically >> that it reduced patch conflicts. This patch sorts >> the headers alphabetically and also place linux >> header first, then uapi header and finally the >> lustre kernel headers. > >I still don't agree, when did you last have a patch conflict with this >file in the .h section? And exactly how hard was it to fix it up? > >I'm all for cleanups, but really, this is useless. And I said so the >last time you sent it... > >greg k-h