xfs
[Top] [All Lists]

Re: [PATCH 1/3] xfs: Remove useless test

To: Dave Chinner <david@xxxxxxxxxxxxx>
Subject: Re: [PATCH 1/3] xfs: Remove useless test
From: Jan Kara <jack@xxxxxxx>
Date: Tue, 20 Jan 2015 23:35:15 +0100
Cc: Jan Kara <jack@xxxxxxx>, Brian Foster <bfoster@xxxxxxxxxx>, xfs@xxxxxxxxxxx
Delivered-to: xfs@xxxxxxxxxxx
In-reply-to: <20150120221453.GE16552@dastard>
References: <1421317085-14350-1-git-send-email-jack@xxxxxxx> <1421317085-14350-2-git-send-email-jack@xxxxxxx> <20150115132754.GA23696@xxxxxxxxxxxxxxx> <20150120164400.GB3648@xxxxxxxxxxxxx> <20150120221453.GE16552@dastard>
User-agent: Mutt/1.5.21 (2010-09-15)
  Hello,

On Wed 21-01-15 09:14:53, Dave Chinner wrote:
> On Tue, Jan 20, 2015 at 05:44:00PM +0100, Jan Kara wrote:
> > On Thu 15-01-15 08:27:54, Brian Foster wrote:
> > > On Thu, Jan 15, 2015 at 11:18:03AM +0100, Jan Kara wrote:
> > > > Q_XQUOTARM is never passed to xfs_fs_set_xstate() so remove the test.
> > > > 
> > > > Signed-off-by: Jan Kara <jack@xxxxxxx>
> > > > ---
> > > 
> > > Reviewed-by: Brian Foster <bfoster@xxxxxxxxxx>
> >   Thanks for reviews! I've pulled the XFS cleanups from this series into my
> > tree.
> 
> I've been at LCA last week and sick all this week as a result, so I
> haven't yet got to pulling these into the XFS tree. Do you still
> want me to give you a stable branch with these changes in it to base
> your series off?
  If you don't mind, I would just carry those three XFS cleanups together
with the rest of the quota interface series in my tree. Currently there
don't seem to be any conflicts with what's in XFS tree so it's easy. And
the series touches XFS in those areas later anyway...

                                                                Honza
-- 
Jan Kara <jack@xxxxxxx>
SUSE Labs, CR

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