public inbox for linux-xfs@vger.kernel.org
 help / color / mirror / Atom feed
From: Jeff Liu <jeff.liu@oracle.com>
To: Ric Wheeler <rwheeler@redhat.com>
Cc: Ben Myers <bpm@sgi.com>,
	elder@kernel.org, Linus Torvalds <torvalds@linux-foundation.org>,
	Mark Tinguely <tinguely@sgi.com>,
	xfs@oss.sgi.com
Subject: Re: xfs: update maintainers file
Date: Fri, 08 Nov 2013 13:24:36 +0800	[thread overview]
Message-ID: <527C7594.1020504@oracle.com> (raw)
In-Reply-To: <527C4B27.6020205@redhat.com>

Hi folks,

I think I'm not the right person to reply this discussion because am still on the road
of how to cooperate with others in open source community, as well as I have not made
much contribution to XFS until now.

However, I'd to show some humble opinions from a user and a trivial developer's point
of view.

First all all, that's fine if we have a co-maintainer who can take the role when Ben
is on leave or busy working on other things, and also, this role may be able to supply
more help to some new comers for patch review, coaching, etc... I felt Mark has already
paid a lot of effort and performed very well in this role.  At least, I got several
offline emails form Mark with the patch review status and comments, which are all helpful
to me. 

Secondly, I want to show respect and admiration to our core developers who are devote
themselves to promote XFS in the past years.  Without those talent guys, I think XFS can
not move ahead rapidly with so much significant performance improvements and new features.
However, how can we credit them for their prominent achievements(i.e, show honor)?
As mentioned above, I have no deep knowledge about the community rule, but I found out
there are several projects are maintained with more than 2 maintainers, e.g, XEN hypervisor
interface, VEM subsystem, a few projects even have 4 maintainers, e.g, TMP driver.
So I was wonder why we can not have more maintainers given that XFS code base become more
and more large(more than one hundred thousand lines via simple `wc -l` though we have much
code comments).

I really felt frustrated from Dave's:"Quite frankly, XFS upstream is completely dysfunctional
right now and, as such, it's no longer a fun thing to work on.", as Dave had made lots of
contributions and going crazy hacking.

I wrote this just because I love this project, I still remember/appreciate Christoph gave me
a quick response to my first email in XFS two years ago, so that I can start to learn kernel
hacking from then on.


Thanks,
-Jeff

On 11/08/2013 10:23 AM, Ric Wheeler wrote:

> Hi Ben,
> 
> How exactly did we decide to add a new co-maintainer? Shouldn't we have
> some discussion on the list and see some substantial history of
> contributions?
> 
> Best regards,
> 
> Ric
> 
> 
> On 11/07/2013 05:08 PM, Mark Tinguely wrote:
>> Updated maintainer info.
>>
>> Signed-off-by: Ben Myers <bpm@sgi.com>
>> Reviewed-by: Mark Tinguely <tinguely@sgi.com>
>> ---
>>  MAINTAINERS |    2 +-
>>  1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> Index: b/MAINTAINERS
>> ===================================================================
>> --- a/MAINTAINERS    2013-11-07 15:42:04.554561805 -0600
>> +++ b/MAINTAINERS    2013-11-07 15:42:59.034889770 -0600
>> @@ -9388,7 +9388,7 @@ F:    drivers/xen/*swiotlb*
>>  XFS FILESYSTEM
>>  P:    Silicon Graphics Inc
>>  M:    Ben Myers <bpm@sgi.com>
>> -M:    Alex Elder <elder@kernel.org>
>> +M:    Mark Tinguely <tinguely@sgi.com>
>>  M:    xfs@oss.sgi.com
>>  L:    xfs@oss.sgi.com
>>  W:    http://oss.sgi.com/projects/xfs
>>
>> _______________________________________________
>> xfs mailing list
>> xfs@oss.sgi.com
>> http://oss.sgi.com/mailman/listinfo/xfs
> 
> _______________________________________________
> xfs mailing list
> xfs@oss.sgi.com
> http://oss.sgi.com/mailman/listinfo/xfs


_______________________________________________
xfs mailing list
xfs@oss.sgi.com
http://oss.sgi.com/mailman/listinfo/xfs

  parent reply	other threads:[~2013-11-08  5:23 UTC|newest]

Thread overview: 34+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-11-07 22:02 xfs: update maintainers file Ben Myers
2013-11-07 22:08 ` Mark Tinguely
2013-11-07 22:32   ` [PATCH v2] " Ben Myers
2013-11-07 23:18     ` Alex Elder
2013-11-08  2:23   ` xfs: " Ric Wheeler
2013-11-08  2:30     ` Zhi Yong Wu
2013-11-08  5:24     ` Jeff Liu [this message]
2013-11-08 11:03     ` XFS leadership and a new co-maintainer candidate Ric Wheeler
2013-11-08 18:03       ` Ben Myers
2013-11-08 18:09         ` Ric Wheeler
2013-11-08 19:45           ` Ben Myers
2013-11-08 19:34         ` Christoph Hellwig
2013-11-08 20:32           ` Ric Wheeler
2013-11-08 20:46           ` Ben Myers
2013-11-08 20:50             ` Ric Wheeler
2013-11-08 22:03               ` [PATCH] update xfs maintainers Ben Myers
2013-11-08 22:07                 ` Ric Wheeler
2013-11-08 22:17                   ` Ben Myers
2013-11-08 23:32                     ` Ric Wheeler
2013-11-09 23:17                       ` Ben Myers
2013-11-08 22:59                 ` Zhi Yong Wu
2013-11-08 23:44                   ` NeilBrown
2013-11-09 23:51                     ` Ben Myers
2013-11-10  0:30                       ` Ben Myers
2013-11-10  3:13                         ` Stan Hoeppner
2013-11-10 23:32                           ` Dave Chinner
2013-11-10 23:12                         ` Dave Chinner
2013-11-11  3:49                           ` Ben Myers
2013-11-14  6:22                 ` Dave Chinner
2013-11-15  0:33                   ` Ben Myers
2013-11-14 18:24                 ` Alex Elder
2013-11-14 18:25                 ` Alex Elder
2013-11-12 17:32             ` XFS leadership and a new co-maintainer candidate Christoph Hellwig
2013-11-12 19:30               ` Ben Myers

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=527C7594.1020504@oracle.com \
    --to=jeff.liu@oracle.com \
    --cc=bpm@sgi.com \
    --cc=elder@kernel.org \
    --cc=rwheeler@redhat.com \
    --cc=tinguely@sgi.com \
    --cc=torvalds@linux-foundation.org \
    --cc=xfs@oss.sgi.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox