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 <linkinj...@gmail.com> > > To: Lukáš Czerner <lczer...@redhat.com> > > Cc: v...@zeniv.linux.org.uk, da...@fromorbit.com, b...@sgi.com, > > ty...@mit.edu, > > adilger.ker...@dilger.ca, j...@suse.cz, mtk.manpa...@gmail.com, > > linux-fsde...@vger.kernel.org, x...@oss.sgi.com, > > linux-e...@vger.kernel.org, linux-kernel@vger.kernel.org, > > Namjae Jeon <namjae.j...@samsung.com>, > > Ashish Sangwan <a.sang...@samsung.com> > > 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 <lczer...@redhat.com>: > > > On Sun, 2 Feb 2014, Namjae Jeon wrote: > > > > > >> Date: Sun, 2 Feb 2014 14:44:34 +0900 > > >> From: Namjae Jeon <linkinj...@gmail.com> > > >> To: v...@zeniv.linux.org.uk, da...@fromorbit.com, b...@sgi.com, > > >> ty...@mit.edu, > > >> adilger.ker...@dilger.ca, j...@suse.cz, mtk.manpa...@gmail.com > > >> Cc: linux-fsde...@vger.kernel.org, x...@oss.sgi.com, > > >> linux-e...@vger.kernel.org, linux-kernel@vger.kernel.org, > > >> Namjae Jeon <linkinj...@gmail.com>, Namjae Jeon > > >> <namjae.j...@samsung.com>, > > >> Ashish Sangwan <a.sang...@samsung.com> > > >> Subject: [PATCH RESEND 3/10] ext4: Add support FALLOC_FL_COLLAPSE_RANGE > > >> for > > >> fallocate > > >> > > >> From: Namjae Jeon <namjae.j...@samsung.com> > > >> > > >> 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 da...@fromorbit.com -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
