xfs
[Top] [All Lists]

Re: [PATCH 34/76] xfs: implement XFS_IOC_SWAPEXT when rmap btree is enab

To: Christoph Hellwig <hch@xxxxxxxxxxxxx>
Subject: Re: [PATCH 34/76] xfs: implement XFS_IOC_SWAPEXT when rmap btree is enabled
From: "Darrick J. Wong" <darrick.wong@xxxxxxxxxx>
Date: Mon, 4 Jan 2016 15:40:33 -0800
Cc: xfs@xxxxxxxxxxx
Delivered-to: xfs@xxxxxxxxxxx
In-reply-to: <20160103121728.GC28429@xxxxxxxxxxxxx>
References: <20151219085622.12713.88678.stgit@xxxxxxxxxxxxxxxx> <20151219090008.12713.25755.stgit@xxxxxxxxxxxxxxxx> <20160103121728.GC28429@xxxxxxxxxxxxx>
User-agent: Mutt/1.5.21 (2010-09-15)
On Sun, Jan 03, 2016 at 04:17:28AM -0800, Christoph Hellwig wrote:
> On Sat, Dec 19, 2015 at 01:00:08AM -0800, Darrick J. Wong wrote:
> > Implement extent swapping when reverse-mapping is enabled.
> 
> Can you just fold this into the previous patch?  There seem to be a lot
> of patches in the series that would benefit from merging, but this
> is the one that really screams for it :)

The only reason I'm keeping Dave's original rmap patches separate is because
the last time I checked with Dave, he said that he'd made a few bugfixes to his
patchset that hadn't yet been sent out, so I'd like to merge those fixes into
the original patches before starting any combining work.  It also might be more
convenient (for him, anyway) to keep my rmapbt changes separate for review
purposes.

At this point, though, I've made so many changes that I'd like to pull in
Dave's changes and combine them with mine into fewer patches.  (AFAIK, Dave
hasn't sent any further reverse-mapping patches since 3 June 2015.)

--D

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

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