Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754044AbcDAP6c (ORCPT ); Fri, 1 Apr 2016 11:58:32 -0400 Received: from mta02.ornl.gov ([128.219.177.12]:51852 "EHLO mta02.ornl.gov" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751313AbcDAP6b convert rfc822-to-8bit (ORCPT ); Fri, 1 Apr 2016 11:58:31 -0400 X-SG: RELAYLIST X-IronPort-AV: E=Sophos;i="5.24,427,1454994000"; d="scan'208";a="101035601" From: "Simmons, James A." To: "'Joe Perches'" , "linux-kernel@vger.kernel.org" CC: Oleg Drokin , Greg Kroah-Hartman , "devel@driverdev.osuosl.org" , "lustre-devel@lists.lustre.org" Subject: RE: [lustre-devel] [RFC PATCH 0/3] staging: lustre: detypedef Thread-Topic: [lustre-devel] [RFC PATCH 0/3] staging: lustre: detypedef Thread-Index: AQHRjBbXL6kXhST1hkaZiyGv2CGxP591Qmeg Date: Fri, 1 Apr 2016 15:58:20 +0000 Message-ID: References: In-Reply-To: Accept-Language: en-US Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: x-ms-exchange-transport-fromentityheader: Hosted x-originating-ip: [128.219.12.132] Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 8BIT MIME-Version: 1.0 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1451 Lines: 28 >Question about removing lustre typedefs. > >Various bits of lustre code use a mix of struct foo and foo_t. > >When would be an appropriate time to submit patches similar to >below that individually remove various typedefs from lustre code? > >These are pretty trivial to produce and verify so there's no >particular hurry to do them now but applying them will require >resync points for active and actually useful developers. Actually could you hold off for the LNet core and LND drivers these changes. I have plans to push a few more LNet patches soon. I have been just waiting for everyone to figure out how to deal with the latest changes to the infinband layer first. So the plan is to push FMR support for the ko2iblnd driver. Also we have additional work too handle setting the size of the DMA pools for o2iblnd but that patch touches some of the core LNet code as well. Once those are landed we can look at removing most of the typedefs. When its time for the typedef to be cleaned up lets do just the structs first. There are a few typedefs like lnet_nid_t I like to keep or if it has to be changed turn it into a struct then. Things like lnet_nid_t act like a cookie handle. Now the best place to do this cleanup right now is LNet selftest. No new code is planned for landing. We have lots of typedefs to remove and I was planning to do that cleanup but if you want to do it just CC me, jsimmons@infradead.org, so I can test the changes.