[Top] [All Lists]

Re: [PATCH 48/49] xfs: Add read-only support for dirent filetype field

To: Ric Wheeler <rwheeler@xxxxxxxxxx>
Subject: Re: [PATCH 48/49] xfs: Add read-only support for dirent filetype field
From: Geoffrey Wehrman <gwehrman@xxxxxxx>
Date: Thu, 15 Aug 2013 13:04:00 -0500
Cc: Dave Chinner <david@xxxxxxxxxxxxx>, Mark Tinguely <tinguely@xxxxxxx>, xfs@xxxxxxxxxxx
Delivered-to: xfs@xxxxxxxxxxx
In-reply-to: <520C6E39.7030501@xxxxxxxxxx>
References: <1374215120-7271-1-git-send-email-david@xxxxxxxxxxxxx> <1374215120-7271-49-git-send-email-david@xxxxxxxxxxxxx> <51F80FA8.4060304@xxxxxxx> <20130812005905.GK12779@dastard> <5208E243.9080403@xxxxxxx> <20130813005024.GS12779@dastard> <520A53E8.6030604@xxxxxxx> <20130814075042.GE6023@dastard> <20130814184729.GA4604@xxxxxxx> <520C6E39.7030501@xxxxxxxxxx>
User-agent: Mutt/1.5.14 (2007-02-12)
On Thu, Aug 15, 2013 at 06:59:21AM +0100, Ric Wheeler wrote:
| As much as the community admires your brave willingness to protect us from 
| code that was entirely developed by Dave, that is not really how the 
| upstream kernel works.
| Dave is pretty much without equal in moving XFS along these days and this 
| is a key feature that we are depending on.
| Please don't try to create hurdles to other people getting work into 
| upstream, that kind of thing will lead to a fork of the XFS code base and 
| ultimately that will be harder for all of us.
| It sounds like what you really should look at doing is to work inside of 
| SGI to create a private, internal branch of the upstream code that you 
| deliver in your product. Upstream code is all about innovation and new 
| features, we don't let vendor specific, non-upstream branches become the 
| place for hardening our code.

I agree Dave is a gifted, dedicated, passionate and prolific contributor
to the XFS community.  His contributions are greatly appreciated as are
the contributions of all other members of the XFS community.  Dave does
stand out as the most significant code contributor to the project.  He
has played a significant role moving XFS forward both from within SGI
and from outside SGI.  This does not place his contributions beyond
review or discussion however.

My objection to the current readdir filetype field feature code is that
the feature is not being made available to mainstream XFS users, only
to those willing to run experimental code.  Dave has clearly stated that
he will not do the back-porting work.  I am not expecting Dave to do the
back-porting work.  Mark has volunteered to fill the back-porting role.
Th porting work cannot be completed as the feature is not yet complete.
The associated user space code changes have not been submitted,
and tests to validate the new feature have not been submitted.  I do
not consider this to be an unreasonable request, even if there were
back-porting required.

If Red Hat is depending on the readdir filetype field feature from Dave
and do not wish to wait for the feature to be completed before pulling
in the code, then perhaps Red Hat should create a internal branch of
the upstream code that you can deliver in your products.  The upstream
code should be stable and fully functional, not a place for incomplete,
partially tested, half finished work.

Geoffrey Wehrman  651-683-5496  gwehrman@xxxxxxx

<Prev in Thread] Current Thread [Next in Thread>