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

Dave Chinner
2015-01-20T23:21:28Z

On Tue, Jan 20, 2015 at 11:35:15PM +0100, Jan Kara wrote:
> 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@suse.cz>
> > > > > ---
> > > >
> > > > Reviewed-by: Brian Foster <bfoster@redhat.com>
> > > 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...

Fine by me.

Cheers,

Dave.
--
Dave Chinner
david@fromorbit.com