From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1755195AbcDAT5W (ORCPT ); Fri, 1 Apr 2016 15:57:22 -0400 Received: from casper.infradead.org ([85.118.1.10]:56082 "EHLO casper.infradead.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1754747AbcDAT5U (ORCPT ); Fri, 1 Apr 2016 15:57:20 -0400 Date: Fri, 1 Apr 2016 20:57:18 +0100 (BST) From: James Simmons To: Joe Perches cc: "Drokin, Oleg" , "" , Greg Kroah-Hartman , "" , "" Subject: Re: [lustre-devel] [RFC PATCH 0/3] staging: lustre: detypedef In-Reply-To: <1459536253.1744.26.camel@perches.com> Message-ID: References: <0586F4EF-3E2B-419A-8343-3A4A7E34628D@intel.com> <1459536253.1744.26.camel@perches.com> User-Agent: Alpine 2.20 (LFD 67 2015-01-07) MIME-Version: 1.0 Content-Type: multipart/mixed; BOUNDARY="1985284609-1875626543-1459540583=:17701" X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20160401_205718_346666_5697EB62 X-CRM114-Status: GOOD ( 24.40 ) X-Spam-Score: -1.9 (-) X-Spam-Report: SpamAssassin version 3.4.0 on casper.infradead.org summary: Content analysis details: (-1.9 points, 5.0 required) pts rule name description ---- ---------------------- -------------------------------------------------- -0.0 NO_RELAYS Informational: message was not relayed via SMTP -1.9 BAYES_00 BODY: Bayes spam probability is 0 to 1% [score: 0.0000] Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org This message is in MIME format. The first part should be readable text, while the remaining parts are likely unreadable without MIME-aware tools. --1985284609-1875626543-1459540583=:17701 Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: 8BIT > > I know James is working on cleaning up LNet, but I don't know if he has > > anything this would be conflicting at this moment or not. > > > > Thanks for the patches. I wonder if you are generating them automatically? > > Because it would be great if it also fixes the alignment issues > > It's pretty automatic. > > It's a trivial variant of the detypedef perl script I wrote awhile ago: > http://permalink.gmane.org/gmane.linux.drivers.driver-project.devel/18603 > > > I think changing the alignment issues is better done in a > separate patch. > > James isn't cc'd on these patches as he's not a listed > maintainer.  Maybe he should be added for all of it or > some part of it? Yes please add me to the maintainer list. Currently the work is divided between Oleg and I. I also actively test this client to find any regressions. I have been working on the LNet, libcfs cleanups and bug fixes. Any libcfs/LNet changes please send my way. Also now that LNet is close to being synced with our production code it means I have free cycles to find all the earlier dropped patches in the Lustre code. We have gaps in various places where patches were dropped. So I will be working on pushing those patches as well. Oleg still is the main person for the Lustre core client code. > I don't know what email address James prefers as there > are several in git log for him. > > James Simmons > James Simmons > James Simmons > "Simmons, James A." The infradead.org account is the main one. The ornl.gov is my day job account. The yahoo and gmail accounts are used, for openid, to push patches to the OpenSFS Lustre branch which is why you see those Signed-Off with various patch ports. > but maybe something like this: > --- >  MAINTAINERS | 1 + >  1 file changed, 1 insertion(+) > > diff --git a/MAINTAINERS b/MAINTAINERS > index 16ccda4..d585631 100644 > --- a/MAINTAINERS > +++ b/MAINTAINERS > @@ -10578,6 +10578,7 @@ F: drivers/staging/media/lirc/ >  STAGING - LUSTRE PARALLEL FILESYSTEM >  M: Oleg Drokin >  M: Andreas Dilger > +R: James Simmons >  L: lustre-devel@lists.lustre.org (moderated for non-subscribers) >  W: http://wiki.lustre.org/ >  S: Maintained --1985284609-1875626543-1459540583=:17701-- From mboxrd@z Thu Jan 1 00:00:00 1970 From: James Simmons Date: Fri, 1 Apr 2016 20:57:18 +0100 (BST) Subject: [lustre-devel] [RFC PATCH 0/3] staging: lustre: detypedef In-Reply-To: <1459536253.1744.26.camel@perches.com> References: <0586F4EF-3E2B-419A-8343-3A4A7E34628D@intel.com> <1459536253.1744.26.camel@perches.com> Message-ID: List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: Joe Perches Cc: "Drokin, Oleg" , "" , Greg Kroah-Hartman , "" , "" > > I know James is working on cleaning up LNet, but I don't know if he has > > anything this would be conflicting at this moment or not. > > > > Thanks for the patches. I wonder if you are generating them automatically? > > Because it would be great if it also fixes the alignment issues > > It's pretty automatic. > > It's a trivial variant of the detypedef perl script I wrote awhile ago: > http://permalink.gmane.org/gmane.linux.drivers.driver-project.devel/18603 > > > I think changing the alignment issues is better done in a > separate patch. > > James isn't cc'd on these patches as he's not a listed > maintainer. ?Maybe he should be added for all of it or > some part of it? Yes please add me to the maintainer list. Currently the work is divided between Oleg and I. I also actively test this client to find any regressions. I have been working on the LNet, libcfs cleanups and bug fixes. Any libcfs/LNet changes please send my way. Also now that LNet is close to being synced with our production code it means I have free cycles to find all the earlier dropped patches in the Lustre code. We have gaps in various places where patches were dropped. So I will be working on pushing those patches as well. Oleg still is the main person for the Lustre core client code. > I don't know what email address James prefers as there > are several in git log for him. > > James Simmons > James Simmons > James Simmons > "Simmons, James A." The infradead.org account is the main one. The ornl.gov is my day job account. The yahoo and gmail accounts are used, for openid, to push patches to the OpenSFS Lustre branch which is why you see those Signed-Off with various patch ports. > but maybe something like this: > --- > ?MAINTAINERS | 1 + > ?1 file changed, 1 insertion(+) > > diff --git a/MAINTAINERS b/MAINTAINERS > index 16ccda4..d585631 100644 > --- a/MAINTAINERS > +++ b/MAINTAINERS > @@ -10578,6 +10578,7 @@ F: drivers/staging/media/lirc/ > ?STAGING - LUSTRE PARALLEL FILESYSTEM > ?M: Oleg Drokin > ?M: Andreas Dilger > +R: James Simmons > ?L: lustre-devel at lists.lustre.org (moderated for non-subscribers) > ?W: http://wiki.lustre.org/ > ?S: Maintained