From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1755751AbcDANCp (ORCPT ); Fri, 1 Apr 2016 09:02:45 -0400 Received: from smtprelay0139.hostedemail.com ([216.40.44.139]:56972 "EHLO smtprelay.hostedemail.com" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1754066AbcDANCo (ORCPT ); Fri, 1 Apr 2016 09:02:44 -0400 X-Session-Marker: 6A6F6540706572636865732E636F6D X-Spam-Summary: 2,0,0,,d41d8cd98f00b204,joe@perches.com,:::::::::::,RULES_HIT:41:69:355:379:541:968:973:982:988:989:1260:1345:1437:1534:1541:1711:1730:1747:1777:1792:2393:2559:2562:2693:2915:3138:3139:3140:3141:3142:3352:3866:3867:3868:3874:4605:5007:6120:6261:10004:10848:11026:11657:11658:11914:12043:12295:12517:12519:12555:12679:13069:13161:13229:13311:13357:14096:14721:21080:30012:30041:30054:30064:30069:30070,0,RBL:none,CacheIP:none,Bayesian:0.5,0.5,0.5,Netcheck:none,DomainCache:0,MSF:not bulk,SPF:fn,MSBL:0,DNSBL:none,Custom_rules:0:0:0,LFtime:5,LUA_SUMMARY:none X-HE-Tag: smile84_6ede55c2ff71d X-Filterd-Recvd-Size: 2400 From: Joe Perches To: linux-kernel@vger.kernel.org Cc: Oleg Drokin , Andreas Dilger , Greg Kroah-Hartman , lustre-devel@lists.lustre.org, devel@driverdev.osuosl.org Subject: [RFC PATCH 0/3] staging: lustre: detypedef Date: Fri, 1 Apr 2016 06:02:08 -0700 Message-Id: X-Mailer: git-send-email 2.8.0.rc4.16.g56331f8 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org 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. A few scripted examples are done: Joe Perches (3): staging: lustre: Remove typedef lnet_peer_t staging: lustre: Remove typedef lnd_t staging: lustre: Remove typedef lnet_msg_t .../staging/lustre/include/linux/lnet/lib-lnet.h | 62 +++++++-------- .../staging/lustre/include/linux/lnet/lib-types.h | 22 +++--- .../staging/lustre/lnet/klnds/o2iblnd/o2iblnd.c | 4 +- .../staging/lustre/lnet/klnds/o2iblnd/o2iblnd.h | 6 +- .../staging/lustre/lnet/klnds/o2iblnd/o2iblnd_cb.c | 8 +- .../staging/lustre/lnet/klnds/socklnd/socklnd.c | 2 +- .../staging/lustre/lnet/klnds/socklnd/socklnd.h | 6 +- .../staging/lustre/lnet/klnds/socklnd/socklnd_cb.c | 6 +- drivers/staging/lustre/lnet/lnet/api-ni.c | 16 ++-- drivers/staging/lustre/lnet/lnet/lib-move.c | 92 +++++++++++----------- drivers/staging/lustre/lnet/lnet/lib-msg.c | 24 +++--- drivers/staging/lustre/lnet/lnet/lib-ptl.c | 4 +- drivers/staging/lustre/lnet/lnet/lo.c | 8 +- drivers/staging/lustre/lnet/lnet/peer.c | 30 +++---- drivers/staging/lustre/lnet/lnet/router.c | 28 +++---- drivers/staging/lustre/lnet/lnet/router_proc.c | 4 +- 16 files changed, 161 insertions(+), 161 deletions(-) -- 2.8.0.rc4.16.g56331f8 From mboxrd@z Thu Jan 1 00:00:00 1970 From: Joe Perches Date: Fri, 1 Apr 2016 06:02:08 -0700 Subject: [lustre-devel] [RFC PATCH 0/3] staging: lustre: detypedef Message-ID: List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: linux-kernel@vger.kernel.org Cc: Oleg Drokin , Andreas Dilger , Greg Kroah-Hartman , lustre-devel@lists.lustre.org, devel@driverdev.osuosl.org 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. A few scripted examples are done: Joe Perches (3): staging: lustre: Remove typedef lnet_peer_t staging: lustre: Remove typedef lnd_t staging: lustre: Remove typedef lnet_msg_t .../staging/lustre/include/linux/lnet/lib-lnet.h | 62 +++++++-------- .../staging/lustre/include/linux/lnet/lib-types.h | 22 +++--- .../staging/lustre/lnet/klnds/o2iblnd/o2iblnd.c | 4 +- .../staging/lustre/lnet/klnds/o2iblnd/o2iblnd.h | 6 +- .../staging/lustre/lnet/klnds/o2iblnd/o2iblnd_cb.c | 8 +- .../staging/lustre/lnet/klnds/socklnd/socklnd.c | 2 +- .../staging/lustre/lnet/klnds/socklnd/socklnd.h | 6 +- .../staging/lustre/lnet/klnds/socklnd/socklnd_cb.c | 6 +- drivers/staging/lustre/lnet/lnet/api-ni.c | 16 ++-- drivers/staging/lustre/lnet/lnet/lib-move.c | 92 +++++++++++----------- drivers/staging/lustre/lnet/lnet/lib-msg.c | 24 +++--- drivers/staging/lustre/lnet/lnet/lib-ptl.c | 4 +- drivers/staging/lustre/lnet/lnet/lo.c | 8 +- drivers/staging/lustre/lnet/lnet/peer.c | 30 +++---- drivers/staging/lustre/lnet/lnet/router.c | 28 +++---- drivers/staging/lustre/lnet/lnet/router_proc.c | 4 +- 16 files changed, 161 insertions(+), 161 deletions(-) -- 2.8.0.rc4.16.g56331f8