On Tue, Feb 18, 2014 at 10:05:49AM +0100, Lukáš Czerner wrote: > On Wed, 12 Feb 2014, Namjae Jeon wrote: > > > Date: Wed, 12 Feb 2014 11:28:35 +0900 > > From: Namjae Jeon <[email protected]> > > To: Lukáš Czerner <[email protected]> > > Cc: [email protected], [email protected], [email protected], > > [email protected], > > [email protected], [email protected], [email protected], > > [email protected], [email protected], > > [email protected], [email protected], > > Namjae Jeon <[email protected]>, > > Ashish Sangwan <[email protected]> > > Subject: Re: [PATCH RESEND 3/10] ext4: Add support FALLOC_FL_COLLAPSE_RANGE > > for fallocate > > > > 2014-02-11 20:59 GMT+09:00, Lukáš Czerner <[email protected]>: > > > On Sun, 2 Feb 2014, Namjae Jeon wrote: > > > > > >> Date: Sun, 2 Feb 2014 14:44:34 +0900 > > >> From: Namjae Jeon <[email protected]> > > >> To: [email protected], [email protected], [email protected], > > >> [email protected], > > >> [email protected], [email protected], [email protected] > > >> Cc: [email protected], [email protected], > > >> [email protected], [email protected], > > >> Namjae Jeon <[email protected]>, Namjae Jeon > > >> <[email protected]>, > > >> Ashish Sangwan <[email protected]> > > >> Subject: [PATCH RESEND 3/10] ext4: Add support FALLOC_FL_COLLAPSE_RANGE > > >> for > > >> fallocate > > >> > > >> From: Namjae Jeon <[email protected]> > > >> > > >> Add support FALLOC_FL_COLLAPSE_RANGE for fallocate > > Hi Lukas. > > > > > > Description is missing here, please provide FALLOC_FL_COLLAPSE_RANGE > > > description so people know what it's supposed to be doing. > > > > > > more comments bellow. > > Okay, I will udpate. > > > > > > Thanks! > > > -Lukas > > Hi, > > you may have noticed my patches for new FALLOC_FL_ZERO_RANGE > fallocate flag. This changes things around the same area as your > patches does so we should probably figure out which are going to > land in first and then rebase the other on top of that. > > One concern I have is that I have not seen any tests provided to > verify the feature. But I just may have missed it. Do you have any > xfstests test or at least fsx and fsstress patches to provide > support for testing FALLOC_FL_COLLAPSE_RANGE ? Patches for > util_linux might also be handy.
There were 5 xfstests patches posted that used the _test_generic_punch infrastructure to test this functionality. See the december 08 index here, search for "shared/00[1-5]": http://oss.sgi.com/archives/xfs/2013-12/index.html They need a little rework, but otherwise the test code is there. Having the feature added to fsx and fsstress would probably be a good idea, though. Cheers, Dave. -- Dave Chinner [email protected] -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
