On Tue, Aug 29, 2017 at 09:41:02PM -0700, Darrick J. Wong wrote: > From: Darrick J. Wong <darrick.wong@xxxxxxxxxx> > > Make sure that we update the rmapbt correctly when we collapse-range a > file and the extents on both sides of the hole can be merged. We can > construct this pretty trivially with insert-range and write, so test > that too. > > Signed-off-by: Darrick J. Wong <darrick.wong@xxxxxxxxxx> > --- > src/punch-alternating.c | 15 +++++- > tests/xfs/706 | 127 +++++++++++++++++++++++++++++++++++++++++++++++ > tests/xfs/706.out | 9 +++ > tests/xfs/group | 1 > 4 files changed, 150 insertions(+), 2 deletions(-) > create mode 100755 tests/xfs/706 > create mode 100644 tests/xfs/706.out > > > diff --git a/src/punch-alternating.c b/src/punch-alternating.c > index 25228cd..281e283 100644 > --- a/src/punch-alternating.c > +++ b/src/punch-alternating.c > @@ -24,6 +24,7 @@ int main(int argc, char *argv[]) > struct stat s; > struct statfs sf; > off_t offset; > + off_t start_offset = 0; > int fd; > blksize_t blksz; > off_t sz; > @@ -33,11 +34,19 @@ int main(int argc, char *argv[]) > int size = 1; /* punch $SIZE blocks ... */ > int interval = 2; /* every $INTERVAL blocks */ > > - while ((c = getopt(argc, argv, "i:s:")) != EOF) { > + while ((c = getopt(argc, argv, "i:o:s:")) != EOF) { Update usage info too? Otherwise looks fine to me. Thanks, Eryu -- To unsubscribe from this list: send the line "unsubscribe linux-xfs" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html