xfs
[Top] [All Lists]

Re: [PATCH 0/3] Add a new quotactl command to support 3 quota types in X

To: Steven Whitehouse <swhiteho@xxxxxxxxxx>
Subject: Re: [PATCH 0/3] Add a new quotactl command to support 3 quota types in XFS
From: Ben Myers <bpm@xxxxxxx>
Date: Wed, 21 Aug 2013 12:44:10 -0500
Cc: Chandra Seetharaman <sekharan@xxxxxxxxxx>, xfs@xxxxxxxxxxx, linux-fsdevel <linux-fsdevel@xxxxxxxxxxxxxxx>, Jan Kara <jack@xxxxxxx>, Abhijith Das <adas@xxxxxxxxxx>
Delivered-to: xfs@xxxxxxxxxxx
In-reply-to: <1377088082.2738.19.camel@menhir>
References: <1375828029-26360-1-git-send-email-sekharan@xxxxxxxxxx> <20130820222528.GG5262@xxxxxxx> <1377088082.2738.19.camel@menhir>
User-agent: Mutt/1.5.20 (2009-06-14)
Hi Steven,

On Wed, Aug 21, 2013 at 01:28:02PM +0100, Steven Whitehouse wrote:
> Hi,
> 
> On Tue, 2013-08-20 at 17:25 -0500, Ben Myers wrote:
> > Hi Chandra,
> > 
> > On Tue, Aug 06, 2013 at 05:27:06PM -0500, Chandra Seetharaman wrote:
> > > XFS now supports simultaneous use of 3 quota types (user, group,
> > > and project).
> > > 
> > > In order to support this a new quotactl command is introduced
> > > in this patch series.
> > 
> > I've applied patches 1 and 2 at git://oss.sgi.com/xfs/xfs.git, master 
> > branch.
> > I think patch 3 should go through the gfs2 tree.
> > 
> > Steven, I think if you want to pull in patch 3 you'll also need patch 1.  
> > Maybe
> > the easiest way is to cherry pick it from the xfs tree?  If you do we'll 
> > have a
> > trivial merge conflict.  Anyway, it's af30cb446dd5 if you want.
> > 
> > Thanks,
> >     Ben
> 
> I think it would be better if all the patches went through the same tree
> to save issues with dependencies. I don't mind if you want to put the
> gfs2 patches in the xfs tree to keep everything in the same place. There
> are no current quota changes in the gfs2 tree, so there should not be a
> conflict due to that.

Ok, we'll take that route.  Thanks.

> Also, it would be good if Christoph's comments could be addressed too,

Agreed.

-Ben

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